From: Thomas Rast Date: Fri, 24 Oct 2008 21:48:36 GMT Subject: [PATCH] add -p: warn if only binary changes present Message-ID: <1224884916-20369-1-git-send-email-trast@student.ethz.ch> Current 'git add -p' will say "No changes." if there are no changes to text files, which can be confusing if there _are_ changes to binary files. Add some code to distinguish the two cases, and give a different message in the latter one. Signed-off-by: Thomas Rast --- git-add--interactive.perl | 9 +++++++-- 1 files changed, 7 insertions(+), 2 deletions(-) diff --git a/git-add--interactive.perl b/git-add--interactive.perl index da768ee..a560133 100755 --- a/git-add--interactive.perl +++ b/git-add--interactive.perl @@ -811,11 +811,16 @@ EOF } sub patch_update_cmd { - my @mods = grep { !($_->{BINARY}) } list_modified('file-only'); + my @all_mods = list_modified('file-only'); + my @mods = grep { !($_->{BINARY}) } @all_mods; my @them; if (!@mods) { - print STDERR "No changes.\n"; + if (@all_mods) { + print STDERR "No changes except to binary files.\n"; + } else { + print STDERR "No changes.\n"; + } return 0; } if ($patch_mode) { -- tg: (759ad19..) t/add-p-binary-warn (depends on: origin/master)