[PATCH] Deb packaging needs two more configuration files

2005-07-22 Thread Ryan Anderson
The deb package building needs these two new files to work correctly. debian/compat sets the rules under which the debhelper scripts (dh_*) operate. debian/git-core.install tells dh_install what files to install in each package that is generated. There is only one package being generated, so

[PATCH 0/2] apply.c: a fix and an enhancement

2005-07-22 Thread Junio C Hamano
In mid April, Ingo announced availability of his conversion from CVS to a flat patchset format: From: Ingo Molnar [EMAIL PROTECTED] Subject: full kernel history, in patchset format Message-ID: [EMAIL PROTECTED] the history data starts at 2.4.0 and ends at 2.6.12-rc2. I've

[PATCH 1/2] apply.c: handle incomplete lines correctly.

2005-07-22 Thread Junio C Hamano
The parsing code had a bug that failed to recognize an incomplete line at the end of a fragment, and the fragment application code had a comparison bug to recognize such. Fix them to handle incomplete lines correctly. Add a test script for patches with various combinations of complete and

[PATCH 2/2] apply.c: --exclude=fnmatch-pattern option.

2005-07-22 Thread Junio C Hamano
Adds --exclude=pattern option to the git-apply command. This was useful while reimporting the BKCVS patchset dump of the Linux kernel, starting at 2.4.0 and ending at 2.6.12-rc2 Ingo announced some time ago to exclude BitKeeper directory. Signed-off-by: Junio C Hamano [EMAIL PROTECTED] ---

Re: [PATCH 1/1] Tell vim the textwidth is 75.

2005-07-22 Thread Sam Ravnborg
I would use a neutral commit template, only that it should have a neutral prefix as well for the lines to be removed (neither STG nor CG but GIT maybe). The $GIT_DIR/commit-template is fine as a file name. How about $GIT_DIR/commit-template-`basename $EDITOR` Then we could have different

Re: [PATCH 0/2] apply.c: a fix and an enhancement

2005-07-22 Thread Ryan Anderson
On Fri, Jul 22, 2005 at 09:56:19AM -0700, Junio C Hamano wrote: Now if we had a mechanism to graft a later history which starts at 2.6.12-rc2 on top of this earlier history leading up to it,... ;-) We do - it's not even very hard, we just end up with 2 commits for every change/merge that's

qgit-0.8

2005-07-22 Thread Marco Costalba
Hi, here is qgit-0.8: http://prdownloads.sourceforge.net/qgit/qgit-0.8.tar.bz2?download This release shows a big GUI rewrite with added menus, buttons, help, settings page, etc. Some new features: - Possibility to view diffs against current checked-out tree, i.e GUI interface to

Re: [PATCH 0/2] apply.c: a fix and an enhancement

2005-07-22 Thread Junio C Hamano
Ryan Anderson [EMAIL PROTECTED] writes: On Fri, Jul 22, 2005 at 09:56:19AM -0700, Junio C Hamano wrote: Now if we had a mechanism to graft a later history which starts at 2.6.12-rc2 on top of this earlier history leading up to it,... ;-) We do - it's not even very hard, we just end up with

Re: [PATCH 0/2] apply.c: a fix and an enhancement

2005-07-22 Thread A Large Angry SCM
Junio C Hamano wrote: Ryan Anderson [EMAIL PROTECTED] writes: On Fri, Jul 22, 2005 at 09:56:19AM -0700, Junio C Hamano wrote: Now if we had a mechanism to graft a later history which starts at 2.6.12-rc2 on top of this earlier history leading up to it,... ;-) We do - it's not even very

Re: [PATCH 1/1] Tell vim the textwidth is 75.

