alpinelinux / alpine-chroot-install

Install Alpine Linux in chroot with a breeze. Build ARM on Travis CI or any other x86_64 CI.
MIT License
290 stars 59 forks source link

After running script - all the local /proc /dev/ /sys /run filesystems remain mounted #7

Closed julian1 closed 3 years ago

julian1 commented 6 years ago

shouldn't the cgroup mounts be unmounted on exit?

jirutka commented 6 years ago

The enter-chroot script currently doesn’t unmount anything, i.e. it’s not a bug, but missing feature. I’ll add it.

hoshsadiq commented 5 years ago

@jirutka I see that you have done the work in the dev branch, could I ask if there's a reason it's not merged into the master branch?

anjandev commented 4 years ago

It might be worthwhile to put warning about this in the readme on how you have to manually umount everything. I ran this script today on my machine and it worked fine. However, after I was done, I wanted to remove /alpine. Since I didnt know you had to umount, I ended up nuking my system by running sudo rm -rf /alpine....

This is mostly my incompetence but a warning would be good to put in the readme.

If this seems like a worthwhile comment, I will send a pull request.

hoshsadiq commented 4 years ago

@anjandev I've raised an PR to add a destroy script that can automatically delete the chroot as well.

anjandev commented 4 years ago

@hoshsadiq thank you. I think it might be worthwhile to document (in the README perhaps?) deleting using the destroy script you made rather than merely rm -rf /alpine.

hoshsadiq commented 4 years ago

It's already mentioned at the end when the script is finished, same way the enter-chroot script is mentioned.

anjandev commented 4 years ago

@hoshsadiq okay. Thanks!