From 8d682b7f6135c42c647c36f14c384db6453c3813 Mon Sep 17 00:00:00 2001 From: nekral-guest Date: Sat, 24 May 2008 19:20:33 +0000 Subject: [PATCH] Remove the transition proposal email for bug 276419. The transition is now over one year. --- 276419 | 168 --------------------------------------------------------- 1 file changed, 168 deletions(-) delete mode 100644 276419 diff --git a/276419 b/276419 deleted file mode 100644 index fdb122cc..00000000 --- a/276419 +++ /dev/null @@ -1,168 +0,0 @@ -Introduction -============ -As reported in #276419, su in the login Debian package doesn't permit to -specify options to the invoked shell and doesn't respect quoted arguments. -We plan to revert this behavior and follow su's documentation and other -implementations. - - -Short details -============= -Packages passing a command in argument to su must use su's -c option -and must quote the command if it contains a space. -For example: - su - root -c "ls -l /" - -The following commands won't work anymore: - su - root -c ls -l / - su - root "ls -l /" - su - root ls -l / - -There will be no problems for backports. -c can be used and arguments -quoted, with the past and future versions. - -Needed adaptations -================== -We tried to find the packages that will be affected by this transition. -We did not audit the full archive, but focused on [1]: - * maintainer scripts [2] - * packages with an init.d script (based on a sid Contents-i386) - * packages with an cron script (based on a sid Contents-i386) - * native packages (on sid i386) -(In general, archives embedded in source packages were not checked) - -Package needing changes ------------------------ -Micah Anderson - backupninja-0.9.2/handlers/pgsql - backupninja-0.9.2/handlers/mysql - backupninja-0.9.2/examples/example.rdiff -Raphael Bossek - python-4suite-0.99cvs20051115/debian/python-4suite-server.init.d -Arnaud Kyheng - gnunet-0.7.0b/contrib/init_gnunet_ubuntu -Brian May - amavisd-new-2.3.3/debian/amavisd-new.cron.daily -Peter Palfrader - echolot-2.1.8/debian/echolot.init - Fixed in 2.1.8-4 -Javier Fernandez-Sanguino Pen~a - samhain-2.0.10a/init/samhain.start.in - Not in Debian - -To be checked -------------- -Roderick Schertler - debget-1.5/debget -(It should be OK. According to the code, it works with GNU su) - -maybe ------ -Stefan Hornburg (Racke) - courier-0.52.1/courier.lpspec(.in)? (maybe not used on Debian) - courier-0.52.1/courier.spec(.in)? (maybe not used on Debian) -Kenneth J. Pronovici - cedar-backup2-2.7.2/CedarBackup2/peer.py (depends on executeCommand) -Arnaud Quette - nut-2.0.2/scripts/HP-UX/nut-drvctl.sh (maybe not used on Debian) - nut-2.0.2/scripts/HP-UX/nut-upsd.sh (maybe not used on Debian) -Taku YASUI - murasaki-0.8.11/scripts/printer (su $USER -c $CMD, $CMD may have a space) -Debian Webmin maintainers - usermin-1.160/cron/config-aix (maybe not used on Debian) - usermin-1.160/web-lib-funcs.pl - usermin-1.160/shell/index.cgi - usermin-1.160/fetchmail/check.pl - usermin-1.160/commands/run.cgi - usermin-1.160/postgresql/postgresql-lib.pl - webmin-1.230/web-lib-funcs.pl - webmin-1.230/cron/config-aix - webmin-1.230/custom/run.cgi - -In comments or documentation ----------------------------- -Clint Adams - bricolage-1.8.8/bin/bric_ftpd -Joel Aelwyn - debpool-0.2.2/debian/README.User -Phil Brooke - yiff-2.14.2/configure -Debian Qt/KDE Maintainers - kdenetwork-3.5.0/kopete/protocols/meanwhile/README -Henrique de Moraes Holschuh - cyrus21-imapd-2.1.18/debian/cyrus21-common.postinst -Robert Jordens - remstats-1.0.13a/INSTALL - remstats-1.0.13a/docs/book.tex (and other formats) - remstats-1.0.13a/docs/install-user.pod - remstats-1.0.13a/docs/install.pod - remstats-1.0.13a/docs/install.txt -Matthias Klose - sqlrelay-0.36.4/doc/gettingstarted/interbase.html -Guus Sliepen - dhis-client-5.3/README -Craig Small - lprng-3.8.28/DOCS/LPRng-Reference.html - lprng-3.8.28/DOCS/LPRng-Reference.sgml - lprng-3.8.28/DOCS/LPRng-Reference-Multipart/x9198.htm -Jonas Smedegaard - pop-before-smtp-1.36/contrib/README.rootless-install - -Transition plan -=============== -Date: 1 month after the announcement - -The SU_NO_SHELL_ARGS environment variable will restore the previous -behavior. The support for this variable should be dropped after Etch. - -login will conflict with the package of the first category. When fixed, -these packages do not need a versionned dependency on login. - - -Recommandation -============== -You should follow the following synopsis for your su commands. -(This will give you more chance to be portable and to work on -POSIXLY_CORRECT environments) - - su [options] [-] [username [args]] - -[args] are arguments passed to the shell - -Specifically: - * It is preferable to provide -c in [args] rather than in [options]. - * su - root -p doesn't work if the POSIXLY_CORRECT environment - variable is set. - -The following packages don't follow these rules: -Stefan Hornburg (Racke) - interchange-5.3.2/debian/interchange.cron.daily - interchange-5.3.2/scripts/restart.PL -Michael Biebl - powersave-0.9.25/scripts/wm_shutdown - powersave-0.9.25/scripts/do_screen_saver - powersave-0.9.25/scripts/wm_logout - powersave-0.9.25/scripts/x_helper_functions -Popularity Contest Developers - popularity-contest-1.31/debian/cron.weekly - popularity-contest-1.31/FAQ -Robert Luberda - dwww-1.9.26/dwww-format-man -Andreas Metzler - findutils-4.2.26/locate/updatedb.sh -Paul Waite - axyl-2.1.9/db/postgres/install-db.sh -Debian Webmin maintainers - usermin-1.160/web-lib-funcs.pl - usermin-1.160/commands/run.cgi - webmin: ditto - - - -[1] The rationale is that we consider there is a greater chance to find - problems on Debian specific packages/scripts since it would have fail - on other OS (on RedHat, Gentoo, Mandriva, SunOS). - Probably 10% of the archive was audited. - -[2] Thanks to Bill Allombert - http://lists.debian.org/debian-devel/2005/11/msg01215.html