Re: [PATCH 3/4 v2] Add tests for .git file

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Feb 6, 2008 11:13 AM, Junio C Hamano <gitster@xxxxxxxxx> wrote:
> "Lars Hjemli" <hjemli@xxxxxxxxx> writes:
>
> > On Feb 6, 2008 9:40 AM, Junio C Hamano <gitster@xxxxxxxxx> wrote:
> >> Lars Hjemli <hjemli@xxxxxxxxx> writes:
> >> > +objpath() {
> >> > +    echo $1 | sed -re 's|(..)(.+)|\1/\2|'
> >> > +}
> >>
> >> "sed -r"???  Please limit ourselves to the basics.
> >
> > Sorry, I didn't realize 'sed -r' wasn't kosher. Should I resend or is
> > this something you'd --amend?
>
> If we live in POSIX only world it is fine, but I try to be
> conservative when I am able.

Ok, good to know that my lack of experience doesn't pollute the codebase ;-)

> No need to resend; I've done something like this (there are
> other minor amends).
>...
>     +objpath() {
>     +    echo "$1" | sed -e 's|\(..\)|\1/|'
>     +}

Thanks.
--
larsh
-
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux