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

[PATCH] sha1_file.c;prepare_packed_git_one() - fix DIR leak

2005-07-05 Thread Junio C Hamano
The function calls opendir() without a matching closedir(). Signed-off-by: Junio C Hamano <[EMAIL PROTECTED]> --- sha1_file.c |1 + 1 files changed, 1 insertions(+), 0 deletions(-) 7a47797264a0be7955f3422a6e79e3311ab2775f diff --git a/sha1_file.c b/sha1_file.c --- a/sha1_file.c +++ b/sha1_f

Re: git-update-cache:

2005-07-05 Thread Sam Ravnborg
On Wed, Jul 06, 2005 at 12:42:20AM +0200, Sam Ravnborg wrote: > > > > I receive the following error: > > git-update-cache: symbol lookup error: git-update-cache: undefined > > symbol: deflateBound > > > > > open("/usr/lib/libz.so.1", O_RDONLY)= 3 > > This is the reason. > > For a str

Re: Merging and parents

2005-07-05 Thread Linus Torvalds
On Tue, 5 Jul 2005, Matthias Urlichs wrote: > > I just had this ugly merge situation: > > M > |\ > | \ > A B > |\/| > |/\| > C D > | / > |/ > E > > Suppose both the EC and the ED branch add files (not with conflicting > filenames!) which then get modified somewhere between C/D and M

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: expensive local git clone

2005-07-05 Thread Linus Torvalds
On Mon, 4 Jul 2005, Linus Torvalds wrote: > > I'm making something based on pack-files. Ok, as of a few minutes ago there's now a pack-based "git clone" there. Of course, it hasn't percolated out to the mirrors yet, so you'll not see it for a while unless you have a master.kernel.org account

Re: zlib experts, please..

2005-07-05 Thread Linus Torvalds
On Tue, 5 Jul 2005, Linus Torvalds wrote: > > But what I care about is that when git-unpack-objects sees that it wants a > zero-byte object, and asks zlib to unpack it, zlib will not actually use > the bytes it wrote - it will just say "oh, you wanted zero bytes, here's > zero bytes". Which m

zlib experts, please..

2005-07-05 Thread Linus Torvalds
I just hit an interesting pack failure because of how git (mis-)uses zlib, and I'm wondering what to do about it. In particular, the "git-unpack-objects" code gets a data stream, and only knows the _unpacked_ size of each object, because writing packed size is extremely inconvenient in many ways

Re: [PATCH] Restore expected list order for --merge-order switch

2005-07-05 Thread Linus Torvalds
On Wed, 6 Jul 2005, Jon Seymour wrote: > > Is there some reason why this didn't get applied? I think it's horribly horribly ugly. I really don't see the point of argument ordering mattering, and I think the test is broken. Convince me otherwise. Linus - To unsubscribe from t

Re: [PATCH] Restore expected list order for --merge-order switch

2005-07-05 Thread Jon Seymour
G'day Linus, Is there some reason why this didn't get applied? jon. On 7/5/05, Jon Seymour <[EMAIL PROTECTED]> wrote: > > A recent change to rev-list altered the order in which start points > are presented to the merge-order sort algorithm. This caused > breaks in the t/t6001 unit tests. > > T

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: git-update-cache:

2005-07-05 Thread Sam Ravnborg
> > I receive the following error: > git-update-cache: symbol lookup error: git-update-cache: undefined > symbol: deflateBound > > open("/usr/lib/libz.so.1", O_RDONLY)= 3 This is the reason. For a strange reason when git-update-chache was compiled is was linked dynamically towards

git-update-cache:

2005-07-05 Thread Sam Ravnborg
While trying to execute: git-update-cache -- drivers/Kconfig drivers/net/Kconfig net/Kconfig I receive the following error: git-update-cache: symbol lookup error: git-update-cache: undefined symbol: deflateBound I have fetched a clean tree from kernel.org, and cloned it using cg-clone. Modificat

[PATCH] Add script for patch submission via e-mail.

2005-07-05 Thread Junio C Hamano
[PATCH] git-format-patch: Prepare patches for e-mail submission. This is the script I use to prepare patches for e-mail submission. Signed-off-by: Junio C Hamano <[EMAIL PROTECTED]> --- *** The latest incarnation. Uses --merge-order while generating *** the list of patches to number them better

[PATCH] Short-cut git-fetch-pack while cloning locally.

2005-07-05 Thread Junio C Hamano
> "LT" == Linus Torvalds <[EMAIL PROTECTED]> writes: LT> Well, I'm not working on it, but tested patches... Prodded by the hint... The git-fetch-pack command used internally by git-fetch-script which in turn is used by git-clone-script refuses to run if there is no common head.

Re: Tags

2005-07-05 Thread Junio C Hamano
> "LT" == Linus Torvalds <[EMAIL PROTECTED]> writes: LT> On Tue, 5 Jul 2005, Eric W. Biederman wrote: >> >> True but if you can you will get multiple tags with the >> same suggested name. So you need so way to find the one you >> care about. LT> I do agree that it would make sense to have a

Re: Tags

