gdm

Comment 68 for bug 1652282

Revision history for this message
In , Mike Fleetwood (mfleetwo) wrote :

Created attachment 358405
Interim workaround for display access (v1)

Hi Curtis,

Here is patchset v1 ready for fully review. Compared to draft 1 from
comment #11 the changes are:
* Incrementally updates the README file in a number of patches for
  various changes.
* P4/9 "Add required polkit action file"
** Installs polkit action file into $(datadir)/polkit-1/actions so
   make distcheck works, but needs to be manually installed when prefix
   is other than /usr.
** Documents this in the README file.
** Mark relevant strings in and the file org.gnome.gparted.policy.in.in
   for translation.
** Increases minimum intltool to 0.36.0 in configure.ac because of the
   use of INTLTOOL_POLICY_RULE in Makefile.am.
** Adds .gitignore entries for new build files
   org.gnome.gparted.policy{,.in}.
* P6/9 "Check for pkexec >= 0.102 which supports execution of X11..."
  moves before P8/9 "Only when configured, grant root access to the..."
* Adds new patches:
  P7/9 "Remove unnecessary autoconf check for pkexec --disable-inter..."
  P9/9 "Add .dirstamp to .gitignore"
* Minor commit wording updates and corrections.

Testing:
1) On Ubuntu 16.10 using Wayland display that configuring with
   --enable-xhost-root allows gparted run as normal user to display and
   without the config option gparted is not allowed to display.
   Note:
   sudo doesn't grant access to the X11 server under Wayland display so
   sudo gparted doesn't work. Have to run gparted as normal user to
   allow it to use xhost to grant root access to the display.
2) On Fedora 26 using Wayland display same results when configured with
   and without --enable-xhost-root.
3) On CentOS 7 with X11 display and configured without
   --enable-xhost-root allows gparted run as normal user and as root to
   display.
4) That make && make distcheck works for every commit.

Thanks,
Mike