Your message dated Fri, 23 Dec 2011 10:17:59 +0000
with message-id <[email protected]>
and subject line Bug#628077: fixed in piuparts 0.42
has caused the Debian Bug report #628077,
regarding piuparts: piuparts does not execute post_install script when testing
package by name (-a switch)
to be marked as done.
This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.
(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact [email protected]
immediately.)
--
628077: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=628077
Debian Bug Tracking System
Contact [email protected] with problems
--- Begin Message ---
Package: piuparts
Version: 0.38 ( from reportbug, but actual executing code is from SVN r889)
Severity: normal
A. Compare using -a to install package by name ...
$ sudo python piuparts.py -b TARBALL.tgz --scriptsdir=... -a openssl >
openssl.log 2>&1
$ grep "Running scripts" openssl.log
0m36.5s INFO: Running scripts post_setup
0m38.2s INFO: Running scripts pre_install
0m52.7s INFO: Running scripts pre_remove **** no post_install
script run
0m53.7s INFO: Running scripts post_remove
0m55.6s INFO: Running scripts post_purge
B. ... to installing from package file
$ sudo python piuparts.py -b TARBALL.tgz --scriptsdir=...
/var/cache/apt/archives/openssl_1.0.0d-2_i386.deb > openssl.log 2>&1
$ grep "Running scripts" openssl.log
0m40.0s INFO: Running scripts post_setup
0m42.2s INFO: Running scripts pre_install
1m0.4s INFO: Running scripts post_install *** post_install is
run here
1m1.6s INFO: Running scripts pre_remove
1m2.5s INFO: Running scripts post_remove
1m4.4s INFO: Running scripts post_purge
1m7.8s INFO: Running scripts pre_install
1m16.2s INFO: Running scripts pre_upgrade
1m16.6s INFO: Running scripts pre_install *** should this be
called again
1m20.8s INFO: Running scripts post_install
1m21.0s INFO: Running scripts pre_remove
1m22.0s INFO: Running scripts post_remove
1m24.7s INFO: Running scripts post_purge
The patch below would cause the script to run using -a, but I am not
certain it has undesirable consequences.
Furthermore, I am not certain whether calling the pre-install script(s)
both before and after pre-upgrade is correct.
diff --git a/piuparts.py b/piuparts.py
--- a/piuparts.py
+++ b/piuparts.py
@@ -963,7 +963,10 @@
self.list_installed_files (pre_info,
self.save_meta_data())
else:
self.run(["apt-get", "-y", "install"] + packages)
-
+
+ if packages:
+ if settings.scriptsdir is not None:
+ self.run_scripts("post_install")
def check_for_no_processes(self):
"""Check there are no processes running inside the chroot."""
s
-- System Information:
Debian Release: 6.0.1
APT prefers stable
APT policy: (990, 'stable')
Architecture: i386 (i686)
Kernel: Linux 2.6.32-5-686 (SMP w/4 CPU cores)
Locale: LANG=en_US.utf8, LC_CTYPE=en_US.utf8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash
Versions of packages piuparts depends on:
ii apt 0.8.10.3 Advanced front-end for dpkg
ii debootstrap 1.0.26+squeeze1 Bootstrap a basic Debian system
ii lsb-release 3.2-23.2squeeze1 Linux Standard Base version
report
ii lsof 4.81.dfsg.1-1 List open files
ii python 2.6.6-3+squeeze6 interactive high-level
object-orie
ii python-debian 0.1.18 Python modules to work with
Debian
piuparts recommends no packages.
Versions of packages piuparts suggests:
ii ghostscript 8.71~dfsg2-9 The GPL Ghostscript
PostScript/PDF
pn python-rpy <none> (no description available)
-- no debconf information
--- End Message ---
--- Begin Message ---
Source: piuparts
Source-Version: 0.42
We believe that the bug you reported is fixed in the latest version of
piuparts, which is due to be installed in the Debian FTP archive:
piuparts_0.42.dsc
to main/p/piuparts/piuparts_0.42.dsc
piuparts_0.42.tar.gz
to main/p/piuparts/piuparts_0.42.tar.gz
piuparts_0.42_all.deb
to main/p/piuparts/piuparts_0.42_all.deb
A summary of the changes between this version and the previous one is
attached.
Thank you for reporting the bug, which will now be closed. If you
have further comments please address them to [email protected],
and the maintainer will reopen the bug report if appropriate.
Debian distribution maintenance software
pp.
Holger Levsen <[email protected]> (supplier of updated piuparts package)
(This message was generated automatically at their request; if you
believe that there is a problem with it please contact the archive
administrators by mailing [email protected])
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA256
Format: 1.8
Date: Fri, 23 Dec 2011 10:51:28 +0100
Source: piuparts
Binary: piuparts
Architecture: source all
Version: 0.42
Distribution: unstable
Urgency: low
Maintainer: piuparts developers team <[email protected]>
Changed-By: Holger Levsen <[email protected]>
Description:
piuparts - .deb package installation, upgrading, and removal testing tool
Closes: 387428 522918 588313 589498 628077 638831 638832 640121 640646 640647
640648 647752 648162 648177 648193 648423 648707 648708 648784 649335 649967
Changes:
piuparts (0.42) unstable; urgency=low
.
[ Holger Levsen ]
* piuparts.py:
- add to self.ignored_files: /etc/blkid.tab (Closes: #638831)
- add to self.ignored_patterns: /var/lib/apt/lists/.*
- apply patch by Gregor Herrmann to fix --minimize. (Closes: #648423)
* Remove Debian.NEWS entry about source in git. (Closes: #640121)
* piuparts.py, piuparts-report.py, ChangeLog: Expand tabs to spaces.
* Remove whitespaces from whitespace-only lines.
* piuparts-report:
- create maintainer subdirs if they don't exist.
- detect tests being terminated due to excessive output.
* Add git to Build-Depends-Indep: as git describe is used to generate
part of the version string for development builds.
* Add debian/.gitignore
.
[ Mika Pflüger ]
* piuparts-analyze.py:
- Rewrite to use python-debianbts to analyze if bugs are filed already.
- The BTS only tracks source versions, so remove binNMU-part from
package versions when comparing with versions from the BTS.
- Reduce noise in the output by only printing one action/advise per
package.
- Fix extraction of package versions from bug reports. Thanks to
Andreas Beckmann for catching and solving the error.
* debian/control: Add python-apt and python-debianbts to piuparts depends.
.
[ Scott Schaefer ]
* debian/copyright: Make it compliant with DEP-5.
* piuparts-slave.py:
- Replace deprecated os.popen2 with subprocess.Popen. (Closes: #640646)
- Add some more logging.
- Kill children (hard-coded value, 45 minutes) to ensure test doesn't
run "forever" (Closes: #640647, #387428)
* piuparty.py:
- Kill leftover processes (via SIGTERM, then if that fails, via SIGKILL).
(Closes: #522918)
- Test for incorrect diversion handling: (Closes: #588313)
a) Existing diversions removed/modified, and/or
b) Installed diversions not removed by purge.
* piupartslib/packagesdb.py: Modify several functions in PackagesDB class
to use python 'set' type, instead of a list. This permitted replacing
inefficient combination of unique() function and random.shuffle() with
python set.pop() method. Since python prohibits storing non-hashable
object in a set, minor modifications to piuparts-report and to
piuparts-master required. (Closes: #640648)
.
[ Andreas Beckmann ]
* *.py: Add vim modeline.
* piuparts.py:
- Add unittests for misbehaving symlinks.
- Fix resolving absolute symlinks of intermediate directory components,
i.e. /var/run -> /run while checking /etc/motd -> /var/run/motd.
Solves about 30000 false positives of
'Broken symlinks: /etc/motd -> /var/run/motd'. (Closes: #648784)
- When running commands in the chroot, redirect stdin from /dev/null.
- Stop using Popen.communicate() as it may run out of memory.
- Terminate commands producing more than 2 MB of output. (Closes: #649335)
- Create /etc/dpkg/dpkg.cfg.d/ if missing inside the chroot (e.g. on
lenny). (Closes: #647752)
- Remove logrotate and its dependencies only once.
- Only run 'apt-get update' after updating the sources.list.
- Only run 'apt-get clean' before creating tarballs or saving meta data.
- Do the same checks for running processes and broken symlinks in all
tests. (Closes: #648708)
- Create piupart's apt config in the chroot as /etc/apt.conf.d/piuparts
instead of /etc/apt.conf in order to allow overriding the settings from
custom scripts by just dropping new config bits in e.g.
/etc/apt/apt.conf.d/piuparts-foobar.
- Integrate diversion checks with Chroot.check_result().
- Use 'apt-get remove' to remove all packages at once with proper
dependency ordering. (Closes: #648162)
- Purge all packages at once instead of doing it one-by-one.
- Make restore_selections() reinstall missing packages. (Closes: #648707)
- Set more environment variables to let custom scripts know where and
when they are being run: PIUPARTS_TEST, PIUPARTS_PHASE,
PIUPARTS_DISTRIBUTION{,_PREV,_NEXT}. See the README for details.
(Closes: #589498)
- Add missing post_install_* custom scripts to install_packages_by_name().
(Closes: #628077)
- Remove pre_upgrade_* custom scripts, they can be replaced with
pre_install_* scripts that check for PIUPARTS_PHASE=upgrade.
- Add pre_test_* custom scripts. These are run exactly once at the
beginning of each test (after recording the initial chroot state).
- Allow multiple --scriptsdir options, the contents will be merged.
- Exclude /tmp/scripts when creating a tarball.
- Use --one-file-system when creating a tarball to exclude bindmounts etc.
- Restore base system from temp_tgz instead of running debootstrap again.
(Closes: #648193)
- Do not fail upgrade/distupgrade tests for a set of packages where not
all packages exist in the start distribution. This happens e.g. when
testing .changes files and packages were split/added. Only install the
old packages that actually exist according to 'apt-cache show'.
- Add --extra-old-packages option to intall additional old packages that
are not in the set of new packages to be tested. Useful for testing
upgrades with Conflicts/Replaces relationships, e.g. in package renames
or merges.
- Use consistent variable names for package lists. (Closes: #648177)
- Compute the changes in restore_selections().
- Check for settings.scriptsdir inside run_scripts().
- Consistently use chroot.relative() to build filenames inside the chroot.
* piupartslib/packagesdb.py:
- Handle 'unknown-preferred-alternative' state correctly.
- Add 'does-not-exist' state for dependency resolution to distinguish this
from 'unknown' state so that the latter only indicates 'unresolvable' or
'not yet resolved'.
- Handle virtual packages separately from real packages.
- Use get_package_state() internally which 'resolves' (purely) virtual
packages by default (can be disabled).
* piuparts-master.py:
- Add a 'status' command that reports package counts for all states.
* piuparts-slave.py:
- Fix triggering tarball recreation.
- Check tarball age regularily.
- Log tarball creation in *.tgz.log.
- Request and print package counts from master.
- Reload section config every time a section is being run.
- Add precedence attribute to allow prioritizing different sections and to
suspend processing of low priority ones while there are packages waiting
in more important sections.
* piuparts-report.py:
- state-*.html: Sort package lists by name, display state of all
alternative dependencies and packages providing a virtual dependency.
- source/?/*.html: Sort binary packages by name.
- maintainer/?/*.html: Sort source packages by name.
- Update list of error states to be highlighted.
- Archive logs of packages that were removed from the distribution.
- Speedup generating maintainer summaries.
* Makefile: Use 'git describe' to get an exact stamp for development
versions.
.
[ Dave Steele ]
* piuparts-slave.py: make Section.run() report the number of packages
processed and use this to decide whether a slave should sleep.
(Closes: #649967)
.
[ Stefano Rivera ]
* piuparts.py:
- Properly install and remove logrotate. (Closes: #638832)
- Use eatmydata by default, add option --no-eatmydata. (This was discussed
in #633033.)
Checksums-Sha1:
d3cd7494c9a52439a5cc90b0e3f017088f79adf2 1672 piuparts_0.42.dsc
7a9654f7a333f1110b84e95c799f936c13e69e56 88394 piuparts_0.42.tar.gz
e7d56a58c228fe95214e89c906d7413c47270854 100664 piuparts_0.42_all.deb
Checksums-Sha256:
617dacf440620f22ac782979d238e11189d34cff94240f658afa436e619ea74e 1672
piuparts_0.42.dsc
0d20f6a6caa071870cdfb2f0359c74ad5a27be6a201349afb87db5b06dcc9334 88394
piuparts_0.42.tar.gz
d00b7b6308bf4751ab041a482ee738726e0a80ae19c2be3a4403c5855f4833e2 100664
piuparts_0.42_all.deb
Files:
0c6e07f7d48236dd978888d28b53a166 1672 devel extra piuparts_0.42.dsc
6e8fdf936068b9beba580f0b9f548052 88394 devel extra piuparts_0.42.tar.gz
dee2702ded5d2e6cc4052d8e718d3ed0 100664 devel extra piuparts_0.42_all.deb
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.10 (GNU/Linux)
iQIVAwUBTvRTmwkauFYGmqocAQiI5w//dyIkENEQT+lTeSG8vCtaxhjQYdv1/XZv
IMd+1mcEfNhwVUYZ8oc6mByK5GzUmCDIHd/9sBHoEdYQqwGJFNj16aLV7TpGKf2M
y+Jscltw7W2yo2ydKgJG2PVbiMUXo7tHASVRLRTi9jPxyt+e3l7q1pVYqQE95FNk
+tizo6aXvgMftouj/6Zf6M+ZDGEOJYKu48HXgVW4dLu11dwwwgUAI1oM9URMHw66
jiEwPj+M5kZFZBZWCVA/AjALd2AlvdhuYn5HjgxxMubQyimmd2yr122v4wNVcK3y
owZHltHNQqTwA/fh5rP9skgeuIJdu3mxBleY0eYloVBqRoLH7rZF1hVz+VdIXSZ0
A/389BI22seUHyWMIPNXjfkdrLdGq03Eqd8huXaCwRgcrtHPMCJRmxGzMyPwuyzV
mqpLlxzgnFnAmELa8nE1+GUVDhiSATI9dNj3BW4pW1scN0TQ0uhS4BofKeEuTA1u
0Bot8FPLq8OEWpMdppaSAX2rcfdPGLFErgTAhzm4HEaTNvY4z07tJMgPh37JEKcT
LgIETxH042kyyja7k6eVsTh57nORHd5awzXZweC0mIUq76vCgsQzuCONejpwaRow
ZfDHHq5OzRQsvI2XrQK+3ZV5EVNjbJXneVnwIJl3mF+OwrIBxvVRBdcmmyxEzvYl
G9kd9vPGTyo=
=ieNq
-----END PGP SIGNATURE-----
--- End Message ---
