I'm not sure which news you're referring to. :)
To answer Jeremy's request: briefly, git doesn't track renames at all
(which are conceptually differences between versions) which makes
sense when you consider what it does track (conceptually, only
collections of files and an ordering between collections). Instead,
tools like "git diff" know how to infer renames from the content, and
then the Rietveld uploader sorta knows how to munge them into a
Subversion-compatible format. (And when you dcommit to svn it
converts those inferred renames into real renames.) It usually works
pretty well except when it doesn't.
If you can convince "git diff" through some combination of flags that
a rename or copy happened, then that info can be communicated to svn.
So two sources of glitches: (1) can't convince git that two files are
related, typically because the text between them is too different; (2)
the Rietveld uploader isn't synthesizing Subversion renames well
enough. (2) is a bug that I (or realistically, you) can fix, while
(1) frequently means that tracking that history doesn't really matter
too much.
On Mon, Nov 2, 2009 at 10:28 PM, Paweł Hajdan Jr.
<phajdan...@chromium.org> wrote:
> That's a good news. Can that info be put somewhere in the UsingGit wiki
> page?
>
> On Mon, Nov 2, 2009 at 23:56, Evan Martin <e...@chromium.org> wrote:
>>
>> On Mon, Nov 2, 2009 at 2:47 PM, Drew Wilson <atwil...@chromium.org> wrote:
>> > I'm doing some refactoring in the chromium worker code, and I got the
>> > following review feedback:
>> > "small request: can the files that were branched from existing files be
>> > "svn
>> > copy'd" instead of copying manually and "svn added"? This will preserve
>> > history
>> > and will make it easier to see what changed in the review."
>> > Is there a way to do this with git? I do want to make my reviewers happy
>> > :)
>>
>> Git has an ...odd... way of thinking about renames. The pieces you
>> need to know are:
>> 1) Does git believe you did a rename? Consult "git diff -M -C --stat
>> origin..." and see if it prints renames.
>> 2) If no, ping me off-list and I'll help you out.
>> 3) If yes, Rietveld should be able to display the diff properly.
>> However, this code is relatively untested (we have to convert the Git
>> renames into SVN-style "renames" -- SVN doesn't actually support true
>> renames, oddly enough) so it could be wrong. In that case, also ping
>> me off-list. :)
>>
>> >>
>
>
--~--~---------~--~----~------------~-------~--~----~
Chromium Developers mailing list: chromium-dev@googlegroups.com
View archives, change email options, or unsubscribe:
http://groups.google.com/group/chromium-dev
-~----------~----~----~----~------~----~------~--~---
