Re: [PATCH v2] git-config support for diff.relative setting

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

 



"Philip Oakley" <philipoakley@xxxxxxx> wrote:
Shouldn't this `(identical to "git diff --relative")` also be
included in the documentation change? It would truly clarify the
intenbt for the reader.

Updated as follows:
+diff.relative::
+	Show pathnames relative to the current directory and exclude
+	changes outside this directory; equivalent to the 'git diff'
+	option '--relative'.

"Junio C Hamano" <gitster@xxxxxxxxx> wrote:
There should be a way to run non-relative diff in a repository that
has the configuration set, perhaps by overriding with some command
line option (e.g. "--no-relative").

Good idea; I'll put together a second patch for a "--no-relative" option to disable diff.relative.

-----Original Message-----
From: Junio C Hamano <gitster@xxxxxxxxx>
Sent: 12/20/2014 09:58 PM
To: Philip Oakley <philipoakley@xxxxxxx>
CC: <kelson@xxxxxxxxxxxxxxx>, "Git Mailing List" <git@xxxxxxxxxxxxxxx>, "Duy Nguyen" <pclouds@xxxxxxxxx>, "Jonathan Nieder" <jrnieder@xxxxxxxxx>
Subject: Re: [PATCH v2] git-config support for diff.relative setting

"Philip Oakley" <philipoakley@xxxxxxx> writes:
From: <kelson@xxxxxxxxxxxxxxx>
By default, git-diff shows changes and pathnames relative to the
repository root. Setting the diff.relative config option to "true"
shows pathnames relative to the current directory and excludes
changes outside this directory (identical to "git diff --relative").
Shouldn't this `(identical to "git diff --relative")` also be included
in the documentation change? It would truly clarify the intenbt for
the reader.

A configuration that sticks a non-default behaviour without a way to
revert to the default is a no-no.

There should be a way to run non-relative diff in a repository that
has the configuration set, perhaps by overriding with some command
line option (e.g. "--no-relative").



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