On Wed, Dec 08, 2010 at 08:44:48PM +0000, Adam D. Barratt wrote:
> On Tue, 2010-12-07 at 17:31 +0000, Roger Leigh wrote:
> > I've made a new upload of schroot to unstable which fixes a few
> > security- and upgrade-related bugs. The bulk of the changes are
> > documentation (manual pages, release notes and changelogs). The
> > code changes are tiny, but are important to have to upgrade from
> > lenny cleanly and fully document security issues and program
> > behaviour.
>
> Unblocked; thanks.
>
> As discussed on IRC, it's a shame that this resulted in a new
> translatable (and currently untranslated) string, although the fact that
> the string should rarely be seen and is in the program translation
> rather than a debconf template mitigates this slightly.
I've allowed a fortnight for translation updates to come back, and
made a new release (1.4.17-1) containing five translation updates
and an additional RC bugfix. Please could you unblock this to
allow it into squeeze?
Changes:
http://git.debian.org/?p=buildd-tools/schroot.git;a=commitdiff;h=debian/schroot-1.4.17-1;hp=debian/schroot-1.4.16-1
Changes with generated files included:
http://git.debian.org/?p=buildd-tools/schroot.git;a=commitdiff;h=distribution/schroot-1.4.17;hp=distribution/schroot-1.4.16
Many thanks,
Roger
schroot (1.4.17-1) unstable; urgency=low
* New upstream stable release.
* 15killprocs: Don't kill processes in other sessions
(Closes: #608054). Compare full chroot path in addition to device
and inode numbers, since the device and inode are not sufficiently
unique (they are shared between non-cloned sessions such as for
directory type chroots).
* Updated translations:
- da (Closes: #606305). Thanks to Joe Hansen.
- de (Closes: #606245). Thanks to Holger Wansing.
- fr (Closes: #606394). Thanks to Thomas Blein.
- it. Thanks to Vincenzo Campanella.
- zh_CN. Thanks to Ji ZhengYu.
--
.''`. Roger Leigh
: :' : Debian GNU/Linux http://people.debian.org/~rleigh/
`. `' Printing on GNU/Linux? http://gutenprint.sourceforge.net/
`- GPG Public Key: 0x25BFB848 Please GPG sign your mail.
diff --git a/NEWS b/NEWS
index 8fa9bba..bdc5677 100644
--- a/NEWS
+++ b/NEWS
@@ -1,12 +1,16 @@
NEWS -*- outline -*-
----
-Welcome to schroot 1.4.16. Please read these release notes carefully.
+Welcome to schroot 1.4.17. Please read these release notes carefully.
Full installation instructions are provided in the INSTALL file. The
README file also contains more specific notes regarding building and
configuration.
+* Major changes in 1.4.17:
+
+ None.
+
* Major changes in 1.4.16:
1) Chroot naming restrictions introduced in 1.4.0 have been relaxed
diff --git a/debian/changelog b/debian/changelog
index 19022bf..fdb1c7f 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,20 @@
+schroot (1.4.17-1) unstable; urgency=low
+
+ * New upstream stable release.
+ * 15killprocs: Don't kill processes in other sessions
+ (Closes: #608054). Compare full chroot path in addition to device
+ and inode numbers, since the device and inode are not sufficiently
+ unique (they are shared between non-cloned sessions such as for
+ directory type chroots).
+ * Updated translations:
+ - da (Closes: #606305). Thanks to Joe Hansen.
+ - de (Closes: #606245). Thanks to Holger Wansing.
+ - fr (Closes: #606394). Thanks to Thomas Blein.
+ - it. Thanks to Vincenzo Campanella.
+ - zh_CN. Thanks to Ji ZhengYu.
+
+ -- Roger Leigh <rleigh@debian.org> Wed, 29 Dec 2010 16:41:30 +0000
+
schroot (1.4.16-1) unstable; urgency=low
* New upstream stable release.
diff --git a/etc/setup.d/15killprocs b/etc/setup.d/15killprocs
index 619035e..1f246b0 100755
--- a/etc/setup.d/15killprocs
+++ b/etc/setup.d/15killprocs
@@ -40,28 +40,35 @@ do_kill_all()
info "Killing processes run inside $1"
ls /proc | egrep '^[[:digit:]]+$' |
while read pid; do
+ # Check if process root are the same device/inode as chroot
+ # root (for efficiency)
if [ /proc/"$pid"/root -ef "$1" ]; then
- exe=$(readlink /proc/"$pid"/exe || true)
- info "Killing left-over pid $pid (${exe##$1})"
- info " Sending SIGTERM to pid $pid"
+ # Check if process and chroot root are the same (may be
+ # different even if device/inode match).
+ root=$(readlink /proc/"$pid"/root || true)
+ if [ "$root" = "$1" ]; then
+ exe=$(readlink /proc/"$pid"/exe || true)
+ info "Killing left-over pid $pid (${exe##$1})"
+ info " Sending SIGTERM to pid $pid"
- /bin/kill -TERM "$pid" 2>/dev/null
+ /bin/kill -TERM "$pid" 2>/dev/null
- count=0
- max=5
- while [ -d /proc/"$pid" ]; do
- count=$(( $count + 1 ))
- info " Waiting for pid $pid to shut down... ($count/$max)"
- sleep 1
- # Wait for $max seconds for process to die before -9'ing it
- if [ "$count" -eq "$max" ]; then
- info " Sending SIGKILL to pid $pid"
- /bin/kill -KILL "$pid" 2>/dev/null
+ count=0
+ max=5
+ while [ -d /proc/"$pid" ]; do
+ count=$(( $count + 1 ))
+ info " Waiting for pid $pid to shut down... ($count/$max)"
sleep 1
- break
- fi
- done
- fi
+ # Wait for $max seconds for process to die before -9'ing it
+ if [ "$count" -eq "$max" ]; then
+ info " Sending SIGKILL to pid $pid"
+ /bin/kill -KILL "$pid" 2>/dev/null
+ sleep 1
+ break
+ fi
+ done
+ fi
+ fi
done
}
diff --git a/po/da.po b/po/da.po
index 5c5ec18..9bf7a48 100644
--- a/po/da.po
+++ b/po/da.po
@@ -19,7 +19,7 @@ msgstr ""
"Project-Id-Version: schroot\n"
"Report-Msgid-Bugs-To: Roger Leigh <rleigh@debian.org>\n"
"POT-Creation-Date: 2010-12-07 14:00+0000\n"
-"PO-Revision-Date: 2010-09-07 17:34+0000\n"
+"PO-Revision-Date: 2010-12-08 17:34+0000\n"
"Last-Translator: Joe Hansen <joedalton2@yahoo.dk>\n"
"Language-Team: Danish <dansk@dansk-gruppen.dk>\n"
"Language: da\n"
@@ -915,7 +915,7 @@ msgid "Namespace separator '%1%' may not be used in a chroot name"
msgstr "Navnerumsadskiller '%1%' må ikke bruges i et chrootnavn"
msgid "Naming restrictions are documented in schroot.conf(5)"
-msgstr ""
+msgstr "Navnebegrænsninger er dokumenteret i schroot.conf(5)"
#, boost-format
msgid "Namespace separator '%1%' may not be used in an alias name"
diff --git a/po/de.po b/po/de.po
index 109869a..5d9e03c 100644
--- a/po/de.po
+++ b/po/de.po
@@ -10,7 +10,7 @@ msgstr ""
"Project-Id-Version: schroot 1.4.12-1\n"
"Report-Msgid-Bugs-To: Roger Leigh <rleigh@debian.org>\n"
"POT-Creation-Date: 2010-12-07 14:00+0000\n"
-"PO-Revision-Date: 2010-09-12 21:20+0200\n"
+"PO-Revision-Date: 2010-12-07 20:15+0100\n"
"Last-Translator: Holger Wansing <linux@wansing-online.de>\n"
"Language-Team: German <debian-l10n-german@lists.debian.org>\n"
"Language: de\n"
@@ -917,6 +917,8 @@ msgstr ""
msgid "Naming restrictions are documented in schroot.conf(5)"
msgstr ""
+"Einschränkungen bezüglich der Namensgebung sind in schroot.conf(5) "
+"dokumentiert"
#, boost-format
msgid "Namespace separator '%1%' may not be used in an alias name"
diff --git a/po/fr.po b/po/fr.po
index b40cc58..dc7b5ab 100644
--- a/po/fr.po
+++ b/po/fr.po
@@ -9,14 +9,15 @@ msgstr ""
"Project-Id-Version: schroot\n"
"Report-Msgid-Bugs-To: Roger Leigh <rleigh@debian.org>\n"
"POT-Creation-Date: 2010-12-07 14:00+0000\n"
-"PO-Revision-Date: 2010-09-26 11:32+0100\n"
+"PO-Revision-Date: 2010-12-07 21:14+0100\n"
"Last-Translator: Thomas Blein <tblein@tblein.eu>\n"
"Language-Team: French <debian-l10n-french@lists.debian.org>\n"
"Language: fr\n"
"MIME-Version: 1.0\n"
"Content-Type: text/plain; charset=UTF-8\n"
"Content-Transfer-Encoding: 8bit\n"
-"X-Generator: KBabel 1.11.4\n"
+"X-Generator: Lokalize 1.0\n"
+"Plural-Forms: nplurals=2; plural=(n > 1);\n"
#. TRANSLATORS: %1% = file
#, boost-format
@@ -904,7 +905,7 @@ msgstr ""
"de schroot."
msgid "Naming restrictions are documented in schroot.conf(5)"
-msgstr ""
+msgstr "Les restrictions de nommage sont documentées dans schroot.conf(5)"
#, boost-format
msgid "Namespace separator '%1%' may not be used in an alias name"
diff --git a/po/it.po b/po/it.po
index 82073f5..015d532 100644
--- a/po/it.po
+++ b/po/it.po
@@ -6,10 +6,10 @@
# Vincenzo Campanella <vinz65@gmail.com>, 2009, 2010.
msgid ""
msgstr ""
-"Project-Id-Version: schroot 1.4.12-1\n"
+"Project-Id-Version: schroot 1.4.16-1\n"
"Report-Msgid-Bugs-To: Roger Leigh <rleigh@debian.org>\n"
"POT-Creation-Date: 2010-12-07 14:00+0000\n"
-"PO-Revision-Date: 2010-09-11 08:46+0200\n"
+"PO-Revision-Date: 2010-12-07 20:01+0100\n"
"Last-Translator: Vincenzo Campanella <vinz65@gmail.com>\n"
"Language-Team: Italian <tp@lists.linux.it>\n"
"Language: it\n"
@@ -905,6 +905,8 @@ msgstr ""
msgid "Naming restrictions are documented in schroot.conf(5)"
msgstr ""
+"Le restrizioni relative alle denominazioni sono documentate in schroot.conf"
+"(5)"
#, boost-format
msgid "Namespace separator '%1%' may not be used in an alias name"
diff --git a/po/zh_CN.po b/po/zh_CN.po
index 5ad926e..e866f74 100644
--- a/po/zh_CN.po
+++ b/po/zh_CN.po
@@ -8,7 +8,7 @@ msgstr ""
"Project-Id-Version: schroot VERSION\n"
"Report-Msgid-Bugs-To: Roger Leigh <rleigh@debian.org>\n"
"POT-Creation-Date: 2010-12-07 14:00+0000\n"
-"PO-Revision-Date: 2010-09-08 00:34+0800\n"
+"PO-Revision-Date: 2010-12-09 15:47+0800\n"
"Last-Translator: Ji ZhengYu <zhengyuji@gmail.com>\n"
"Language-Team: Chinese (simplified) <i18n-zh@googlegroups.com>\n"
"Language: zh_CN\n"
@@ -875,7 +875,7 @@ msgid "Namespace separator '%1%' may not be used in a chroot name"
msgstr "名字空间分隔符\"%1%\"未在 chroot 环境中使用"
msgid "Naming restrictions are documented in schroot.conf(5)"
-msgstr ""
+msgstr "schroot.conf(5) 中有命名规则介绍"
#, boost-format
msgid "Namespace separator '%1%' may not be used in an alias name"
Attachment:
signature.asc
Description: Digital signature