Skip to content

bugfix: chroot_exit_teardown#95

Open
Nschembr wants to merge 1 commit into
mvallim:masterfrom
Nschembr:bugfix/chroot_exit_teardown
Open

bugfix: chroot_exit_teardown#95
Nschembr wants to merge 1 commit into
mvallim:masterfrom
Nschembr:bugfix/chroot_exit_teardown

Conversation

@Nschembr
Copy link
Copy Markdown

Description: Ubuntu 24.04.4 LTS
Release: 24.04
Codename: noble

I'm running live-custom-ubuntu-from-scratch on a desktop host build for testing.  I'm in the process of building iso's for automation. I found a minor issue while testing build.sh,  chroot_exit_teardown.  It looks like proc is needed for the mount command to unmount /sys and /dev/pts. The build script fails to clean up the chroot.   I have added export -f to allow the functions to be reused in other scripts. 

# test
# ./scripts/chroot is in place 
# nested bash needed to keep active shell from closing  on error
bash
bash
# source the 3 chroot functions (paste in a shell)
chroot_enter_setup
chroot_enter_debug
chroot_exit_teardown

The 3 functions can be executed repeatedly without error, and the chroot_enter_debug will have access to networking each time.

Description:	Ubuntu 24.04.4 LTS
Release:	24.04
Codename:	noble

I'm running live-custom-ubuntu-from-scratch on a desktop host build for testing.  I'm in the process of building iso's for automation. I found a minor issue while testing build.sh,  chroot_exit_teardown.  It looks like proc is needed for the mount command to unmount /sys and /dev/pts. The build script fails to clean up the chroot.  
I have added export -f to allow the functions to be reused in other scripts. 

\# test
\# ./scripts/chroot is in place 
\# nested bash needed to keep active shell from closing 
bash
bash
\# source the 3 chroot functions (paste in a shell)
chroot_enter_setup
chroot_enter_debug
chroot_exit_teardown

The 3 functions can be executed repeatedly without error, and the chroot_enter_debug will have access to networking each time.
@Nschembr Nschembr marked this pull request as draft May 28, 2026 15:58
@Nschembr Nschembr marked this pull request as ready for review May 28, 2026 16:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant