create-targz script fixes #1
Conversation
There was a problem hiding this comment.
This script is pretty old. Check against https://github.com/WhitewaterFoundry/pengwin-rootfs-builds/blob/master/create-targz.sh if anything important is left. Recently we migrated pengwin to Debian 12. I agree to have another script apart from the main build system to customize for user builds.
| @@ -31,10 +35,10 @@ sudo chroot $DIST /bin/bash -c "echo 'Defaults lecture_file = /etc/sudoers.lectu | |||
| sudo chroot $DIST /bin/bash -c "echo 'Enter your UNIX password below. This is not your Windows password.' > /etc/sudoers.lecture" | |||
|
|
|||
| # remove unnecessary packages in initial image | |||
| sudo chroot $DIST apt-get -y -q remove systemd dmidecode --allow-remove-essential | |||
| #sudo chroot $DIST apt-get -y -q remove systemd dmidecode --allow-remove-essential | |||
There was a problem hiding this comment.
| #sudo chroot $DIST apt-get -y -q remove systemd dmidecode --allow-remove-essential |
| @@ -31,10 +35,10 @@ sudo chroot $DIST /bin/bash -c "echo 'Defaults lecture_file = /etc/sudoers.lectu | |||
| sudo chroot $DIST /bin/bash -c "echo 'Enter your UNIX password below. This is not your Windows password.' > /etc/sudoers.lecture" | |||
|
|
|||
| # remove unnecessary packages in initial image | |||
There was a problem hiding this comment.
| # remove unnecessary packages in initial image |
|
|
||
| # clean up any orphaned apt dependencies | ||
| sudo chroot $DIST apt-get -y -q autoremove | ||
| #sudo chroot $DIST apt-get -y -q autoremove |
There was a problem hiding this comment.
This helps to have a smaller image, why is being commented?
There was a problem hiding this comment.
It was trying to remove required packages/dependencies that was causing apt to throw errors. There's probably a better resolution than commenting out autoremove, but I'm going to work on getting the build process to work with the non-legacy rootfs repo you mentioned before working on this PR further. If I can't get it to work, I'll fall back on this one (which uses cdebootstrap instead of debootstrap)
| @@ -6,6 +6,7 @@ BUILDIR=$(pwd) | |||
| TMPDIR=$(mktemp -d) | |||
| ARCH="" | |||
| DIST="testing" | |||
| mkdir -p $TMPDIR/$DIST | |||
| cd $TMPDIR | |||
|
|
|||
| function build { | |||
There was a problem hiding this comment.
Make bash / sh compliant
| function build { | |
| function build() { |
Related to WhitewaterFoundry/Pengwin#771