Bug #12684

Update config/build-manifest-extra-packages.yml

Added by intrigeri 2017-06-12 12:18:16 . Updated 2017-09-28 18:49:50 .

Status:
Resolved
Priority:
Normal
Assignee:
Category:
Build system
Target version:
Start date:
2017-06-12
Due date:
% Done:

100%

Feature Branch:
bugfix/12684-update-build-manifest-extra-packages
Type of work:
Code
Blueprint:

Starter:
Affected tool:
Deliverable for:

Description

E.g. it references a version of squashfs-tools (1:4.2+20130409-2) that we probably don’t use anymore.


Subtasks


Related issues

Blocks Tails - Feature #13234: Core work 2017Q3: Foundations Team Resolved 2017-06-29

History

#1 Updated by intrigeri 2017-06-13 07:20:12

  • Status changed from Confirmed to In Progress
  • Target version changed from Tails_3.1 to Tails_3.2
  • % Done changed from 0 to 10
  • Feature Branch set to bugfix/12684-update-build-manifest-extra-packages

Indeed, at first glance it seems we have a problem here:

reprepro-tagged-snapshots@apt:~/repositories/3.0$ find . -name squashfs*
./debian/pool/main/s/squashfs-tools
./debian/pool/main/s/squashfs-tools/squashfs-tools_4.2+20130409-2.dsc
./debian/pool/main/s/squashfs-tools/squashfs-tools_4.2+20130409-2_i386.deb
./debian/pool/main/s/squashfs-tools/squashfs-tools_4.2+20130409.orig.tar.xz
./debian/pool/main/s/squashfs-tools/squashfs-tools_4.2+20130409-2.debian.tar.xz
./debian/pool/main/s/squashfs-tools/squashfs-tools_4.2+20130409-2_amd64.deb

OTOH we don’t include packages coming from our custom APT repo in tagged snapshots, so as long as we use a custom squashfs-tools package, we’re good. And even once we switch to a squashfs-tools from Debian, we’re good as well since the version we really use is actually identified by our apt-get wrapper, and included in the build manifest:

  - arch: amd64
    package: squashfs-tools
    version: 1:4.2+20130409-2
  - arch: amd64
    package: squashfs-tools
    version: 1:4.3-3.0tails4

I think this demonstrates that the “pulled by lb_binary_rootfs, outside of the reach of our apt-get wrapper” comment and the rationale for commit:0dcf467e25dda7d2ee33142806e45ca236b35927 are actually wrong: we wrap apt-get before live-build creates the chroot, so it’ll save whatever packages are installed in there, including those installed outside of the chroot copy (chroot/chroot), e.g. by lb_binary_rootfs and lb_binary_iso. I admit I’m afraid I might be missing something though: I’m pretty sure these changes did fix a real problem when I applied them a year ago, but OTOH I don’t think reverting them can break anything on the short term, as we’ll likely keep using a custom squashfs-tools for a while, so I’ll go ahead.

#2 Updated by intrigeri 2017-06-13 07:21:13

Let’s not take any bet for 3.1 though: I’d rather not see an inexperienced RM hit any problem caused by my changes :)

#4 Updated by intrigeri 2017-06-29 10:33:15

#5 Updated by anonym 2017-08-15 14:16:06

  • Status changed from In Progress to Fix committed
  • Assignee deleted (anonym)
  • % Done changed from 50 to 100
  • QA Check changed from Ready for QA to Pass

#6 Updated by anonym 2017-09-28 18:49:50

  • Status changed from Fix committed to Resolved