On Mon, Dec 10, 2007 at 11:42:35AM +0100, Peter Baumann wrote: > On Sun, Dec 09, 2007 at 07:24:40PM -0500, Marcus D. Hanwell wrote: > > Hi, > > > > I am quite new to git and git-svn but have been using both for my > > development work recently. I imported the Avogadro subversion repository > > (hosted on Sourceforge) using the following commands, > > > > git svn init -t tags -b branches -T trunk > > https://avogadro.svn.sourceforge.net/svnroot/avogadro > > > > git svn fetch > > > > > > The files avogadro.pro and README in the trunk/ directory appear in my > > imported git repository but not in Avogadro subversion trunk. We also had > > trunk/src/ and all its files/subdirectories appearing in the git checkout > > but not in subversion trunk. We deleted this using git and git svn which > > removed it from the git checkouts too after r858. > > > > I have been talking to Peter who confirmed this and pointed out that the > > repo was reorganised several times in the past. Please CC me on replies as > > I am not on the list. There is a copy of my git repo at > > http://platinum.cryos.net/avogadro.git/ if you would rather skip the > > import. Other than that everything has been working great. It would be good > > to get rid of this bug if possible. Let me know if there is anything else I > > can do to help. > > > > [ Eric Cc'ed, as the author of git-svn ] > > I can confirm that this looks like an import problem. > (e.g see svn log -v -r33:78 https://avogadro.svn.sourceforge.net/svnroot/avogadro) > > I did my analysis on the file /trunk/avogadro.pro, because the error > happens really early in the history so could just import up to revision 76 > to see what goes wrong. (The file gets never deleted in the import, as it > should be!) > > > > trunk/avogadro.pro is added here: > > ------------------------------------------------------------------------ > r33 | dcurtis3 | 2006-08-21 07:34:10 +0200 (Mon, 21 Aug 2006) | 3 lines > Changed paths: > A /trunk/avogadro.pro > > Does a recursive build. > > > Here the refactoring starts (a new directory not tracked by git is added): > ------------------------------------------------------------------------ > r66 | dcurtis3 | 2007-01-03 06:42:45 +0100 (Wed, 03 Jan 2007) | 3 lines > Changed paths: > A /avogadro > > Making room for libavogadro. > > > /branches is moved to /avogadro/branches > ------------------------------------------------------------------------ > r67 | dcurtis3 | 2007-01-03 06:47:11 +0100 (Wed, 03 Jan 2007) | 3 lines > Changed paths: > A /avogadro/branches (from /branches:66) > D /branches > > Making room for libavogadro > > > > > ERROR HAPPENS HERE: > ==================== > > /trunk and /tags are moved, too. > (/trunk/avogadro.pro becomes /avogadro/trunk/avogadro.pro): > ------------------------------------------------------------------------ > r68 | dcurtis3 | 2007-01-03 06:47:34 +0100 (Wed, 03 Jan 2007) | 3 lines > Changed paths: > A /avogadro/tags (from /tags:66) > A /avogadro/trunk (from /trunk:66) > D /tags > D /trunk > > Making room for libavogadro. > > The above delete/move of trunk isn't recorded anywhere in the git svn import. > 'git-svn find-rev r66' doesn't produce any output! Sorry. I meant r67 and r68 instead of r66. > And later git-svn thinks that /trunk and all its files are still there, so > e.g. /trunk/avogadro.pro stays in the repo forever. > > > -Peter > > This is just for consistency here and for anyone interested in further > analyses. > > Here is the rest of the import analyses I've done. Look how "interesting" the > repo is and how many times the files are moved. > > > > > /libavogadro is added > ------------------------------------------------------------------------ > r69 | dcurtis3 | 2007-01-03 06:50:58 +0100 (Wed, 03 Jan 2007) | 3 lines > Changed paths: > A /libavogadro > A /libavogadro/CMakeLists.txt > A /libavogadro/FindOpenBabel.cmake > A /libavogadro/src > A /libavogadro/src/CMakeLists.txt > A /libavogadro/src/color.cpp > A /libavogadro/src/color.h > A /libavogadro/src/engine.h > A /libavogadro/src/engines > A /libavogadro/src/engines/CMakeLists.txt > A /libavogadro/src/engines/ResidueSphereEngine.cpp > A /libavogadro/src/engines/ResidueSphereEngine.h > A /libavogadro/src/engines/SphereEngine.cpp > A /libavogadro/src/engines/SphereEngine.h > A /libavogadro/src/engines/StickEngine.cpp > A /libavogadro/src/engines/StickEngine.h > A /libavogadro/src/engines/WireframeEngine.cpp > A /libavogadro/src/engines/WireframeEngine.h > A /libavogadro/src/engines/bsengine.cpp > A /libavogadro/src/engines/bsengine.h > A /libavogadro/src/engines/vertexarray.cpp > A /libavogadro/src/engines/vertexarray.h > A /libavogadro/src/glwidget.cpp > A /libavogadro/src/glwidget.h > A /libavogadro/src/primitives.cpp > A /libavogadro/src/primitives.h > > Added libavogadro. > > > Create /libavogadro/{trunk,branches,tags} and move anything below the > former /libavogadro into /libavogadro/trunk > ------------------------------------------------------------------------ > r70 | dcurtis3 | 2007-01-03 06:53:13 +0100 (Wed, 03 Jan 2007) | 3 lines > Changed paths: > D /libavogadro/CMakeLists.txt > D /libavogadro/FindOpenBabel.cmake > A /libavogadro/branches > D /libavogadro/src > A /libavogadro/tags > A /libavogadro/trunk > A /libavogadro/trunk/CMakeLists.txt (from /libavogadro/CMakeLists.txt:69) > A /libavogadro/trunk/FindOpenBabel.cmake (from /libavogadro/FindOpenBabel.cmake:69) > A /libavogadro/trunk/src (from /libavogadro/src:69) > > Putting in trunk / branches / tags. > > > ------------------------------------------------------------------------ > r72 | dcurtis3 | 2007-01-03 08:55:32 +0100 (Wed, 03 Jan 2007) | 3 lines > Changed paths: > A /avogadro/trunk/CMakeLists.txt > A /avogadro/trunk/FindAvogadro.cmake > A /avogadro/trunk/FindOpenBabel.cmake > D /avogadro/trunk/src/Engine.h > D /avogadro/trunk/src/GLWidget.cpp > D /avogadro/trunk/src/GLWidget.h > D /avogadro/trunk/src/MainWindow.cpp > D /avogadro/trunk/src/MainWindow.h > D /avogadro/trunk/src/Primitives.cpp > D /avogadro/trunk/src/Primitives.h > D /avogadro/trunk/src/Renderer.h > D /avogadro/trunk/src/Tool.cpp > D /avogadro/trunk/src/Tool.h > D /avogadro/trunk/src/Views.cpp > D /avogadro/trunk/src/Views.h > D /avogadro/trunk/src/color.cpp > D /avogadro/trunk/src/color.h > D /avogadro/trunk/src/engines > M /avogadro/trunk/src/main.cpp > A /avogadro/trunk/src/mainwindow.cpp (from /avogadro/trunk/src/MainWindow.cpp:70) > A /avogadro/trunk/src/mainwindow.h (from /avogadro/trunk/src/MainWindow.h:70) > A /avogadro/trunk/src/tool.cpp (from /avogadro/trunk/src/Tool.cpp:70) > A /avogadro/trunk/src/tool.h (from /avogadro/trunk/src/Tool.h:70) > A /avogadro/trunk/untitled01.gpr > > Now uses libavogadro. Starting to think we should package this a > little different. > > > > Renamed /libavogadro to /avogadro-lib > ------------------------------------------------------------------------ > r73 | dcurtis3 | 2007-01-03 16:48:02 +0100 (Wed, 03 Jan 2007) | 3 lines > Changed paths: > A /avogadro-lib (from /libavogadro:72) > D /libavogadro > > Refactor > > > mv /avogadro/trunk/avogadro.pro /trunk/avogadro/avogadro.pro > ------------------------------------------------------------------------ > r75 | dcurtis3 | 2007-01-03 20:49:35 +0100 (Wed, 03 Jan 2007) | 2 lines > Changed paths: > D /avogadro > D /avogadro-lib > A /branches > A /tags > A /trunk > A /trunk/avogadro (from /avogadro/trunk:74) > A /trunk/libavogadro (from /avogadro-lib/trunk:74) > > Moving things around. Conforming to a more KDE-like layout in SVN. > I'm going nuts. SOrry - 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