<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sat, 29 Jun 2019 at 18:37, Rob Landley <<a href="mailto:rob@landley.net">rob@landley.net</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On 6/29/19 7:52 AM, Eric Molitor wrote:<br>
> I know that you are working on toysh which I'm looking forward to. In the<br>
> meantime below is a patch to improve compatibility with older/odd versions of<br>
> bash. This fixed a minor build issue I was having on MacOS (which was using zsh<br>
> in emulated bash mode) as well as an oddball embedded SDK using a non-gnu<br>
> version of bash. I believe these changes are minimal and should be safe to<br>
> apply, if not I wanted to at least get them on the mailing list in case others<br>
> ran into these issues.<br>
<br>
I applied this, on the theory you have a testing environment I don't and I<br>
didn't cause me obvious regressions when I tried it, but I'm not happy with it.<br>
<br>
> -if [ generated/mkflags -ot scripts/mkflags.c ]<br>
> +if [ ! -f generated/mkflags ] || [ generated/mkflags -ot scripts/mkflags.c ]<br>
<br>
In which bash version does comparing a file that doesn't exist return true?<br>
<br></blockquote><div>I believe this is a bug in the zsh version of bash emulation. I originally ran into this on MacOS but also reproduced with zsh 5.7 running in emulated bash mode on Linux. I was not able to reproduce it with modern versions of bash or mksh.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
>  if [ -z "$SED" ]<br>
>  then<br>
> -  [ ! -z "$(which gsed 2>/dev/null)" ] && SED=gsed || SED=sed<br>
> +  [ ! -z "$(command -v gsed 2>/dev/null)" ] && SED=gsed || SED=sed<br>
>  fi<br>
<br>
Toybox provides a which this works with. What problem did you hit?<br>
<br></blockquote><div> </div><div>On an old arm11 sdk this was evaluating to true even though gsed was not present. I had originally worked around this by specifying SED=sed when building until I realised that command -v worked reliably. <br></div><div><br></div><div>- Eric<br></div></div></div>