Bug#886328: live-boot: Please use /run/live instead of /lib/live/mount
On Thu, 15 Mar 2018, Benjamin Drung wrote: > I have finally tested git master and found one missing rename of > /live/overlay (probably caused by applying my patches in a different > order which causes merge conflicts). See > https://salsa.debian.org/live-team/live-boot/merge_requests/6 > for a fix. Merged, thanks. Cheers, -- Raphaël Hertzog ◈ Debian Developer Support Debian LTS: https://www.freexian.com/services/debian-lts.html Learn to master Debian: https://debian-handbook.info/get/
Processed: Bug #884355 in live-boot marked as pending
Processing control commands: > tag -1 pending Bug #884355 [src:live-boot] live-boot: Please support live-{top,premount,bottom} hooks Added tag(s) pending. -- 884355: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=884355 Debian Bug Tracking System Contact ow...@bugs.debian.org with problems
Bug#886328: live-boot: Please use /run/live instead of /lib/live/mount
Am Freitag, den 23.02.2018, 19:24 +0100 schrieb Raphael Hertzog: > Hello, > > On Fri, 05 Jan 2018, intrigeri wrote: > > Benjamin Drung: > > > Therefore move /lib/live/mount to /run/live and skip the > > > intermedia > > > /live mount points. This reduces code and complexity. > > > > As someone who had to repeatedly bang his head against exactly this > > part of the live-boot code (last time earlier this week), I can > > only > > agree with the proposed simplification idea. I didn't do a full > > code > > review though. > > I'm not familiar enough with this part either and I am unlikely to > find > any obvious mistake. But I committed the patch anyway > > It would be nice if we could test the live-boot in git before I > upload > it. I have finally tested git master and found one missing rename of /live/overlay (probably caused by applying my patches in a different order which causes merge conflicts). See https://salsa.debian.org/live-team/live-boot/merge_requests/6 for a fix. > Benjamin, did you test your changes with persistence enabled? No, I haven't. Thanks for everyone else who does. Am Dienstag, den 13.03.2018, 13:05 + schrieb Luca Boccassi: > On Tue, 2018-03-13 at 10:49 +0100, Raphael Hertzog wrote: > > On Mon, 12 Mar 2018, Luca Boccassi wrote: > > > Or maybe have a backward-compatible symlinks? > > > > This seems entirely reasonable. Can you work on this? > > Yes no problem, I'll give it a shot and send a PR for review before > the > end of the week. Thanks for doing it. I will be happy to review it. -- Benjamin Drung System Developer Debian & Ubuntu Developer ProfitBricks GmbH Greifswalder Str. 207 D - 10405 Berlin Email: benjamin.dr...@profitbricks.com URL: https://www.profitbricks.de Sitz der Gesellschaft: Berlin Registergericht: Amtsgericht Charlottenburg, HRB 125506 B Geschäftsführer: Achim Weiss, Matthias Steinberg
Bug#884355: live-boot: Please support live-{top,premount,bottom} hooks
Am Freitag, den 23.02.2018, 19:20 +0100 schrieb Raphael Hertzog: > Hi, > > On Thu, 14 Dec 2017, Benjamin Drung wrote: > > initramfs-tools support hooks for top, premount, bottom when > > booting > > with boot=local or boot=nfs. Please add similar support to live- > > boot > > when booting with boot=live. A patch (for git) is attached. > > I will gladly apply your patch but I think it's time to clean up the > mess with this: > > > --- a/components/9990-initramfs-tools.sh > > +++ b/components/9990-initramfs-tools.sh > > @@ -73,3 +73,25 @@ panic() > > . /scripts/functions > > panic "$@" > > } > > + > > +# Note: Other components source /scripts/functions before sourcing > > this file. > > +# /scripts/functions overrides the mount_* functions (to no-ops). > > Thus fix > > +# the mount_* options here again. > > This kind of duplication is just ugly. I looked around and saw this: > $ grep -r -E 'functions|9990-initramfs-tools.sh' components/ > components/9990-initramfs-tools.sh:# Override maybe_break from > scripts/functions > components/9990-initramfs-tools.sh: . > /scripts/functions > components/9990-initramfs-tools.sh:# Override panic from > scripts/functions > components/9990-initramfs-tools.sh: . /scripts/functions > components/9990-fstab.sh: # FIXME: stop hardcoding overloading > of initramfs-tools functions > components/9990-fstab.sh: . /scripts/functions > components/9990-fstab.sh: . /lib/live/boot/9990-initramfs- > tools.sh > components/9990-misc-helpers.sh: # the output of setup_loop is > evaluated in other functions, > components/9990-netbase.sh: # FIXME: stop hardcoding > overloading of initramfs-tools functions > components/9990-netbase.sh: . /scripts/functions > components/9990-netbase.sh: . /lib/live/boot/9990-initramfs- > tools.sh > > I have the feeling that all those problematic include are no longer > necessary > nowadays because we first source all the files and then we execute > functions > in the expected order. My review has not been thorough but can you > try to remove > all those include and see if things still work? > > /script/functions is already sourced in backend/initramfs- > tools/live.script > so it should not be required to have it in the components, right? I have removed the includes (except the ones that are needed to access the panic function) and the plugin system still works. The tested pull request: https://salsa.debian.org/live-team/live-boot/merge_requests/6 I have just tested my use case. So I haven't verified that removing the problematic includes doesn't break anything. > BTW, I merged all you other patches. For the future, please send > patches > as merge requests on the salsa repositories. Yes. Thanks for merging. -- Benjamin Drung System Developer Debian & Ubuntu Developer ProfitBricks GmbH Greifswalder Str. 207 D - 10405 Berlin Email: benjamin.dr...@profitbricks.com URL: https://www.profitbricks.de Sitz der Gesellschaft: Berlin Registergericht: Amtsgericht Charlottenburg, HRB 125506 B Geschäftsführer: Achim Weiss, Matthias Steinberg