2005-07-22 Thread Junio C Hamano
Sam Ravnborg [EMAIL PROTECTED] writes: I would use a neutral commit template, only that it should have a neutral prefix as well for the lines to be removed (neither STG nor CG but GIT maybe). The $GIT_DIR/commit-template is fine as a file name. How about $GIT_DIR/commit-template-`basename

Re: [PATCH 1/1] Tell vim the textwidth is 75.

2005-07-22 Thread Petr Baudis
Dear diary, on Fri, Jul 22, 2005 at 12:37:05PM CEST, I got a letter where Catalin Marinas [EMAIL PROTECTED] told me that... Cogito seems to use $GIT_DIR/commit-template for that purpose. Can't users put that vim: hint there, and if StGIT does not use a commit template, patch it to use the

Re: [PATCH 0/2] apply.c: a fix and an enhancement

2005-07-22 Thread Linus Torvalds
On Fri, 22 Jul 2005, A Large Angry SCM wrote: To do it without the history rewrite, create an alternate_history directory under .git with it's own objects tree. And populate that object tree with alternative content for the objects in the normal trees. Then teach the things the

Re: [PATCH 1/1] Tell vim the textwidth is 75.

2005-07-22 Thread Catalin Marinas
On Fri, 2005-07-22 at 19:24 +, Sam Ravnborg wrote: I would use a neutral commit template, only that it should have a neutral prefix as well for the lines to be removed (neither STG nor CG but GIT maybe). The $GIT_DIR/commit-template is fine as a file name. How about

Re: Should cg-mkpatch output be usable with cg-patch?

2005-07-22 Thread Petr Baudis
Dear diary, on Thu, Jul 21, 2005 at 05:57:49AM CEST, I got a letter where Junio C Hamano [EMAIL PROTECTED] told me that... I only briefly looked at cg-patch, but I suspect that it can lose 90% lines of its code by just using git-apply --index. Can git-apply already deal with fuzzy patches? --

Re: Should cg-mkpatch output be usable with cg-patch?

2005-07-22 Thread Petr Baudis
Dear diary, on Thu, Jul 21, 2005 at 01:49:04AM CEST, I got a letter where Wolfgang Denk [EMAIL PROTECTED] told me that... I wander what I should do with cg-mkpatch generated output; I had the impression that this should be usable with cg-patch, but these are incompatible with each other.

Re: [PATCH 1/1] Tell vim the textwidth is 75.

2005-07-22 Thread Junio C Hamano
Wonderful start. Later on, Porcelains could agree on what @TOKEN@ are generally available, and even start using a common script to pre-fill the templates, like: $ git-fill-template-script template output-file var=val var=val... In your example, I see AUTHOR_NAME, AUTHOR_EMAIL, and AUTHOR_DATE

Re: [PATCH 0/2] apply.c: a fix and an enhancement

2005-07-22 Thread Junio C Hamano
Linus Torvalds [EMAIL PROTECTED] writes: Just teach parse_commit() to look at a .git/fake_parents file, and insert fake extra parents for commits that way - you can graft any tree on top of any other tree that way, and it's probably a nice idea for testing things out. Nicely put, thanks.

Re: [PATCH 2/2] GIT: Listen on IPv6 as well, if available.

2005-07-22 Thread Petr Baudis
Dear diary, on Thu, Jul 21, 2005 at 03:10:49PM CEST, I got a letter where YOSHIFUJI Hideaki / ?$B5HF#1QL@ [EMAIL PROTECTED] told me that... Hello. Hello from an IPv6 fan, Listen on IPv6 as well, if available. Signed-off-by: Hideaki YOSHIFUJI [EMAIL PROTECTED] diff --git a/daemon.c

Re: [PATCH 1/2] GIT: Try all addresses for given remote name

2005-07-22 Thread YOSHIFUJI Hideaki / 吉藤英明
In article [EMAIL PROTECTED] (at Fri, 22 Jul 2005 23:09:13 +0200), Petr Baudis [EMAIL PROTECTED] says: -} +#define STR_(s)# s +#define STR(s) STR_(s) Uh-huh? Why two macros? Well, why any macros at all? : + char *colon, *end; + char *port = STR(DEFAULT_GIT_PORT); +

Re: [PATCH 1/1] Tell vim the textwidth is 75.

2005-07-22 Thread Petr Baudis
Dear diary, on Fri, Jul 22, 2005 at 11:16:51PM CEST, I got a letter where Junio C Hamano [EMAIL PROTECTED] told me that... Wonderful start. Later on, Porcelains could agree on what @TOKEN@ are generally available, and even start using a common script to pre-fill the templates, like: $

Re: [PATCH 2/2] GIT: Listen on IPv6 as well, if available.

2005-07-22 Thread YOSHIFUJI Hideaki / 吉藤英明
In article [EMAIL PROTECTED] (at Fri, 22 Jul 2005 23:21:51 +0200), Petr Baudis [EMAIL PROTECTED] says: this whole getaddrinfo() magic looks horribly complicated. What's wrong on just adding a similar code (or factoring it out to a function) for IPv6 as there is for IPv4, just s/INET/INET6/?

Re: [PATCH 1/2] GIT: Try all addresses for given remote name

2005-07-22 Thread Petr Baudis
Dear diary, on Fri, Jul 22, 2005 at 11:26:29PM CEST, I got a letter where YOSHIFUJI Hideaki / ?$B5HF#1QL@ [EMAIL PROTECTED] told me that... In article [EMAIL PROTECTED] (at Fri, 22 Jul 2005 23:09:13 +0200), Petr Baudis [EMAIL PROTECTED] says: -} +#define STR_(s) # s +#define STR(s)

Re: [PATCH 1/1] Tell vim the textwidth is 75.

2005-07-22 Thread Catalin Marinas
On Fri, 2005-07-22 at 13:39 -0700, Junio C Hamano wrote: I would like to see Porcelains stay compatible when the do not have to differ. The commit template [*2*] is one example of such. For StGIT it is not a problem to use any commit template with any prefix. It doesn't generate extra

Re: [PATCH 2/2] GIT: Listen on IPv6 as well, if available.

2005-07-22 Thread Petr Baudis
Dear diary, on Fri, Jul 22, 2005 at 11:35:17PM CEST, I got a letter where YOSHIFUJI Hideaki / ?$B5HF#1QL@ [EMAIL PROTECTED] told me that... In article [EMAIL PROTECTED] (at Fri, 22 Jul 2005 23:21:51 +0200), Petr Baudis [EMAIL PROTECTED] says: this whole getaddrinfo() magic looks horribly

Re: [PATCH] Make debian/rules executable, and correct the spelling of rsync in debian/control

2005-07-22 Thread Petr Baudis
Dear diary, on Wed, Jul 20, 2005 at 03:17:47AM CEST, I got a letter where Ryan Anderson [EMAIL PROTECTED] told me that... Make debian/rules executable, and correct the spelling of rsync in debian/control Any progress with applying of this one? Linus, do you want me to ack even trivial patches

Re: [PATCH] Make debian/rules executable, and correct the spelling of rsync in debian/control

2005-07-22 Thread Linus Torvalds
On Sat, 23 Jul 2005, Petr Baudis wrote: Dear diary, on Wed, Jul 20, 2005 at 03:17:47AM CEST, I got a letter where Ryan Anderson [EMAIL PROTECTED] told me that... Make debian/rules executable, and correct the spelling of rsync in debian/control Any progress with applying of this one?

Re: [PATCH 1/2] GIT: Try all addresses for given remote name

2005-07-22 Thread Sam Ravnborg
On Fri, Jul 22, 2005 at 05:26:29PM -0400, YOSHIFUJI Hideaki / ?$B5HF#1QL@ wrote: In article [EMAIL PROTECTED] (at Fri, 22 Jul 2005 23:09:13 +0200), Petr Baudis [EMAIL PROTECTED] says: -} +#define STR_(s) # s +#define STR(s) STR_(s) Uh-huh? Why two macros? Well, why any macros

Re: [PATCH 0/2] apply.c: a fix and an enhancement

2005-07-22 Thread Junio C Hamano
Linus Torvalds [EMAIL PROTECTED] writes: I'd _really_ prefer to not have any preferences or other metadata files under version control within that same project. Don't you think that would be a per-project decision? Is it acceptable if I make sure that .gitinfo/* is _optional_ and things do

[PATCH 0/9] applymbox updates and assorted trivial patches.

2005-07-22 Thread Junio C Hamano
(Resend after you said you prefer people resending). I was experimenting with applypatch and applymbox to see how it feels like being Linus ;-) and also I saw some trivially correct patches on the list floating unmerged during KS/OLS. Since I rebase my repostitory to fix up the history often, I

[PATCH] tools/applymbox: allow manual fixing and continuing after a failure.

2005-07-22 Thread Junio C Hamano
With -c .dotest/0002 flag, the applymbox command can be told to use existing .dotest/patch file after hand-fixing the patch conflicts for the second patch in the mailbox, and continue on from that message, skipping the already applied first patch in .dotest/0001. Signed-off-by: Junio C Hamano

[PATCH] git-format-patch-script and mailinfo updates.

2005-07-22 Thread Junio C Hamano
- avoid duplicating [PATCH] in the commit message body if the original commit has it already (happens for commits done from mails via applymbox). - check if the commit author is different from the one who is running the script, and emit an appropriate From: and Date: lines to the

[PATCH] format-patch: --mbox and --check.

2005-07-22 Thread Junio C Hamano
Add --mbox option to export patches in a format resembling UNIX mbox, so that later they can be concatenated and fed to applymbox. Add --check to look for lines that introduce bogus whitespaces. Signed-off-by: Junio C Hamano [EMAIL PROTECTED] --- git-format-patch-script | 49

Re: [PATCH 1/1] Tell vim the textwidth is 75.

2005-07-22 Thread Junio C Hamano
Catalin Marinas [EMAIL PROTECTED] writes: Would such a template only have 'GIT:' prefixed lines? I usually put another line like 'Signed-off-by:', for convenience. The problem with StGIT appears when one wants to re-edit the patch description (stg refresh -e), in which case the existing

Re: [PATCH 1/1] Tell vim the textwidth is 75.

2005-07-22 Thread Junio C Hamano
Petr Baudis [EMAIL PROTECTED] writes: Cogito shows '[NMD] filename' in place of @[EMAIL PROTECTED] Sounds sensible. Does it parse it to limit the files to be committed? This brings me to another subject, M and N are pretty hard to distinguish visually without close inspection of the output.

Re: [PATCH 0/2] apply.c: a fix and an enhancement

2005-07-22 Thread Linus Torvalds
On Fri, 22 Jul 2005, Junio C Hamano wrote: Linus Torvalds [EMAIL PROTECTED] writes: I'd _really_ prefer to not have any preferences or other metadata files under version control within that same project. Don't you think that would be a per-project decision? Is it acceptable if I make

Re: [PATCH 0/2] apply.c: a fix and an enhancement

2005-07-22 Thread Petr Baudis
Dear diary, on Fri, Jul 22, 2005 at 11:53:41PM CEST, I got a letter where Linus Torvalds [EMAIL PROTECTED] told me that... On Fri, 22 Jul 2005, Junio C Hamano wrote: And the file would obviously be per-project, so according to Pasky's suggestion that would be .gitinfo/fake_parents ;-).

Re: [PATCH 0/2] apply.c: a fix and an enhancement

2005-07-22 Thread Petr Baudis
Dear diary, on Sat, Jul 23, 2005 at 01:26:07AM CEST, I got a letter where Linus Torvalds [EMAIL PROTECTED] told me that... On Fri, 22 Jul 2005, Junio C Hamano wrote: Linus Torvalds [EMAIL PROTECTED] writes: I'd _really_ prefer to not have any preferences or other metadata files under

Re: [PATCH 0/2] apply.c: a fix and an enhancement

2005-07-22 Thread Petr Baudis
Dear diary, on Sat, Jul 23, 2005 at 01:50:09AM CEST, I got a letter where Linus Torvalds [EMAIL PROTECTED] told me that... On Sat, 23 Jul 2005, Petr Baudis wrote: Yes, but this stuff is not for personal preferences. It is for project-wide preferences and policies, which can be still

Re: [PATCH 0/2] apply.c: a fix and an enhancement

2005-07-22 Thread Junio C Hamano
Linus Torvalds [EMAIL PROTECTED] writes: In other words, if it's per-project, then that implies that every single developer has to agree on the same thing. Which just not possible - it makes no sense. I agree 75%. See the bottom for the rest 25%. In contrast, if you have a separate local

[PATCH 6/9] Cleanup: git-verify-tag-script

2005-07-22 Thread Junio C Hamano
From: Eric W. Biederman [EMAIL PROTECTED] Date: 1121545120 -0600 - Use git-rev-parse to allow sha1 tags references - When the tag does not verify set an appropriate exit status - Use git-sh-setup-script to verify the .git directory Signed-off-by: Eric W. Biederman [EMAIL PROTECTED]

[PATCH 9/9] Document curl requirements.

2005-07-22 Thread Junio C Hamano
Not just libcurl, but now we require curl executable as well. Signed-off-by: Junio C Hamano [EMAIL PROTECTED] --- INSTALL |6 -- 1 files changed, 4 insertions(+), 2 deletions(-) b300e6d28fc160ddc6685bde96b6bc9a710e01c3 diff --git a/INSTALL b/INSTALL --- a/INSTALL +++ b/INSTALL @@ -41,8

[PATCH 4/9] debian/ fixes

2005-07-22 Thread Junio C Hamano
From: Ryan Anderson [EMAIL PROTECTED] Date: 1121822267 -0400 Make debian/rules executable, and correct the spelling of rsync in debian/control. Signed-off-by: Junio C Hamano [EMAIL PROTECTED] --- debian/control |2 +- debian/rules |0 2 files changed, 1 insertions(+), 1 deletions(-)

[PATCH 5/9] Fix a typo in git-unpack-objects documentation.

2005-07-22 Thread Junio C Hamano
From: Jan Veldeman [EMAIL PROTECTED] Date: 1121810349 -0700 Fix a typo in git-unpack-objects documentation. Signed-off-by: Jan Veldeman [EMAIL PROTECTED] Signed-off-by: Junio C Hamano [EMAIL PROTECTED] --- Documentation/git-unpack-objects.txt |2 +- 1 files changed, 1 insertions(+), 1

Various fixes and cleanups.

2005-07-22 Thread Junio C Hamano
I'll be sending these next: [PATCH] Help scripts that use git-rev-parse to grok args with SP/TAB/LF [PATCH] git-branch: avoid getting confused by empty or nonexisting branches. [PATCH] Audit rev-parse users. [PATCH] Install tools with make install-tools. [PATCH] Test framework:

[PATCH] Help scripts that use git-rev-parse to grok args with SP/TAB/LF

2005-07-22 Thread Junio C Hamano
The git-rev-parse command uses LF to separate each argument it parses, so its users at least need to set IFS to LF to be able to handle filenames with embedded SPs and TABs. Some commands, however, can take and do expect arguments with embedded LF, notably, -S (pickaxe) of diff family, so even

[PATCH] git-branch: avoid getting confused by empty or nonexisting branches.

2005-07-22 Thread Junio C Hamano
When not specifying the start point explicitly, we ended up emitting ^0 in addition to the default HEAD. Be careful to see if we have $2 before finding out which commit to base the new branch on. Signed-off-by: Junio C Hamano [EMAIL PROTECTED] --- git-branch-script |8 +++- 1 files

[PATCH] Audit rev-parse users.

2005-07-22 Thread Junio C Hamano
This patch changes rev-parse users that pass a single argument that is supposed to be a rev parameter to use --verify. Signed-off-by: Junio C Hamano [EMAIL PROTECTED] --- git-checkout-script |2 +- git-cherry |8 git-commit-script |2 +- git-rebase-script |

[PATCH] Test framework: prettyprint the failed command.

2005-07-22 Thread Junio C Hamano
The output from a failure case had the test description message and the first line of the actual test script concatenated on the same line, which was ugly. Correct the output routine a bit to make it more readable. Signed-off-by: Junio C Hamano [EMAIL PROTECTED] --- t/test-lib.sh |4 +++-

[PATCH] Install tools with make install-tools.

2005-07-22 Thread Junio C Hamano
Match the main Makefile by separating COPTS from CFLAGS, defining INSTALL, prefix, and bin. Add a new target 'install-tools' to the main Makefile to install them. Signed-off-by: Junio C Hamano [EMAIL PROTECTED] --- Makefile |4 tools/Makefile | 13 +++-- 2 files

[PATCH] Deb packages should include the binaries (Try 2 - this time it actually applies)

2005-07-22 Thread Junio C Hamano
From: Ryan Anderson [EMAIL PROTECTED] Date: 1122011756 -0400 The Deb packages were missing a dependency on build install from the binary target - this fixes that, and cleans up some inconsistencies elsewhere in the rulesets. Traditionally, Debian packaging uses a file called build-stamp (or

[PATCH] Deb packaging needs two more configuration files

2005-07-22 Thread Junio C Hamano
From: Ryan Anderson [EMAIL PROTECTED] Date: 1122014167 -0400 The deb package building needs these two new files to work correctly. debian/compat sets the rules under which the debhelper scripts (dh_*) operate. debian/git-core.install tells dh_install what files to install in each package that

[PATCH] GIT: Try all addresses for given remote name

2005-07-22 Thread Junio C Hamano
From: YOSHIFUJI Hideaki [EMAIL PROTECTED] Date: 1121951436 -0400 Try all addresses for given remote name until it succeeds. Also supports IPv6. Signed-off-by: Hideaki YOSHIFUJI [EMAIL PROTECTED] Signed-off-by: Junio C Hamano [EMAIL PROTECTED] --- *** Linus, I am not ACKing Hideaki's other change