Hi Ian, On Tue, Jun 11, 2019 at 04:21:06PM +0100, Ian Abbott wrote: > On 11/06/2019 15:52, Ian Abbott wrote: > > Since commit 90df2a955e3c ("defaultenv: Convert init script to C"), if > > the legacy /env/bin/init script is still being used, any environment > > variables that were set in the script (such as the PS1 prompt) disappear > > when the script exits. I guess it is because the script is now run > > directly by run_command(initfile) (initfile is "/env/bin/init") rather > > than by run_command("source /env/bin/init"). > > This seems easy to fix in "common/startup.c", assuming it is indeed a bug. > Which of the following fixes would be preferable? Indeed this behaviour was not intended and should be fixed. > > 1. Call run_command with a string literal: > > run_command("source /env/bin/init"); I'd probably do something like this: #define INITFILE "/env/bin/init" run_command("source " INITFILE); But I might decide differently if you ask me tomorrow, so do whatever looks best to you ;) Sascha -- Pengutronix e.K. | | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | _______________________________________________ barebox mailing list barebox@xxxxxxxxxxxxxxxxxxx http://lists.infradead.org/mailman/listinfo/barebox