Re: cvsimport: rewritten in Perl

2005-07-06 Thread Wolfgang Denk
In message <[EMAIL PROTECTED]> Linus Torvalds wrote: > > If you make it print out its and then pause, you can use > > ls -l /proc//fd/ > > to get an idea of what the files may be. Looks like the new perl version > is leaking file descriptors.. It does. In case it's still of interest:

Re: cvsimport: rewritten in Perl

2005-07-06 Thread Matthias Urlichs
Hi, Sven Verdoolaege: > > to get an idea of what the files may be. Looks like the new perl version > > is leaking file descriptors.. > > > > Matthias? > > That was my mistake, actually. > Thanks for spotting this. > Ouch. For me, the main danger of lots of Python programming is that I tend not

Re: cvsimport: rewritten in Perl

2005-07-05 Thread Sven Verdoolaege
On Tue, Jul 05, 2005 at 07:41:30PM -0700, Linus Torvalds wrote: > If you make it print out its and then pause, you can use > > ls -l /proc//fd/ > > to get an idea of what the files may be. Looks like the new perl version > is leaking file descriptors.. > > Matthias? That was my mistake

Re: cvsimport: rewritten in Perl

2005-07-05 Thread Linus Torvalds
On Wed, 6 Jul 2005, Wolfgang Denk wrote: > > I tested this on the U-Boot CVS repository (available at > http://cvs.sourceforge.net/cvstarballs/u-boot-cvsroot.tar.bz2). > > Committing initial tree 2183da96d548c02bd4f99b05e673395f81212878 > Cannot create object: Too many open files If you make it

Re: cvsimport: rewritten in Perl

2005-07-05 Thread Wolfgang Denk
In message <[EMAIL PROTECTED]> you wrote: > > Linus: Please grab http://netz.smurf.noris.de/git/git.git/#cvs2git. I tested this on the U-Boot CVS repository (available at http://cvs.sourceforge.net/cvstarballs/u-boot-cvsroot.tar.bz2). I got: -> ~/test/git-cvsimport-script -d ~/git/u-boot-SF/u-b

Re: [PATCH] cvsimport: rewritten in Perl

2005-07-04 Thread Sven Verdoolaege
On Mon, Jul 04, 2005 at 04:36:37PM +0200, Matthias Urlichs wrote: > Ideally, I'd prefer to recycle standard CVS options as much as possible, > but given that the confusion is already there (worse: cvs' -z wants an > argument (compression level), cvsps' -Z doesn't) that may not actually > make sens

Re: [PATCH] cvsimport: rewritten in Perl

2005-07-04 Thread Matthias Urlichs
Hi, Sven Verdoolaege: > > Because my code doesn't support compressed cvs connections: > > a -z that doesn't work except for the rlog part would be a lie. > > I was talking about the cvsps '-z' option (see current git-cvsimport-script). > Are you saying you want to reserve that option to signify c

Re: [PATCH] cvsimport: rewritten in Perl

2005-07-04 Thread Sven Verdoolaege
On Mon, Jul 04, 2005 at 03:53:27PM +0200, Matthias Urlichs wrote: > Hi, > > Sven Verdoolaege: > > Why not an explicit '-z' option as in the current git-cvsimport-script ? > > Because my code doesn't support compressed cvs connections: > a -z that doesn't work except for the rlog part would be a l

Re: [PATCH] cvsimport: rewritten in Perl

2005-07-04 Thread Matthias Urlichs
Hi, Sven Verdoolaege: > Why not an explicit '-z' option as in the current git-cvsimport-script ? Because my code doesn't support compressed cvs connections: a -z that doesn't work except for the rlog part would be a lie. Feel free to add that code. ;-) -- Matthias Urlichs | {M:U} IT Design

Re: [PATCH] cvsimport: rewritten in Perl

2005-07-04 Thread Sven Verdoolaege
On Thu, Jun 30, 2005 at 11:00:23PM +0200, Matthias Urlichs wrote: > Sven Verdoolaege: > > It would also be nice if the user could pass extra options > > to cvsps (notably '-z'). > > > Ditto. ;-) Why not an explicit '-z' option as in the current git-cvsimport-script ? skimo - To unsubscribe from

Re: [PATCH] cvsimport: rewritten in Perl

2005-07-04 Thread Sven Verdoolaege
Hi, On Mon, Jul 04, 2005 at 03:49:48AM +0200, Matthias Urlichs wrote: > Sven Verdoolaege: > > Note how the patchsets with the same date have somehow > > been reversed. Any ideas ? > > > No. I process the lines from cvsps in the order I get them... > Ok, I reran cvs2git and now I get the same o