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

Catalin Marinas wrote: It seems I inadvertantly kicked off the discussion I wanted to kick off, but I didn't excpect this patch to do so! I prepared a patch adding the following information into git/Documentation to kick off discussion. Obviously Catalin is more likely to be accurate.

[PATCH 1/1] Support configurable SMTP port for stg mail.

The example configuration file makes it appear that the SMTP port is configurable. Make it so. Signed-off-by: Bryan Larsen [EMAIL PROTECTED] --- stgit/commands/mail.py |6 ++ 1 files changed, 6 insertions(+), 0 deletions(-) diff --git a/stgit/commands/mail.py b/stgit/commands/mail.py

[PATCH 1/1] Add authentication capability to the mail command.

Add authentication capability to the mail command. Added 'smtppassword' and 'smtpuser' to config file and --smtp-user, --smtp-password to command line. Signed-off-by: Bryan Larsen [EMAIL PROTECTED] --- stgit/commands/mail.py | 34 ++ 1 files changed, 30

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

When invoking EDITOR, add some metadata to tell vim the textwidth is 75. Signed-off-by: Bryan Larsen [EMAIL PROTECTED] --- stgit/stack.py |2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/stgit/stack.py b/stgit/stack.py --- a/stgit/stack.py +++ b/stgit/stack.py @@ -66,6

OLS?

Hi gitsters, I'm currently living in Ottawa, and will be attending OLS. I would like to meet people involved with git. I can offer my house (walking distance from OLS), my BBQ, my beer or my recommendations on good restaurants to facilitate an informal BOF. Bryan - To unsubscribe from

Re: git, porcelain, darcs, and version 1.0

Junio C Hamano wrote: I fully agree that supporting C-level linkage is worthy, and should be one of our longer term goals. Excellent. A similar 1.0 goal would be to document porcelain's use of the .git directory. For instance, stacked git uses .git/patches, .git/patchdescr.tmpl and

[PATCH 0/6] cogito: compatibility with OS X

This is a resend of my previous set of patches. I have updated these patches with Junio's suggestion. I have also added some documentation, a simple README.osx. Once you have applied these patches, could you choose one of these 4 options, Junio? 1) send me the result of make Portfile on next

[PATCH 1/6] cogito: remove use of xargs -r, a non-portable GNU extension

Remove usage of xargs -r, a non-portable gnu extension. Signed-off-by: Bryan Larsen [EMAIL PROTECTED] --- cg-add |6 +++--- cg-init |2 +- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/cg-add b/cg-add --- a/cg-add +++ b/cg-add @@ -25,8 +25,6 @@ USAGE=cg-add [-N] FILE

[PATCH 3/6] cogito: try harder to find gnu stat

Look harder for gnu stat. Cogito has code to use awk if gnu stat is missing. Look harder for gnu stat under alternate names such as gstat and gnustat, avoiding the use of awk if possible. Signed-off-by: Bryan Larsen [EMAIL PROTECTED] --- cg-Xlib | 13 + 1 files changed, 9

[PATCH 4/6] cogito: try harder to find gnu date

