Re: [PATCH] sparse-checkout: improve OS ls compatibility

From: Ed Maste <emaste@freebsd.org>
Date: Dec 19, 2019, 16:15 UTC
Message-ID: <CAPyFy2AV2NG66LqBJr_Wb1_V5XhKnM+44m0H8FYa_3K4XupLow@mail.gmail.com>
In-Reply-To: <c4fef89a-2275-b4bc-b5c2-7bc647cd9bf6@gmail.com>

On Thu, 19 Dec 2019 at 08:56, Derrick Stolee <stolee@gmail.com> wrote:

Show 9 quoted lines

>
> On 12/18/2019 9:45 PM, Eric Wong wrote:
> > This goes on top of your patch:
> ...
> > +     ( cd "$1" && for i in *; do echo "$i"; done )
>
> Could we drop the "cd" and "echo" processes with this line instead?
>
>         for i in "$1"/*; do printf "$i\n"; done

That would output repo/a, but we could do something like: for i in "$1"/*; do echo "${i#$1/}"; done

echo's a builtin on any /bin/sh I'm aware of - do you have a /bin/sh with builtin printf but not echo?