user/sven/git.git/git-cvsimport.perl, branch v1.4.4.4

Run "git repack -a -d" once more at end, if there's 1MB or more of not-packed data.

2006-11-15T20:50:29Z

Although I converted upstream coreutils to git last month, I just reconverted coreutils once again, as a test, and ended up with a git repository of about 130MB (contrast with my packed git repo of size 52MB). That was because there were a lot of commits (but < 1024) after the final automatic "git-repack -a -d". Running a final git-repack -a -d && git-prune-packed cut the final repository size down to the expected size. So this looks like an easy way to improve git-cvsimport. Just run "git repack ..." at the end if there's more than some reasonable amount of not-packed data. My choice of 1MB is a little arbitrarily. I wouldn't mind missing the minimal repo size by 1MB. At the other end of the spectrum, it's probably not worthwhile to pack everything when the total repository size is less than 1MB. Here's the patch: Signed-off-by: Jim Meyering Signed-off-by: Junio C Hamano

cvsimport: move over to using git-for-each-ref to read refs.

2006-09-20T17:21:46Z

cvsimport opens all of the files in $GIT_DIR/refs/heads and reads out the sha1's in order to work out what time the last commit on that branch was made (in CVS) thus allowing incremental updates. However, this takes no account of hierachical refs naming producing the following error for each directory in $GIT_DIR/refs: Use of uninitialized value in chomp at /usr/bin/git-cvsimport line 503. Use of uninitialized value in concatenation (.) or string at /usr/bin/git-cvsimport line 505. usage: git-cat-file [-t|-s|-e|-p|] Take advantage of the new packed refs work to use the new for-each-ref iterator to get this information. Signed-off-by: Andy Whitcroft Signed-off-by: Junio C Hamano

cvsimport - cleanup of the multi-indexes handling

2006-06-28T10:53:37Z

Indexes are only needed when we are about preparing to commit. Prime them inside commit() when we have all the info we need, and remove all the redundant index setups. While we are at it, make sure that index handling is correct when opening new branches, and on initial import. Signed-off-by: Martin Langhoff Signed-off-by: Junio C Hamano

cvsimport: always set $ENV{GIT_INDEX_FILE} to $index{$branch}

2006-06-25T03:08:25Z

Also, make sure that the initial git-read-tree is performed. Signed-off-by: Johannes Schindelin

cvsimport: setup indexes correctly for ancestors and incremental imports

2006-06-24T12:30:06Z

Two bugs had slipped in the "keep one index per branch during import" patch. Both incremental imports and new branches would see an empty tree for their initial commit. Now we cover all the relevant cases, checking whether we actually need to setup the index before preparing the actual commit, and doing it. Signed-off-by: Martin Langhoff Signed-off-by: Junio C Hamano

cvsimport: keep one index per branch during import

2006-06-17T05:45:11Z

With this patch we have a speedup and much lower IO when importing trees with many branches. Instead of forcing index re-population for each branch switch, we keep many index files around, one per branch. Signed-off-by: Martin Langhoff Signed-off-by: Junio C Hamano

cvsimport: complete the cvsps run before starting the import

2006-06-17T05:45:11Z

We now capture the output of cvsps to a tempfile, and then read it in. cvsps 2.1 works quite a bit "in memory", and only prints its patchset info once it has finished talking with cvs, but apparently retaining all that memory allocation. With this patch, cvsps is finished and reaped before cvsimport start working (and growing). So the footprint of the whole process is much lower. Signed-off-by: Martin Langhoff Signed-off-by: Junio C Hamano

cvsimport: ignore CVSPS_NO_BRANCH and impossible branches

2006-06-17T05:45:11Z

cvsps output often contains references to CVSPS_NO_BRANCH, commits that it could not trace to a branch. Ignore that branch. Additionally, cvsps will sometimes draw circular relationships between branches -- where two branches are recorded as opening from the other. In those cases, and where the ancestor branch hasn't been seen, ignore it. Signed-off-by: Martin Langhoff Signed-off-by: Junio C Hamano

cvsimport: avoid "use" with :tag

2006-05-25T07:18:42Z

Avoid "use POSIX qw(strftime dup2 :errno_h)"; it was reported that a Perl installations on Mandrake 9.1 did not like it, even though it understood "use POSIX qw(:errno_h)". Funny. Signed-off-by: Junio C Hamano

cvsimport: set up commit environment in perl instead of using env

2006-05-23T23:43:12Z

Signed-off-by: Junio C Hamano