After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 687813 - Use kdesudo on KDE, as gksu is not installed by default
Use kdesudo on KDE, as gksu is not installed by default
Status: RESOLVED FIXED
Product: gparted
Classification: Other
Component: application
0.14.0
Other Linux
: Normal normal
: ---
Assigned To: gparted maintainers alias
gparted maintainers alias
Depends on:
Blocks:
 
 
Reported: 2012-11-06 22:22 UTC by Curtis Gedak
Modified: 2012-12-12 18:11 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
kdesudo patch set 1 (4.23 KB, patch)
2012-11-07 00:57 UTC, Curtis Gedak
none Details | Review

Description Curtis Gedak 2012-11-06 22:22:57 UTC
When gparted is compiled and installed from a source tarball, the application does not start when invoked from the "K --> System --> GParted (Partition Editor)".  Instead a message dialog is displayed indicating:

   Root privileges are required for running GParted

   Since GParted is a powerful tool capable of destroying partition
   tables and vast amounts of data, only root may run it.

This problem occurrs on a kubuntu 12.04 LTS system.
Comment 1 Curtis Gedak 2012-11-07 00:57:14 UTC
Created attachment 228330 [details] [review]
kdesudo patch set 1

The included set of patches adds kdesudo and gksudo to the search list of possible commands to use to invoke gparted with root privilege.

This patch set has been successfully tested on the following systems:

Tested with and without gksu package on:
kubuntu 12.04 LTS
kubuntu 10.04 LTS
kubuntu  8.04 LTS

Tested with gksu package on:
ubuntu 12.04 LTS
ubuntu 10.04 LTS
ubuntu  8.04 LTS

I also tried testing on Fedora, but this distribution does not appear to include gksu or gksudo (at least I did not find these commands).  Most likely Fedora will be better addressed by the following report:
Bug #552637 - PolicyKit support
Comment 2 Mike Fleetwood 2012-11-10 20:02:48 UTC
Hi Curtis,

Passed review.

As you've done so much testing on Ubuntu I though I would try something
different.  My Fedora boxes weren't going to be of much use as they
don't include any of gksu, gksudo and kdesudo.  So I installed Debian 6
inside VirtualBox.  Tested under GNOME and KDE with gksu, gksudo and
kdesudo.  All worked.

Let me know and I'll commit it.

Thanks,
Mike
Comment 3 Curtis Gedak 2012-11-10 21:40:55 UTC
Thank you Mike for reviewing and testing this patch set.

With Fedora I think the best path forward is to add PolicyKit support.  Distributions, such as Ubuntu, can also use PolicyKit, so that might become the default "gain root privilege" method, with a fall-back being gksudo, gksu, and kdesudo.

You have my okay to commit this patch set.

Thanks,
Curtis
Comment 4 Mike Fleetwood 2012-11-11 12:22:44 UTC
Hi Curtis,

Patchset "kdesudo patch set 1" from comment #1 has been committed to git
repository for inclusion in the next release of GParted.

The relevant git commits can be viewed at the following links:

Use kdesudo in gparted.desktop when gksu is not installed (#687813)
http://git.gnome.org/browse/gparted/commit/?id=ec4a2ac09b28b0db2acea02fb314d6be73909e4b

Use gksudo, if available, in gparted.desktop
http://git.gnome.org/browse/gparted/commit/?id=88a293988027fdc351404ac50c76caea3cc97b64

Thanks,
Mike
Comment 5 Curtis Gedak 2012-12-12 18:11:23 UTC
The enhancements to address this bug report have been included in GParted 0.14.1 released on December 12, 2012.