Johannes Sixt <j.sixt@xxxxxxxxxxxxx> writes: > Jonathan del Strother schrieb: >> - svn import -m 'import for git-svn' . $svnrepo >/dev/null && >> + svn import -m 'import for git-svn' . '$svnrepo' >/dev/null && >> cd .. && >> rm -rf import && >> - git-svn init $svnrepo" >> + git-svn init '$svnrepo'" > > I don't see the point in changing an incorrect quoting to a different > incorrect quoting that you fix up in a follow-up patch. It's *two* > large patches to review instead of just one. I'm stopping the review > here. Since I consider it unlikely that Jonathan is making your life harder on purpose, it might be somewhat more helpful to offer submission advice: Jonathan, try git rebase -i HEAD~3 or so in order to consolidate the last 3 patches you did. Interactive rebase is one useful manner of munging history until it looks reasonably nice for submission. One major point of git's distributed operation is that one can clean up the development history locally before handing things out. That makes the project repositories cleaner to understand. -- David Kastrup - 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
