Re: Busybox 1.27 breaks kernel cmdline preseeding

2017-11-28 Thread Raphael Hertzog
On Mon, 27 Nov 2017, Philip Hands wrote: > I'm not sure its important, but when testing a similar script I came up > with something that seems to give better results on the (perhaps > demented) test case I was playing with. Here's mine: > > 's/\([^= ]\+\|\([^= ]*=\("[^"]*"\|'"'[^']*'"'\|[^"'"'"

Re: Busybox 1.27 breaks kernel cmdline preseeding

2017-11-27 Thread Philip Hands
On Mon, 27 Nov 2017, Mathieu Trudel-Lapierre wrote: > On Mon, Nov 27, 2017 at 3:08 PM, Raphael Hertzog wrote: > [...] >> I pushed a pu/kernel-cmdline-preseed branch implementing the preseeding >> out of /proc/cmdline. It's more elaborate than Mathieu's patch >> (https://paste.ubuntu.com/26034695

Re: Busybox 1.27 breaks kernel cmdline preseeding

2017-11-27 Thread Mathieu Trudel-Lapierre
On Mon, Nov 27, 2017 at 4:22 PM, Raphael Hertzog wrote: [...] > We wants to stop using the environment because busybox hides it from us... > I don't see the point of continuing to use it. I understand that; I tend to agree, modulo things not being listed in /proc/cmdline as below... > > Can you

Re: Busybox 1.27 breaks kernel cmdline preseeding

2017-11-27 Thread Raphael Hertzog
Hi, On Mon, 27 Nov 2017, Mathieu Trudel-Lapierre wrote: > On Mon, Nov 27, 2017 at 3:08 PM, Raphael Hertzog wrote: > [...] > > I pushed a pu/kernel-cmdline-preseed branch implementing the preseeding > > out of /proc/cmdline. It's more elaborate than Mathieu's patch > > (https://paste.ubuntu.com/26

Re: Busybox 1.27 breaks kernel cmdline preseeding

2017-11-27 Thread Mathieu Trudel-Lapierre
On Mon, Nov 27, 2017 at 3:08 PM, Raphael Hertzog wrote: [...] > I pushed a pu/kernel-cmdline-preseed branch implementing the preseeding > out of /proc/cmdline. It's more elaborate than Mathieu's patch > (https://paste.ubuntu.com/26034695/) in that it is able to handle > multi-word values. > > I te

Re: Busybox 1.27 breaks kernel cmdline preseeding

2017-11-27 Thread Raphael Hertzog
On Fri, 24 Nov 2017, Raphael Hertzog wrote: > Apparently, kfreebsd has a working /proc/cmdline (thanks to linprocfs mounted > by default) so we might want to switch to that. I pushed a pu/kernel-cmdline-preseed branch implementing the preseeding out of /proc/cmdline. It's more elaborate than Mathi

Re: Busybox 1.27 breaks kernel cmdline preseeding

2017-11-26 Thread Cyril Brulebois
Hi Chris, Chris Boot (2017-11-26): > On 25/11/17 16:24, Cyril Brulebois wrote: > > Busybox maintainers: are you OK with a revert until we figure out what > > to do? (Meaning we can possibly release Buster Alpha 2 without thinking > > + rewriting things…) > > I think I'd prefer not to just revert

Re: Busybox 1.27 breaks kernel cmdline preseeding

2017-11-26 Thread Chris Boot
On 25/11/17 16:24, Cyril Brulebois wrote: > Hi, > > Cyril Brulebois (2017-11-24): >> Raphael Hertzog (2017-11-24): >>> It looks like it has been partially reverted upstream: >>> https://bugs.busybox.net/show_bug.cgi?id=10231 >>> https://git.busybox.net/busybox/commit/?id=9c143ce52da11ec3d21a3491

Re: Busybox 1.27 breaks kernel cmdline preseeding

2017-11-25 Thread Cyril Brulebois
Hi, Cyril Brulebois (2017-11-24): > Raphael Hertzog (2017-11-24): > > It looks like it has been partially reverted upstream: > > https://bugs.busybox.net/show_bug.cgi?id=10231 > > https://git.busybox.net/busybox/commit/?id=9c143ce52da11ec3d21a3491c3749841d3dc10f0 > > > > However this doesn't wo

Re: Busybox 1.27 breaks kernel cmdline preseeding

2017-11-24 Thread Mathieu Trudel-Lapierre
On Fri, Nov 24, 2017 at 9:33 AM, Raphael Hertzog wrote: > Apparently, kfreebsd has a working /proc/cmdline (thanks to linprocfs > mounted > by default) so we might want to switch to that. I don't know about the > hurd though. > Sorry. If I wrote that /proc/cmdline didn't work, I misspoke. What

Re: Busybox 1.27 breaks kernel cmdline preseeding

2017-11-24 Thread Cyril Brulebois
Raphael Hertzog (2017-11-24): > It looks like it has been partially reverted upstream: > https://bugs.busybox.net/show_bug.cgi?id=10231 > https://git.busybox.net/busybox/commit/?id=9c143ce52da11ec3d21a3491c3749841d3dc10f0 > > However this doesn't work for us either because the changes to "showvar

Re: Busybox 1.27 breaks kernel cmdline preseeding

2017-11-24 Thread Raphael Hertzog
Hello, On Fri, 24 Nov 2017, Cyril Brulebois wrote: > > Now, how should we go about fixing this? Adam pointed out we could always > > just parse /proc/cmdline, but this does not just work on kfreebsd. It's > > also possible to consider both /proc/cmdline and the environment; or just > > revert the

Re: Busybox 1.27 breaks kernel cmdline preseeding

2017-11-24 Thread Cyril Brulebois
Hi Mathieu, Mathieu Trudel-Lapierre (2017-11-22): > In Ubuntu we recently noticed the following: with busybox 1.27.0; d-i now > does not take into account some preseeding that happens at the kernel > cmdline. > > For instance: > linux /vmlinux etc. etc. etc. debconf/priority=critical > netcfg/ge

Busybox 1.27 breaks kernel cmdline preseeding

2017-11-22 Thread Mathieu Trudel-Lapierre
Hi, In Ubuntu we recently noticed the following: with busybox 1.27.0; d-i now does not take into account some preseeding that happens at the kernel cmdline. For instance: linux /vmlinux etc. etc. etc. debconf/priority=critical netcfg/get_hostname=blah Here, debconf/priority and netcfg/get_hostna