Re: git add to ignore whitespaces, some day?

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Feb 27, 2015 at 01:09:30AM +0100, Marc-André Lureau wrote:

> It would be nice if git-add could be told to ignore whitespace
> changes, wouldn't it?
> 
> According to SO, I am not the one to think so:
> http://stackoverflow.com/questions/3515597/git-add-only-non-whitespace-changes
> 
> A change to add--interactive would be as simple as adding the diff -b
> or -w option like:
> my @diff = run_cmd_pipe("git", @diff_cmd, "-w", "--", $path);

What would it mean to stage such a hunk? For example, consider this
situation:

    git init

    echo 'foo();' >file
    git add file

    {
      echo 'if (something) {'
      echo '    foo();'
      echo '}'
    } >file

A regular diff shows:

    diff --git a/file b/file
    index a280f9a..ce0eeda 100644
    --- a/file
    +++ b/file
    @@ -1 +1,3 @@
    -foo();
    +if (something) {
    +    foo();
    +}

but "diff -w" would show:

    diff --git a/file b/file
    index a280f9a..ce0eeda 100644
    --- a/file
    +++ b/file
    @@ -1 +1,3 @@
    +if (something) {
         foo();
    +}

If we try to apply that hunk to what is in the index, it will not work.
The context line does not exist in the index file. Even if you could
convince git-apply to massage it into place, it still does not update
the whitespace in the 'foo();' line. IOW, we did not stage the full hunk
at all; running "git add -p" again would show that we still have the
whitespace change to stage.

So if you were to pursue this, it would have to have two copies of each
hunk: the one to apply, and the "display" copy that we show the user. We
do this already for colorization. However, I think we rely there on the
fact that the two versions of the diff match up, line for line. Whereas
here, you would not even necessarily have the same number of hunks
between the regular and "-b" versions.

-Peff
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]