Re: [PATCH] diff: add config option relative

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

 



On Fri, May 15, 2020 at 05:04:43PM -0700, Junio C Hamano wrote:
> "brian m. carlson" <sandals@xxxxxxxxxxxxxxxxxxxx> writes:
> 
> > On 2020-05-15 at 15:57:06, Laurent Arnoud wrote:
> >> The `diff.relative` boolean option set to `true` to show only changes on the
> >> current directory and show relative pathnames.
> >
> > Usually when we implement configuration settings like this, we implement
> > an option value, such as --no-relative, so that users or scripting tools
> > can disable this feature if they need to.  However, I don't see that in
> > this series.  Would adding such a feature be possible?
> 
> I think I saw a variant that does have --[no-]foobar support on the
> list, but I may be hallucinating.  You definitely would want it, or
> you've broken "format-patch" forever without a knob to countermand
> the configuration (well, you can still say "git -c diff.foobar=no"
> but that's kind of cheating).

Yes I've submitted a v2 patch with the `--no-relative` I will send a v3 to this
thread for review

-- 
Laurent



[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