: revert sudo upgrade to fix rootless X
cover : revert sudo upgrade to fix rootless X
I found the recent sudo upgrade to be the cause of Yocto Bug 1211. I have reviewed the sudo changelogs and didn't notice any changes to indicate that the xserver-nodm init script was misusing sudo. Reverting the upgrade resolves the issue. As the commit message doesn't indicate it was anything other than a routine upgrade (and not a bug fix) I suggest we revert it until it can be shown to work properly with xserver-nodm for rootless X.
The following changes since commit fa4bcfdb73167f8159b88e5a4d711c0d37627a70:
bb-matrix: correct BB and PM number canonicalization (2011-07-14 22:23:09 +0100)
are available in the git repository at:
git://git.pokylinux.org/poky-contrib dvhart/sudo
http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=dvhart/sudo
Darren Hart (1):
Revert "sudo: upgrade to v1.8.1p2"
meta/recipes-extended/sudo/sudo.inc | 27 +++++++++++--------
.../sudo/{sudo_1.8.1p2.bb => sudo_1.7.4p6.bb} | 4 +-
2 files changed, 18 insertions(+), 13 deletions(-)
rename meta/recipes-extended/sudo/{sudo_1.8.1p2.bb => sudo_1.7.4p6.bb} (52%)PATCH 1/1 Revert "sudo: upgrade to v1.8.1p2"
Fixes [YOCTO 1211]
This reverts commit 7c6f47b13c55e5c1d15d9ab9c2cee3da065fb592.
The upgrade caused rootless X to fail on at least the n450 BSP. X would fail to start with:
xf86OpenConsole: Cannot open /dev/tty0 (No such file or directory)
The file does indeed exist. If the permissions were changed to a+rw, X would get further, and then abort with permissions issues with /dev/dri/card0. Setting those permissions to a+rw would allow it to start on boot without issue. However, if the init script was run after logging in as root, X would start fine (no changes necessary to the files it complained about on boot).
Reverting this patch restores the rootless X functionality.
---
meta/recipes-extended/sudo/sudo.inc | 27 +++++++++++--------
.../sudo/{sudo_1.8.1p2.bb => sudo_1.7.4p6.bb} | 4 +-
2 files changed, 18 insertions(+), 13 deletions(-)
rename meta/recipes-extended/sudo/{sudo_1.8.1p2.bb => sudo_1.7.4p6.bb} (52%)meta/recipes-extended/sudo/sudo.inc +16 -11
@@ -3,12 +3,14 @@ DESCRIPTION = "Sudo (superuser do) allows a system administrator to give certain HOMEPAGE = "http://www.sudo.ws" BUGTRACKER = "http://www.sudo.ws/bugs/" SECTION = "admin" -LICENSE = "ISC & UCB & Zlib" -LIC_FILES_CHKSUM = "file://doc/LICENSE;md5=54f1b46c2459ecec3d892618eab44302 \ - file://compat/fnmatch.c;beginline=6;endline=31;md5=5872733146b9eb0deb79e1f664815b85 \ - file://compat/getcwd.c;beginline=5;endline=27;md5=449af4cc57fc7d46f42090608ba3e681 \ - file://compat/glob.c;beginline=6;endline=31;md5=5872733146b9eb0deb79e1f664815b85 \ - file://compat/snprintf.c;beginline=6;endline=31;md5=c98b24f02967c095d7a70ae2e4d4d4ea" +LICENSE = "ISC & UCB & MIT" +LIC_FILES_CHKSUM = "file://LICENSE;md5=0b07397b2fca3fb8b71f08cd85c6eb3f \ + file://nonunix.h;beginline=4;endline=28;md5=af7d361f47ae60769cac77e4fca0dbb0 \ + file://vasgroups.c;beginline=4;endline=28;md5=af7d361f47ae60769cac77e4fca0dbb0 \ + file://fnmatch.c;beginline=6;endline=31;md5=5872733146b9eb0deb79e1f664815b85 \ + file://getcwd.c;beginline=5;endline=27;md5=449af4cc57fc7d46f42090608ba3e681 \ + file://glob.c;beginline=6;endline=31;md5=5872733146b9eb0deb79e1f664815b85 \ + file://snprintf.c;beginline=6;endline=31;md5=c98b24f02967c095d7a70ae2e4d4d4ea" inherit autotools
@@ -20,14 +22,17 @@ do_configure_prepend () { fi } -# Explicitly create ${localstatedir}/lib before do_install to ensure -# the directory is accessible by all users. Otherwise the mkinstalldirs -# script (from sudo) will recursively create ${localstatedir}/lib/sudo -# and then chmod each directory with 0700 permissions, which isn't what -# we want (i.e, users would not be able to access /var/lib). +# The script "mkinstalldirs" from package "sudo" will create directory +# "/var/lib/sudo" by recursion with mode "0700" during installing files. +# That is to say, "var", "var/lib" and "var/lib/sudo" will possess access authority +# with mode "0700". It cause that directory "var" and "var/lib" +# can't be accessed by common user. Creating directory "/var/lib" before +# installing files can resolve this problem. + do_install_prepend (){ mkdir -p ${D}/${localstatedir}/lib } + pkg_postinst_${PN} () { if [ "x$D" != "x" ]; then
meta/recipes-extended/sudo/sudo_1.7.4p6.bb renamed from meta/recipes-extended/sudo/sudo_1.8.1p2.bb +2 -2
@@ -5,7 +5,7 @@ PR = "r0" SRC_URI = "http://ftp.sudo.ws/sudo/dist/sudo-${PV}.tar.gz \ file://libtool.patch" -SRC_URI[md5sum] = "e8330f0e63b0ecb2e12b5c76922818cc" -SRC_URI[sha256sum] = "281f90c80547cf22132e351e7f61c25ba4ba9cf393438468f318f9a7884026fb" +SRC_URI[md5sum] = "1ae12d3d22e7ffedbf2db26f957676f0" +SRC_URI[sha256sum] = "20091ef71018698c674c779f4b57178b2ecb4275fa34909b06219d2688ad14d5" EXTRA_OECONF += " --with-pam=no"
Signature · 1 line
1.7.1