Re: [PATCH] bisect: remove Perl use by implementing "git bisect--helper --sq-quote"

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

 



Christian Couder <chriscool@xxxxxxxxxxxxx> writes:

> The sq() function in "git-bisect.sh" was the only place where Perl
> was needed. This patch remove this use of Perl by implementing
> a new "--sq-quote" option in "builtin-bisect--helper.c".

Isn't it because only git-bisect.sh for whatever reason reimplements sq
using Perl while original implementation in am used sed for more
portability?

I would suspect, if it were 3 years ago, that any serious porcelain
writers would have very much appreciated such a feature that gives shell
programmers an easy access to a quoting function that allows a safe eval,
and I would have strongly suggested that the feature to be implemented in
a more permanent place rather than in bisect--helper; perhaps a better
place would be the kitchen-sink "rev-parse".

But given that nobody seems competent enough to do serious programming in
shell these days, I am not sure if the choice between bisect--helper and
rev-parse makes much of a difference ;-).


--
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]