Look harder for gnu date, use if available. Signed-off-by: Bryan Larsen [EMAIL PROTECTED] --- cg-Xlib | 12 ++-- 1 files changed, 10 insertions(+), 2 deletions(-) diff --git a/cg-Xlib b/cg-Xlib --- a/cg-Xlib +++ b/cg-Xlib @@ -70,7 +70,7 @@ showdate () { secs=$(($secs

[PATCH 5/6] cogito: remove findutils dependency from Portfile

Gnu findutils (xargs) is no longer required; remove the dependency. Gnu coreutils is still required, but only if awk is not installed. Signed-off-by: Bryan Larsen [EMAIL PROTECTED] --- Portfile.in |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/Portfile.in b

[PATCH 6/6] cogito: update documentation

Update the documentation to add a README.osx and update requirements. Signed-off-by: Bryan Larsen [EMAIL PROTECTED] --- README |2 ++ README.osx | 31 +++ 2 files changed, 33 insertions(+), 0 deletions(-) diff --git a/README b/README --- a/README +++ b

[PATCH 1/6 RESEND] cogito: remove use of xargs -r, a non-portable GNU extension

Remove usage of xargs -r, a non-portable gnu extension. Resent with nasty bug fixed. Signed-off-by: Bryan Larsen [EMAIL PROTECTED] --- cg-add |6 +++--- cg-init |2 +- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/cg-add b/cg-add --- a/cg-add +++ b/cg-add @@ -25,8

[PATCH] cg-commit chokes when given a very large list of files

is cleaner. Signed-off-by: Bryan Larsen [EMAIL PROTECTED] --- cg-commit |6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/cg-commit b/cg-commit --- a/cg-commit +++ b/cg-commit @@ -289,9 +289,9 @@ precommit_update () { eval [EMAIL PROTECTED]\$fname\ done

Re: [PATCH] cg-commit chokes when given a very large list of files

This patch is broken. The original patch still works. Bryan Bryan Larsen wrote: cg-commit currently chokes when passed a very large list of files. Fix it. This patch depends on your filenames not containing line feeds. No big deal, other parts of cogito break on filenames containing line

Re: Barebone Porcelain. Where to stop?

Junio C Hamano wrote: I have been somewhat disturbed and confused by the fact that the line between what Porcelain like Cogito does and what we ship as part of core GIT is getting more and more blurred. This was especially so while I was working on the $GIT_DIR/branches/ patch. I have also

git, porcelain, darcs, and version 1.0

Juliusz Chroboczek wrote: There are three ways to do that: (1) require that the users put a suitable libgit.a in /usr/local/lib before building Darcs, and distribute a tarball of Git from darcs.net; I was under the impression that the stablest interface to git was the command

[PATCH] git: remove usage of xargs -r, a non-portable gnu extension

Remove usage of xargs -r, a non-portable gnu extension. Signed-off-by: Bryan Larsen [EMAIL PROTECTED] --- git-prune-script |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/git-prune-script b/git-prune-script --- a/git-prune-script +++ b/git-prune-script @@ -20,6 +20,6

[PATCH 0/5] cogito: compatibility with OS X

Since Linus and Junio were unhappy with my previous set of patches to make git and cogito compatible with OS X and other BSD unices, I have redone my patches to cogito along a similar line. There is still one outstanding incompatibility: cogito makes use of cp -u. I have this message from

[PATCH 2/5] cogito: remove use of cp -a, a non-portable GNU extension

Remove usage of cp -a, a non-portable gnu extension. Signed-off-by: Bryan Larsen [EMAIL PROTECTED] --- cg-pull |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/cg-pull b/cg-pull --- a/cg-pull +++ b/cg-pull @@ -217,7 +217,7 @@ fetch_local () { [ $1 = -i ] shift

[PATCH 5/5] cogito: remove findutils dependency from Portfile

Gnu findutils (xargs) is no longer required; remove the dependency. Gnu coreutils is still required, but only if awk is not installed. Signed-off-by: Bryan Larsen [EMAIL PROTECTED] --- Portfile.in |3 +-- 1 files changed, 1 insertions(+), 2 deletions(-) diff --git a/Portfile.in b

[PATCH 3/5] cogito: try harder to find gnu stat

Look harder for gnu stat. Cogito has code to use awk if gnu stat is missing. Look harder for gnu stat under alternate names such as gstat and gnustat, avoiding the use of awk if possible. Signed-off-by: Bryan Larsen [EMAIL PROTECTED] --- cg-Xlib |6 +++--- 1 files changed, 3 insertions

[PATCH] stgit: allow spaces in filenames (second try)

The current version of stgit does not allow whitespace in filenames. This patch fixes that. It also speeds up operations on large filesets considerably. Signed-off-by: Bryan Larsen [EMAIL PROTECTED] --- stgit/commands/export.py |4 - stgit/git.py | 203

[PATCH] stgit: export without --range does not export the last patch

The top patch is never exported. Signed-off-by: Bryan Larsen [EMAIL PROTECTED] --- stgit/commands/export.py |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/stgit/commands/export.py b/stgit/commands/export.py --- a/stgit/commands/export.py +++ b/stgit/commands/export.py

Re: [PATCH] stgit: allow spaces in filenames

Does it make that big difference if the commands are invoked via the shell? I haven't run any tests. It wasn't for the time difference that I bypassed the shell, it was to support spaces and other strange characters in parameters. It's easy to use spawnvp than it is to escape the

[PATCH] stgit: new --message does not work

Stacked GIT: Using the --message option with the new command fails. Fix it. Signed-off-by: Bryan Larsen [EMAIL PROTECTED] --- stgit/stack.py |2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/stgit/stack.py b/stgit/stack.py --- a/stgit/stack.py +++ b/stgit/stack.py

Re: [PATCH] stgit: allow spaces in filenames

Catalin Marinas wrote: An additional thing, can you please convert all the tabs to spaces? That's a better convention for a language like Python where you delimit blocks by indentation. I would have hoped that emacs py-mode would do the right thing. Anybody know how to make it do what Catalin

Re: Stacked GIT 0.4

Catalin Marinas wrote: Stacked GIT 0.4 release is available from http://procode.org/stgit/ Very nice. Here's my wishlist. Hopefully I'll be able to dig in and help out. import: the complement to export template files for the series output of export, to put it into a format that

[PATCH] cg-commit chokes when given a very large list of files

cg-commit currently chokes when passed a very large list of files. Fix it. This patch depends on your filenames not containing line feeds. No big deal, lots of other parts of cogito break on filenames containing line feeds. Signed-off-by: Bryan Larsen [EMAIL PROTECTED] --- cg-commit |8

Re: Stacked GIT 0.4

Catalin Marinas wrote: Bryan Larsen [EMAIL PROTECTED] wrote: template files for the series output of export, to put it into a format that sendpatchset understands. I thought about integrating sendpatchset into stgit but it is much simpler to just generate a control file (especially if you

[PATCH 3/6] git-gnu-progs-Makefile: git Makefile update

Update the git Makefile to put the results of config.sh into the scripts. config.sh searches for gnu utilities cp, stat, date and xargs. Signed-off-by: Bryan Larsen [EMAIL PROTECTED] --- Makefile | 13 +++-- 1 files changed, 11 insertions(+), 2 deletions(-) diff --git a/Makefile b

[PATCH 6/6] darwinports-Portfile: Portfile for cogito

A Portfile for darwinports. Signed-off-by: Bryan Larsen [EMAIL PROTECTED] --- Makefile|2 ++ Portfile.in | 25 + 2 files changed, 27 insertions(+), 0 deletions(-) diff --git a/Makefile b/Makefile --- a/Makefile +++ b/Makefile @@ -240,6 +240,8 @@ dist

Re: [PATCH 3/6] git-gnu-progs-Makefile: git Makefile update

Linus Torvalds wrote: On Mon, 11 Jul 2005, Junio C Hamano wrote: I am not yet convinced one variable per GNU program is the right way to do (I do agree it is a problem and I appreciate your trying to solving it; an obvious alternative cop-out would be to fix this in the user's environment,

Re: [PATCH 3/6] git-gnu-progs-Makefile: git Makefile update

The only user of cp -l in the Linus GIT is git-clone-script local optimization. I could revert it to the version that I originally sent to the list, which uses cpio -pld, if your cpio groks that flag. Those options are in the man page, at least. Bryan - To unsubscribe from this list: send

[PATCH 3/4] add --missing-ok option to write-tree

Add --missing-ok option to git-write.tree. This option allows a write-tree even if the referenced objects are not in the database. Signed-off-by: Bryan Larsen [EMAIL PROTECTED] --- diff --git a/Documentation/git-write-tree.txt b/Documentation/git-write-tree.txt --- a/Documentation/git-write

[PATCH 4/4] switch cg-commit -N to use --missing-ok instead of --no-check

Make cg-commit aware of the rename of git-write-tree --no-check to --missing-ok. Signed-off-by: Bryan Larsen [EMAIL PROTECTED] --- diff --git a/cg-commit b/cg-commit --- a/cg-commit +++ b/cg-commit @@ -111,13 +111,13 @@ forceeditor= ignorecache= infoonly= commitalways= -nocheck= +missingok

Re: Commit 7c5977297f07c2a52c8c0f486309076b2a795d33 appears to have broken cg-init

Yes, I meant to use $ARGS. I included the fix in the patchset I just sent. Sorry, Bryan Joel Becker wrote: Petr, Commit 7c5977297f07c2a52c8c0f486309076b2a795d33 contains the chunk: -uri=$1 + +infoonly= +while optparse; do + if optparse -N; then + infoonly=-N +

[PATCH 1/3] add -N option to cg-add (resent)

(resending cogito patches) Add the -N option to cg-add. Signed-off-by: Bryan Larsen [EMAIL PROTECTED] --- cg-add | 21 ++--- 1 files changed, 18 insertions(+), 3 deletions(-) diff --git a/cg-add b/cg-add --- a/cg-add +++ b/cg-add @@ -13,20 +13,35 @@ # is that 'Cogito

[PATCH 3/3] add -N option to cg-init (resent)

(resending cogito patches) Add the -N option to cg-add. Signed-off-by: Bryan Larsen [EMAIL PROTECTED] --- cg-add | 21 ++--- 1 files changed, 18 insertions(+), 3 deletions(-) diff --git a/cg-add b/cg-add --- a/cg-add +++ b/cg-add @@ -13,20 +13,35 @@ # is that 'Cogito

[PATCH 3/3] add -N option to cg-init (resent again)

(resending cogito patches) add the -N option to cg-init Signed-off-by: Bryan Larsen [EMAIL PROTECTED] --- cg-init | 22 +++--- 1 files changed, 19 insertions(+), 3 deletions(-) diff --git a/cg-init b/cg-init --- a/cg-init +++ b/cg-init @@ -10,13 +10,29 @@ # # If `cg-init

Re: cogito Mac OS X compatibility

Sounds like you're missing stat from coreutils. skimo Thanks. I've got it working to the point where it passes make test now. All I had to do was install coreutils and findutils from darwinports, and then make gcp, gstat and gnuxargs available under their standard names. Do the BSD's

[PATCH] add --no-check option to git-write-tree

Add --no-check option to git-write-tree. Signed-off-by: Bryan Larsen [EMAIL PROTECTED] --- Documentation/git-write-tree.txt |6 ++ write-tree.c | 14 +- 2 files changed, 19 insertions(+), 1 deletions(-) diff --git a/Documentation/git-write-tree.txt b

patches to support working without the object database

Sometimes you may wish to keep an audit trail of what changed, where, and by whom. You do not need to know the exact details of the change, and the files are so large that keeping an extra copy of the data in the object database cache is prohibitively expensive. Git is (almost) ideally

[PATCH] 7/7 add -N option to cg-init

add -N option to cg-init Signed-off-by: Bryan Larsen [EMAIL PROTECTED] --- cg-init | 22 +++--- 1 files changed, 19 insertions(+), 3 deletions(-) diff --git a/cg-init b/cg-init --- a/cg-init +++ b/cg-init @@ -10,13 +10,29 @@ # # If `cg-init` is run in a non-empty directory

[PATCH] 2/7 Prepare for the next two patches

and the subsequent patches minimally intrusive. Signed-off-by: Bryan Larsen [EMAIL PROTECTED] --- cache.h|3 ++- sha1_file.c| 13 + update-cache.c |2 +- write-blob.c |2 +- 4 files changed, 13 insertions(+), 7 deletions(-) diff --git a/cache.h b/cache.h

[PATCH] 6/7 Add the -N option to cg-commit.

Add the -N option to cg-commit. Signed-off-by: Bryan Larsen [EMAIL PROTECTED] --- cg-commit | 15 --- 1 files changed, 12 insertions(+), 3 deletions(-) diff --git a/cg-commit b/cg-commit --- a/cg-commit +++ b/cg-commit @@ -39,6 +39,10 @@ # the tree is the same

[PATCH] 5/7 Add the -N option to cg-add.

Add the -N option to cg-add. Signed-off-by: Bryan Larsen [EMAIL PROTECTED] --- cg-add | 21 ++--- 1 files changed, 18 insertions(+), 3 deletions(-) diff --git a/cg-add b/cg-add --- a/cg-add +++ b/cg-add @@ -13,20 +13,35 @@ # is that 'Cogito' manages content and empty

cogito Mac OS X compatibility

On Mac OS X $ cg-clone http://www.kernel.org/pub/scm/cogito/cogito.git defaulting to local storage area 19:11:10 URL:http://www.kernel.org/pub/scm/cogito/cogito.git/refs/heads/master [41/41] - refs/heads/origin [1] /Users/blarsen/bin/cg-pull: line 82: 0 + : syntax error: operand expected