On Thu, Jul 10, 2014 at 8:31 PM, David Turner dtur...@twopensource.com wrote:
Add tests to confirm that invalidation of subdirectories neither over-
nor under-invalidates.
Signed-off-by: David Turner dtur...@twitter.com
---
t/t0090-cache-tree.sh | 26 +++---
1 file
I've used git-p4 for several years now and it's generally working well
for me.
The only thing that bugs me at this time is having to re-clone
regularly. Here is how this happens:
* Say my p4 client maps //foo/bar/... to /home/jdoe/perforce/foo/bar/...
(I don't want to clone the entire repo,
Tested-by: Dennis Kaarsemaker den...@kaarsemaker.net
I've been using this branch for a little while now and have no breakages
to report. Max Kirillov reported some bugs in the interaction with
submodules which I plan to chase when I have some time unless someone
beats me to it :)
On wo,
On Fri, Jul 11, 2014 at 01:59:53AM +0100, Ramsay Jones wrote:
The code you're touching here was trying to make sure that each commit
gets a unique index, under the assumption that commits only get
allocated via alloc_commit_node. But I think that assumption is wrong.
We can also get
Here's a series to address the bug I mentioned earlier by catching the
conversion of OBJ_NONE to OBJ_COMMIT in a central location and setting
the index there.
I've included your patch 1/2 unchanged in the beginning, as I build on
top of it (and your patch 2/2 is no longer applicable). The rest
From: Ramsay Jones ram...@ramsay1.demon.co.uk
In order to encapsulate the setting of the unique commit index, commit
969eba63 (commit: push commit_index update into alloc_commit_node,
10-06-2014) introduced a (logically private) intermediary allocator
function. However, this function
The only way that obj can be non-NULL is if it came from
one of the lookup_* functions. These functions always ensure
that the object has the expected type (and return NULL
otherwise), so there is no need for us to set the type.
Signed-off-by: Jeff King p...@peff.net
---
object.c | 2 --
1 file
The struct object type implements basic object
polymorphism. Individual instances are allocated as
concrete types (or as a union type that can store any
object), and a struct object * can be cast into its real
type after examining its type enum. This means it is
dangerous to have a type field
When we call lookup_commit, lookup_tree, etc, the logic goes
something like:
1. Look for an existing object struct. If we don't have
one, allocate and return a new one.
2. Double check that any object we have is the expected
type (and complain and return NULL otherwise).
3.
We keep a static counter to set the commit index on newly
allocated objects. However, since we also need to set the
index on any_objects which are converted to commits, let's
make the counter available as a public function.
While we're moving it, let's make sure the counter is
allocated as an
The point of the index field of struct commit is that
every allocated commit would have a uniquely allocated
value. It is supposed to be an invariant that whenever
object-type is set to OBJ_COMMIT, we have a unique index.
Commit 969eba6 (commit: push commit_index update into
alloc_commit_node,
We generally want to avoid lookup_unknown_object, because it
results in allocating more memory for the object than may be
strictly necessary.
In this case, it is used to check whether we have an
already-parsed object before calling parse_object, to save
us from reading the object from disk. Using
On Thu, Jul 10, 2014 at 7:36 PM, Junio C Hamano gits...@pobox.com wrote:
Junio C Hamano gits...@pobox.com writes:
As the user might expect that a new replace ref was created on success
(0 exit code), and as we should at least warn if we would create a
commit that is the same as an existing
2014-07-09 19:40 GMT+09:00 Peter Krefting pe...@softwolves.pp.se:
Yi EungJun:
Example:
LANGUAGE= -
LANGUAGE=ko - Accept-Language: ko; q=1.000, *; q=0.001
LANGUAGE=ko:en - Accept-Language: ko; q=1.000, en; q=0.999, *; q=0.001
Avoid adding q=1.000. It is redundant (the default for any
Hi,
git seems to have issues with alternates when cycles are present (repo A
has B/objects as alternates, B has A/objects as alternates). In such
cases, gc and repack might delete objects that are present in only one
of the alternates, leading to data loss.
I understand that this is no big
On 11/07/14 09:32, Jeff King wrote:
On Fri, Jul 11, 2014 at 01:59:53AM +0100, Ramsay Jones wrote:
The code you're touching here was trying to make sure that each commit
gets a unique index, under the assumption that commits only get
allocated via alloc_commit_node. But I think that
2014-07-11 5:10 GMT+09:00 Jeff King p...@peff.net:
On Wed, Jul 09, 2014 at 11:46:14AM +0100, Peter Krefting wrote:
Jeff King:
I did some digging, and I think the public API is setlocale with a NULL
parameter, like:
printf(%s\n, setlocale(LC_MESSAGES, NULL));
That still will end up
From: Yi EungJun eungjun...@navercorp.com
Add an Accept-Language header which indicates the user's preferred
languages defined by $LANGUAGE, $LC_ALL, $LC_MESSAGES and $LANG.
Examples:
LANGUAGE= -
LANGUAGE=ko:en - Accept-Language: ko, en; q=0.9, *; q=0.1
LANGUAGE=ko LANG=en_US.UTF-8 -
There is an incorrect message when doing git rebase -i remote/branch.
I have it only in german, see below. what happend is:
#01 make changes on another host
#02 copy patchfile to localhost
#03 apply patchfile
#04 git commit -avs # create commit#1
#05 sleep 123456
#06 make different changes on
On 11/07/14 09:41, Jeff King wrote:
Here's a series to address the bug I mentioned earlier by catching the
conversion of OBJ_NONE to OBJ_COMMIT in a central location and setting
the index there.
I've included your patch 1/2 unchanged in the beginning, as I build on
top of it (and your patch
On 11/07/14 09:48, Jeff King wrote:
[snip]
diff --git a/object.c b/object.c
index 472aa8d..b2319f6 100644
--- a/object.c
+++ b/object.c
@@ -158,6 +158,23 @@ void *create_object(const unsigned char *sha1, void *o)
return obj;
}
+void *object_as_type(struct object *obj, enum
Hi all,
I am running git on Windows 8.1 (with all the latest updates installed),
and it consequently crashes when I run git pull in my cloned working copy.
I attached a screen shot of the message window (it is in Dutch...)
This is my git version:
$ git --version
git version 1.9.4.msysgit.0
wrote to:
Dear Beneficiary,
This is to inform you that you have been awarded the sum of (USD$1,200,000.00)
as charity donations/aid from the Qatar Foundation, held on 7th of July 2014 in
Qatar. Reply for more information via e-mail: qatarharit...@gmail.com
Yours Sincerely,
Sheikh Saad Al
On 2014-07-11 12.49, Ronald Bos wrote:
Hi all,
I am running git on Windows 8.1 (with all the latest updates installed), and
it consequently crashes when I run git pull in my cloned working copy.
I attached a screen shot of the message window (it is in Dutch...)
This is my git version:
Hi Chris,
you're the original author of the code touched by this patch. Is the
second -q option really a simple copy-and-paste of the first or am I
overlooking something here? I'd like to confirm this as, in retrospect,
I feel a bit uncertain about the hasty claim in the log message.
Kind
Hi,
I had a closer look at error management (once more, sorry: I should have
done this earlier...), and it seems to me that:
* Not all errors are managed properly
* Most error cases are untested
Among the cases I can think of:
* Syntax error when parsing the file
* Non-existant file
*
Christian Couder christian.cou...@gmail.com writes:
On Thu, Jul 10, 2014 at 7:36 PM, Junio C Hamano gits...@pobox.com wrote:
Making sure A's parent is B would be an
idempotent operation, no? Why not just make sure A's parent is
already B and report Your wish has been granted to the user?
Tanay Abhra tanay...@gmail.com writes:
diff --git a/test-config.c b/test-config.c
new file mode 100644
index 000..dc313c2
--- /dev/null
+++ b/test-config.c
+int main(int argc, char **argv)
+{
+ int i, val;
+ const char *v;
+ const struct string_list *strptr;
+
Signed-off-by: Matthieu Moy matthieu@imag.fr
---
Consider squashing this into PATCH 2/2
Probably not sufficient.
t/t1308-config-set.sh | 22 ++
test-config.c | 8 ++--
2 files changed, 28 insertions(+), 2 deletions(-)
diff --git a/t/t1308-config-set.sh
Signed-off-by: Matthieu Moy matthieu@imag.fr
---
Consider squashing this into PATCH 2/2.
test-config.c | 42 +++---
1 file changed, 27 insertions(+), 15 deletions(-)
diff --git a/test-config.c b/test-config.c
index dc313c2..07b61ef 100644
---
Jacob Keller jacob.e.kel...@intel.com writes:
Add support for configuring default sort ordering for git tags. Command
line option will override this configured value, using the exact same
syntax.
Cc: Jeff King p...@peff.net
Signed-off-by: Jacob Keller jacob.e.kel...@intel.com
---
- v4
*
Eric Sunshine sunsh...@sunshineco.com writes:
On Thu, Jul 10, 2014 at 8:31 PM, David Turner dtur...@twopensource.com
wrote:
Add tests to confirm that invalidation of subdirectories neither over-
nor under-invalidates.
Signed-off-by: David Turner dtur...@twitter.com
---
Junio C Hamano gits...@pobox.com writes:
+ sed -n -e s/[0-9]* subtrees// -e '/#(ref)/d' -e '/^invalid /p'
actual
Is the second one to remove #(ref), which appears for a good
reference cache tree entry shown for comparison, necessary? Do
they ever begin with invalid? If they ever
Ephrim Khong dr.kh...@gmail.com writes:
git seems to have issues with alternates when cycles are present (repo
A has B/objects as alternates, B has A/objects as alternates).
Yeah, don't do that. A thinks eh, the other guy must have it and
B thinks the same. In general, do not prune or gc a
On Fri, 2014-07-11 at 08:04 -0700, Junio C Hamano wrote:
Jacob Keller jacob.e.kel...@intel.com writes:
Add support for configuring default sort ordering for git tags. Command
line option will override this configured value, using the exact same
syntax.
Cc: Jeff King p...@peff.net
On Fri, Jul 11, 2014 at 4:22 PM, Junio C Hamano gits...@pobox.com wrote:
Christian Couder christian.cou...@gmail.com writes:
On Thu, Jul 10, 2014 at 7:36 PM, Junio C Hamano gits...@pobox.com wrote:
Making sure A's parent is B would be an
idempotent operation, no? Why not just make sure A's
Keller, Jacob E jacob.e.kel...@intel.com writes:
On Fri, 2014-07-11 at 08:04 -0700, Junio C Hamano wrote:
...
+static int tag_sort = 0;
Please do not initialize variables in bss segment to 0 by hand.
If this variable is meant to take one of these *CMP_SORT values
defined as macro later
2014-07-12 1:24 GMT+09:00 Eric Sunshine sunsh...@sunshineco.com:
On Fri, Jul 11, 2014 at 5:22 AM, Yi, EungJun semtlen...@gmail.com wrote:
2014-07-09 6:52 GMT+09:00 Eric Sunshine sunsh...@sunshineco.com:
+ grep ^Accept-Language: ko; q=1.000, en; q=0.999, \*; q=0.001
actual
Do you want
On Fri, 2014-07-11 at 08:04 -0700, Junio C Hamano wrote:
Jacob Keller jacob.e.kel...@intel.com writes:
Add support for configuring default sort ordering for git tags. Command
line option will override this configured value, using the exact same
syntax.
Cc: Jeff King p...@peff.net
Add an Accept-Language header which indicates the user's preferred
languages defined by $LANGUAGE, $LC_ALL, $LC_MESSAGES and $LANG.
Examples:
LANGUAGE= -
LANGUAGE=ko:en - Accept-Language: ko, en; q=0.9, *; q=0.1
LANGUAGE=ko LANG=en_US.UTF-8 - Accept-Language: ko, *; q=0.1
LANGUAGE=
The --sort tests should use the better format for expect to maintain
indenting and ensure that no substitution is occurring. This makes
parsing and understanding the tests a bit easier.
Signed-off-by: Jacob Keller jacob.e.kel...@intel.com
---
t/t7004-tag.sh | 40
Make the parsing of the --sort parameter more readable by having
skip_prefix keep our pointer up to date.
Authored-by: Jeff King p...@peff.net
Signed-off-by: Jacob Keller jacob.e.kel...@intel.com
---
builtin/tag.c | 14 --
1 file changed, 4 insertions(+), 10 deletions(-)
diff --git
Tanay Abhra tanay...@gmail.com writes:
diff --git a/config.c b/config.c
index ba882a1..aa58275 100644
--- a/config.c
+++ b/config.c
@@ -9,6 +9,8 @@
#include exec_cmd.h
#include strbuf.h
#include quote.h
+#include hashmap.h
+#include string-list.h
struct config_source {
On Sat, Jul 12, 2014 at 01:52:53AM +0900, Yi EungJun wrote:
Add an Accept-Language header which indicates the user's preferred
languages defined by $LANGUAGE, $LC_ALL, $LC_MESSAGES and $LANG.
Examples:
LANGUAGE= -
LANGUAGE=ko:en - Accept-Language: ko, en; q=0.9, *; q=0.1
More data points. I have reproduced the problem on
$ git --version
git version 1.8.5.2 (Apple Git-48)
$ python --version
Python 2.7.5
$ uname -a
Darwin Kernel Version 13.3.0: Tue Jun 3 21:27:35 PDT 2014;
root:xnu-2422.110.17~1/RELEASE_X86_64 x86_64
However, it the command is used on a new empty
On Fri, Jul 11, 2014 at 10:24:07AM -0700, Jacob Keller wrote:
Updated to include changes due to Junio's feedback. This has not resolved
whether we should fail on a configuration error or simply warn. It appears
that
we actually seem to error out more than warn, so I am unsure what the
On Fri, Jul 11, 2014 at 10:24:05AM -0700, Jacob Keller wrote:
Make the parsing of the --sort parameter more readable by having
skip_prefix keep our pointer up to date.
Authored-by: Jeff King p...@peff.net
I suspect Junio may just apply this on the version of the commit he has
upstream, so
On Fri, 2014-07-11 at 12:15 +0200, Olaf Hering wrote:
There is an incorrect message when doing git rebase -i remote/branch.
I have it only in german, see below. what happend is:
#01 make changes on another host
#02 copy patchfile to localhost
#03 apply patchfile
#04 git commit -avs #
On Fri, 2014-07-11 at 09:01 -0700, Junio C Hamano wrote:
Ephrim Khong dr.kh...@gmail.com writes:
git seems to have issues with alternates when cycles are present (repo
A has B/objects as alternates, B has A/objects as alternates).
Yeah, don't do that. A thinks eh, the other guy must
On Fri, 2014-07-11 at 13:50 -0400, Jeff King wrote:
On Fri, Jul 11, 2014 at 10:24:05AM -0700, Jacob Keller wrote:
Make the parsing of the --sort parameter more readable by having
skip_prefix keep our pointer up to date.
Authored-by: Jeff King p...@peff.net
I suspect Junio may just
On Fri, 2014-07-11 at 13:46 -0400, Jeff King wrote:
On Fri, Jul 11, 2014 at 10:24:07AM -0700, Jacob Keller wrote:
Updated to include changes due to Junio's feedback. This has not resolved
whether we should fail on a configuration error or simply warn. It appears
that
we actually seem
Tanay Abhra tanay...@gmail.com writes:
diff --git a/t/t1308-config-set.sh b/t/t1308-config-set.sh
new file mode 100755
index 000..87a29f1
--- /dev/null
+++ b/t/t1308-config-set.sh
@@ -0,0 +1,170 @@
+#!/bin/sh
+
+test_description='Test git config-set API in different settings'
+
+.
On Fri, Jul 11, 2014 at 06:11:08PM +, Keller, Jacob E wrote:
I personally prefer error out on options, even though it can make it a
bit more difficult, though as far as I know unknown fields simply warn
or are ignored. (ie: old versions of git just ignore unknown fields in
configuration).
Christian Couder christian.cou...@gmail.com writes:
On Fri, Jul 11, 2014 at 4:22 PM, Junio C Hamano gits...@pobox.com wrote:
Christian Couder christian.cou...@gmail.com writes:
On Thu, Jul 10, 2014 at 7:36 PM, Junio C Hamano gits...@pobox.com wrote:
Making sure A's parent is B would be an
On Fri, Jul 11, 2014 at 12:15:47PM +0200, Olaf Hering wrote:
Could not apply 6c5842320acc797d395afb5cdf373c2bfaebfa34... revert
Its not clear what '--allow-empty' refers to, git rebase does not seem to
understand this option.
I think this is the same problem discussed recently in:
On Fri, Jul 11, 2014 at 11:25:43AM -0700, Junio C Hamano wrote:
Christian Couder christian.cou...@gmail.com writes:
On Fri, Jul 11, 2014 at 4:22 PM, Junio C Hamano gits...@pobox.com wrote:
Christian Couder christian.cou...@gmail.com writes:
On Thu, Jul 10, 2014 at 7:36 PM, Junio C
Keller, Jacob E jacob.e.kel...@intel.com writes:
This is not how the rest of the current tests work. I will submit a
patch which fixes up the current --sort tests (but not every test, for
now) as well.
I do not want to pile more work that is unrelated to the task at
hand on your plate, i.e.
Jeff King p...@peff.net writes:
On Fri, Jul 11, 2014 at 10:24:05AM -0700, Jacob Keller wrote:
Make the parsing of the --sort parameter more readable by having
skip_prefix keep our pointer up to date.
Authored-by: Jeff King p...@peff.net
I suspect Junio may just apply this on the version
Am 05.07.2014 12:48, schrieb Duy Nguyen:
On Sat, Jul 5, 2014 at 5:42 AM, Karsten Blees karsten.bl...@gmail.com wrote:
'git status' segfaults if a directory is longer than PATH_MAX, because
processing .gitignore files in prep_exclude() writes past the end of a
PATH_MAX-bounded buffer.
Remove
Am 07.07.2014 20:30, schrieb Junio C Hamano:
Karsten Blees karsten.bl...@gmail.com writes:
The above cache_def_free(cache) does not free the cache itself, but
only its associated data, so the name cache_def_free() is somewhat
misleading.
You already merged this to master
Am 07.07.2014 19:43, schrieb Junio C Hamano:
Karsten Blees karsten.bl...@gmail.com writes:
Hashmap entries are typically looked up by just a key. The hashmap_get()
API expects an initialized entry structure instead, to support compound
keys. This flexibility is currently only needed by
Am 09.07.2014 18:33, schrieb Junio C Hamano:
Karsten Blees karsten.bl...@gmail.com writes:
'git status' segfaults if a directory is longer than PATH_MAX, because
processing .gitignore files in prep_exclude() writes past the end of a
PATH_MAX-bounded buffer.
Remove the limitation by using
Am 10.07.2014 22:05, schrieb Johannes Sixt:
It looks like I totally missed the topic sk/mingw-unicode-spawn-args.
Now it's in master, and it breaks lots of test cases for me:
t0050-filesystem
t0110-urlmatch-normalization
t4014-format-patch
t4041-diff-submodule-option
t4120-apply-popt
On Fri, 2014-07-11 at 14:22 -0400, Jeff King wrote:
On Fri, Jul 11, 2014 at 06:11:08PM +, Keller, Jacob E wrote:
I personally prefer error out on options, even though it can make it a
bit more difficult, though as far as I know unknown fields simply warn
or are ignored. (ie: old
On Fri, 2014-07-11 at 11:29 -0700, Junio C Hamano wrote:
Keller, Jacob E jacob.e.kel...@intel.com writes:
This is not how the rest of the current tests work. I will submit a
patch which fixes up the current --sort tests (but not every test, for
now) as well.
I do not want to pile more
The --sort tests should use the better format for expect to maintain
indenting and ensure that no substitution is occurring. This makes
parsing and understanding the tests a bit easier.
Signed-off-by: Jacob Keller jacob.e.kel...@intel.com
---
t/t7004-tag.sh | 40
Add support for configuring default sort ordering for git tags. Command
line option will override this configured value, using the exact same
syntax.
Cc: Jeff King p...@peff.net
Signed-off-by: Jacob Keller jacob.e.kel...@intel.com
---
Updated based on Junio's suggestions, as well as making sure
From: Jeff King p...@peff.net
Make the parsing of the --sort parameter more readable by having
skip_prefix keep our pointer up to date.
Signed-off-by: Jeff King p...@peff.net
Signed-off-by: Jacob Keller jacob.e.kel...@intel.com
---
Fixed authorship. I don't expect this version to be taken, but
On Fri, 2014-07-11 at 13:51 -0700, Jacob Keller wrote:
Add support for configuring default sort ordering for git tags. Command
line option will override this configured value, using the exact same
syntax.
Cc: Jeff King p...@peff.net
Signed-off-by: Jacob Keller jacob.e.kel...@intel.com
---
On Fri, Jul 11, 2014 at 01:51:35PM -0700, Jacob Keller wrote:
+ if (!strcmp(var, tag.sort)) {
+ if (!value)
+ return config_error_nonbool(var);
+ status = parse_sort_string(value, tag_sort);
+ if (status) {
+
On Fri, 2014-07-11 at 17:06 -0400, Jeff King wrote:
On Fri, Jul 11, 2014 at 01:51:35PM -0700, Jacob Keller wrote:
+ if (!strcmp(var, tag.sort)) {
+ if (!value)
+ return config_error_nonbool(var);
+ status = parse_sort_string(value, tag_sort);
+
The --sort tests should use the better format for expect to maintain
indenting and ensure that no substitution is occurring. This makes
parsing and understanding the tests a bit easier.
Signed-off-by: Jacob Keller jacob.e.kel...@intel.com
---
t/t7004-tag.sh | 40
From: Jeff King p...@peff.net
Make the parsing of the --sort parameter more readable by having
skip_prefix keep our pointer up to date.
Signed-off-by: Jeff King p...@peff.net
Signed-off-by: Jacob Keller jacob.e.kel...@intel.com
---
builtin/tag.c | 14 --
1 file changed, 4
Add support for configuring default sort ordering for git tags. Command
line option will override this configured value, using the exact same
syntax.
Cc: Jeff King p...@peff.net
Signed-off-by: Jacob Keller jacob.e.kel...@intel.com
---
Updated warning texts based on Jeff's feedback. Also added
Jeff King p...@peff.net writes:
On Fri, Jul 11, 2014 at 10:24:07AM -0700, Jacob Keller wrote:
Updated to include changes due to Junio's feedback. This has not resolved
whether we should fail on a configuration error or simply warn. It appears
that
we actually seem to error out more than
git-fast-import is not writing a commit even after a checkpoint/progress
command.
See my previous message git p4 diff-tree ambiguous argument error.
The error in git-p4 is caused by git not writing the commit even after
git-fast-import has been given a checkpoint and progress command.
On
On Fri, 2014-07-11 at 14:54 -0700, Junio C Hamano wrote:
Jeff King p...@peff.net writes:
On Fri, Jul 11, 2014 at 10:24:07AM -0700, Jacob Keller wrote:
Updated to include changes due to Junio's feedback. This has not resolved
whether we should fail on a configuration error or simply
Jeff King p...@peff.net writes:
On Fri, Jul 11, 2014 at 01:51:35PM -0700, Jacob Keller wrote:
+if (!strcmp(var, tag.sort)) {
+if (!value)
+return config_error_nonbool(var);
+status = parse_sort_string(value, tag_sort);
+if
Karsten Blees karsten.bl...@gmail.com writes:
Am 07.07.2014 20:30, schrieb Junio C Hamano:
Karsten Blees karsten.bl...@gmail.com writes:
The above cache_def_free(cache) does not free the cache itself, but
only its associated data, so the name cache_def_free() is somewhat
misleading.
Karsten Blees karsten.bl...@gmail.com writes:
In other words, why isn't hashmap_get() more like this:
...
with hashmap_entry_init() purely a static helper in hashmap.c?
1. Performance
OK.
2. Simplicity
Hashmap clients will typically provide small, type safe wrappers around the
On Fri, Jul 11, 2014 at 1:35 PM, Jeff King p...@peff.net wrote:
On Sat, Jul 12, 2014 at 01:52:53AM +0900, Yi EungJun wrote:
Add an Accept-Language header which indicates the user's preferred
languages defined by $LANGUAGE, $LC_ALL, $LC_MESSAGES and $LANG.
Examples:
LANGUAGE= -
Karsten Blees karsten.bl...@gmail.com writes:
Anyways, I'd like to kindly withdraw this patch in favor of Duy's version.
http://article.gmane.org/gmane.comp.version-control.git/248310
Thanks; I've already reverted it from 'next'.
Is Duy's patch still viable?
--
To unsubscribe from this list:
On Fri, 2014-07-11 at 15:17 -0700, Junio C Hamano wrote:
Jeff King p...@peff.net writes:
On Fri, Jul 11, 2014 at 01:51:35PM -0700, Jacob Keller wrote:
+ if (!strcmp(var, tag.sort)) {
+ if (!value)
+ return config_error_nonbool(var);
+ status =
Jacob Keller jacob.e.kel...@intel.com writes:
From: Jeff King p...@peff.net
Make the parsing of the --sort parameter more readable by having
skip_prefix keep our pointer up to date.
Signed-off-by: Jeff King p...@peff.net
Signed-off-by: Jacob Keller jacob.e.kel...@intel.com
---
On Fri, 2014-07-11 at 08:40 -0700, Junio C Hamano wrote:
Junio C Hamano gits...@pobox.com writes:
+ sed -n -e s/[0-9]* subtrees// -e '/#(ref)/d' -e '/^invalid /p'
actual
Is the second one to remove #(ref), which appears for a good
reference cache tree entry shown for
On Fri, 2014-07-11 at 08:27 -0700, Junio C Hamano wrote:
Eric Sunshine sunsh...@sunshineco.com writes:
On Thu, Jul 10, 2014 at 8:31 PM, David Turner dtur...@twopensource.com
wrote:
Add tests to confirm that invalidation of subdirectories neither over-
nor under-invalidates.
On Fri, 2014-07-11 at 15:44 -0700, Junio C Hamano wrote:
Jacob Keller jacob.e.kel...@intel.com writes:
From: Jeff King p...@peff.net
Make the parsing of the --sort parameter more readable by having
skip_prefix keep our pointer up to date.
Signed-off-by: Jeff King p...@peff.net
The --sort tests should use the better format for expect to maintain
indenting and ensure that no substitution is occurring. This makes
parsing and understanding the tests a bit easier.
Signed-off-by: Jacob Keller jacob.e.kel...@intel.com
---
t/t7004-tag.sh | 40
From: Jeff King p...@peff.net
Make the parsing of the --sort parameter more readable by having
skip_prefix keep our pointer up to date.
Signed-off-by: Jeff King p...@peff.net
Signed-off-by: Jacob Keller jacob.e.kel...@intel.com
---
Fixed issue with patch in that we dropped the reset to
When git checkout checks out a branch, create or update the
cache-tree so that subsequent operations are faster.
update_main_cache_tree learned a new flag, WRITE_TREE_REPAIR. When
WRITE_TREE_REPAIR is set, portions of the cache-tree which do not
correspond to existing tree objects are
Do not treat known-invalid trees as errors even when their subtree_nr is
incorrect. Because git already knows that these trees are invalid,
an incorrect subtree_nr will not cause problems.
Add a couple of comments.
Signed-off-by: David Turner dtur...@twitter.com
---
test-dump-cache-tree.c | 4
Add tests to confirm that invalidation of subdirectories neither over-
nor under-invalidates.
Signed-off-by: David Turner dtur...@twitter.com
---
t/t0090-cache-tree.sh | 26 +++---
1 file changed, 23 insertions(+), 3 deletions(-)
diff --git a/t/t0090-cache-tree.sh
On Fri, 2014-07-11 at 08:52 -0700, Junio C Hamano wrote:
David Turner dtur...@twopensource.com writes:
@@ -16,8 +16,34 @@ cmp_cache_tree () {
# We don't bother with actually checking the SHA1:
# test-dump-cache-tree already verifies that all existing data is
# correct.
Is this
Am 12.07.2014 00:29, schrieb Junio C Hamano:
Karsten Blees karsten.bl...@gmail.com writes:
Anyways, I'd like to kindly withdraw this patch in favor of Duy's version.
http://article.gmane.org/gmane.comp.version-control.git/248310
Thanks; I've already reverted it from 'next'.
Is Duy's
On Fri, Jul 11, 2014 at 7:22 PM, David Turner dtur...@twopensource.com wrote:
During the commit process, update the cache-tree. Write this updated
cache-tree so that it's ready for subsequent commands.
Add test code which demonstrates that git commit now writes the cache
tree. Make all tests
Changes since v7:
[04]: Fixed -Wextra compiler warnings, thanks to Ramsay Jones.
[11]: Added #ifndef TRACE_CONTEXT, explained why __FILE__ :
__FUNCTION__ doesn't work.
[17]: New Documentation/technical/api-trace.txt
Karsten Blees (17):
trace: move trace declarations from cache.h to new
Also include direct dependencies (strbuf.h and git-compat-util.h for
__attribute__) so that trace.h can be used independently of cache.h, e.g.
in test programs.
Signed-off-by: Karsten Blees bl...@dcon.de
Signed-off-by: Junio C Hamano gits...@pobox.com
---
cache.h | 13 ++---
trace.h | 17
The format parameter to trace_printf functions is sometimes abbreviated
'fmt'. Rename to 'format' everywhere (consistent with POSIX' printf
specification).
Signed-off-by: Karsten Blees bl...@dcon.de
Signed-off-by: Junio C Hamano gits...@pobox.com
---
trace.c | 22 +++---
trace.h
trace_printf_key() is the only non-static function that duplicates the
printf format attribute in the .c file, remove it for consistency.
Signed-off-by: Karsten Blees bl...@dcon.de
Signed-off-by: Junio C Hamano gits...@pobox.com
---
trace.c | 1 -
1 file changed, 1 deletion(-)
diff --git
The trace API currently rechecks the environment variable and reopens the
trace file on every API call. This has the ugly side effect that errors
(e.g. file cannot be opened, or the user specified a relative path) are
also reported on every call. Performance can be improved by about factor
three
1 - 100 of 108 matches
Mail list logo