Merge lp://staging/~jibel/ubiquity/lp1719734-oem-prepare-wayland into lp://staging/ubiquity

Proposed by Jean-Baptiste Lallement
Status: Merged
Approved by: Mathieu Trudel-Lapierre
Approved revision: 6564
Merged at revision: 6568
Proposed branch: lp://staging/~jibel/ubiquity/lp1719734-oem-prepare-wayland
Merge into: lp://staging/ubiquity
Diff against target: 14 lines (+3/-1)
1 file modified
bin/oem-config-prepare (+3/-1)
To merge this branch: bzr merge lp://staging/~jibel/ubiquity/lp1719734-oem-prepare-wayland
Reviewer Review Type Date Requested Status
Mathieu Trudel-Lapierre Approve
Review via email: mp+331385@code.staging.launchpad.net

Description of the change

Under wayland, the confirmation dialog of oem prepare is not displayed because oem-config-prepare is pkexec-ed as root.
This fix drop privileges to the oem user so zenity can be executed and display the dialog.

To post a comment you must log in.
Revision history for this message
Mathieu Trudel-Lapierre (cyphermox) wrote :

Yes please.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to status/vote changes: