Phillip Wood <phillip.wood123@xxxxxxxxx> writes: >> @@ -5494,6 +5501,9 @@ static void prep_parse_options(struct diff_options *options) >> N_("when run from subdir, exclude changes outside and show relative paths"), >> PARSE_OPT_NONEG | PARSE_OPT_OPTARG, >> diff_opt_relative), >> + OPT_SET_INT_F(0, "no-relative", &options->flags.relative_name, >> + N_("disable diff.relative config option"), >> + 0, PARSE_OPT_NONEG), > > Rather than adding a new option, it would be better to modify the > existing --relative option. If you remove PARSE_OPT_NONEG from the > declaration above it will support --no-relative. You will also need to > modify diff_opt_relative() to handle clearing the option. Excellent.