Merge ~mwhudson/ubiquity:lp-1841672 into ubiquity:master

Proposed by Michael Hudson-Doyle
Status: Rejected
Rejected by: Michael Hudson-Doyle
Proposed branch: ~mwhudson/ubiquity:lp-1841672
Merge into: ubiquity:master
Diff against target: 29 lines (+12/-0)
2 files modified
compat/apt-install (+5/-0)
debian/changelog (+7/-0)
Reviewer Review Type Date Requested Status
Ubuntu Installer Team Pending
Review via email: mp+373794@code.launchpad.net

Description of the change

Or we could just revert https://git.launchpad.net/ubiquity/commit/?id=c8a1cb57f77d8acdea350a549d1929c4b423a15d.

Not tested, and don't have time to test today. I have uploaded it to ppa:mwhudson/devirt though.

To post a comment you must log in.
Revision history for this message
Michael Hudson-Doyle (mwhudson) wrote :

This doesn't seem to work, although I don't really understand why not :( Is /usr/lib/partman/finish.d/70crypto_aptinstall not executed for some reason? It gets mentioned in /var/log/partman though...

~mwhudson/ubiquity:lp-1841672 updated
54a6d01... by Michael Hudson-Doyle

suppress output from apt-mark

Revision history for this message
Michael Hudson-Doyle (mwhudson) wrote :

Ah this doesn't work because 70crypto_aptinstall is called before the target filesystem is populated... but apt-install does write the file to /var/lib/ubiquity/apt-installed and the logic to install those packages can be fixed in the same way. Will have another MP momentarily.

Revision history for this message
Michael Hudson-Doyle (mwhudson) wrote :

Unmerged commits

54a6d01... by Michael Hudson-Doyle

suppress output from apt-mark

1cb42ed... by Michael Hudson-Doyle

compat/apt-install: Mark any packages already installed as manually installed. (LP: #1841672)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/compat/apt-install b/compat/apt-install
2index 81340ba..e6dbfe9 100755
3--- a/compat/apt-install
4+++ b/compat/apt-install
5@@ -36,4 +36,9 @@ if [ -f /var/lib/ubiquity/apt-install-direct ]; then
6 setup_env
7 chroot /target apt-get $apt_get_args install $toinstall </dev/null
8 fi
9+ # Mark any packages that are already installed as manually installed.
10+ tomark="$(echo "$@" | xargs -n1 | sort | comm -12 - "$installed")"
11+ if [ "$tomark" ]; then
12+ chroot /target apt-mark manual $tomark </dev/null >/dev/null
13+ fi
14 fi
15diff --git a/debian/changelog b/debian/changelog
16index 3d67582..7ad4eac 100644
17--- a/debian/changelog
18+++ b/debian/changelog
19@@ -1,3 +1,10 @@
20+ubiquity (19.10.16) UNRELEASED; urgency=medium
21+
22+ * compat/apt-install: Mark any packages already installed as manually
23+ installed. (LP: #1841672)
24+
25+ -- Michael Hudson-Doyle <michael.hudson@ubuntu.com> Tue, 08 Oct 2019 16:04:00 +1300
26+
27 ubiquity (19.10.15) eoan; urgency=medium
28
29 [ Jean-Baptiste Lallement ]

Subscribers

People subscribed via source and target branches