Philippe Blain wrote:
> Le 2021-05-23 à 14:11, Junio C Hamano a écrit :
> I do not have a clear opinion on a migration path; if consensus can be reached
> that not implying '--patch' is a better behaviour, then changing the behaviour
> would be OK. If some people use scripts that parse 'git log -L' ouptut expecting
> that '-p' is implied, I would expect it's pretty easy to notice the breakage and
> add the now-required switch... but I'll let others be the judge of that.
I agree; it should be easy to notice the breakage. I don't think any
migration path would help, we should just switch. But the git project
has a tendency to worry about hypothetical users.
This is the only thing I can imagine doing for those users:
--- a/builtin/log.c
+++ b/builtin/log.c
@@ -262,8 +262,13 @@ static void cmd_log_init_finish(int argc, const char **argv, const char *prefix,
load_ref_decorations(&decoration_filter, decoration_style);
}
- if (rev->line_level_traverse)
+ if (rev->line_level_traverse) {
+ if (!(rev->diffopt.output_format & DIFF_FORMAT_NO_OUTPUT) && !rev->diff) {
+ advise("In the future you will need to specify --patch. Hit Enter to proceed.");
+ getc(stdin);
+ }
line_log_init(rev, line_cb.prefix, &line_cb.args);
+ }
setup_pager();
}
--
Felipe Contreras
