On 7/11/26 20:59, Michael Tokarev wrote:
...
What the mess.
So I really wonder where this bug #925979 come from.
Now, by reverting the mentioned busybox commit, we change one more
place, in showvars(), where - after reverting - only the "valid"
names are shown, the rest are filtered.
Cyrill, can you think how #925979 happened? How about trying d-i
with busybox without these two patches applied?
Re-reading #925979 again, it feels like at the time, things were in
hurry for the upcoming buster release, and "something" went wrong.
But I really don't see where the breakage might come from, - it is
just wrong. Upstream busybox without the hacks Just Works, and it
works better than our hacked version (because of the changes in
showenv() to display the "bad" variabled).
Maybe that was the issue here? -- somewhere in d-i it compared the
output of `set` with something? But it is very unlikel.
The regular upstream version should work just fine...
I think it should be okay to drop the two patches now and see what
happens next.
I'll do that in a couple days, if no one objects.
Thanks,
/mjt
