Re: [9fans] Git/fs: Possibly Usable
is this an equivalent fix for 9legacy env? (i'm guessing the answer is no)
% diff clone /bin/git/clone
75c75
< for(f in `$nl{walk -f $tree | sed 's@^'$tree'/*@@'}){
---
> for(f in `{ifs=$nl walk -f $tree | sed 's@^'$tree'/*@@'}){
also, the earliest reference to `$split{...}` notation i found is in 9atom;
can anyone confirm?
On Thu, Jul 25, 2019 at 3:01 PM Ori Bernstein wrote:
> On Sun, 21 Jul 2019 16:06:54 -0500
> clue...@tonymendoza.us wrote:
>
> > Thanks Ori!
> >
> > I pulled it down today and started using it against a few GitHub repos I
> have. For those who are thinking about using it, it only uses 'git' and
> 'git+ssh' style URLs, but they work so far without incident.
> >
> > git/clone git+ssh://g...@github.com:tmendoza/9front-user
> >
> > Once I got keys generated and pushed to github.com, worked just fine.
> My updates show up as expected.
>
> Good to hear!
>
> --
> Ori Bernstein
>
>
Re: [9fans] Git/fs: Possibly Usable
yes, that's my change and it was introduced in 9atom, along with allowing newlines within lists.- erikOn Aug 5, 2019 11:43, Skip Tavakkolian wrote:is this an equivalent fix for 9legacy env? (i'm guessing the answer is no)% diff clone /bin/git/clone75c75< for(f in `$nl{walk -f $tree | sed 's@^'$tree'/*@@'}){---> for(f in `{ifs=$nl walk -f $tree | sed 's@^'$tree'/*@@'}){also, the earliest reference to `$split{...}` notation i found is in 9atom; can anyone confirm?On Thu, Jul 25, 2019 at 3:01 PM Ori Bernstein wrote:On Sun, 21 Jul 2019 16:06:54 -0500
clueelf@tonymendoza.us wrote:
> Thanks Ori!
>
> I pulled it down today and started using it against a few GitHub repos I have. For those who are thinking about using it, it only uses 'git' and 'git+ssh' style URLs, but they work so far without incident.
>
> git/clone git+ssh://git@github.com:tmendoza/9front-user
>
> Once I got keys generated and pushed to github.com, worked just fine. My updates show up as expected.
Good to hear!
--
Ori Bernstein
Re: [9fans] Git/fs: Possibly Usable
the reason for this addition was a bug I found in coraid usage of rc scrips where one script set ifs, and called another which did not reset ifs. in retrospect this appears to be a regression over the Bourne she'll due to the elimination of export. anyway, rather than forcing all scripts to defensively use ifs, I thought it would be easier to sunset the use of ifs.- erik
