Merge lp:~rcj/livecd-rootfs/zesty-proposed into lp:~ubuntu-core-dev/livecd-rootfs/zesty-proposed

Proposed by Robert C Jennings
Status: Merged
Merged at revision: 1478
Proposed branch: lp:~rcj/livecd-rootfs/zesty-proposed
Merge into: lp:~ubuntu-core-dev/livecd-rootfs/zesty-proposed
Diff against target: 77 lines (+27/-13)
3 files modified
debian/changelog (+12/-0)
live-build/functions (+14/-8)
live-build/ubuntu-cpc/hooks/032-root-squashfs.binary (+1/-5)
To merge this branch: bzr merge lp:~rcj/livecd-rootfs/zesty-proposed
Reviewer Review Type Date Requested Status
Ubuntu Core Development Team Pending
Review via email: mp+330699@code.launchpad.net
To post a comment you must log in.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'debian/changelog'
2--- debian/changelog 2017-09-11 19:00:02 +0000
3+++ debian/changelog 2017-09-13 18:40:21 +0000
4@@ -1,3 +1,15 @@
5+livecd-rootfs (2.441.6) UNRELEASED; urgency=medium
6+
7+ [ Robert C Jennings ]
8+ * live-build/ubuntu-cpc/functions: Add a function, teardown_mountpoint,
9+ to reverse the work done in setup_mountpoint. Lack of this function
10+ has forced users of setup_mountpoint to implement this separately
11+ and the implementations have diverged. (LP: #1716992)
12+ * live-build/ubuntu-cpc/functions: Remove umount_settle function.
13+ The was only used where teardown_mountpoint was lacking.
14+
15+ -- Robert C Jennings <robert.jennings@canonical.com> Wed, 13 Sep 2017 13:36:40 -0500
16+
17 livecd-rootfs (2.441.5) zesty; urgency=medium
18
19 [ Colin Watson ]
20
21=== modified file 'live-build/functions'
22--- live-build/functions 2017-09-11 18:59:56 +0000
23+++ live-build/functions 2017-09-13 18:40:21 +0000
24@@ -91,6 +91,19 @@
25
26 }
27
28+teardown_mountpoint() {
29+ # Reverse the operations from setup_mountpoint
30+ local mountpoint="$1"
31+
32+ umount "$mountpoint/tmp"
33+ umount "$mountpoint/sys"
34+ umount "$mountpoint/proc"
35+ umount -R "$mountpoint/dev"
36+ udevadm settle
37+ sleep 3
38+ mv resolv.conf.tmp "$mountpoint/etc/resolv.conf"
39+}
40+
41 mount_partition() {
42 partition="$1"
43 mountpoint="$2"
44@@ -134,16 +147,9 @@
45
46 }
47
48-umount_settle() {
49- # Unmount device, and let it settle
50- umount $1
51- udevadm settle
52-}
53-
54 umount_partition() {
55 local mountpoint=${1}
56- mv resolv.conf.tmp "$mountpoint/etc/resolv.conf"
57- umount -R $mountpoint
58+ teardown_mountpoint $mountpoint
59 udevadm settle
60
61 if [ -n "${rootfs_dev_mapper}" -a -b "${rootfs_dev_mapper}" ]; then
62
63=== modified file 'live-build/ubuntu-cpc/hooks/032-root-squashfs.binary'
64--- live-build/ubuntu-cpc/hooks/032-root-squashfs.binary 2017-09-11 18:51:09 +0000
65+++ live-build/ubuntu-cpc/hooks/032-root-squashfs.binary 2017-09-13 18:40:21 +0000
66@@ -36,11 +36,7 @@
67 chroot binary/boot/squashfs.dir rm /usr/sbin/grub-probe
68 chroot binary/boot/squashfs.dir dpkg-divert --remove --local --rename /usr/sbin/grub-probe
69
70-mv resolv.conf.tmp "binary/boot/squashfs.dir/etc/resolv.conf"
71-umount "binary/boot/squashfs.dir/proc"
72-umount "binary/boot/squashfs.dir/sys"
73-umount -R "binary/boot/squashfs.dir/dev"
74-umount "binary/boot/squashfs.dir/tmp"
75+teardown_mountpoint binary/boot/squashfs.dir
76
77 apt-get -qqy install squashfs-tools
78

Subscribers

People subscribed via source and target branches