releasing package livecd-rootfs version 2.542.2

minimize-manual would be creating those files again, and other
parts may be creating them, so best to do the cleanup at the
very end.

LP: #1826377

debian/changelog

@@ -1,8 +1,16 @@
-livecd-rootfs (2.542.1) UNRELEASED; urgency=medium
+livecd-rootfs (2.542.2) cosmic; urgency=medium
 
+  * Remove crufty files after minimize-manual (LP: #1826377)
+
+ -- Julian Andres Klode <juliank@ubuntu.com>  Fri, 26 Apr 2019 10:36:33 +0200
+
+livecd-rootfs (2.542.1) cosmic; urgency=medium
+
+  [ Cody Shepherd ]
   * Ensure pre-seeded snaps are now published in the image manifests.
+  * Include grub efi packages in manifests for uefi images.
 
- -- Cody Shepherd <cody.shepherd@canonical.com>  Mon, 22 Oct 2018 10:16:19 -0700
+ -- Steve Langasek <steve.langasek@ubuntu.com>  Thu, 20 Dec 2018 13:12:42 -0600
 
 livecd-rootfs (2.542) cosmic; urgency=medium
 

debian/control

@@ -4,7 +4,7 @@ Priority: optional
 Build-Depends: debhelper (>= 7)
 Maintainer: Ubuntu Developers <ubuntu-devel-discuss@lists.ubuntu.com>
 Standards-Version: 3.9.6
-Vcs-Bzr: http://bazaar.launchpad.net/~ubuntu-core-dev/livecd-rootfs/trunk
+Vcs-Git: https://git.launchpad.net/livecd-rootfs -b ubuntu/cosmic
 
 Package: livecd-rootfs
 Architecture: any

live-build/auto/build

@@ -259,9 +259,6 @@ EOF
 			apt-get -y --purge autoremove"
 	fi
 
-	# remove crufty files that shouldn't be left in an image
-	rm -f chroot/var/cache/debconf/*-old chroot/var/lib/dpkg/*-old
-	Chroot chroot apt clean
 	if [ "${PROJECT}:${SUBPROJECT:-}" = "ubuntu-base:minimized" ]; then
 		# Save even more size by removing apt lists (that are currently removed
 		# downstream anyway)
@@ -452,6 +449,10 @@ EOF
 
 	/usr/share/livecd-rootfs/minimize-manual chroot
 
+	# remove crufty files that shouldn't be left in an image
+	rm -f chroot/var/cache/debconf/*-old chroot/var/lib/dpkg/*-old
+	Chroot chroot apt clean
+
 	lb binary "$@"
 	touch binary.success
 ) 2>&1 | tee binary.log
@@ -551,7 +552,7 @@ fi
 
 # '--initramfs none' produces different manifest names.
 if [ -e "binary/$INITFS/filesystem.packages" ]; then
-	./config/snap-seed-parse "chroot/" >> "binary/${INITFS}/filesystem.packages"
+	./config/snap-seed-parse "chroot/" "binary/${INITFS}/filesystem.packages"
 	ln "binary/$INITFS/filesystem.packages" "$PREFIX.manifest"
 	chmod 644 "$PREFIX.manifest"
 fi

live-build/functions

@@ -49,7 +49,7 @@ create_manifest() {
     echo "create_manifest chroot_root: ${chroot_root}"
     dpkg-query --show --admindir="${chroot_root}/var/lib/dpkg" > ${target_file}
     echo "create_manifest call to dpkg-query finished."
-    ./config/snap-seed-parse "${chroot_root}" >> ${target_file}
+    ./config/snap-seed-parse "${chroot_root}" "${target_file}"
     echo "create_manifest call to snap_seed_parse finished."
     echo "create_manifest finished"
 }

live-build/snap-seed-parse.py

@@ -1,26 +1,35 @@
 #!/usr/bin/python3
 
 """
-Usage: snap-seed-parse ${chroot_dir} > somefile.manifest
+Usage: snap-seed-parse [${chroot_dir}] <output file>
 
 This script looks for a seed.yaml path in the given root directory, parsing
-it and printing generated manifest lines to stdout for easy redirection.
+it and appending the parsed lines to the given output file.
+
+The $chroot_dir argument is optional and will default to the empty string.
 """
 
-import re
+import argparse
-import sys
-import yaml
 import os.path
+import re
+import yaml
 
 
 def log(msg):
-    sys.stderr.write("snap-seed-parse: {}\n".format(msg))
+    print("snap-seed-parse: {}".format(msg))
 
 
 log("Parsing seed.yaml")
 
-CHROOT_ROOT = sys.argv[1] if len(sys.argv) > 1 and len(sys.argv[1]) > 0 \
+parser = argparse.ArgumentParser()
-                          else ''
+parser.add_argument('chroot', nargs='?', default='',
+                    help='root dir for the chroot from which to generate the '
+                         'manifest')
+parser.add_argument('file', help='Output manifest to this file')
+
+ARGS = parser.parse_args()
+CHROOT_ROOT = ARGS.chroot
+FNAME = ARGS.file
 
 # Trim any trailing slashes for correct appending
 log("CHROOT_ROOT: {}".format(CHROOT_ROOT))
@@ -43,14 +52,17 @@ else:
 with open(YAML_PATH, 'r') as fh:
     yaml_lines = yaml.safe_load(fh)['snaps']
 
-# Loop over dict items, outputting one manifest line from each triplet
+log('Writing manifest to {}'.format(FNAME))
-for item in yaml_lines:
+
-    filestring = item['file']
+with open(FNAME, 'a+') as fh:
-    # Pull the revision number off the file name
+    for item in yaml_lines:
-    revision = filestring[filestring.rindex('_')+1:]
+        filestring = item['file']
-    revision = re.sub(r'[^0-9]', '', revision)
+        # Pull the revision number off the file name
-    print("{}{}\t{}\t{}".format(LINE_PREFIX,
+        revision = filestring[filestring.rindex('_')+1:]
-                                        item['name'],
+        revision = re.sub(r'[^0-9]', '', revision)
-                                        item['channel'],
+        fh.write("{}{}\t{}\t{}\n".format(LINE_PREFIX,
-                                        revision,
+                                         item['name'],
-                                        ))
+                                         item['channel'],
+                                         revision,
+                                         ))
+log('Manifest output finished.')