[PATCH] shell: fix $(()) precedence bug in "X=A?B:C" (it is _not_ "(X=A)?..)"

Denys Vlasenko vda.linux at googlemail.com
Mon Sep 5 17:07:59 UTC 2022


On Mon, Sep 5, 2022 at 2:46 PM Steffen Nurpmeso <steffen at sdaoden.eu> wrote:
>
> Denys Vlasenko wrote in
>  <CAK1hOcO5whb+rKErS-taZLA+_b6Fqu=D_eyPBWzO=h3=VwHC2g at mail.gmail.com>:
>  |On Wed, Aug 31, 2022 at 1:35 AM Steffen Nurpmeso <steffen at sdaoden.eu> \
>  |wrote:
>  |>
>  |> Denys Vlasenko wrote in
>  |>  <CAK1hOcOU1R8PaxN26b5Pc_Ob-XoSrwNsEz1y888pujmfmDZgjw at mail.gmail.com>:
>  |>|Your patches seem to be against dash, not busybox git tree?
>  |>
>  |> No?  I think dash uses a yacc thing..?  (That only supports the
>  |> most basic things, as per POSIX, which is two steps back for
>  |> busybox.  But smaller i presume.)
>  |
>  |The patches need to be against the git tree obtained via
>  |    git clone https://git.busybox.net/busybox/
>  |
>  |Your patches are not.
>
> Of course they are.
>
>   [remote "origin"]
>           url = git://git.busybox.net/busybox/
>           fetch = +refs/heads/master:refs/remotes/origin/master
>

busybox tree has no file named "shexp-arith.h",
yet your patch is against that file.

Do you see this file here?
https://git.busybox.net/busybox/tree/shell


More information about the busybox mailing list