Trying to complete the command
git show master:./file
would cause a Not a valid object name error to be output on standard
error. Silence the error so it won't appear on the command line.
Signed-off-by: Dylan Smith dylan.ah.sm...@gmail.com
---
contrib/completion/git-completion.bash |2 +-
The user can do --depth=2147483647 (*) for restoring complete repo
now. But it's hard to remember. Any other numbers larger than the
longest commit chain in the repository would also do, but some
guessing may be involved. Make easy-to-remember --unshallow an alias
for --depth=2147483647.
Make
get_shallow_commits() is used to determine the cut points at a given
depth (i.e. the number of commits in a chain that the user likes to
get). However we count current depth up to the commit commit but we
do the cutting at its parents (i.e. current depth + 1). This makes
upload-pack always return
--depth is explained as deepen, but the way it's applied, it can
shorten the history as well. Keen users may have noticed the
implication by the phrase the specified number of commits from the
tip of each remote branch. Put shorten in the description to make
it clearer.
Signed-off-by: Nguyễn Thái
On Fri, 11 Jan 2013 11:30:01 +0400
Ishayahu Lastov meoc...@mail.ru wrote:
[...]
As I understand after last git checkout in git status I should see
that I gave no changes. It looks like an bug, isn't it?
Looks like an EOL-conversion problem rather typical to Windows, see
On Fri, Jan 11, 2013 at 12:58 AM, Junio C Hamano gits...@pobox.com wrote:
I can see why it is wrong to let pathspec.raw be rewritten without
making matching change to the containing pathspec, but I find it
strange why it matters only on case-insensitive codepath.
Yeah, I don't get it either. I
On Fri, Jan 11, 2013 at 6:09 AM, Junio C Hamano gits...@pobox.com wrote:
Or I could hold off nd/parse-pathspec if this series has a better
chance of graduation first. Decision?
I am greedy and want to have both ;-)
Apparently I have no problems with your being greedy.
There is no textual
Currently to fill a struct pathspec, we do:
const char **paths;
paths = get_pathspec(prefix, argv);
...
init_pathspec(pathspec, paths);
paths can only carry bare strings, which loses information from
command line arguments such as pathspec magic or the prefix part's
length for each
These call sites follow the pattern:
paths = get_pathspec(prefix, argv);
init_pathspec(pathspec, paths);
which can be converted into a single parse_pathspec() call.
Signed-off-by: Nguyễn Thái Ngọc Duy pclo...@gmail.com
---
builtin/grep.c | 4 +---
builtin/ls-tree.c | 2 +-
Signed-off-by: Nguyễn Thái Ngọc Duy pclo...@gmail.com
---
builtin/clean.c | 20 ++--
1 file changed, 10 insertions(+), 10 deletions(-)
diff --git a/builtin/clean.c b/builtin/clean.c
index 69c1cda..788ad8c 100644
--- a/builtin/clean.c
+++ b/builtin/clean.c
@@ -42,7 +42,7 @@ int
Signed-off-by: Nguyễn Thái Ngọc Duy pclo...@gmail.com
---
builtin/commit.c | 18 +-
cache.h | 3 +++
setup.c | 3 +++
3 files changed, 15 insertions(+), 9 deletions(-)
diff --git a/builtin/commit.c b/builtin/commit.c
index d6dd3df..444ae1d 100644
---
Signed-off-by: Nguyễn Thái Ngọc Duy pclo...@gmail.com
---
builtin/commit.c | 9 +
wt-status.c | 17 +++--
wt-status.h | 2 +-
3 files changed, 13 insertions(+), 15 deletions(-)
diff --git a/builtin/commit.c b/builtin/commit.c
index 444ae1d..196dfab 100644
---
Signed-off-by: Nguyễn Thái Ngọc Duy pclo...@gmail.com
---
builtin/rerere.c | 6 +++---
rerere.c | 8
rerere.h | 4 +++-
3 files changed, 10 insertions(+), 8 deletions(-)
diff --git a/builtin/rerere.c b/builtin/rerere.c
index dc1708e..a573c4a 100644
---
Signed-off-by: Nguyễn Thái Ngọc Duy pclo...@gmail.com
---
builtin/checkout.c | 30 ++
1 file changed, 14 insertions(+), 16 deletions(-)
diff --git a/builtin/checkout.c b/builtin/checkout.c
index a9c1b5a..da25298 100644
--- a/builtin/checkout.c
+++
Signed-off-by: Nguyễn Thái Ngọc Duy pclo...@gmail.com
---
builtin/rm.c | 16 +++-
1 file changed, 7 insertions(+), 9 deletions(-)
diff --git a/builtin/rm.c b/builtin/rm.c
index dabfcf6..d719d95 100644
--- a/builtin/rm.c
+++ b/builtin/rm.c
@@ -216,7 +216,7 @@ static struct option
strip_trailing_slash_from_submodules() modifies pathspec and is moved
to dir.c, close to other pathspec code. It'll be removed later when
parse_pathspec() learns to take over its job.
Signed-off-by: Nguyễn Thái Ngọc Duy pclo...@gmail.com
---
builtin/ls-files.c | 42
Signed-off-by: Nguyễn Thái Ngọc Duy pclo...@gmail.com
---
archive.c | 10 --
archive.h | 2 +-
2 files changed, 5 insertions(+), 7 deletions(-)
diff --git a/archive.c b/archive.c
index 4666404..530badb 100644
--- a/archive.c
+++ b/archive.c
@@ -150,7 +150,6 @@ int
treat_gitlinks() modifies pathspec and is moved to dir.c, close to
other pathspec code. It'll be removed later when parse_pathspec()
learns to take over its job. Note that treat_gitlinks() and
strip_trailing_slash_from_submodules() do not perform exactly the same
thing. But that does not matter
Signed-off-by: Nguyễn Thái Ngọc Duy pclo...@gmail.com
---
builtin/checkout.c | 2 +-
builtin/commit.c | 4 ++--
builtin/diff-files.c | 2 +-
builtin/diff-index.c | 2 +-
builtin/diff.c | 4 ++--
cache.h | 4 +++-
preload-index.c | 20 +++-
7
Signed-off-by: Nguyễn Thái Ngọc Duy pclo...@gmail.com
---
builtin/checkout.c | 2 +-
rerere.c | 2 +-
resolve-undo.c | 4 ++--
resolve-undo.h | 2 +-
4 files changed, 5 insertions(+), 5 deletions(-)
diff --git a/builtin/checkout.c b/builtin/checkout.c
index 00910dc..aa399d6
Signed-off-by: Nguyễn Thái Ngọc Duy pclo...@gmail.com
---
builtin/checkout.c | 9 +++--
tree.c | 4 ++--
tree.h | 2 +-
3 files changed, 6 insertions(+), 9 deletions(-)
diff --git a/builtin/checkout.c b/builtin/checkout.c
index aa399d6..a7ddb35 100644
---
Signed-off-by: Nguyễn Thái Ngọc Duy pclo...@gmail.com
---
builtin/checkout.c | 2 +-
builtin/commit.c | 14 ++
builtin/ls-files.c | 14 --
cache.h| 2 +-
4 files changed, 16 insertions(+), 16 deletions(-)
diff --git a/builtin/checkout.c
Signed-off-by: Nguyễn Thái Ngọc Duy pclo...@gmail.com
---
builtin/add.c| 14 ++
builtin/commit.c | 2 +-
builtin/rm.c | 2 +-
cache.h | 2 +-
read-cache.c | 5 +++--
5 files changed, 12 insertions(+), 13 deletions(-)
diff --git a/builtin/add.c
Signed-off-by: Nguyễn Thái Ngọc Duy pclo...@gmail.com
---
builtin/add.c | 2 +-
builtin/clean.c| 2 +-
builtin/grep.c | 2 +-
builtin/ls-files.c | 2 +-
dir.c | 10 +-
dir.h | 4 ++--
wt-status.c| 4 ++--
7 files changed, 13
Signed-off-by: Nguyễn Thái Ngọc Duy pclo...@gmail.com
---
builtin/add.c| 8 +---
builtin/commit.c | 2 +-
cache.h | 2 +-
3 files changed, 7 insertions(+), 5 deletions(-)
diff --git a/builtin/add.c b/builtin/add.c
index 0295cef..5d262eb 100644
--- a/builtin/add.c
+++
init_pathspec() was introduced to work with the result from
get_pathspec(). init_pathspec() will be removed eventually after
parse_pathspec() takes over, so that there is only place that
initializes struct pathspec.
Signed-off-by: Nguyễn Thái Ngọc Duy pclo...@gmail.com
---
archive.c |
This patch is essentially no-op. It helps catching new use of this
field though. This field is introduced as an intermediate step for the
pathspec conversion and will be removed eventually. At this stage no
more access sites should be introduced.
Signed-off-by: Nguyễn Thái Ngọc Duy
On Fri, Jan 11, 2013 at 1:46 PM, Ishayahu Lastov meoc...@mail.ru wrote:
This is my session on Win7 x64:
Microsoft Windows [Version 6.1.7601]
(c) Корпорация Майкрософт (Microsoft Corp.), 2009. Все права защищены.
C:\Dropbox\Dropbox\Wesnoth\Apocryphscd Apokryphs.Orks
On Thu, Jan 10, 2013 at 4:04 AM, Junio C Hamano gits...@pobox.com wrote:
Here are the topics that have been cooking. Commits prefixed with
'-' are only in 'pu' (proposed updates) while commits prefixed with
'+' are in 'next'.
I see you start to use no-change commits as a way to keep notes in
On Fri, Jan 11, 2013 at 06:21:11PM +0700, Nguyễn Thái Ngọc Duy wrote:
- for (i = 0; i specs; i++) {
+ for (i = 0; i pathspec-nr; i++) {
if (!seen[i])
- die(_(pathspec '%s' did not match any files),
pathspec[i]);
+
Signed-off-by: Nguyễn Thái Ngọc Duy pclo...@gmail.com
---
I was digging the t7400.38 failure and found that add.c is the last
place that uses match_pathspec(). This is the first yay moment for me
since 'struct pathspec' was introduced. I wanted to remove this
function for so long!
Good luck
On Fri, Jan 11, 2013 at 03:06:22AM -0500, Dylan Smith wrote:
Trying to complete the command
git show master:./file
would cause a Not a valid object name error to be output on standard
error. Silence the error so it won't appear on the command line.
Signed-off-by: Dylan Smith
Duy Nguyen pclo...@gmail.com writes:
On Thu, Jan 10, 2013 at 4:04 AM, Junio C Hamano gits...@pobox.com wrote:
Here are the topics that have been cooking. Commits prefixed with
'-' are only in 'pu' (proposed updates) while commits prefixed with
'+' are in 'next'.
I see you start to use
On Thu, Jan 10, 2013 at 1:10 PM, Karsten Blees karsten.bl...@gmail.com wrote:
Changes since initial version (see attached diff for details):
- split in two patches
- removed unused variables
- improved the dll error message
- changed ?: to if else
- added comments
Also available here:
From: Eric S. Raymond e...@thyrsus.com
...
diff --git a/git-cvsimport.perl b/git-cvsimport-fallback.perl
similarity index 98%
rename from git-cvsimport.perl
rename to git-cvsimport-fallback.perl
index 0a31ebd..4bc0717 100755
--- a/git-cvsimport.perl
+++ b/git-cvsimport-fallback.perl
@@
[raising this UI wart to the git list]
On 01/11/2013 01:42 AM, Peter Krempa wrote:
On 01/11/13 07:31, Chunyan Liu wrote:
This patch series is to...
[snip]
Please don't answer y when git send email shows the following prompt:
Message-ID to be used as In-Reply-To for the first email?
On Fri, Jan 11, 2013 at 04:40:38PM +0530, Sitaram Chamarty wrote:
I find a lot of info on how to recover from and/or repair a repo that
has missing (or corrupted) objects.
What I need is info on common reasons (other than disk errors -- we've
checked for those) for such errors to occur, any
Dylan Smith dylan.ah.sm...@gmail.com writes:
Trying to complete the command
git show master:./file
would cause a Not a valid object name error to be output on standard
error. Silence the error so it won't appear on the command line.
Signed-off-by: Dylan Smith dylan.ah.sm...@gmail.com
On Fri, Jan 11, 2013 at 09:39:06AM -0700, Eric Blake wrote:
Please don't answer y when git send email shows the following prompt:
Message-ID to be used as In-Reply-To for the first email?
you should respond with a message ID there. Unfortunately we have a
growing thread that
Martin von Zweigbergk martinv...@gmail.com writes:
Justification?
Clairvoyance ...
;-)
--
To unsubscribe from this list: send the line unsubscribe git in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
On 01/11/2013 09:47 AM, Jeff King wrote:
On Fri, Jan 11, 2013 at 09:39:06AM -0700, Eric Blake wrote:
Please don't answer y when git send email shows the following prompt:
Anyone willing to patch upstream 'git send-email' to reject a simple 'y'
What version of git? Commit 51bbccf is in
On Fri, Jan 11, 2013 at 06:20:57PM +0700, Nguyễn Thái Ngọc Duy wrote:
+#define PATHSPEC_FROMTOP(10)
The previous commit introduces a use of this macro in get_pathspec.
Should this be defined by that commit instead?
@@ -266,9 +266,9 @@ static int pathspec_item_cmp(const void *a_, const void
Very cool! I'm glad you got it doing what you wanted; I'll be
interested to see how parsecvs compares in quality and performance to
cvs2git and cvsps. --Bart
On Fri, Jan 11, 2013 at 3:21 AM, Eric S. Raymond e...@thyrsus.com wrote:
Since Heiko Voigt and others were concerned about this, I report
On 11 January 2013 08:47, Jeff King p...@peff.net wrote:
On Fri, Jan 11, 2013 at 09:39:06AM -0700, Eric Blake wrote:
Please don't answer y when git send email shows the following prompt:
Message-ID to be used as In-Reply-To for the first email?
you should respond with a message ID
The git-completion.bash script did not implemented full, git aware,
support to complete paths, for git commands that operate on files within
the current working directory or the index.
As an example:
git add TAB
will suggest all files in the current working directory, including
ignored
On Fri, Jan 11, 2013 at 10:43:39AM -0800, Hilco Wijbenga wrote:
People answer 'y' to Who should the emails appear to be from? and
'n' to Message-ID to be used as In-Reply-To for the first email?
for some unknown reason. While it is possible that your local
username really is y
Eric S. Raymond e...@thyrsus.com writes:
Junio C Hamano gits...@pobox.com:
I think the prevalent style in this script is to write print
without parentheses:
print STDERR msg\n;
That can be easily fixed.
This looks lazy and unsafe quoting. Is there anything that makes
sure
Eric S. Raymond e...@thyrsus.com writes:
Junio C Hamano gits...@pobox.com:
...
The other is a design-level problem - these options were a bad idea to
begin with. In earlier list mail I said
An example of the batchiness mistake close to home is the -m and -M
options in the old
This short discussion on GitHub (file git-compat-util.h) might be relevant:
https://github.com/msysgit/git/commit/435bdf8c7ffa493f8f6f2e8f329f8cc22db16ce6#commitcomment-2407194
The change suggested there (to remove an inclusion of windows.h in
git-compat-util.h) might simplify the solution a
On Fri, Jan 11, 2013 at 9:08 PM, Alex Riesen raa.l...@gmail.com wrote:
This short discussion on GitHub (file git-compat-util.h) might be relevant:
https://github.com/msysgit/git/commit/435bdf8c7ffa493f8f6f2e8f329f8cc22db16ce6#commitcomment-2407194
The change suggested there (to remove an
On Thu, Jan 10, 2013 at 9:51 PM, Herry Wang tech.he...@gmail.com wrote:
OS:
Darwin ... 11.4.2 Darwin Kernel Version 11.4.2: Thu Aug 23 16:25:48 PDT
2012; root:xnu-1699.32.7~1/RELEASE_X86_64 x86_64
Git: version: (via homebrew)
git --version
git version 1.8.0
curl version:
curl
Jeff King p...@peff.net wrote in message
news:2013085417.ga12...@sigill.intra.peff.net...
On Fri, Jan 11, 2013 at 10:43:39AM -0800, Hilco Wijbenga wrote:
How about What Message-ID to use as In-Reply-To for the first email?
or Provide the Message-ID to use as In-Reply-To for the first
On Wed, Jan 09, 2013 at 10:52:48AM -0800, Bruce Lysik wrote:
Trying to run git-archive fails against smart-http based repos. Example:
$ git archive --verbose --format=zip
--remote=http://code.toofishes.net/git/dan/initscripts.git
fatal: Operation not supported by protocol.
Unexpected end
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
Il 11/01/2013 09:06, Dylan Smith ha scritto:
Trying to complete the command
git show master:./file
would cause a Not a valid object name error to be output on standard
error. Silence the error so it won't appear on the command line.
I
On Fri, Jan 11, 2013 at 08:13:57PM +, Matt Seitz (matseitz) wrote:
How about What Message-ID to use as In-Reply-To for the first email?
or Provide the Message-ID to use as In-Reply-To for the first
email:.
seem fine to me. Maybe somebody who has been confused by it can offer
On Fri, Jan 11, 2013 at 10:23 PM, Jeff King p...@peff.net wrote:
On Fri, Jan 11, 2013 at 08:13:57PM +, Matt Seitz (matseitz) wrote:
How about What Message-ID to use as In-Reply-To for the first email?
or Provide the Message-ID to use as In-Reply-To for the first
email:.
seem
Manlio Perillo manlio.peri...@gmail.com writes:
+# Process path list returned by ls-files and diff-index --name-only
+# commands, in order to list only file names relative to a specified
+# directory, and append a slash to directory names.
+__git_index_file_list_filter ()
+{
+ # Default
All the other callers of logmsg_reencode() pass return value of
get_commit_output_encoding() or get_log_output_encoding(). Teach
the function to optionally take NULL as a synonym to aka no
conversion requested so that we can simplify the only remaining
calling site.
Signed-off-by: Junio C
Please disregard this...
--
To unsubscribe from this list: send the line unsubscribe git in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Antoine Pelisse apeli...@gmail.com writes:
On Fri, Jan 11, 2013 at 10:23 PM, Jeff King p...@peff.net wrote:
On Fri, Jan 11, 2013 at 08:13:57PM +, Matt Seitz (matseitz) wrote:
How about What Message-ID to use as In-Reply-To for the first email?
or Provide the Message-ID to use as
On Fri, Jan 11, 2013 at 11:18 PM, Junio C Hamano gits...@pobox.com wrote:
The exchange, when you do not have a configuration, goes like this:
$ git send-email 0001-filename-of-the-patch.patch
0001-filename-of-the-patch.patch
Who should the emails be sent to (if any)? junio
Antoine Pelisse apeli...@gmail.com writes:
I would simply go for:
What Message-ID are you replying to (if any)?
If I don't know what to answer, I would definitely not say y/yes/n/no,
but press enter directly.
Sounds sensible (even though technically you reply to a message
that has that
Here are the topics that have been cooking. Commits prefixed with
'-' are only in 'pu' (proposed updates) while commits prefixed with
'+' are in 'next'.
As usual, this cycle is expected to last for 8 to 10 weeks, with a
preview -rc0 sometime in the middle of next month.
You can find the changes
Thanks for the very detailed answer.
On Fri, Jan 11, 2013 at 10:12 PM, Jeff King p...@peff.net wrote:
On Fri, Jan 11, 2013 at 04:40:38PM +0530, Sitaram Chamarty wrote:
I find a lot of info on how to recover from and/or repair a repo that
has missing (or corrupted) objects.
What I need is
On 12/01/2013 10:54 AM, Junio C Hamano wrote:
Antoine Pelisse apeli...@gmail.com writes:
I would simply go for:
What Message-ID are you replying to (if any)?
If I don't know what to answer, I would definitely not say y/yes/n/no,
but press enter directly.
Sounds sensible (even though
Ben Aveling bena@optusnet.com.au writes:
On 12/01/2013 10:54 AM, Junio C Hamano wrote:
Antoine Pelisse apeli...@gmail.com writes:
I would simply go for:
What Message-ID are you replying to (if any)?
If I don't know what to answer, I would definitely not say y/yes/n/no,
but press
Import of a trivial CVS repository fails due to a cvsps bug. Given the
following series of commits:
timestamp abc message
--- --- --- --- ---
2012/12/12 21:09:39 1.1changes are done
2012/12/12 21:09:441.1
Import of a trivial CVS repository fails due to a cvsps bug. Given the
following series of commits:
timestamp abc message
--- --- --- --- ---
2012/12/12 21:09:39 1.1changes are done
2012/12/12 21:09:441.1
I cloned git://gitorious.org/cvsps/cvsps.git and installed cvsps-3.7
at c2ce6cc (More fun with test loads, sigh. Timezones suck.,
2013-01-09) earlier on my $PATH, and tried to run t96xx series with
this patch applied on top of Git 1.8.1.
The first thing I noticed was that all the tests were
The rewrite patch was supposed to remove these scripts, but somehow
we ended up removing only the supporting files for them but not the
test script themselves. Remove them for real.
Signed-off-by: Junio C Hamano gits...@pobox.com
---
* I'll queue this on top of your patch together with a few
Hi,
I believe `remote set-url --add --push` has a bug. Performed tests
with v1.8.0.1 and v1.8.1 (Mac OS X).
Quoting the relevant part of the documentation:
set-url
Changes URL remote points to. Sets first URL remote points to matching
regex oldurl (first URL if no oldurl is given) to
The test Makefile has a default set of lint tests which are run
as part of make test.
The macro TEST_LINT defaults to test-lint-duplicates test-lint-executable.
Add test-lint-shell-syntax here, to detect non-portable shell syntax early.
Signed-off-by: Torsten Bögershausen tbo...@web.de
---
Torsten Bögershausen tbo...@web.de writes:
The test Makefile has a default set of lint tests which are run
as part of make test.
The macro TEST_LINT defaults to test-lint-duplicates test-lint-executable.
Add test-lint-shell-syntax here, to detect non-portable shell syntax early.
On Sat, Jan 12, 2013 at 12:56 AM, Matt Kraai kr...@ftbfs.org wrote:
On Fri, Jan 11, 2013 at 06:20:57PM +0700, Nguyễn Thái Ngọc Duy wrote:
+#define PATHSPEC_FROMTOP(10)
The previous commit introduces a use of this macro in get_pathspec.
Should this be defined by that commit instead?
This
On Fri, Jan 11, 2013 at 11:38 PM, Junio C Hamano gits...@pobox.com wrote:
By the way, Chris, we'll need your Sign-off on the three paches
(t/lib-cvs.sh fix to allow cvsps v3, t9600 fix and t9604 fix).
Sure. I was just maintaining them for myself but thought I'd share
when I saw the
Reroll w/ sign-off.
Chris Rorvick (3):
t/lib-cvs.sh: allow cvsps version 3.x.
t9600: fixup for new cvsimport
t9604: fixup for new cvsimport
t/lib-cvs.sh| 2 +-
t/t9600-cvsimport.sh| 10 --
t/t9604-cvsimport-timestamps.sh | 5 ++---
3 files
cvsimport no longer supports -a (import all commits including recent ones)
and no longer uses the 'origin' branch by default for imports.
Signed-off-by: Chris Rorvick ch...@rorvick.com
---
t/t9600-cvsimport.sh | 10 --
1 file changed, 4 insertions(+), 6 deletions(-)
diff --git
cvsps no longer writes a cache file and therefore no longer can be told
to ignore it with -x.
Signed-off-by: Chris Rorvick ch...@rorvick.com
---
t/t9604-cvsimport-timestamps.sh | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/t/t9604-cvsimport-timestamps.sh
Chris Rorvick ch...@rorvick.com writes:
Reroll w/ sign-off.
Chris Rorvick (3):
t/lib-cvs.sh: allow cvsps version 3.x.
t9600: fixup for new cvsimport
t9604: fixup for new cvsimport
t/lib-cvs.sh| 2 +-
t/t9600-cvsimport.sh| 10 --
On Sat, Jan 12, 2013 at 12:36 AM, Junio C Hamano gits...@pobox.com wrote:
I too noticed the droppage of -a support, which may not be a big
deal (people can drop it from their script, run cvsimport and they
can drop newer commits from the resulting Git history to emulate the
old behaviour
Jardel Weyrich jweyr...@gmail.com writes:
I believe `remote set-url --add --push` has a bug. Performed tests
with v1.8.0.1 and v1.8.1 (Mac OS X).
Quoting the relevant part of the documentation:
set-url
Changes URL remote points to. Sets first URL remote points to matching
regex
81 matches
Mail list logo
