Junio C Hamano wrote:
> Brandon Casey <brandon.casey.ctr@xxxxxxxxxxxxxxx> writes:
>
>>> This seems to break t9001. Near the tip of 'pu' I have a iffy
>>> workaround.
>> Can you squash this into your 'iffy' workaround to help platforms
>> (Solaris 7, IRIX 6.5) without the 'yes' utility?
>
> Not in this form, for two reasons ;-)
>
> (1) t7610-mergetool.sh,also seems to use "yes". Perhaps define something
> in test-lib.sh?
>
> (2) The implementation is iffy.
Looks good, I'll rework it sometime if you don't beat me to it.
-brandon
>> +yes () {
>> + test -n "$*" && y="$*" || y='y'
>
> Shouldn't it be
>
> if test $# = 0
> then
> y=y
> else
> y="$*"
> fi
>
> so that
>
> yes ""
>
> would give runs of empty lines?
--
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