# [PATCH] Teach --text option to diff 6 messages from 2006-07-07 to 2006-07-07. Participants: Stephan Feder, Junio C Hamano, sf. Thread: https://gitlist.dev/t/4794 ## Stephan Feder, 2006-07-07 10:33 Subject: [PATCH] Teach --text option to diff Message-ID: <11522684373987-git-send-email-sf@b-i-t.de> URL: https://gitlist.dev/e/11522684373987-git-send-email-sf%40b-i-t.de ``` Add new item text to struct diff_options. If set then do not try to detect binary files. Signed-off-by: Stephan Feder --- I have to send patches of binary data to a customer but the builtin diff was no help in this case. Notes: 1. The shorthand -a for --text is not implemented. Is there a conflicting shorthand? 2. For diffstat --text is ignored. It seems pointless because binary patch data is not for human consumption anyway. 3. No documentation yet. If the patch is accepted I will add a short description. To Documentation/diff-options.txt? Regards Stephan diff.c | 5 ++++- diff.h | 1 + 2 files changed, 5 insertions(+), 1 deletions(-) diff --git a/diff.c b/diff.c index f0450a8..1f0219d 100644 --- a/diff.c +++ b/diff.c @@ -723,7 +723,7 @@ static void builtin_diff(const char *nam if (fill_mmfile(&mf1, one) < 0 || fill_mmfile(&mf2, two) < 0) die("unable to read files to diff"); - if (mmfile_is_binary(&mf1) || mmfile_is_binary(&mf2)) { + if (!o->text && (mmfile_is_binary(&mf1) || mmfile_is_binary(&mf2))) { /* Quite common confusing case */ if (mf1.size == mf2.size && !memcmp(mf1.ptr, mf2.ptr, mf1.size)) @@ -1561,6 +1561,9 @@ int diff_opt_parse(struct diff_options * options->output_format |= DIFF_FORMAT_PATCH; options->full_index = options->binary = 1; } + else if (!strcmp(arg, "--text")) { + options->text = 1; + } else if (!strcmp(arg, "--name-only")) options->output_format |= DIFF_FORMAT_NAME; else if (!strcmp(arg, "--name-status")) diff --git a/diff.h b/diff.h index d557394..f80f646 100644 --- a/diff.h +++ b/diff.h @@ -42,6 +42,7 @@ struct diff_options { unsigned recursive:1, tree_in_recursive:1, binary:1, + text:1, full_index:1, silent_on_remove:1, find_copies_harder:1, -- 1.4.1.gbc483 ``` ## Junio C Hamano, 2006-07-07 11:06 Subject: Re: [PATCH] Teach --text option to diff Message-ID: <7v64i91xow.fsf@assigned-by-dhcp.cox.net> URL: https://gitlist.dev/e/7v64i91xow.fsf%40assigned-by-dhcp.cox.net In-Reply-To: <11522684373987-git-send-email-sf@b-i-t.de> ``` Stephan Feder writes: > I have to send patches of binary data to a customer but the builtin diff > was no help in this case. Given the previous patch, and also your point #2 below, I would have expected you to introduce an option to force files to be treated as binary even when they are otherwise misidentified as text, but this patch is going the other way. Interesting. > 1. The shorthand -a for --text is not implemented. Is there a conflicting > shorthand? I do not think of one offhand, but it's the responsibility for the party to propose such an enhancement to do the study ;-) > 2. For diffstat --text is ignored. It seems pointless because binary > patch data is not for human consumption anyway. > 3. No documentation yet. If the patch is accepted I will add a short > description. To Documentation/diff-options.txt? Most likely that would be the place. ``` ## sf, 2006-07-07 11:53 Subject: Re: [PATCH] Teach --text option to diff Message-ID: <44AE4B35.6060607@b-i-t.de> URL: https://gitlist.dev/e/44AE4B35.6060607%40b-i-t.de In-Reply-To: <7v64i91xow.fsf@assigned-by-dhcp.cox.net> ``` Junio C Hamano wrote: > Stephan Feder writes: > >> I have to send patches of binary data to a customer but the builtin diff >> was no help in this case. > > Given the previous patch, and also your point #2 below, I would > have expected you to introduce an option to force files to be > treated as binary even when they are otherwise misidentified as > text, but this patch is going the other way. > > Interesting. Not really. I was surprised that the GNU diff option --text is unsupported in the builtin diff. > >> 1. The shorthand -a for --text is not implemented. Is there a conflicting >> shorthand? > > I do not think of one offhand, but it's the responsibility for > the party to propose such an enhancement to do the study ;-) Of course. I did not find any conflict but as the builtin diff and its options are used by quite a lot of git commands I wanted to make sure. If no objections arise I am going to add the shorthand. >> 2. For diffstat --text is ignored. It seems pointless because binary >> patch data is not for human consumption anyway. > >> 3. No documentation yet. If the patch is accepted I will add a short >> description. To Documentation/diff-options.txt? > > Most likely that would be the place. Good. ``` ## Stephan Feder, 2006-07-07 13:57 Subject: [PATCH 1/3] Teach diff -a as shorthand for --text Message-ID: <11522806291956-git-send-email-sf@b-i-t.de> URL: https://gitlist.dev/e/11522806291956-git-send-email-sf%40b-i-t.de In-Reply-To: <44AE4B35.6060607@b-i-t.de> ``` Signed-off-by: Stephan Feder --- diff.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/diff.c b/diff.c index 1f0219d..b423491 100644 --- a/diff.c +++ b/diff.c @@ -1561,7 +1561,7 @@ int diff_opt_parse(struct diff_options * options->output_format |= DIFF_FORMAT_PATCH; options->full_index = options->binary = 1; } - else if (!strcmp(arg, "--text")) { + else if (!strcmp(arg, "-a") || !strcmp(arg, "--text")) { options->text = 1; } else if (!strcmp(arg, "--name-only")) -- 1.4.1.gbc483 ``` ## Stephan Feder, 2006-07-07 13:57 Subject: [PATCH 2/3] Add -a and --text to common diff options help Message-ID: <11522806292871-git-send-email-sf@b-i-t.de> URL: https://gitlist.dev/e/11522806292871-git-send-email-sf%40b-i-t.de In-Reply-To: <44AE4B35.6060607@b-i-t.de> ``` Signed-off-by: Stephan Feder --- diff.h | 3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/diff.h b/diff.h index f80f646..8ab0448 100644 --- a/diff.h +++ b/diff.h @@ -162,7 +162,8 @@ #define COMMON_DIFF_OPTIONS_HELP \ " -O reorder diffs according to the .\n" \ " -S find filepair whose only one side contains the string.\n" \ " --pickaxe-all\n" \ -" show all files diff when -S is used and hit is found.\n" +" show all files diff when -S is used and hit is found.\n" \ +" -a --text treat all files as text.\n" extern int diff_queue_is_empty(void); extern void diff_flush(struct diff_options*); -- 1.4.1.gbc483 ``` ## Stephan Feder, 2006-07-07 13:57 Subject: [PATCH 3/3] diff-options: Explain --text and -a Message-ID: <11522806292890-git-send-email-sf@b-i-t.de> URL: https://gitlist.dev/e/11522806292890-git-send-email-sf%40b-i-t.de In-Reply-To: <44AE4B35.6060607@b-i-t.de> ``` Signed-off-by: Stephan Feder --- Documentation/diff-options.txt | 6 ++++++ 1 files changed, 6 insertions(+), 0 deletions(-) diff --git a/Documentation/diff-options.txt b/Documentation/diff-options.txt index f523ec2..1a93629 100644 --- a/Documentation/diff-options.txt +++ b/Documentation/diff-options.txt @@ -94,5 +94,11 @@ Swap two inputs; that is, show differences from index or on-disk file to tree contents. +--text:: + Treat all files as text. + +-a:: + Shorthand for "--text". + For more detailed explanation on these common options, see also link:diffcore.html[diffcore documentation]. -- 1.4.1.gbc483 ```