Thread (10 messages) flat view 10 messages, 3 authors, 2016-06-15

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

From: Kelson <hidden>
Date: 2016-06-15 23:03:16

That is quite manageable. I was concerned that --relative changes the UI 
(relative paths) and behavior (excluding files outside the current 
directory), which might not be clear if placed in just the UI component. 
You make a great point that git_diff_basic_config drives other commands 
though, like git-bisect, which --relative would not effect.

-----Original Message-----
From: Duy Nguyen <redacted>
Sent: 12/11/2014 08:37 AM
To: Kelson <redacted>
CC: Git Mailing List <redacted>
Subject: Re: [PATCH] added git-config support for diff.relative setting

On Thu, Dec 11, 2014 at 2:28 PM, Kelson [off-list ref] wrote:
quoted hunk ↗ jump to hunk
@@ -270,6 +270,14 @@ int git_diff_basic_config(const char *var, const char
*value, void *cb)
                 return 0;
         }

+       if (!strcmp(var, "diff.relative")) {
+               if (git_config_bool(var, value))
+                       DIFF_OPT_SET(&default_diff_options, RELATIVE_NAME);
+               else
+                       DIFF_OPT_CLR(&default_diff_options, RELATIVE_NAME);
+               return 0;
+       }
+
         if (starts_with(var, "submodule."))
                 return parse_submodule_config_option(var, value);
This affects more than just git-diff. git_diff_ui_config() may be a
better place.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help