<div dir="ltr">James, thanks for working.<div><br></div><div>As far as I have checked through trunk, there is no major problem.</div><div>I saw a few differences in author name, but I assume they would be intentional, since you say you double-checked :)</div><div><br></div><div>I haven't checked branches yet, but I think we can easily rewind branches at the moment.</div><div><br></div><div>Takumi</div></div><br><div class="gmail_quote"><div dir="ltr">On Wed, Nov 7, 2018 at 12:00 AM James Y Knight <<a href="mailto:jyknight@google.com">jyknight@google.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div dir="ltr"><div>I've regenerated the repository set at <a href="https://github.com/jyknight/llvm-git-migration" target="_blank">https://github.com/jyknight/llvm-git-migration</a> again, with these fixes.</div></div></div><div dir="ltr"><div dir="ltr"><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">James, could you prune r70891? It cuts the history.</blockquote><div><br></div></div></div><div dir="ltr"><div dir="ltr">Done.<div><br><div class="gmail_quote"></div></div></div></div><div dir="ltr"><div dir="ltr"><div><div class="gmail_quote"><div dir="ltr">On Tue, Oct 23, 2018 at 2:44 PM James Y Knight <<a href="mailto:jyknight@google.com" target="_blank">jyknight@google.com</a>> wrote:</div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div>My preference is "all or nothing" :)</div></div></blockquote><div><br></div><div>Yes, I think it's probably best to preserve the empty directory add+modify commits. I'll fix this...</div><div></div></div></div></div></div></div></div></div></div></blockquote><div><br></div></div></div></div></div><div dir="ltr"><div dir="ltr"><div><div class="gmail_quote"><div>Done.</div></div></div></div></div><div dir="ltr"><div dir="ltr"><div><div class="gmail_quote"><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div>2) Glitches of authors who have changed their name</div><div><br></div><div>I found too many commits that have incompatible author filed.</div><div>I heard from you, you have sought through commits, mailing lists.</div><div>But I don't think all of glitches would be intentional.</div></div></blockquote><div><br></div><div>Indeed -- oops! I managed to insert an off-by-one error into the author mapping function, so there's a mismatch at every revision where the mapping changes. Fixing that...</div></div></div></div></div></div></div></div></div></blockquote><div><br></div></div></div></div></div><div dir="ltr"><div dir="ltr"><div><div class="gmail_quote"><div>Done, and double-checked again against the old git conversion.</div><div><br></div><div><div>I also took the opportunity to add some more fixups for CVS issues back to the 1.0 release. I've now also now verified that the file moving done by these CVS fixes didn't end up accidentally losing changes, by iterating over all the commits and ensuring that file hashes in a recursive ls-tree have not disappeared from the commit (just changed their path).</div><div><br></div><br class="m_8405952876700584036gmail-Apple-interchange-newline"></div></div></div></div></div>
</blockquote></div>