[PATCH] Better option parsing for gitdiff.sh

2005-04-19 Thread Pavel Roskin
Hello! This patch improves option handling for gitdiff.sh. Now -p doesn't need to precede -r, although all options still have to be placed before the file names. Also, the patch introduces a minimal usage info for the script. The patch is against current git-pasky. Signed-off-by: Pavel Roskin

[PATCH] gittrack.sh accepts invalid branch names

2005-04-20 Thread Pavel Roskin
) correctly. So I decided to use tab directly in the sed expression. I cannot think of any portable way to avoid grep completely (q is a GNU sed extension, and we want to support BSD, I think), so it's still there, looking for any output from sed. Signed-off-by: Pavel Roskin [EMAIL PROTECTED

Re: [PATCH] gittrack.sh accepts invalid branch names

2005-04-20 Thread Pavel Roskin
Hi, Petr! On Thu, 2005-04-21 at 01:21 +0200, Petr Baudis wrote: Dear diary, on Wed, Apr 20, 2005 at 09:48:30PM CEST, I got a letter where Pavel Roskin [EMAIL PROTECTED] told me that... --- a/gittrack.sh +++ b/gittrack.sh @@ -35,7 +35,7 @@ die () { mkdir -p .git/heads if [ $name

Switching between branches

2005-04-20 Thread Pavel Roskin
to specify -b pasky, as if the currently checked out branch is unknown. I'm using git-pasky 0.6.2. -- Regards, Pavel Roskin - To unsubscribe from this list: send the line unsubscribe git in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo

[PATCH] Colored log on any ANSI capable terminal

2005-04-21 Thread Pavel Roskin
of $'...' but it may not work in bash 1.x. Signed-off-by: Pavel Roskin [EMAIL PROTECTED] --- a/gitlog.sh +++ b/gitlog.sh @@ -12,11 +12,11 @@ if [ $1 = -c ]; then shift - colheader=$(setterm -foreground green) - colauthor=$(setterm -foreground cyan) - colcommitter=$(setterm -foreground

Re: [PATCH] mmap error handling

2005-07-29 Thread Pavel Roskin
Hi, Linus! On Thu, 2005-07-28 at 17:30 -0700, Linus Torvalds wrote: _always_ save the value of errno before doing any other calls. Even successful calls are perfectly allowed to change errno. OK. Fixed patch below. Signed-off-by: Pavel Roskin [EMAIL PROTECTED] diff --git a/diff.c b/diff.c

[PATCH] Making CFLAGS compilant with GNU Coding Standards

2005-08-05 Thread Pavel Roskin
the implicit rules use CFLAGS rather than ALL_CFLAGS. I believe that serious projects should not rely on implicit rules anyway. Percent rules are used because they are used already and because they don't need the .SUFFIXES target. Signed-off-by: Pavel Roskin [EMAIL PROTECTED] diff --git

Re: New script: cg-clean

2005-08-06 Thread Pavel Roskin
Hello, Petr! Sorry for delay. On Sun, 2005-07-10 at 17:46 +0200, Petr Baudis wrote: Dear diary, on Sat, Jul 09, 2005 at 12:34:44AM CEST, I got a letter where Pavel Roskin [EMAIL PROTECTED] told me that... Hello, Petr! Hello, Please consider this script for Cogito. Signed-off

[PATCH] Warning fix for gcc 4

2005-08-09 Thread Pavel Roskin
Hello! This patch fixes the only warning reported by gcc 4.0.1 on Fedora Core 4 for x86_64: sha1_file.c:1391: warning: pointer targets in assignment differ in signedness Signed-off-by: Pavel Roskin [EMAIL PROTECTED] diff --git a/sha1_file.c b/sha1_file.c --- a/sha1_file.c +++ b/sha1_file.c

[PATCH] Need to set PAGER in tests

2005-08-10 Thread Pavel Roskin
Hello! t5400-send-pack.sh --verbose stops waiting for user input. It happens because git log uses less for output now. To prevent this, PAGER should be set to cat. Signed-off-by: Pavel Roskin [EMAIL PROTECTED] diff --git a/t/test-lib.sh b/t/test-lib.sh --- a/t/test-lib.sh +++ b/t/test-lib.sh

[PATCH] Trapping exit in tests, using return for errors

2005-08-10 Thread Pavel Roskin
script: not 2 parameters to test-expect-success say 3 expecting success: $2 - if eval 3 24 $2 + test_run_ $2 + if [ $? = 0 -a $eval_ret = 0 ] then test_ok_ $1 else -- Regards, Pavel Roskin - To unsubscribe from this list: send the line

Re: [PATCH] Trapping exit in tests, using return for errors

2005-08-11 Thread Pavel Roskin
that is set when test-lib.sh is sourced and unset either by test_done(), error() or by test_failure_() with --immediate. This patch also depends on the patch that adds test_done() the the scripts that don't have it. Signed-off-by: Pavel Roskin [EMAIL PROTECTED] diff --git a/t/test-lib.sh b/t/test

Re: New script: cg-clean

2005-08-11 Thread Pavel Roskin
show everything that gets deleted. return I suppose you mean continue? I'm not really sure what does return do here, if it jumps out of the do block or what, and continue is nicely explicit. :-) My error, it was fixed soon after I posted the script. -- Regards, Pavel Roskin

[PATCH] cg-admin-cat ignoring -r

2005-08-16 Thread Pavel Roskin
Hello! cg-admin-cat ignores the argument for the -r option because it uses optparse incorrectly. For OPTARG to be set, -r= should be used instead of -r. Signed-off-by: Pavel Roskin [EMAIL PROTECTED] diff --git a/cg-admin-cat b/cg-admin-cat --- a/cg-admin-cat +++ b/cg-admin-cat @@ -27,7 +27,7

[PATCH] commit-id, tree-id fail on tags

2005-08-16 Thread Pavel Roskin
Hello! Tag names don't work with current cogito because commit-id and tree-id don't parse the cg-Xnormid output properly. Namely, if $type is empty (which is the case for tags), $normid is used before the trailing space is stripped from it. Signed-off-by: Pavel Roskin [EMAIL PROTECTED] diff

[PATCH] Improve can_hardlink diagnostics, remove suggest_hardlink

2005-08-23 Thread Pavel Roskin
Hello! suggest_hardlink is write-only in cg-pull - remove it. can_hardlink should not be shown to the user as is (it's either l or empty) - we should output something meaningful instead. Signed-off-by: Pavel Roskin [EMAIL PROTECTED] diff --git a/cg-pull b/cg-pull --- a/cg-pull +++ b/cg-pull

[PATCH] cg-pull to stop treating master specially, fix fetch_local for .git/HEAD

2005-08-23 Thread Pavel Roskin
option handling. Signed-off-by: Pavel Roskin [EMAIL PROTECTED] diff --git a/cg-pull b/cg-pull --- a/cg-pull +++ b/cg-pull @@ -67,6 +67,8 @@ pull_progress() fetch_rsync() { + [ $1 = -b ] shift + redir= if [ $1 = -i ]; then # ignore-errors redir=2/dev/null

[PATCH] Catch more exceptions in compat_log_entry()

2013-10-22 Thread Pavel Roskin
Catch exceptions in default_repo(). Catch git.RepositoryException. This suppresses stack trace in stg pull on detached head and outside the repository. Signed-off-by: Pavel Roskin pro...@gnu.org --- stgit/lib/log.py |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git