Re: [PATCH] Avoid bug in Solaris xpg4/sed as used in submodule

Excerpts from Junio C Hamano's message of Tue Apr 10 12:10:22 -0400 2012:

Andreas Schwab [off-list ref] writes:

quoted

Junio C Hamano [off-list ref] writes:

quoted

FWIW, it bothers me a lot more that expression does not anchor
matches against path elements with explicit '/' than the issue your patch
addresses, i.e.

    sed -e 's|[^/][^/]*/|../|g'

Note that this requires readding a trailing slash to the path, which
complicates the whole thing a bit.

Or simply swapping order of the stripping and sed invocation, which
is not a big deal.  I personally find the pattern with trailing '/'
easier to grok but it is also not a big deal.

Sorry for the delay here, I haven't had a chance to look at this in a
few days.  I tried the version with [^/][^/]*/ tonight and it was more
invasive than required for such a simple fix, I think.

Switching the order of the calls is ok, but then you need to use a
rela and relb as the second substitution relies on $a being
/-terminated and the creation of gitdir:... requires the opposite.  So
now, you're switching the order, moving other lines, renaming
variables and also altering the lines used to spit out the paths in
order to avoid also stripping the / from the rel* variables.

Updated patch to follow, but please don't hesitate to request the
alternate version if it really is what you'd like to see.  (I've done
the work already.)

Thanks
-Ben
--
Ben Walton
Systems Programmer - CHASS
University of Toronto
C:416.407.5610 | W:416.978.4302