2005-07-05 Thread Linus Torvalds
On Tue, 5 Jul 2005, Eric W. Biederman wrote: > > True but if you can you will get multiple tags with the > same suggested name. So you need so way to find the one you > care about. I do agree that it would make sense to have a "tagger" field with the same semantics as the "committer" in a com

Re: Tags

2005-07-05 Thread Eric W. Biederman
Daniel Barkalow <[EMAIL PROTECTED]> writes: > On Tue, 5 Jul 2005, Eric W. Biederman wrote: > >> Could you include the person who generated the tag and the time the >> tag was generated in the tag object? >> >> For a tag like "latest" it would help quite a bit if you could actually >> find out whi

Merging and parents

2005-07-05 Thread Matthias Urlichs
I just had this ugly merge situation: M |\ | \ A B |\/| |/\| C D | / |/ E Suppose both the EC and the ED branch add files (not with conflicting filenames!) which then get modified somewhere between C/D and M. No matter which node gets picked as the parent, some files will end up as "

Re: [PATCH] Add script for patch submission via e-mail.

2005-07-05 Thread Jon Seymour
On 7/6/05, Junio C Hamano <[EMAIL PROTECTED]> wrote: > > "PB" == Petr Baudis <[EMAIL PROTECTED]> writes: > > PB> Any reason why this was not applied? It appears kind of cool. > > FYI, the one in <[EMAIL PROTECTED]> is > newer than what you quoted. > > One thing _I_ am unhappy about what it d

Re: Tags

2005-07-05 Thread Daniel Barkalow
On Tue, 5 Jul 2005, Eric W. Biederman wrote: > Could you include the person who generated the tag and the time the > tag was generated in the tag object? > > For a tag like "latest" it would help quite a bit if you could actually > find out which was the latest version of it :) Actually, what yo

Re: [PATCH] Add script for patch submission via e-mail.

2005-07-05 Thread Junio C Hamano
> "PB" == Petr Baudis <[EMAIL PROTECTED]> writes: PB> Any reason why this was not applied? It appears kind of cool. FYI, the one in <[EMAIL PROTECTED]> is newer than what you quoted. One thing _I_ am unhappy about what it does is that it does not try to be intelligent about merges (I haven't

Re: Last mile for 1.0 again

2005-07-05 Thread Marco Costalba
--- Junio C Hamano <[EMAIL PROTECTED]> wrote: > > "DB" == Daniel Barkalow <[EMAIL PROTECTED]> writes: > > DB> [perl script] > > >> How does this work, and what do we do about merges? > Checking diffs of all the parents can be computational expensive. I'am developing a different alghori

Re: Tags

2005-07-05 Thread Eric W. Biederman
Linus Torvalds <[EMAIL PROTECTED]> writes: > (And you might also change tag contents occasionally. One reason might be > a bug and you decide to re-tag something else. But a more common reason > might be because you want to have tags like "latest" that don't actually > update with development, but

Re: [PATCH] Add script for patch submission via e-mail.

2005-07-05 Thread Jon Seymour
On 7/5/05, Petr Baudis <[EMAIL PROTECTED]> wrote: > > Any reason why this was not applied? It appears kind of cool. Well, I > will probably take it and extend cg-mkpatch with it so I don't need it > in Git, but I'm so altruistic and want to bring at least a bit of light > to the gloomy dark world

Re: [PATCH] Add script for patch submission via e-mail.

2005-07-05 Thread Petr Baudis
Dear diary, on Sat, Jun 11, 2005 at 03:32:30AM CEST, I got a letter where Junio C Hamano <[EMAIL PROTECTED]> told me that... > This git-format-patch-script is what I use to prepare patches > for e-mail submission. > > Typical usage is: > > $ git-format-patch-script -B -C --find-copies-harder HEAD

Re: Proposed script: cg-info

2005-07-05 Thread Petr Baudis
Dear diary, on Tue, Jul 05, 2005 at 07:46:12AM CEST, I got a letter where Pavel Roskin <[EMAIL PROTECTED]> told me that... > Hello! Hello, > It would be useful to have a script to display current status of the > repository - branches, current branch and merge status. Maybe I'm > missing somethin

Re: [PATCH] Fixlet for argument parsing in rev-list.

2005-07-05 Thread Jon Seymour
I ack the patch, though --show-breaks could use the same treatment. jon. On 7/5/05, Junio C Hamano <[EMAIL PROTECTED]> wrote: > The --merge-order flag does not take parameter so there is no > point doing strncmp with the length. > > Signed-off-by: Junio C Hamano <[EMAIL PROTECTED]> > --- > > r

[PATCH] Fixlet for argument parsing in rev-list.

2005-07-05 Thread Junio C Hamano
The --merge-order flag does not take parameter so there is no point doing strncmp with the length. Signed-off-by: Junio C Hamano <[EMAIL PROTECTED]> --- rev-list.c |2 +- 1 file changed, 1 insertions(+), 1 deletions(-) diff --git a/rev-list.c b/rev-list.c --- a/rev-list.c +++ b/rev-list.c @