Re: [PATCH 1/2] git-svn: don't append extra newlines at the end of commit messages.

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

 



On 6/13/08, Andreas Ericsson <ae@xxxxxx> wrote:
>  I'm not familiar with git-svn, and my perl is pretty weak as it is.
>  Are you proposing to remove extra whitespace from git commits when they are
>  sent back to svn via dcommit? If so, wouldn't it be better to always strip
>  extra newlines when importing from svn so they're never there in the first
>  place?

I thought of doing it that way (in fact, I *did* do it that way the
first time).  But I figured it would be too bad if you couldn't always
reconstruct the precise svn commit message from the git commit
message.  Currently you can, even though you sometimes have to remove
multiple newlines.  Interestingly, the addition of the git-svn-id:
line seems to be the only reason the commit messages aren't corrupted,
since git normally trims whitespace from the end of its own commits.

So for the record, whether the sha1's would be affected after my patch
didn't even occur to me.  But it's very convenient that they're not.

Avery
--
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]

  Powered by Linux