Re: [PATCH] git-rebase.sh: Use POSIX/Susv command substitution instead of backticks

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

 



Hi,

On Wed, 6 Feb 2008, Jari Aalto wrote:

> * Tue 2008-02-05 Johannes Schindelin <Johannes.Schindelin@xxxxxx>
> * Message-Id: alpine.LSU.1.00.0802052226340.8543@xxxxxxxxxx
>
> > On Wed, 6 Feb 2008, Jari Aalto wrote:
> >
> >> Use redable $(<command>) syntax instead of backtics in code.  See The 
> >> Open Group Base Specifications Issue 6, IEEE Std 1003.1, 2004 Edition
> >
> > Sorry, I am not quite sure if this patch is worth it: either you want 
> > to clean this up in _all_ of our shell scripts, or you leave it.
> 
> Yes, that's the plan. This is just a start.

Again, you failed to address me.  FYI my email address is 
johannes.schindelin@xxxxxxx  Thank you very much.

"This is just a start" is not good enough.  First, you have to defend why 
this is a good change.  It does not really fix anything.

And then you have to do it for all scripts in one go.  Mind you, it is not 
really complicated: just one call to perl.

Hth,
Dscho



-
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