* [PATCH 3/7] Add notify and warn to util.py
From: Sverre Rabbelier @ 2009-10-29 6:40 UTC (permalink / raw)
To: Git List, Johannes Schindelin, Daniel Barkalow, Johan Herland
Cc: Sverre Rabbelier
In-Reply-To: <1256798426-21816-1-git-send-email-srabbelier@gmail.com>
Signed-off-by: Sverre Rabbelier <srabbelier@gmail.com>
---
These turned out to be quite useful for my git-remote-hg, if
desired this patch can move to later in the series, but I
figured that it might be useful to someone else already.
git_remote_helpers/util.py | 8 ++++++++
1 files changed, 8 insertions(+), 0 deletions(-)
diff --git a/git_remote_helpers/util.py b/git_remote_helpers/util.py
index 7d6adb4..d3ca487 100644
--- a/git_remote_helpers/util.py
+++ b/git_remote_helpers/util.py
@@ -15,6 +15,10 @@ import subprocess
# Whether or not to show debug messages
DEBUG = False
+def notify(msg, *args):
+ """Print a message to stderr."""
+ print >> sys.stderr, msg % args
+
def debug (msg, *args):
"""Print a debug message to stderr when DEBUG is enabled."""
if DEBUG:
@@ -24,6 +28,10 @@ def error (msg, *args):
"""Print an error message to stderr."""
print >> sys.stderr, "ERROR:", msg % args
+def warn(msg, *args):
+ """Print a warning message to stderr."""
+ print >> sys.stderr, "warning:", msg % args
+
def die (msg, *args):
"""Print as error message to stderr and exit the program."""
error(msg, *args)
--
1.6.5.2.291.gf76a3
^ permalink raw reply related
* [PATCH 0/2] Re: packaging vs default pager
From: Jonathan Nieder @ 2009-10-29 7:32 UTC (permalink / raw)
To: Junio C Hamano; +Cc: Ben Walton, GIT List
In-Reply-To: <7vskd3o11t.fsf@alter.siamese.dyndns.org>
Junio C Hamano wrote:
> Ben Walton <bwalton@artsci.utoronto.ca> writes:
>> Would it be completely out of line to provide a knob so that the
>> fallback $pager could be set to something more specific/appropriate
>> during the build?
>
> I think that is a sensible thing to do. Something like this?
>
> Makefile | 6 ++++++
> pager.c | 6 +++++-
> 2 files changed, 11 insertions(+), 1 deletions(-)
That looks good to me, but it’s missing a corresponding change to
git-svn. I think something like this could be useful for Debian to
avoid patching to comply with the distro policy of falling back to
generic "pager" and "editor" symlinks. How about the following two
patches?
Jonathan Nieder (1):
Provide a build time default-editor setting
Junio C Hamano (1):
Provide a build time default-pager setting
Makefile | 18 ++++++++++++++++++
editor.c | 2 +-
git-add--interactive.perl | 3 ++-
git-sh-setup.sh | 6 ++++--
git-svn.perl | 8 +++++---
pager.c | 2 +-
t/Makefile | 2 ++
t/t7005-editor.sh | 29 ++++++++++++++++++++++-------
8 files changed, 55 insertions(+), 15 deletions(-)
^ permalink raw reply
* Re: [RFC PATCH v2 0/2] git-gui: (un)stage a range of changes at once
From: Peter Baumann @ 2009-10-29 7:34 UTC (permalink / raw)
To: Jeff Epler; +Cc: Shawn O. Pearce, git
In-Reply-To: <1256160023-29629-1-git-send-email-jepler@unpythonic.net>
On Wed, Oct 21, 2009 at 04:20:21PM -0500, Jeff Epler wrote:
> Compared to the first version, I fixed a bug concerning staging line(s)
> when all following lines are deletions (a preexisting bug in git-gui).
> This version is made based off the master branch of git-gui.git, rather
> than the master branch of git.git.
>
> The first change fixes a long-standing git-gui bug in an area that the
> new feature is rewriting anyway. If there's interest in the new feature
> then maybe the two should just be squashed (using the message from the
> second). If not, it'd be nice to see the bugfix applied anyway.
>
>
> Jeff Epler (2):
> Fix applying a line when all following lines are deletions
> Make it possible to apply a range of changes at once
>
> git-gui.sh | 15 +++-
> lib/diff.tcl | 224 ++++++++++++++++++++++++++++++++--------------------------
> 2 files changed, 135 insertions(+), 104 deletions(-)
Cc ing Shawn as the git gui maintainer, as he might have missed this series
during his away time.
The original series including user comments can be found at
http://thread.gmane.org/gmane.comp.version-control.git/130732
whereas the newest version is here:
http://thread.gmane.org/gmane.comp.version-control.git/130968
--
Peter
^ permalink raw reply
* [PATCH 1/2] Provide a build time default-pager setting
From: Jonathan Nieder @ 2009-10-29 7:45 UTC (permalink / raw)
To: Junio C Hamano; +Cc: Ben Walton, GIT List
In-Reply-To: <20091029073224.GA15403@progeny.tock>
From: Junio C Hamano <gitster@pobox.com>
On (old) solaris systems, /usr/bin/less (typically the first less
found) doesn't understand the default arguments (FXRS), which
forces users to alter their environment (PATH, GIT_PAGER, LESS,
etc) or have a local or global gitconfig before paging works as
expected.
On Debian systems, by policy packages must fall back to the
'pager' command, so that changing the target of the
/usr/bin/pager symlink changes the default pager for all packages
at once.
Provide a DEFAULT_PAGER knob so packagers can set the fallback
pager to something appropriate during the build.
This puts the "less" default in the Makefile instead of pager.c,
since it is needed for git-svn, too. This means that the
DEFAULT_PAGER preprocessor token _has_ to be defined on the
command line for git to build. I was worried about this for a
moment, but GIT_VERSION already works this way without trouble.
Probably the DEFAULT_PAGER setting should be added to something
like TRACK_CFLAGS as well. Actually, some other settings that
can change without forcing files to be rebuilt (e.g. SHELL_PATH),
too. This should be probably be addressed separately.
Reported-by: Ben Walton <bwalton@artsci.utoronto.ca>
Signed-off-by: Jonathan Nieder <jrnieder@gmail.com>
---
Makefile | 8 ++++++++
git-svn.perl | 5 +++--
| 2 +-
3 files changed, 12 insertions(+), 3 deletions(-)
diff --git a/Makefile b/Makefile
index 268aede..fc1a461 100644
--- a/Makefile
+++ b/Makefile
@@ -200,6 +200,9 @@ all::
# memory allocators with the nedmalloc allocator written by Niall Douglas.
#
# Define NO_REGEX if you have no or inferior regex support in your C library.
+#
+# Define DEFAULT_PAGER to the path of a sensible pager (defaults to "less") if
+# you want to use something different.
GIT-VERSION-FILE: .FORCE-GIT-VERSION-FILE
@$(SHELL_PATH) ./GIT-VERSION-GEN
@@ -1294,6 +1297,10 @@ ifdef NO_REGEX
COMPAT_CFLAGS += -Icompat/regex
COMPAT_OBJS += compat/regex/regex.o
endif
+ifndef DEFAULT_PAGER
+ DEFAULT_PAGER = less
+endif
+BASIC_CFLAGS += -DDEFAULT_PAGER='"$(DEFAULT_PAGER)"'
ifdef USE_NED_ALLOCATOR
COMPAT_CFLAGS += -DUSE_NED_ALLOCATOR -DOVERRIDE_STRDUP -DNDEBUG -DREPLACE_SYSTEM_ALLOCATOR -Icompat/nedmalloc
@@ -1451,6 +1458,7 @@ $(patsubst %.perl,%,$(SCRIPT_PERL)): % : %.perl
-e '}' \
-e 's|@@INSTLIBDIR@@|'"$$INSTLIBDIR"'|g' \
-e 's/@@GIT_VERSION@@/$(GIT_VERSION)/g' \
+ -e 's/@@DEFAULT_PAGER@@/$(DEFAULT_PAGER)/g' \
$@.perl >$@+ && \
chmod +x $@+ && \
mv $@+ $@
diff --git a/git-svn.perl b/git-svn.perl
index 6a3b501..c270b23 100755
--- a/git-svn.perl
+++ b/git-svn.perl
@@ -3,11 +3,12 @@
# License: GPL v2 or later
use warnings;
use strict;
-use vars qw/ $AUTHOR $VERSION
+use vars qw/ $AUTHOR $VERSION $DEFAULT_PAGER
$sha1 $sha1_short $_revision $_repository
$_q $_authors $_authors_prog %users/;
$AUTHOR = 'Eric Wong <normalperson@yhbt.net>';
$VERSION = '@@GIT_VERSION@@';
+$DEFAULT_PAGER = '@@DEFAULT_PAGER@@';
# From which subdir have we been invoked?
my $cmd_dir_prefix = eval {
@@ -5174,7 +5175,7 @@ sub git_svn_log_cmd {
sub config_pager {
$pager ||= $ENV{GIT_PAGER} || $ENV{PAGER};
if (!defined $pager) {
- $pager = 'less';
+ $pager = $DEFAULT_PAGER;
} elsif (length $pager == 0 || $pager eq 'cat') {
$pager = undef;
}
--git a/pager.c b/pager.c
index 86facec..416a796 100644
--- a/pager.c
+++ b/pager.c
@@ -58,7 +58,7 @@ void setup_pager(void)
if (!pager)
pager = getenv("PAGER");
if (!pager)
- pager = "less";
+ pager = DEFAULT_PAGER;
else if (!*pager || !strcmp(pager, "cat"))
return;
--
1.6.5.2
^ permalink raw reply related
* Re: [PATCH v3 1/2] filter-branch: stop special-casing $filter_subdir argument
From: Johannes Sixt @ 2009-10-29 7:35 UTC (permalink / raw)
To: Thomas Rast; +Cc: git, Junio C Hamano
In-Reply-To: <6e01558f719f4bfcd12f3c6dc5657790e86c874d.1256770377.git.trast@student.ethz.ch>
Thomas Rast schrieb:
> diff --git a/git-filter-branch.sh b/git-filter-branch.sh
> index a480d6f..da23b99 100755
> --- a/git-filter-branch.sh
> +++ b/git-filter-branch.sh
> @@ -257,15 +257,23 @@ git read-tree || die "Could not seed the index"
> # map old->new commit ids for rewriting parents
> mkdir ../map || die "Could not create map/ directory"
>
> +dashdash=
> +test -z "$(git rev-parse --no-revs "$@")" && dashdash=--
Hmm, if the user runs
$ git filter-branch does-not-exist
then this would be the first call to rev-parse that fails with "fatal:
ambiguous argument 'does-not-exist': unknown revision or path...", but
this code wouldn't catch the failure. I suggest this instead:
# we need "--" only if there are no path arguments in $@
nonrevs=$(git rev-parse --no-revs "$@") || exit
dashdash=${nonrevs+"--"}
(including the comment; you can drop the dquotes in the dashdash
assignment if you think the result is still readable.)
I was to suggest to move these lines into the case arm below that is the
only user of $dashdash. But since this is a good error check of the
user-supplied arguments, I now prefer to keep the lines outside the case
statement.
> +ref_args=$(git rev-parse --revs-only "$@")
"ref_args"? Shouldn't it be "rev_args"? It's about "revisions", not
"references".
> case "$filter_subdir" in
> "")
> - git rev-list --reverse --topo-order --default HEAD \
> - --parents --simplify-merges "$@"
> + eval set -- "$(git rev-parse --sq --no-revs "$@")"
> ;;
> *)
> - git rev-list --reverse --topo-order --default HEAD \
> - --parents --simplify-merges "$@" -- "$filter_subdir"
> -esac > ../revs || die "Could not get the commits"
> + eval set -- "$(git rev-parse --sq --no-revs "$@")" \
> + $dashdash "$filter_subdir"
This is not correct: $filter_subdir undergoes an extra level of
evaluation. You must write it like this:
eval set -- "$(git rev-parse --sq --no-revs "$@" \
$dashdash "$filter_subdir")"
> + ;;
> +esac
> +
> +git rev-list --reverse --topo-order --default HEAD \
> + --parents --simplify-merges $ref_args "$@" \
> + > ../revs || die "Could not get the commits"
Personally, I would prefer to write this as
git rev-list --reverse --topo-order --default HEAD \
--parents --simplify-merges $ref_args "$@" > ../revs ||
die "Could not get the commits"
to avoid one backslash.
> @@ -356,8 +364,7 @@ then
> do
> sha1=$(git rev-parse "$ref"^0)
> test -f "$workdir"/../map/$sha1 && continue
> - ancestor=$(git rev-list --simplify-merges -1 \
> - $ref -- "$filter_subdir")
> + ancestor=$(git rev-list --simplify-merges -1 "$ref" "$@")
You added dquotes around $ref: while not absolutely necessary, I agree
with this change.
> test "$ancestor" && echo $(map $ancestor) >> "$workdir"/../map/$sha1
> done < "$tempdir"/heads
> fi
-- Hannes
^ permalink raw reply
* Re: [PATCH v3 2/2] filter-branch: nearest-ancestor rewriting outside subdir filter
From: Johannes Sixt @ 2009-10-29 7:38 UTC (permalink / raw)
To: Thomas Rast; +Cc: git, Junio C Hamano
In-Reply-To: <a11809cee976bb42102dbd9b2afb06b9e5b587bf.1256770377.git.trast@student.ethz.ch>
Thomas Rast schrieb:
> @@ -358,7 +365,7 @@ done <../revs
> # revision walker. Fix it by mapping these heads to the unique nearest
> # ancestor that survived the pruning.
>
> -if test "$filter_subdir"
> +if test "$remap_to_ancestor" = t
> then
The comment whose last lines are in the context of this hunk talks about
"subdirectory filter". You may want to revise it.
-- Hannes
^ permalink raw reply
* [PATCH/RFC 2/2] Provide a build time default-editor setting
From: Jonathan Nieder @ 2009-10-29 7:50 UTC (permalink / raw)
To: Junio C Hamano; +Cc: Ben Walton, GIT List
In-Reply-To: <20091029073224.GA15403@progeny.tock>
Provide a DEFAULT_EDITOR knob to allow the fallback editor (to
use instead of vi if VISUAL, EDITOR, and GIT_EDITOR are unset) to
be set at build time according to a system’s policy. For
example, on Debian systems, the default editor should be the
'editor' command.
The contrib/fast-import/git-p4 script still uses vi, since it is
not modified by the Makefile currently, and making it require
build-time modification would create too much trouble for people
deploying that script.
This change makes t7005-editor into a mess. Any ideas for fixing
this?
Signed-off-by: Jonathan Nieder <jrnieder@gmail.com>
---
Makefile | 10 ++++++++++
editor.c | 2 +-
git-add--interactive.perl | 3 ++-
git-sh-setup.sh | 6 ++++--
git-svn.perl | 5 +++--
t/Makefile | 2 ++
t/t7005-editor.sh | 29 ++++++++++++++++++++++-------
7 files changed, 44 insertions(+), 13 deletions(-)
diff --git a/Makefile b/Makefile
index fc1a461..fae8647 100644
--- a/Makefile
+++ b/Makefile
@@ -203,6 +203,9 @@ all::
#
# Define DEFAULT_PAGER to the path of a sensible pager (defaults to "less") if
# you want to use something different.
+#
+# Define DEFAULT_EDITOR to a sensible editor command (defaults to "vi") if you
+# want to use something different.
GIT-VERSION-FILE: .FORCE-GIT-VERSION-FILE
@$(SHELL_PATH) ./GIT-VERSION-GEN
@@ -1301,6 +1304,11 @@ ifndef DEFAULT_PAGER
DEFAULT_PAGER = less
endif
BASIC_CFLAGS += -DDEFAULT_PAGER='"$(DEFAULT_PAGER)"'
+ifndef DEFAULT_EDITOR
+ DEFAULT_EDITOR = vi
+endif
+export DEFAULT_EDITOR
+BASIC_CFLAGS += -DDEFAULT_EDITOR='"$(DEFAULT_EDITOR)"'
ifdef USE_NED_ALLOCATOR
COMPAT_CFLAGS += -DUSE_NED_ALLOCATOR -DOVERRIDE_STRDUP -DNDEBUG -DREPLACE_SYSTEM_ALLOCATOR -Icompat/nedmalloc
@@ -1435,6 +1443,7 @@ $(patsubst %.sh,%,$(SCRIPT_SH)) : % : %.sh
-e 's|@SHELL_PATH@|$(SHELL_PATH_SQ)|' \
-e 's/@@GIT_VERSION@@/$(GIT_VERSION)/g' \
-e 's/@@NO_CURL@@/$(NO_CURL)/g' \
+ -e 's|DEFAULT_EDITOR:=vi|DEFAULT_EDITOR:=$(DEFAULT_EDITOR)|' \
-e $(BROKEN_PATH_FIX) \
$@.sh >$@+ && \
chmod +x $@+ && \
@@ -1459,6 +1468,7 @@ $(patsubst %.perl,%,$(SCRIPT_PERL)): % : %.perl
-e 's|@@INSTLIBDIR@@|'"$$INSTLIBDIR"'|g' \
-e 's/@@GIT_VERSION@@/$(GIT_VERSION)/g' \
-e 's/@@DEFAULT_PAGER@@/$(DEFAULT_PAGER)/g' \
+ -e 's/@@DEFAULT_EDITOR@@/$(DEFAULT_EDITOR)/g' \
$@.perl >$@+ && \
chmod +x $@+ && \
mv $@+ $@
diff --git a/editor.c b/editor.c
index 4d469d0..93b8cbb 100644
--- a/editor.c
+++ b/editor.c
@@ -19,7 +19,7 @@ int launch_editor(const char *path, struct strbuf *buffer, const char *const *en
return error("Terminal is dumb but no VISUAL nor EDITOR defined.");
if (!editor)
- editor = "vi";
+ editor = DEFAULT_EDITOR;
if (strcmp(editor, ":")) {
size_t len = strlen(editor);
diff --git a/git-add--interactive.perl b/git-add--interactive.perl
index 69aeaf0..c3d932c 100755
--- a/git-add--interactive.perl
+++ b/git-add--interactive.perl
@@ -1,6 +1,7 @@
#!/usr/bin/perl -w
use strict;
+use constant DEFAULT_EDITOR => '@@DEFAULT_EDITOR@@';
use Git;
binmode(STDOUT, ":raw");
@@ -988,7 +989,7 @@ EOF
close $fh;
my $editor = $ENV{GIT_EDITOR} || $repo->config("core.editor")
- || $ENV{VISUAL} || $ENV{EDITOR} || "vi";
+ || $ENV{VISUAL} || $ENV{EDITOR} || DEFAULT_EDITOR;
system('sh', '-c', $editor.' "$@"', $editor, $hunkfile);
if ($? != 0) {
diff --git a/git-sh-setup.sh b/git-sh-setup.sh
index c41c2f7..d053d56 100755
--- a/git-sh-setup.sh
+++ b/git-sh-setup.sh
@@ -99,19 +99,21 @@ set_reflog_action() {
}
git_editor() {
+ : "${DEFAULT_EDITOR:=vi}"
: "${GIT_EDITOR:=$(git config core.editor)}"
: "${GIT_EDITOR:=${VISUAL:-${EDITOR}}}"
case "$GIT_EDITOR,$TERM" in
,dumb)
echo >&2 "No editor specified in GIT_EDITOR, core.editor, VISUAL,"
- echo >&2 "or EDITOR. Tried to fall back to vi but terminal is dumb."
+ echo >&2 "or EDITOR. Tried to fall back to $DEFAULT_EDITOR" \
+ "but terminal is dumb."
echo >&2 "Please set one of these variables to an appropriate"
echo >&2 "editor or run $0 with options that will not cause an"
echo >&2 "editor to be invoked (e.g., -m or -F for git-commit)."
exit 1
;;
esac
- eval "${GIT_EDITOR:=vi}" '"$@"'
+ eval "${GIT_EDITOR:=$DEFAULT_EDITOR}" '"$@"'
}
is_bare_repository () {
diff --git a/git-svn.perl b/git-svn.perl
index c270b23..b98d378 100755
--- a/git-svn.perl
+++ b/git-svn.perl
@@ -3,12 +3,13 @@
# License: GPL v2 or later
use warnings;
use strict;
-use vars qw/ $AUTHOR $VERSION $DEFAULT_PAGER
+use vars qw/ $AUTHOR $VERSION $DEFAULT_PAGER $DEFAULT_EDITOR
$sha1 $sha1_short $_revision $_repository
$_q $_authors $_authors_prog %users/;
$AUTHOR = 'Eric Wong <normalperson@yhbt.net>';
$VERSION = '@@GIT_VERSION@@';
$DEFAULT_PAGER = '@@DEFAULT_PAGER@@';
+$DEFAULT_EDITOR = '@@DEFAULT_EDITOR@@';
# From which subdir have we been invoked?
my $cmd_dir_prefix = eval {
@@ -1322,7 +1323,7 @@ sub get_commit_entry {
close $log_fh or croak $!;
if ($_edit || ($type eq 'tree')) {
- my $editor = $ENV{VISUAL} || $ENV{EDITOR} || 'vi';
+ my $editor = $ENV{VISUAL} || $ENV{EDITOR} || $DEFAULT_EDITOR;
# TODO: strip out spaces, comments, like git-commit.sh
system($editor, $commit_editmsg);
}
diff --git a/t/Makefile b/t/Makefile
index bd09390..9174bbb 100644
--- a/t/Makefile
+++ b/t/Makefile
@@ -9,6 +9,8 @@
SHELL_PATH ?= $(SHELL)
TAR ?= $(TAR)
RM ?= rm -f
+DEFAULT_EDITOR ?= vi
+export DEFAULT_EDITOR
# Shell quote;
SHELL_PATH_SQ = $(subst ','\'',$(SHELL_PATH))
diff --git a/t/t7005-editor.sh b/t/t7005-editor.sh
index b647957..2b76f72 100755
--- a/t/t7005-editor.sh
+++ b/t/t7005-editor.sh
@@ -4,7 +4,18 @@ test_description='GIT_EDITOR, core.editor, and stuff'
. ./test-lib.sh
-for i in GIT_EDITOR core_editor EDITOR VISUAL vi
+: ${DEFAULT_EDITOR=vi}
+
+unset EDITOR VISUAL GIT_EDITOR
+
+case "$DEFAULT_EDITOR" in
+*/* | [A-Z]*)
+ DEFAULT_EDITOR=
+ ;;
+esac
+
+for i in GIT_EDITOR core_editor EDITOR VISUAL \
+ ${DEFAULT_EDITOR:+"$DEFAULT_EDITOR"}
do
cat >e-$i.sh <<-EOF
#!$SHELL_PATH
@@ -12,15 +23,17 @@ do
EOF
chmod +x e-$i.sh
done
-unset vi
-mv e-vi.sh vi
-unset EDITOR VISUAL GIT_EDITOR
+
+if test -n "$DEFAULT_EDITOR"
+then
+ mv "e-$DEFAULT_EDITOR.sh" "$DEFAULT_EDITOR"
+fi
test_expect_success setup '
msg="Hand edited" &&
echo "$msg" >expect &&
- git add vi &&
+ git add "e-VISUAL.sh" &&
test_tick &&
git commit -m "$msg" &&
git show -s --pretty=oneline |
@@ -44,7 +57,8 @@ test_expect_success 'dumb should error out when falling back on vi' '
TERM=vt100
export TERM
-for i in vi EDITOR VISUAL core_editor GIT_EDITOR
+for i in ${DEFAULT_EDITOR:+"$DEFAULT_EDITOR"} \
+ EDITOR VISUAL core_editor GIT_EDITOR
do
echo "Edited by $i" >expect
unset EDITOR VISUAL GIT_EDITOR
@@ -68,7 +82,8 @@ done
unset EDITOR VISUAL GIT_EDITOR
git config --unset-all core.editor
-for i in vi EDITOR VISUAL core_editor GIT_EDITOR
+for i in ${DEFAULT_EDITOR:+"$DEFAULT_EDITOR"} \
+ EDITOR VISUAL core_editor GIT_EDITOR
do
echo "Edited by $i" >expect
case "$i" in
--
1.6.5.2
^ permalink raw reply related
* translation help documentation
From: Laszlo Papp @ 2009-10-29 7:40 UTC (permalink / raw)
To: git
Hello,
Isn't there a translation help documentation ?
What's your 'policy' about it ? A lot of projects use such a
documentation, and sometimes translation helper script too. I can send
a sample if you feel so.
Best Regards,
Laszlo Papp
^ permalink raw reply
* [PATCH] clone: detect extra arguments
From: Jonathan Nieder @ 2009-10-29 8:10 UTC (permalink / raw)
To: Junio C Hamano; +Cc: git
If git clone is given more than two non-option arguments, it
silently throws away all but the first one. Complain instead.
Discovered by comparing the new builtin clone to the old
git-clone.sh.
Signed-off-by: Jonathan Nieder <jrnieder@gmail.com>
---
builtin-clone.c | 3 +++
1 files changed, 3 insertions(+), 0 deletions(-)
diff --git a/builtin-clone.c b/builtin-clone.c
index 5762a6f..76ad581 100644
--- a/builtin-clone.c
+++ b/builtin-clone.c
@@ -377,6 +377,9 @@ int cmd_clone(int argc, const char **argv, const char *prefix)
argc = parse_options(argc, argv, prefix, builtin_clone_options,
builtin_clone_usage, 0);
+ if (argc > 2)
+ die("Too many arguments.");
+
if (argc == 0)
die("You must specify a repository to clone.");
--
1.6.5.2
^ permalink raw reply related
* Re: [RFC PATCH v4 00/26] Return of smart HTTP
From: Jakub Narebski @ 2009-10-29 8:12 UTC (permalink / raw)
To: Shawn O. Pearce; +Cc: git
In-Reply-To: <1256774448-7625-1-git-send-email-spearce@spearce.org>
"Shawn O. Pearce" <spearce@spearce.org> writes:
> I think this is the final spin of the smart HTTP series. I've
> collected patches from a few others (thanks folks!) and added
> tests specific to the smart variant of the HTTP transport.
>
> At this point, I think it is "next ready"... but would appreciate
> any additional feedback if folks identify something we should
> address before hitting next.
[...]
> .gitignore | 1 +
> Documentation/config.txt | 8 +
> Documentation/git-http-backend.txt | 170 ++++++++
> Documentation/git-remote-helpers.txt | 85 ++++-
> Makefile | 1 +
[...]
> 32 files changed, 2574 insertions(+), 223 deletions(-)
> create mode 100644 Documentation/git-http-backend.txt
> create mode 100644 http-backend.c
> create mode 100755 t/t5541-http-push.sh
> create mode 100755 t/t5551-http-fetch.sh
If it is a final spin, then what of missing RFC-like documentation of
Git HTTP protocol in Documentation/technical/http-protocol.txt
(it was present only in first version of new series)?
--
Jakub Narebski
Poland
ShadeHawk on #git
^ permalink raw reply
* Re: [PATCH I18N filenames v2 3/3] Provide compatibility with MinGW
From: Peter Krefting @ 2009-10-29 9:01 UTC (permalink / raw)
To: Timur Sufiev; +Cc: Git Mailing List
In-Reply-To: <1256752900-2615-3-git-send-email-timur@iris-comp.ru>
Hi!
Timur Sufiev:
> diff --git a/compat/mingw.c b/compat/mingw.c
> index 6b5b5b2..2a2ebcb 100644
> --- a/compat/mingw.c
> +++ b/compat/mingw.c
> @@ -2,7 +2,7 @@
> #include "win32.h"
> #include <conio.h>
> #include "../strbuf.h"
> -
> +#include "../io-i18n.h"
> #include <shellapi.h>
>
> static int err_win_to_posix(DWORD winerr)
> @@ -132,7 +132,7 @@ int mingw_open (const char *filename, int oflags, ...)
> if (!strcmp(filename, "/dev/null"))
> filename = "nul";
>
> - fd = open(filename, oflags, mode);
> + fd = open_i18n(filename, oflags, mode);
>
> if (fd < 0 && (oflags & O_CREAT) && errno == EACCES) {
> DWORD attrs = GetFileAttributes(filename);
Instead of calling the open_i18n() which converts from UTF-8 to a local
8-bit character set, this should probably call a version that converts from
UTF-8 to UTF-16 and uses _wopen().
Same thing for fopen_i18n() and _wfopen().
I created a small RFC patch for that that changed parts of the system
earlier this year - http://kerneltrap.org/mailarchive/git/2009/3/2/5350814
I did not address readdir() and friends, I'm not sure if they are available
in UTF-16 form or if they need to be rewritten using findfirst()/findnext().
--
\\// Peter - http://www.softwolves.pp.se/
^ permalink raw reply
* Re: [PATCH I18N filenames v2 1/3] Add IO-wrappers for filenames encoding <local encoding> <-> UTF-8
From: Johannes Sixt @ 2009-10-29 9:26 UTC (permalink / raw)
To: Timur Sufiev; +Cc: git
In-Reply-To: <1256752900-2615-1-git-send-email-timur@iris-comp.ru>
Timur Sufiev schrieb:
> +#define opendir(a) opendir_i18n(a)
> +#define fopen(a, b) fopen_i18n(a, b)
> +#define chmod(a, b) chmod_i18n(a, b)
> +#define open open_i18n
> +#define stat(a, b) stat_i18n(a, b)
> +#define lstat(a, b) lstat_i18n(a, b)
> +#define readdir(a) readdir_i18n(a)
> +#define unlink(a) unlink_i18n(a)
> +#define link(a, b) link_i18n(a, b)
I doubt that such a whole-sale replacement is necessary. For example,
names of object files (in .git/objects/XX) need not be translated, nor ref
names.
Please review the infrastructure patch that Linus provided (as pointed out
by Jeff) and see how you can build on it. IIRC, there are a handful of
central places where path names must be massaged, and perhaps a few others
as fall-out.
Moreover, once you know what to change, it would be worthwhile to check
how the file name translation can be adjusted to use the UTF16 versions of
the functions on Windows to avoid the UTF8<->UTF16 conversion (that
happens on Windows behind the scenes). Perhaps MacOS can profit from this,
too.
-- Hannes
^ permalink raw reply
* Re: [PATCH I18N filenames v2 2/3] Use I18N-wrappers everywhere in Git
From: Alex Riesen @ 2009-10-29 9:30 UTC (permalink / raw)
To: Timur Sufiev; +Cc: git
In-Reply-To: <1256752900-2615-2-git-send-email-timur@iris-comp.ru>
On Wed, Oct 28, 2009 at 19:01, Timur Sufiev <timur@iris-comp.ru> wrote:
> Signed-off-by: Timur Sufiev <timur@iris-comp.ru>
> ---
> abspath.c | 1 +
> attr.c | 1 +
> bisect.c | 1 +
Instead of modifying all these files you could just have put that
darn header in compat/mingw.h
^ permalink raw reply
* Re: [PATCH] git-rebase -i: improve usage message
From: Thomas Rast @ 2009-10-29 9:40 UTC (permalink / raw)
To: Junio C Hamano; +Cc: Brian Ewins, git, kusmabite
In-Reply-To: <7vtyxiafa9.fsf@alter.siamese.dyndns.org>
Junio C Hamano wrote:
>
> The OPTIONS_SPEC in rebase--interactive is for the interactive mode and
> for nothing else, so it may be a good idea to clearly say so at the
> beginning. The user experience perhaps should look like:
>
> $ git rebase -i -h
> Note: this help is only about the interactive mode;
> see 'git rebase -h' for help on non-interactive mode.
>
> usage: git rebase -i [<options>] [--] <upstream> [<branch>]
> or: git rebase -i (--continue|--abort|--skip)
Nit-pick: the last line is not correct; the user does not have to
specify -i for git-rebase to figure out that an interactive rebase is
in progress.
(I agree with the rest though.)
--
Thomas Rast
trast@{inf,student}.ethz.ch
^ permalink raw reply
* Re: [PATCH/RFC 2/2] Provide a build time default-editor setting
From: David Roundy @ 2009-10-29 10:36 UTC (permalink / raw)
To: Jonathan Nieder; +Cc: Junio C Hamano, Ben Walton, GIT List
In-Reply-To: <20091029075021.GC15403@progeny.tock>
Any chance this will be exported as plumbing? I know it's pretty
high-level, but it'd be handy to have be able to write `git editor
$FILENAME` and just have it do the right thing. This would also mean
that the perl scripts below could be simplified.
Same goes for pager, of course...
David
On Thu, Oct 29, 2009 at 3:50 AM, Jonathan Nieder <jrnieder@gmail.com> wrote:
> Provide a DEFAULT_EDITOR knob to allow the fallback editor (to
> use instead of vi if VISUAL, EDITOR, and GIT_EDITOR are unset) to
> be set at build time according to a system’s policy. For
> example, on Debian systems, the default editor should be the
> 'editor' command.
>
> The contrib/fast-import/git-p4 script still uses vi, since it is
> not modified by the Makefile currently, and making it require
> build-time modification would create too much trouble for people
> deploying that script.
>
> This change makes t7005-editor into a mess. Any ideas for fixing
> this?
>
> Signed-off-by: Jonathan Nieder <jrnieder@gmail.com>
> ---
> Makefile | 10 ++++++++++
> editor.c | 2 +-
> git-add--interactive.perl | 3 ++-
> git-sh-setup.sh | 6 ++++--
> git-svn.perl | 5 +++--
> t/Makefile | 2 ++
> t/t7005-editor.sh | 29 ++++++++++++++++++++++-------
> 7 files changed, 44 insertions(+), 13 deletions(-)
>
> diff --git a/Makefile b/Makefile
> index fc1a461..fae8647 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -203,6 +203,9 @@ all::
> #
> # Define DEFAULT_PAGER to the path of a sensible pager (defaults to "less") if
> # you want to use something different.
> +#
> +# Define DEFAULT_EDITOR to a sensible editor command (defaults to "vi") if you
> +# want to use something different.
>
> GIT-VERSION-FILE: .FORCE-GIT-VERSION-FILE
> @$(SHELL_PATH) ./GIT-VERSION-GEN
> @@ -1301,6 +1304,11 @@ ifndef DEFAULT_PAGER
> DEFAULT_PAGER = less
> endif
> BASIC_CFLAGS += -DDEFAULT_PAGER='"$(DEFAULT_PAGER)"'
> +ifndef DEFAULT_EDITOR
> + DEFAULT_EDITOR = vi
> +endif
> +export DEFAULT_EDITOR
> +BASIC_CFLAGS += -DDEFAULT_EDITOR='"$(DEFAULT_EDITOR)"'
>
> ifdef USE_NED_ALLOCATOR
> COMPAT_CFLAGS += -DUSE_NED_ALLOCATOR -DOVERRIDE_STRDUP -DNDEBUG -DREPLACE_SYSTEM_ALLOCATOR -Icompat/nedmalloc
> @@ -1435,6 +1443,7 @@ $(patsubst %.sh,%,$(SCRIPT_SH)) : % : %.sh
> -e 's|@SHELL_PATH@|$(SHELL_PATH_SQ)|' \
> -e 's/@@GIT_VERSION@@/$(GIT_VERSION)/g' \
> -e 's/@@NO_CURL@@/$(NO_CURL)/g' \
> + -e 's|DEFAULT_EDITOR:=vi|DEFAULT_EDITOR:=$(DEFAULT_EDITOR)|' \
> -e $(BROKEN_PATH_FIX) \
> $@.sh >$@+ && \
> chmod +x $@+ && \
> @@ -1459,6 +1468,7 @@ $(patsubst %.perl,%,$(SCRIPT_PERL)): % : %.perl
> -e 's|@@INSTLIBDIR@@|'"$$INSTLIBDIR"'|g' \
> -e 's/@@GIT_VERSION@@/$(GIT_VERSION)/g' \
> -e 's/@@DEFAULT_PAGER@@/$(DEFAULT_PAGER)/g' \
> + -e 's/@@DEFAULT_EDITOR@@/$(DEFAULT_EDITOR)/g' \
> $@.perl >$@+ && \
> chmod +x $@+ && \
> mv $@+ $@
> diff --git a/editor.c b/editor.c
> index 4d469d0..93b8cbb 100644
> --- a/editor.c
> +++ b/editor.c
> @@ -19,7 +19,7 @@ int launch_editor(const char *path, struct strbuf *buffer, const char *const *en
> return error("Terminal is dumb but no VISUAL nor EDITOR defined.");
>
> if (!editor)
> - editor = "vi";
> + editor = DEFAULT_EDITOR;
>
> if (strcmp(editor, ":")) {
> size_t len = strlen(editor);
> diff --git a/git-add--interactive.perl b/git-add--interactive.perl
> index 69aeaf0..c3d932c 100755
> --- a/git-add--interactive.perl
> +++ b/git-add--interactive.perl
> @@ -1,6 +1,7 @@
> #!/usr/bin/perl -w
>
> use strict;
> +use constant DEFAULT_EDITOR => '@@DEFAULT_EDITOR@@';
> use Git;
>
> binmode(STDOUT, ":raw");
> @@ -988,7 +989,7 @@ EOF
> close $fh;
>
> my $editor = $ENV{GIT_EDITOR} || $repo->config("core.editor")
> - || $ENV{VISUAL} || $ENV{EDITOR} || "vi";
> + || $ENV{VISUAL} || $ENV{EDITOR} || DEFAULT_EDITOR;
> system('sh', '-c', $editor.' "$@"', $editor, $hunkfile);
>
> if ($? != 0) {
> diff --git a/git-sh-setup.sh b/git-sh-setup.sh
> index c41c2f7..d053d56 100755
> --- a/git-sh-setup.sh
> +++ b/git-sh-setup.sh
> @@ -99,19 +99,21 @@ set_reflog_action() {
> }
>
> git_editor() {
> + : "${DEFAULT_EDITOR:=vi}"
> : "${GIT_EDITOR:=$(git config core.editor)}"
> : "${GIT_EDITOR:=${VISUAL:-${EDITOR}}}"
> case "$GIT_EDITOR,$TERM" in
> ,dumb)
> echo >&2 "No editor specified in GIT_EDITOR, core.editor, VISUAL,"
> - echo >&2 "or EDITOR. Tried to fall back to vi but terminal is dumb."
> + echo >&2 "or EDITOR. Tried to fall back to $DEFAULT_EDITOR" \
> + "but terminal is dumb."
> echo >&2 "Please set one of these variables to an appropriate"
> echo >&2 "editor or run $0 with options that will not cause an"
> echo >&2 "editor to be invoked (e.g., -m or -F for git-commit)."
> exit 1
> ;;
> esac
> - eval "${GIT_EDITOR:=vi}" '"$@"'
> + eval "${GIT_EDITOR:=$DEFAULT_EDITOR}" '"$@"'
> }
>
> is_bare_repository () {
> diff --git a/git-svn.perl b/git-svn.perl
> index c270b23..b98d378 100755
> --- a/git-svn.perl
> +++ b/git-svn.perl
> @@ -3,12 +3,13 @@
> # License: GPL v2 or later
> use warnings;
> use strict;
> -use vars qw/ $AUTHOR $VERSION $DEFAULT_PAGER
> +use vars qw/ $AUTHOR $VERSION $DEFAULT_PAGER $DEFAULT_EDITOR
> $sha1 $sha1_short $_revision $_repository
> $_q $_authors $_authors_prog %users/;
> $AUTHOR = 'Eric Wong <normalperson@yhbt.net>';
> $VERSION = '@@GIT_VERSION@@';
> $DEFAULT_PAGER = '@@DEFAULT_PAGER@@';
> +$DEFAULT_EDITOR = '@@DEFAULT_EDITOR@@';
>
> # From which subdir have we been invoked?
> my $cmd_dir_prefix = eval {
> @@ -1322,7 +1323,7 @@ sub get_commit_entry {
> close $log_fh or croak $!;
>
> if ($_edit || ($type eq 'tree')) {
> - my $editor = $ENV{VISUAL} || $ENV{EDITOR} || 'vi';
> + my $editor = $ENV{VISUAL} || $ENV{EDITOR} || $DEFAULT_EDITOR;
> # TODO: strip out spaces, comments, like git-commit.sh
> system($editor, $commit_editmsg);
> }
> diff --git a/t/Makefile b/t/Makefile
> index bd09390..9174bbb 100644
> --- a/t/Makefile
> +++ b/t/Makefile
> @@ -9,6 +9,8 @@
> SHELL_PATH ?= $(SHELL)
> TAR ?= $(TAR)
> RM ?= rm -f
> +DEFAULT_EDITOR ?= vi
> +export DEFAULT_EDITOR
>
> # Shell quote;
> SHELL_PATH_SQ = $(subst ','\'',$(SHELL_PATH))
> diff --git a/t/t7005-editor.sh b/t/t7005-editor.sh
> index b647957..2b76f72 100755
> --- a/t/t7005-editor.sh
> +++ b/t/t7005-editor.sh
> @@ -4,7 +4,18 @@ test_description='GIT_EDITOR, core.editor, and stuff'
>
> . ./test-lib.sh
>
> -for i in GIT_EDITOR core_editor EDITOR VISUAL vi
> +: ${DEFAULT_EDITOR=vi}
> +
> +unset EDITOR VISUAL GIT_EDITOR
> +
> +case "$DEFAULT_EDITOR" in
> +*/* | [A-Z]*)
> + DEFAULT_EDITOR=
> + ;;
> +esac
> +
> +for i in GIT_EDITOR core_editor EDITOR VISUAL \
> + ${DEFAULT_EDITOR:+"$DEFAULT_EDITOR"}
> do
> cat >e-$i.sh <<-EOF
> #!$SHELL_PATH
> @@ -12,15 +23,17 @@ do
> EOF
> chmod +x e-$i.sh
> done
> -unset vi
> -mv e-vi.sh vi
> -unset EDITOR VISUAL GIT_EDITOR
> +
> +if test -n "$DEFAULT_EDITOR"
> +then
> + mv "e-$DEFAULT_EDITOR.sh" "$DEFAULT_EDITOR"
> +fi
>
> test_expect_success setup '
>
> msg="Hand edited" &&
> echo "$msg" >expect &&
> - git add vi &&
> + git add "e-VISUAL.sh" &&
> test_tick &&
> git commit -m "$msg" &&
> git show -s --pretty=oneline |
> @@ -44,7 +57,8 @@ test_expect_success 'dumb should error out when falling back on vi' '
>
> TERM=vt100
> export TERM
> -for i in vi EDITOR VISUAL core_editor GIT_EDITOR
> +for i in ${DEFAULT_EDITOR:+"$DEFAULT_EDITOR"} \
> + EDITOR VISUAL core_editor GIT_EDITOR
> do
> echo "Edited by $i" >expect
> unset EDITOR VISUAL GIT_EDITOR
> @@ -68,7 +82,8 @@ done
>
> unset EDITOR VISUAL GIT_EDITOR
> git config --unset-all core.editor
> -for i in vi EDITOR VISUAL core_editor GIT_EDITOR
> +for i in ${DEFAULT_EDITOR:+"$DEFAULT_EDITOR"} \
> + EDITOR VISUAL core_editor GIT_EDITOR
> do
> echo "Edited by $i" >expect
> case "$i" in
> --
> 1.6.5.2
>
> --
> To unsubscribe from this list: send the line "unsubscribe git" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
--
David Roundy
^ permalink raw reply
* [PATCH] diff --color-words -U0: fix the location of hunk headers
From: Johannes Schindelin @ 2009-10-29 10:45 UTC (permalink / raw)
To: Markus Heidelberg; +Cc: Junio C Hamano, git
In-Reply-To: <1256732672-11817-4-git-send-email-markus.heidelberg@web.de>
Colored word diff without context lines firstly printed all the hunk
headers among each other and then printed the diff.
This was due to the code relying on getting at least one context line at
the end of each hunk, where the colored words would be flushed (it is
done that way to be able to ignore rewrapped lines).
Noticed by Markus Heidelberg.
Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
---
I would strongly prefer this fix instead of your 2/3 and 3/3.
diff.c | 6 ++++++
t/t4034-diff-words.sh | 2 +-
2 files changed, 7 insertions(+), 1 deletions(-)
diff --git a/diff.c b/diff.c
index 51b5dbb..4eafaf5 100644
--- a/diff.c
+++ b/diff.c
@@ -656,6 +656,12 @@ static void fn_out_consume(void *priv, char *line, unsigned long len)
for (i = 0; i < len && line[i] == '@'; i++)
;
if (2 <= i && i < len && line[i] == ' ') {
+ /* flush --color-words even for --unified=0 */
+ if (ecbdata->diff_words &&
+ (ecbdata->diff_words->minus.text.size ||
+ ecbdata->diff_words->plus.text.size))
+ diff_words_show(ecbdata->diff_words);
+
ecbdata->nparents = i - 1;
len = sane_truncate_line(ecbdata, line, len);
emit_line(ecbdata->file,
diff --git a/t/t4034-diff-words.sh b/t/t4034-diff-words.sh
index 82240cf..21db6e9 100755
--- a/t/t4034-diff-words.sh
+++ b/t/t4034-diff-words.sh
@@ -77,7 +77,7 @@ cat > expect <<\EOF
<GREEN>aeff = aeff * ( aaa )<RESET>
EOF
-test_expect_failure 'word diff without context' '
+test_expect_success 'word diff without context' '
word_diff --color-words --unified=0
--
1.6.4.GIT
^ permalink raw reply related
* Re: What's cooking in git.git (Oct 2009, #01; Wed, 07)
From: Johannes Schindelin @ 2009-10-29 10:54 UTC (permalink / raw)
To: Sverre Rabbelier; +Cc: Shawn O. Pearce, git, vcs-fast-import-devs
In-Reply-To: <fabb9a1e0910281508m3e9bb8a6g7b39abc29fceae78@mail.gmail.com>
Hi,
On Wed, 28 Oct 2009, Sverre Rabbelier wrote:
> On Thu, Oct 8, 2009 at 10:58, Sverre Rabbelier <srabbelier@gmail.com> wrote:
> > I think it makes to ignore options that are not for our vcs, as long
> > as options that change import behavior (such as marks, date-format)
> > are combined with, say, 'feature tool=git'. This way we can be sure
> > that when outputting out a vcs specific stream, it is only parsed by
> > that vcs.
> >
> > Note: yes, I know that marks and date-format are features now, but
> > there's really no other suitable example that I could think of).
> >
> > vcs fast import devs please ack this idea (and perhaps suggest
> > something other than "feature tool=git" if preferable) so that I can
> > reroll my gfi-options series :).
>
> Shawn, what do you want to do with this, it seems the vcs devs are not
> very interested in this feature, should I implement it as described
> above? That is:
> * If you use any option that is stream-changing you should include
> "feature tool=git" in your stream
> * import-marks and export-marks are made into features
> * "option vcs" is ignored if vcs is a different vcs
> * "option vcs" must be recognised if vcs is this vcs
It would be quite nice if this issue moved forward for a change.
As a consequence of it moving forward, I could nudge Sverre into
continuing with his git-remote-hg work that will allow me to work
transparently on a Mercurial repository using Git.
Transparent as in "no hassles".
It also will serve nicely as a perfect excuse to fix some design mistakes
in the foreign vcs stuff.
Ciao,
Dscho
^ permalink raw reply
* Re: [PATCH 0/3] increase user-friendliness
From: Johannes Schindelin @ 2009-10-29 10:59 UTC (permalink / raw)
To: Jeff King; +Cc: git
In-Reply-To: <20091029002229.GA986@sigill.intra.peff.net>
Hi,
On Wed, 28 Oct 2009, Jeff King wrote:
> Git has a reputation as being unfriendly to users. Let's fix that by
> adding some features implemented by more user-friendly programs.
So the bar for pirate patches has been raised to patch series now?
LOL,
Dscho
^ permalink raw reply
* Re: [PATCH] diff --color-words -U0: fix the location of hunk headers
From: Markus Heidelberg @ 2009-10-29 11:22 UTC (permalink / raw)
To: Johannes Schindelin; +Cc: Junio C Hamano, git
In-Reply-To: <alpine.DEB.1.00.0910291144170.3687@felix-maschine>
Johannes Schindelin, 29.10.2009:
>
> I would strongly prefer this fix instead of your 2/3 and 3/3.
>
> diff.c | 6 ++++++
> t/t4034-diff-words.sh | 2 +-
> 2 files changed, 7 insertions(+), 1 deletions(-)
>
> diff --git a/diff.c b/diff.c
> index 51b5dbb..4eafaf5 100644
> --- a/diff.c
> +++ b/diff.c
> @@ -656,6 +656,12 @@ static void fn_out_consume(void *priv, char *line, unsigned long len)
> for (i = 0; i < len && line[i] == '@'; i++)
> ;
> if (2 <= i && i < len && line[i] == ' ') {
> + /* flush --color-words even for --unified=0 */
> + if (ecbdata->diff_words &&
> + (ecbdata->diff_words->minus.text.size ||
> + ecbdata->diff_words->plus.text.size))
> + diff_words_show(ecbdata->diff_words);
> +
> ecbdata->nparents = i - 1;
> len = sane_truncate_line(ecbdata, line, len);
> emit_line(ecbdata->file,
This seems to apply before commit b8d9c1a (diff.c: the builtin_diff()
deals with only two-file comparison, 2009-09-03).
Indeed my initial fix was in the same fashion:
@@ -772,6 +772,15 @@ static void fn_out_consume(void *priv, char *line, unsigned long len)
}
if (line[0] == '@') {
+ if (ecbdata->diff_words) {
+ /*
+ * The content of the previous hunk, necessary for
+ * 0-context.
+ */
+ if (ecbdata->diff_words->minus.text.size ||
+ ecbdata->diff_words->plus.text.size)
+ diff_words_show(ecbdata->diff_words);
+ }
len = sane_truncate_line(ecbdata, line, len);
find_lno(line, ecbdata);
emit_line(ecbdata->file,
But then I thought I should not put the diff output from --color-words
into the block that deals with the hunk header, but save another place
where diff_words_show() is called.
Markus
^ permalink raw reply
* Re: [PATCH/RFC 2/2] Provide a build time default-editor setting
From: Johannes Sixt @ 2009-10-29 11:50 UTC (permalink / raw)
To: David Roundy; +Cc: Jonathan Nieder, Junio C Hamano, Ben Walton, GIT List
In-Reply-To: <117f2cc80910290336k1e7b5877uc472ad511bb6f5ae@mail.gmail.com>
David Roundy schrieb:
> Any chance this will be exported as plumbing? I know it's pretty
> high-level, but it'd be handy to have be able to write `git editor
> $FILENAME` and just have it do the right thing. This would also mean
> that the perl scripts below could be simplified.
Something like below? Possible usage in shell scripts:
editor=$(git var GIT_EDITOR)
"$editor" "$filename"
-- Hannes
PS: warning: linewrapped.
Subject: [PATCH] Teach git var about GIT_EDITOR
Signed-off-by: Johannes Sixt <j6t@kdbg.org>
---
cache.h | 1 +
editor.c | 13 +++++++++++--
var.c | 6 ++++++
3 files changed, 18 insertions(+), 2 deletions(-)
diff --git a/cache.h b/cache.h
index a5eeead..3103dda 100644
--- a/cache.h
+++ b/cache.h
@@ -750,6 +750,7 @@ extern const char *git_author_info(int);
extern const char *git_committer_info(int);
extern const char *fmt_ident(const char *name, const char *email, const
char *date_str, int);
extern const char *fmt_name(const char *name, const char *email);
+extern const char *git_editor();
struct checkout {
const char *base_dir;
diff --git a/editor.c b/editor.c
index 4d469d0..bd8c828 100644
--- a/editor.c
+++ b/editor.c
@@ -2,7 +2,7 @@
#include "strbuf.h"
#include "run-command.h"
-int launch_editor(const char *path, struct strbuf *buffer, const char
*const *env)
+const char *git_editor()
{
const char *editor, *terminal;
@@ -16,11 +16,20 @@ int launch_editor(const char *path, struct strbuf
terminal = getenv("TERM");
if (!editor && (!terminal || !strcmp(terminal, "dumb")))
- return error("Terminal is dumb but no VISUAL nor EDITOR defined.");
+ return "/dev/null";
if (!editor)
editor = "vi";
+ return editor;
+}
+
+int launch_editor(const char *path, struct strbuf *buffer, const char
*const *env)
+{
+ const char *editor = git_editor();
+
+ if (!strcmp(editor, "/dev/null"))
+ return error("Terminal is dumb but no VISUAL nor EDITOR defined.");
if (strcmp(editor, ":")) {
size_t len = strlen(editor);
int i = 0;
diff --git a/var.c b/var.c
index 125c0d1..48d8b9a 100644
--- a/var.c
+++ b/var.c
@@ -8,6 +8,11 @@
static const char var_usage[] = "git var [-l | <variable>]";
+static const char *editor(int unused)
+{
+ return git_editor();
+}
+
struct git_var {
const char *name;
const char *(*read)(int);
@@ -15,6 +20,7 @@ struct git_var {
static struct git_var git_vars[] = {
{ "GIT_COMMITTER_IDENT", git_committer_info },
{ "GIT_AUTHOR_IDENT", git_author_info },
+ { "GIT_EDITOR", editor },
{ "", NULL },
};
--
1.6.5.rc2.47.g49402
^ permalink raw reply related
* Re: [PATCH 1/7] Refactor git_remote_cvs to a more generic git_remote_helpers
From: Johan Herland @ 2009-10-29 12:05 UTC (permalink / raw)
To: Sverre Rabbelier; +Cc: Git List, Johannes Schindelin, Daniel Barkalow
In-Reply-To: <1256798426-21816-2-git-send-email-srabbelier@gmail.com>
On Thursday 29 October 2009, Sverre Rabbelier wrote:
> This in an effort to allow future remote helpers written in python to
> re-use the non-cvs-specific code.
>
> Signed-off-by: Sverre Rabbelier <srabbelier@gmail.com>
> CC: Johan Herland <johan@herland.net>
> ---
>
> As discussed with Johan Herland, refactored git_remote_cvs into a
> more reusable git_remote_helpers module.
It's hard to review this patch for a couple of reasons:
1. It's over 200K large, which causes it to be blocked by the Git mailing
list (100K limit, I believe).
2. The patch renames some files, but instead of simply stating the rename,
the patch lists the entire file twice (deletion + creation)
Fortunately, you can easily solve both problems by rerolling the patch with
the -M flag to git-format-patch.
...Johan
--
Johan Herland, <johan@herland.net>
www.herland.net
^ permalink raw reply
* msysgit won't install as per instructions
From: Anthony W. Youngman @ 2009-10-29 12:38 UTC (permalink / raw)
To: git
I want to do some work on git-daemon under msysgit. And as a newbie to
git it could well be I'm just being stupid, but ...
I installed msysgit, did a clone on the msysgit repository, and followed
the instructions, namely double-click on msys.bat. It falls over with
"no target for make install" - I know make well enough to know that
*sounds* *like* a messed up makefile. But lack of experience means
trying to track how it got there defeats me. The error output is below.
Cheers,
Wol
-------------------------------------------------------
Building and Installing Git
-------------------------------------------------------
make: *** No rule to make target `install'. Stop.
-------------------------
Hello, dear Git developer.
This is a minimal MSYS environment to work on Git.
Your build failed... Please fix it, and give feedback on the Git list.
Welcome to msysGit
Run 'git help git' to display the help index.
Run 'git help <command>' to display help for specific commands.
Run '/share/msysGit/add-shortcut.tcl' to add a shortcut to msysGit.
bash: /git/contrib/completion/git-completion.bash: No such file or
directory
bash: __git_ps1: command not found
--
Anthony W. Youngman - anthony@thewolery.demon.co.uk
^ permalink raw reply
* Re: [PATCH 1/3] add splash screen
From: Paolo Bonzini @ 2009-10-29 12:48 UTC (permalink / raw)
To: Jeff King; +Cc: git
In-Reply-To: <20091029002400.GA1057@sigill.intra.peff.net>
On 10/29/2009 01:24 AM, Jeff King wrote:
> Because bash completion is so slow to start, we need to
> entertain users with a splash screen, so reuse the one from
> git-gui.
I suggest adding "wm overrideredirect . 1; wm geometry . +300+300".
Paolo
^ permalink raw reply
* Re: date change of commit?
From: Alex K @ 2009-10-29 12:55 UTC (permalink / raw)
To: Miklos Vajna; +Cc: Matthieu Moy, git
In-Reply-To: <20091027122156.GD6115@genesis.frugalware.org>
And how do you actually set those variables? Sorry to ask such a
trivial question but it's been an hour that i'm going through the doc
for such a simple feature. I thought those were environment variables
... but they are not seen under git var -l. Thank you.
2009/10/27 Miklos Vajna <vmiklos@frugalware.org>:
> On Tue, Oct 27, 2009 at 10:41:47AM +0100, Alex K <spaceoutlet@gmail.com> wrote:
>> Thank you. And how would you use git-filter-branch to create another
>> branch with a different time stamp? Is it possible to commit under a
>> different time stamp than the one provided by your default local time?
>
> You can set GIT_AUTHOR_DATE and GIT_COMMITTER_DATE. Both expect a format
> like: "1112911993 -0700" (unix timestamp + timezone info).
>
^ permalink raw reply
* Re: How does format-patch determine the filename of the patch?
From: Timur Tabi @ 2009-10-29 13:05 UTC (permalink / raw)
To: Robin Rosenberg; +Cc: git
In-Reply-To: <200910151959.47778.robin.rosenberg.lists@dewire.com>
On Thu, Oct 15, 2009 at 12:59 PM, Robin Rosenberg
<robin.rosenberg.lists@dewire.com> wrote:
>> The reason I ask is that I'm writing a script which calls
>> git-format-patch to create some patches for post-processing. So I
>> need the name of the file that git-format-patch creates so that I can
>> open it and examine it. I'd liked to see if there's a way to get the
>> name of the patch without actually creating the file.
>
> It tells you the names on stdout.
Is there a way for it to tell me the name on stdout *without* actually
creating the patch?
--
Timur Tabi
Linux kernel developer at Freescale
^ permalink raw reply
page: next (older) | prev (newer) | latest
- recent:[subjects (threaded)|topics (new)|topics (active)]
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox