Merge lp:~rohangarg/ubiquity/ubiquity into lp:ubiquity

Proposed by Rohan Garg
Status: Merged
Merged at revision: 6172
Proposed branch: lp:~rohangarg/ubiquity/ubiquity
Merge into: lp:ubiquity
Diff against target: 20 lines (+4/-3)
1 file modified
ubiquity/plugins/ubi-language.py (+4/-3)
To merge this branch: bzr merge lp:~rohangarg/ubiquity/ubiquity
Reviewer Review Type Date Requested Status
Ubuntu Installer Team Pending
Review via email: mp+215839@code.launchpad.net
To post a comment you must log in.
lp:~rohangarg/ubiquity/ubiquity updated
6172. By Jonathan Riddell

call the appropriate
drop_privileges and regain_privileges functions

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'ubiquity/plugins/ubi-language.py'
--- ubiquity/plugins/ubi-language.py 2014-02-27 15:41:18 +0000
+++ ubiquity/plugins/ubi-language.py 2014-04-15 10:40:26 +0000
@@ -497,12 +497,13 @@
497 def openURL(self, url):497 def openURL(self, url):
498 from PyQt4.QtGui import QDesktopServices498 from PyQt4.QtGui import QDesktopServices
499 from PyQt4.QtCore import QUrl499 from PyQt4.QtCore import QUrl
500 from ubiquity.misc import drop_privileges_save, regain_privileges_save
501500
502 # this nonsense is needed because kde doesn't want to be root501 # this nonsense is needed because kde doesn't want to be root
503 drop_privileges_save()502 misc.drop_privileges()
503 misc.drop_privileges_save()
504 QDesktopServices.openUrl(QUrl(url))504 QDesktopServices.openUrl(QUrl(url))
505 regain_privileges_save()505 misc.regain_privileges()
506 misc.regain_privileges_save()
506507
507 def set_language_choices(self, choices, choice_map):508 def set_language_choices(self, choices, choice_map):
508 PageBase.set_language_choices(self, choices, choice_map)509 PageBase.set_language_choices(self, choices, choice_map)

Subscribers

People subscribed via source and target branches

to status/vote changes: