Tags: u-root/gobusybox
Tags
Lock mvdan/sh to v3.11.0 v3.13 has a serious bug, which is that it names kill as a builtin but does not implement it. Which would not be a problem, save that mkuimage imports gobusybox, and that cascades to u-root, and u-root ends up using v3.13.1 of mvdan/sh, which has the bug. It would be much better, I suspect, to stop using shell.Fields in gobusybox, and just forgo having working shell expansion. Using mvdan/sh in gobusybox causes trouble 2 repos to our left ... mvdan/sh might need to be split up as grpc was a while back. Just getting a bunch of broken code, when all you want is shell expansion, is a problem. Signed-off-by: Ronald G Minnich <rminnich@gmail.com>
Lock mvdan/sh to v3.11.0 v3.13 has a serious bug, which is that it names kill as a builtin but does not implement it. Which would not be a problem, save that mkuimage imports gobusybox, and that cascades to u-root, and u-root ends up using v3.13.1 of mvdan/sh, which has the bug. It would be much better, I suspect, to stop using shell.Fields in gobusybox, and just forgo having working shell expansion. Using mvdan/sh in gobusybox causes trouble 2 repos to our left ... mvdan/sh might need to be split up as grpc was a while back. Just getting a bunch of broken code, when all you want is shell expansion, is a problem. Signed-off-by: Ronald G Minnich <rminnich@gmail.com>