Re: [PATCH 0/5] Use watchman to reduce index refresh time

On Tue, Nov 3, 2015 at 10:21 AM, Duy Nguyen wrote: > On Mon, Nov 2, 2015 at 8:23 PM, Duy Nguyen wrote: >> On Mon, Nov 2, 2015 at 3:54 PM, Paolo Ciarrocchi >> wrote: >>> On Sun, Nov 1, 2015 at 2:55 PM, Nguyễn Thái Ngọc Duy

[ANNOUNCE] Git Rev News edition 9

Hi everyone, I'm happy announce that the 9th edition of Git Rev News is now published: http://git.github.io/rev_news/2015/11/11/edition-9/ Thanks a lot to all the contributors, especially Matthieu! Enjoy, Christian, Thomas and Nicola. -- To unsubscribe from this list: send the line

Re: Watchman/inotify support and other ways to speed up git status

On Tue, Nov 3, 2015 at 6:45 AM, Duy Nguyen <pclo...@gmail.com> wrote: > On Mon, Nov 2, 2015 at 9:56 PM, David Turner <dtur...@twopensource.com> wrote: >> On Thu, 2015-10-29 at 09:10 +0100, Christian Couder wrote: >>> > We're using Watchman at Twitter. A wee

Draft of Git Rev News edition 9

Hi, A draft of Git Rev News edition 9 is available here: https://github.com/git/git.github.io/blob/master/rev_news/drafts/edition-9.md Everyone is welcome to contribute in any section either by editing the above page on GitHub and sending a pull request, or by commenting on this GitHub issue:

[ANNOUNCE] Git Rev News edition 8

Hi everyone, I'm happy announce that the 8th edition of Git Rev News is now published: https://git.github.io/rev_news/2015/10/14/edition-8/ Thanks a lot to all the contributors! Enjoy, Christian, Thomas and Nicola. -- To unsubscribe from this list: send the line "unsubscribe git" in the body

[PATCH v2 2/2] quote: move comment before sq_quote_buf()

A big comment at the beginning of quote.c is really related to sq_quote_buf(), so let's move it in front of this function. Signed-off-by: Christian Couder <chrisc...@tuxfamily.org> --- quote.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/quote.c b/quote.c

[PATCH v2 1/2] quote: fix broken sq_quote_buf() related comment

Since 77d604c (Enhanced sq_quote(), 10 Oct 2005), the comment at the beginning of quote.c is broken. Let's fix it. Signed-off-by: Christian Couder <chrisc...@tuxfamily.org> --- The only change in this v2 is the added Signed-off-by ;-) Sorry for the spam. quote.c | 1 + 1 file chan

Re: [PATCH v2 3/4] stripspace: Implement --count-lines option

On Mon, Oct 19, 2015 at 3:46 PM, Tobias Klauser wrote: > On 2015-10-18 at 19:18:53 +0200, Junio C Hamano wrote: >> Eric Sunshine writes: >> >> > Is there any application beyond git-rebase--interactive where a >> > --count-lines

Re: Draft of Git Rev News edition 8

On Sun, Oct 11, 2015 at 1:45 AM, Eric Sunshine <sunsh...@sunshineco.com> wrote: > On Sat, Oct 10, 2015 at 7:36 PM, Christian Couder > <christian.cou...@gmail.com> wrote: >> A draft of Git Rev News edition 8 is available here: >> https://github.com/git/git.github.

Draft of Git Rev News edition 8

Hi, A draft of Git Rev News edition 8 is available here: https://github.com/git/git.github.io/blob/master/rev_news/drafts/edition-8.md Everyone is welcome to contribute in any section either by editing the above page on GitHub and sending a pull request, or by commenting on this GitHub issue:

[ANNOUNCE] Git Rev News edition 5

Hi, Git Rev News edition 5 is now available: https://git.github.io/rev_news/2015/07/08/edition-5/ Thanks a lot to all the helpers! Enjoy, Christian, Thomas and Nicola. -- To unsubscribe from this list: send the line unsubscribe git in the body of a message to majord...@vger.kernel.org More

Re: Draft of Git Rev News edition 5

On Wed, Jul 8, 2015 at 9:43 AM, Junio C Hamano gits...@pobox.com wrote: Michael J Gruber g...@drmicha.warpmail.net writes: Maybe a matter of taste, but I think in general we could do with a bit less of narrating and more of summarizing. True. I think sometimes the details might be

Re: [PATCH v2 06/10] Documentation/tag: remove double occurance of pattern

On Thu, Jul 9, 2015 at 12:27 PM, Karthik Nayak karthik@gmail.com wrote: Mentored-by: Christian Couder christian.cou...@gmail.com Mentored-by: Matthieu Moy matthieu@grenoble-inp.fr Signed-off-by: Karthik Nayak karthik@gmail.com --- Documentation/git-tag.txt | 1 - 1 file changed

[PATCH v2 1/2] trailer: ignore first line of message

When looking for the start of the trailers in the message we are passed, we should ignore the first line of the message. The reason is that if we are passed a patch or commit message then the first line should be the patch title. If we are passed only trailers we can expect that they start with

[PATCH v2 2/2] trailer: support multiline title

We currently ignore the first line passed to `git interpret-trailers`, when looking for the beginning of the trailers. Unfortunately this does not work well when a commit is created with a line break in the title, using for example the following command: git commit -m 'place of code: change we

Re: What's cooking in git.git (Aug 2015, #05; Fri, 28)

* dt/refs-bisection (2015-08-28) 5 commits - bisect: make bisection refs per-worktree - refs: make refs/worktree/* per-worktree - SQUASH??? - path: optimize common dir checking - refs: clean up common_list Move the refs used during a git bisect session to per-worktree hierarchy

Re: [PATCH v2 2/2] trailer: support multiline title

On Wed, Aug 26, 2015 at 9:48 PM, Junio C Hamano gits...@pobox.com wrote: Christian Couder christian.cou...@gmail.com writes: We currently ignore the first line passed to `git interpret-trailers`, when looking for the beginning of the trailers. Unfortunately this does not work well when

[PATCH v3] trailer: support multiline title

We currently ignore the first line passed to `git interpret-trailers`, when looking for the beginning of the trailers. Unfortunately this does not work well when a commit is created with a line break in the title, using for example the following command: git commit -m 'place of code: change we

Re: [PATCH v2 2/2] trailer: support multiline title

Sorry I sent the part below privately by mistake: On Tue, Aug 25, 2015 at 11:07 PM, Matthieu Moy matthieu@grenoble-inp.fr wrote: Now, I found another issue: I still have this interpret-trailers in my hooks/commit-msg, and it behaves badly when I use git commit -v. With -v, I get a diff in

Re: [PATCH v3] trailer: support multiline title

On Mon, Aug 31, 2015 at 8:13 PM, Junio C Hamano <gits...@pobox.com> wrote: > Christian Couder <christian.cou...@gmail.com> writes: > >> On Mon, Aug 31, 2015 at 10:38 AM, Matthieu Moy >> <matthieu@grenoble-inp.fr> wrote: >>> Christian Couder <chr

Re: What's cooking in git.git (Aug 2015, #05; Fri, 28)

On Mon, Aug 31, 2015 at 10:06 PM, Junio C Hamano wrote: > David Turner writes: > >>> Christian, thanks for raising this one. >>> >>> I do recall the thread and I might be the somebody like Michael you >>> remember, e.g. $gmane/275105---which did

Re: [PUB]What's cooking in git.git (Aug 2015, #05; Fri, 28)

On Mon, Aug 31, 2015 at 9:36 AM, Matthieu Moy wrote: > Junio C Hamano writes: > >> * ad/bisect-terms (2015-08-03) 4 commits >> - bisect: allow setting any user-specified in 'git bisect start' >> - bisect: add 'git bisect terms' to view the

Re: [PATCH v3] trailer: support multiline title

On Mon, Aug 31, 2015 at 10:38 AM, Matthieu Moy <matthieu@grenoble-inp.fr> wrote: > Christian Couder <christian.cou...@gmail.com> writes: > >> That's why instead of ignoring only the first line, it is better to >> ignore the first paragraph. >> --- >

Re: git bisect replay produces wrong result

Hi, On Sun, Aug 30, 2015 at 6:38 AM, Neil Brown wrote: > > Hi, > the following git-bisect log - applied to a recent linux-kernel tree > produced different end results when I use "git bisect replay" > and when I just run it as a shell script. > > $ git bisect replay /tmp/log >

Draft of Git Rev News edition 7

Hi, A draft of Git Rev News edition 7 is available here: https://github.com/git/git.github.io/blob/master/rev_news/drafts/edition-7.md Everyone is welcome to contribute in any section, like Matthieu already did, either by editing the above page on GitHub and sending a pull request, or by

Re: GSoC 2015 is over

Hi, On Thu, Sep 3, 2015 at 12:46 AM, Johannes Schindelin wrote: > Hi, > > On Wed, 2 Sep 2015, Paul Tan wrote: > >> On Wed, Sep 2, 2015 at 12:55 AM, Matthieu Moy >> wrote: >> > I consider this GSoC as a great success and a pleasant

Re: [PATCH] interpret-trailers: allow running outside a repository

On Sat, Sep 5, 2015 at 3:39 PM, John Keeping <j...@keeping.me.uk> wrote: > It may be useful to run git-interpret-trailers without needing to be in > a repository. Yeah, it looks like it works fine outside a repo. Tested-by: Christian Couder <chrisc...@tuxfamily.org>

Re: More builtin git-am issues..

On Sat, Sep 5, 2015 at 9:39 PM, Junio C Hamano wrote: > Junio C Hamano writes: > >> To salvage "interpret-trailers" needs a lot more, as we are >> realizing that the definition that led to its external design does >> not match the way users use footers in

[ANNOUNCE] Git Rev News edition 7

Hi everyone, I'm happy announce that the 7th edition of Git Rev News is now published: https://git.github.io/rev_news/2015/09/09/edition-7/ Thanks a lot to all the contributors! Enjoy, Christian, Thomas and Nicola. -- To unsubscribe from this list: send the line "unsubscribe git" in the body

Re: Developing- Where to Start

Hi, On Mon, Sep 14, 2015 at 10:42 PM, Breanna Devore-McDonald wrote: > Hello all, > > I'm a third year Computer Science student at the University of Notre > Dame, and for the final project of my Data Structures class, my group > and I have to find a way to

[PATCH 1/2] quote: fix broken sq_quote_buf() related comment

Since 77d604c (Enhanced sq_quote(), 10 Oct 2005), the comment at the beginning of quote.c is broken. Let's fix it. --- quote.c | 1 + 1 file changed, 1 insertion(+) diff --git a/quote.c b/quote.c index 7920e18..890885a 100644 --- a/quote.c +++ b/quote.c @@ -7,6 +7,7 @@ int quote_path_fully = 1;

[PATCH 2/2] quote: move comment before sq_quote_buf()

A big comment at the beginning of quote.c is really related to sq_quote_buf(), so let's move it in front of this function. --- quote.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/quote.c b/quote.c index 890885a..fe884d2 100644 --- a/quote.c +++ b/quote.c @@ -4,6

Re: best practices against long git rebase times?

On Mon, Dec 7, 2015 at 11:59 PM, Jeff King wrote: > On Mon, Dec 07, 2015 at 02:56:33PM -0800, Junio C Hamano wrote: > >> Jeff King writes: >> >> > You're computing the patch against the parent for each of those 3000 >> > commits (to get a hash of it to compare

[PATCH 0/8] Untracked cache improvements

there: https://github.com/chriscool/git/tree/uc-notifs14 Christian Couder (8): update-index: add untracked cache notifications update-index: use enum for untracked cache options update-index: add --test-untracked-cache update-index: move 'uc' var declaration dir: add add_untracked_cache() dir

[PATCH 5/8] dir: add add_untracked_cache()

This new function will be used in a later patch. Signed-off-by: Christian Couder <chrisc...@tuxfamily.org> --- builtin/update-index.c | 11 +-- dir.c | 14 ++ dir.h | 1 + 3 files changed, 16 insertions(+), 10 deletions(-) diff

[PATCH 1/8] update-index: add untracked cache notifications

ch "git update-index --untracked-cache" tells the user in which directory tests are performed and in which working directory the untracked cache is allowed. Signed-off-by: Christian Couder <chrisc...@tuxfamily.org> --- builtin/update-index.c | 4 +++- 1 file changed, 3 insertions(+),

[PATCH 7/8] config: add core.untrackedCache

hat after this patch there is no difference any more between `--untracked-cache` and `--force-untracked-cache`. Signed-off-by: Christian Couder <chrisc...@tuxfamily.org> --- Documentation/config.txt | 7 +++ Documentation/git-update-index.txt | 28 ++

[PATCH 2/8] update-index: use enum for untracked cache options

Signed-off-by: Christian Couder <chrisc...@tuxfamily.org> --- builtin/update-index.c | 18 +- 1 file changed, 13 insertions(+), 5 deletions(-) diff --git a/builtin/update-index.c b/builtin/update-index.c index 6f6b289..246b3d3 100644 --- a/builtin/update-index.c +++ b/b

[PATCH 4/8] update-index: move 'uc' var declaration

Signed-off-by: Christian Couder <chrisc...@tuxfamily.org> --- builtin/update-index.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/builtin/update-index.c b/builtin/update-index.c index ecb685d..21f74b2 100644 --- a/builtin/update-index.c +++ b/builtin/update-i

[PATCH 8/8] t7063: add tests for core.untrackedCache

Signed-off-by: Christian Couder <chrisc...@tuxfamily.org> --- t/t7063-status-untracked-cache.sh | 48 +-- 1 file changed, 46 insertions(+), 2 deletions(-) diff --git a/t/t7063-status-untracked-cache.sh b/t/t7063-status-untracked-cache.sh index 2

[PATCH 6/8] dir: add remove_untracked_cache()

This new function will be used in a later patch. Signed-off-by: Christian Couder <chrisc...@tuxfamily.org> --- builtin/update-index.c | 3 +-- dir.c | 6 ++ dir.h | 1 + 3 files changed, 8 insertions(+), 2 deletions(-) diff --git a/builtin/update-ind

[PATCH 3/8] update-index: add --test-untracked-cache

It is nice to just be able to test if untracked cache is supported without enabling it. Signed-off-by: Christian Couder <chrisc...@tuxfamily.org> --- Documentation/git-update-index.txt | 9 - builtin/update-index.c | 5 + 2 files changed, 13 insertions(+), 1 de

[RFC/PATCH 3/8] update-index: move 'uc' var declaration

Signed-off-by: Christian Couder <chrisc...@tuxfamily.org> --- builtin/update-index.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/builtin/update-index.c b/builtin/update-index.c index b7b5108..b54ddc3 100644 --- a/builtin/update-index.c +++ b/builtin/update-i

[RFC/PATCH 8/8] update-index: make core.untrackedCache a bool

and because this avoids the untracked cache being disabled by a kernel change or a directory change. Of course this breaks some backward compatibility, but the simplification and increased useability is worth it. Signed-off-by: Christian Couder <chrisc...@tuxfamily.org> --- Documentation/git-

[RFC/PATCH 4/8] dir: add add_untracked_cache()

This new function will be used in a later patch. Signed-off-by: Christian Couder <chrisc...@tuxfamily.org> --- builtin/update-index.c | 11 +-- dir.c | 14 ++ dir.h | 1 + 3 files changed, 16 insertions(+), 10 deletions(-) diff

[RFC/PATCH 1/8] update-index: add untracked cache notifications

ch "git update-index --untracked-cache" tells the user in which directory tests are performed and in which working directory the untracked cache is allowed. Signed-off-by: Christian Couder <chrisc...@tuxfamily.org> --- builtin/update-index.c | 4 +++- 1 file changed, 3 insertions(+),

[RFC/PATCH 2/8] update-index: add --test-untracked-cache

It is nice to just be able to test if untracked cache is supported without enabling it. Signed-off-by: Christian Couder <chrisc...@tuxfamily.org> --- Documentation/git-update-index.txt | 9 - builtin/update-index.c | 8 ++-- 2 files changed, 14 insertions(+), 3 del

[RFC/PATCH 0/8] Untracked cache improvements

patch 6/8 backward compatibility is preserved. Patchs 7/8 and 8/8 are trying to improve usability by making the untracked cache cli and config options more in line with other git cli and config options, but this sacrifies some backward compatibility. Christian Couder (8): update-index: add untra

[RFC/PATCH 5/8] dir: add remove_untracked_cache()

This new function will be used in a later patch. Signed-off-by: Christian Couder <chrisc...@tuxfamily.org> --- builtin/update-index.c | 3 +-- dir.c | 6 ++ dir.h | 1 + 3 files changed, 8 insertions(+), 2 deletions(-) diff --git a/builtin/update-ind

[RFC/PATCH 7/8] update-index: prevent --untracked-cache from performing tests

them. So to be more consistent with other git commands it's better to make `--untracked-cache` not perform them, which is the purpose of this patch. Note that after this patch there is no difference any more between `--untracked-cache` and `--force-untracked-cache`. Signed-off-by: Christian Couder

[RFC/PATCH 6/8] config: add core.untrackedCache

over a network file system, then we might want 'git update-index --untracked-cache' to fail immediately instead of it testing if it works (because it might work on some systems using the repo over the network file system but not others). Signed-off-by: Christian Couder <chrisc...@tuxfamily.

Re: [RFC/PATCH 6/8] config: add core.untrackedCache

On Fri, Dec 4, 2015 at 6:54 PM, Torsten Bögershausen wrote: >> Current state of affairs: >> >> * Enable on a per-repo basis: git update-index --untracked-cache >> * Disable on a per-repo basis: git update-index --no-cache >> * Enable system-wide: N/A >> * Disable system-wide:

Re: [RFC/PATCH 2/8] update-index: add --test-untracked-cache

On Wed, Dec 2, 2015 at 8:17 PM, Duy Nguyen <pclo...@gmail.com> wrote: > On Tue, Dec 1, 2015 at 9:31 PM, Christian Couder > <christian.cou...@gmail.com> wrote: >> diff --git a/builtin/update-index.c b/builtin/update-index.c >> index e568acc..b7b5108 100644 >> -

Re: [RFC/PATCH] config: add core.trustmtime

On Wed, Dec 2, 2015 at 8:28 PM, Duy Nguyen wrote: > On Wed, Nov 25, 2015 at 10:00 AM, Ævar Arnfjörð Bjarmason > wrote: >> Aside from the slight hassle of enabling this and keeping it enabled >> this feature is great. It's sped up "git status" across the board

Re: [RFC/PATCH 7/8] update-index: prevent --untracked-cache from performing tests

On Wed, Dec 2, 2015 at 8:18 PM, Duy Nguyen <pclo...@gmail.com> wrote: > On Tue, Dec 1, 2015 at 9:31 PM, Christian Couder > <christian.cou...@gmail.com> wrote: > diff --git a/t/t7063-status-untracked-cache.sh > b/t/t7063-status-untracked-cache.sh >> index 0e8d0d4..8

Re: [RFC/PATCH 1/8] update-index: add untracked cache notifications

On Wed, Dec 2, 2015 at 8:16 PM, Duy Nguyen <pclo...@gmail.com> wrote: > On Tue, Dec 1, 2015 at 9:31 PM, Christian Couder > <christian.cou...@gmail.com> wrote: >> Doing: >> >> cd /tmp >> git --git-dir=/git/somewhere/else/.git update-index --untrac

Re: [RFC/PATCH 8/8] update-index: make core.untrackedCache a bool

(Sorry I already sent a private reply to Tosten by mistake.) On Sat, Dec 5, 2015 at 1:44 PM, Torsten Bögershausen <tbo...@web.de> wrote: > On 01.12.15 21:31, Christian Couder wrote: >> Most features in Git can be enabled or disabled using a simple >> bool config variable

Re: [PATCH v4] Documentation/git-update-index: add missing opts to synopsis

On Wed, Nov 25, 2015 at 10:30 AM, Christian Couder <christian.cou...@gmail.com> wrote: > Split index related options should appear in the 'SYNOPSIS' > section. > > These options are already documented in the 'OPTIONS' section. > > Signed-off-by: Christian Couder

Draft of Git Rev News edition 10

Hi, A draft of Git Rev News edition 10 is available here: https://github.com/git/git.github.io/blob/master/rev_news/drafts/edition-10.md Everyone is welcome to contribute in any section either by editing the above page on GitHub and sending a pull request, or by commenting on this GitHub

Re: [RFC/PATCH 6/8] config: add core.untrackedCache

On Thu, Dec 3, 2015 at 5:10 PM, Torsten Bögershausen wrote: > [snip all good stuff] > > First of all: > Thanks for explaining it so well > > I now can see the point in having this patch. > (Do the commit messages reflect all this ? I need to re-read) Maybe not. I will have a look

[ANNOUNCE] Git Rev News edition 10

Hi everyone, I'm happy announce that the 10th edition of Git Rev News is now published: https://git.github.io/rev_news/2015/12/09/edition-10/ It was supposed to be published yesterday but I got busy with other things. Sorry about that. Thanks a lot to all the contributors, especially Stefan!

[PATCH v2 01/10] update-index: use enum for untracked cache options

Signed-off-by: Christian Couder <chrisc...@tuxfamily.org> --- builtin/update-index.c | 18 +- 1 file changed, 13 insertions(+), 5 deletions(-) diff --git a/builtin/update-index.c b/builtin/update-index.c index 7431938..2430a68 100644 --- a/builtin/update-index.c +++ b/b

[PATCH v2 00/10] Untracked cache improvements

and removes code that is now useless. So the changes compared to v1 are mostly small updates, and patchs 7/10 and 10/10. The patch series is also available there: https://github.com/chriscool/git/tree/uc-notifs25 Thanks to the reviewers and helpers. Christian Couder (10): update-index: use enum

[PATCH v2 05/10] dir: add add_untracked_cache()

Factor out code into add_untracked_cache(), which will be used in a later commit. Signed-off-by: Christian Couder <chrisc...@tuxfamily.org> --- builtin/update-index.c | 11 +-- dir.c | 14 ++ dir.h | 1 + 3 files changed, 16 inse

[PATCH v2 03/10] update-index: add untracked cache notifications

tory tests are performed. This makes it easy to spot any problem. Also in verbose mode, let's tell the user when the cache is enabled or disabled. Signed-off-by: Christian Couder <chrisc...@tuxfamily.org> --- builtin/update-index.c | 6 +- 1 file changed, 5 insertions(+), 1 deletio

[PATCH v2 10/10] dir: do not use untracked cache ident anymore

ld indexes that already have an untracked cache with this field. Signed-off-by: Christian Couder <chrisc...@tuxfamily.org> --- dir.c | 38 +- dir.h | 2 +- 2 files changed, 6 insertions(+), 34 deletions(-) diff --git a/dir.c b/dir.c index 0b07ba7..94fba2a 10

[PATCH v2 04/10] update-index: move 'uc' var declaration

Signed-off-by: Christian Couder <chrisc...@tuxfamily.org> --- builtin/update-index.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/builtin/update-index.c b/builtin/update-index.c index e84674f..fffad79 100644 --- a/builtin/update-index.c +++ b/builtin/update-i

[PATCH v2 08/10] config: add core.untrackedCache

ore between `--untracked-cache` and `--force-untracked-cache`. All the changes to `--[no-|force-]untracked-cache` make it possible to deprecate those options in the future. Signed-off-by: Christian Couder <chrisc...@tuxfamily.org> Signed-off-by: Ævar Arnfjörð Bjarmason <ava...@gmail.com> --- D

[PATCH v2 06/10] dir: add remove_untracked_cache()

Factor out code into remove_untracked_cache(), which will be used in a later commit. Signed-off-by: Christian Couder <chrisc...@tuxfamily.org> --- builtin/update-index.c | 3 +-- dir.c | 6 ++ dir.h | 1 + 3 files changed, 8 insertions(+), 2 del

Re: [PATCH 7/8] config: add core.untrackedCache

On Tue, Dec 15, 2015 at 10:49 AM, Torsten Bögershausen <tbo...@web.de> wrote: > On 15.12.15 10:34, Christian Couder wrote: >> On Mon, Dec 14, 2015 at 10:30 PM, Junio C Hamano <gits...@pobox.com> wrote: >>> Junio C Hamano <gits...@pobox.com> writes: >>>

[PATCH v2 09/10] t7063: add tests for core.untrackedCache

Signed-off-by: Christian Couder <chrisc...@tuxfamily.org> --- t/t7063-status-untracked-cache.sh | 48 +-- 1 file changed, 46 insertions(+), 2 deletions(-) diff --git a/t/t7063-status-untracked-cache.sh b/t/t7063-status-untracked-cache.sh index 2

Re: [PATCH 7/8] config: add core.untrackedCache

On Tue, Dec 15, 2015 at 11:02 AM, Duy Nguyen <pclo...@gmail.com> wrote: > On Tue, Dec 15, 2015 at 4:34 PM, Christian Couder > <christian.cou...@gmail.com> wrote: >> On Mon, Dec 14, 2015 at 10:30 PM, Junio C Hamano <gits...@pobox.com> wrote: >>> Ju

[PATCH v2 07/10] dir: free untracked cache when removing it

Signed-off-by: Christian Couder <chrisc...@tuxfamily.org> --- dir.c | 1 + 1 file changed, 1 insertion(+) diff --git a/dir.c b/dir.c index ffc0286..3b83cc0 100644 --- a/dir.c +++ b/dir.c @@ -1954,6 +1954,7 @@ void add_untracked_cache(void) void remove_untracked_cach

[PATCH v2 02/10] update-index: add --test-untracked-cache

It is nice to just be able to test if untracked cache is supported without enabling it. Signed-off-by: Christian Couder <chrisc...@tuxfamily.org> --- Documentation/git-update-index.txt | 9 - builtin/update-index.c | 5 + 2 files changed, 13 insertions(+), 1 de

Re: [PATCH 7/8] config: add core.untrackedCache

On Tue, Dec 8, 2015 at 11:43 PM, Junio C Hamano <gits...@pobox.com> wrote: > Junio C Hamano <gits...@pobox.com> writes: > >> Christian Couder <christian.cou...@gmail.com> writes: >> >>> When we know that mtime is fully supported by the envir

Re: [PATCH 7/8] config: add core.untrackedCache

On Mon, Dec 14, 2015 at 10:30 PM, Junio C Hamano wrote: > Junio C Hamano writes: > > The primary reason why I do not like your "configuration decides" is > it will be a huge source of confusions and bugs. Imagine what > should happen in this sequence, and

Re: [PATCH 2/8] update-index: use enum for untracked cache options

On Fri, Dec 11, 2015 at 6:44 PM, Junio C Hamano <gits...@pobox.com> wrote: > Christian Couder <christian.cou...@gmail.com> writes: > >>> As you know I am bad at bikeshedding; the only suggestion in the >>> above is to have common UC_ prefix ;-) Don't take

Re: [PATCH 2/8] update-index: use enum for untracked cache options

On Thu, Dec 10, 2015 at 7:46 PM, Junio C Hamano <gits...@pobox.com> wrote: > Christian Couder <christian.cou...@gmail.com> writes: > >>>> +/* Untracked cache mode */ >>>> +enum uc_mode { >>>> + UNDEF_UC = -1, &g

Re: [PATCH v2 10/10] dir: do not use untracked cache ident anymore

On Tue, Dec 15, 2015 at 8:49 PM, Junio C Hamano <gits...@pobox.com> wrote: > Christian Couder <christian.cou...@gmail.com> writes: > >> +/* >> + * We used to save the location of the work tree and the kernel version, >> + * but it was not a good idea,

Re: [PATCH v2 10/10] dir: do not use untracked cache ident anymore

On Thu, Dec 17, 2015 at 7:33 PM, Junio C Hamano <gits...@pobox.com> wrote: > Christian Couder <christian.cou...@gmail.com> writes: > >> In the "git worktree" documentation there is: >> >> "If you move a linked working tree to another file syste

Re: [PATCH 7/8] config: add core.untrackedCache

orking wich Christian on getting this > integrated, and we both thought it would be good to have some fresh > input on the matter from me. > >> Christian Couder <christian.cou...@gmail.com> writes: > >>> If you want only some repos to use the UC, you will s

Re: [PATCH 2/8] update-index: use enum for untracked cache options

On Tue, Dec 8, 2015 at 8:11 PM, Junio C Hamano <gits...@pobox.com> wrote: > Christian Couder <christian.cou...@gmail.com> writes: > >> Signed-off-by: Christian Couder <chrisc...@tuxfamily.org> >> --- >> builtin/update-index.c | 18 +---

Re: [PATCH 1/8] update-index: add untracked cache notifications

On Tue, Dec 8, 2015 at 8:03 PM, Junio C Hamano <gits...@pobox.com> wrote: > Christian Couder <christian.cou...@gmail.com> writes: > >> Doing: >> >> cd /tmp >> git --git-dir=/git/somewhere/else/.git update-index --untracked-cache >> >>

Re: [PATCH 5/8] dir: add add_untracked_cache()

On Wed, Dec 9, 2015 at 8:37 AM, Torsten Bögershausen <tbo...@web.de> wrote: > On 08.12.15 18:15, Christian Couder wrote: >> This new function will be used in a later patch. > May be > Factor out code into add_untracked_cache(), which will be used in the next > commit. T

Re: [PATCH v4 09/10] config: add core.untrackedCache

On Thu, Dec 31, 2015 at 12:23 AM, Junio C Hamano <gits...@pobox.com> wrote: > Christian Couder <christian.cou...@gmail.com> writes: > >> On Tue, Dec 29, 2015 at 11:35 PM, Junio C Hamano <gits...@pobox.com> wrote: >> ... >>> While the above is

Re: Feature request: git bisect merge to usable base

Hi, On Wed, Dec 30, 2015 at 11:40 AM, Andy Lutomirski wrote: > Hi- > > I'm currently bisecting a Linux bug on my laptop. The starting good > commit is v4.4-rc3 and the starting bad commit is v4.4-rc7. > Unfortunately, anything much older than v4.4-rc3 doesn't boot at all. > >

Re: [PATCH] dir.h: remove orphaned declaration

Hi Ramsay, On Wed, Dec 30, 2015 at 6:11 PM, Ramsay Jones wrote: > > Signed-off-by: Ramsay Jones > --- > > Hi Christian, > > If you need to re-roll your 'cc/untracked' branch, could you > please squash this into your patches. > > You seem

Re: [PATCH v4 09/10] config: add core.untrackedCache

On Tue, Dec 29, 2015 at 11:35 PM, Junio C Hamano <gits...@pobox.com> wrote: > Christian Couder <christian.cou...@gmail.com> writes: > >> +core.untrackedCache:: >> + Determines if untracked cache will be automatically enabled or >> + disabled. It can

Re: [PATCH v4 08/10] dir: simplify untracked cache "ident" field

On Wed, Dec 30, 2015 at 12:47 PM, Torsten Bögershausen <tbo...@web.de> wrote: > On 2015-12-29 08.09, Christian Couder wrote: >> It is not a good idea to compare kernel versions and disable >> the untracked cache if it changes as people may upgrade and >> still want the

Re: [PATCH v4 08/10] dir: simplify untracked cache "ident" field

On Tue, Dec 29, 2015 at 11:32 PM, Junio C Hamano <gits...@pobox.com> wrote: > Christian Couder <christian.cou...@gmail.com> writes: > >> -static int ident_in_untracked(const struct untracked_cache *uc) >> +static int ident_current_location_in_untracked(const

Re: Git Rebase Issue

Hi, On Tue, Dec 22, 2015 at 6:53 PM, Pierre-Luc Loyer wrote: > Hi, > > I've encountered a situation using rebase for which I don't understand the > results, even after reading the documentation. > I'm currently working in my feature branch and then I want to squash

Re: [PATCH v3 10/11] config: add core.untrackedCache

On Thu, Dec 24, 2015 at 11:13 AM, Duy Nguyen <pclo...@gmail.com> wrote: > On Thu, Dec 24, 2015 at 4:03 AM, Christian Couder > <christian.cou...@gmail.com> wrote: >> --force-untracked-cache:: >> - For safety, `--untracked-cache` performs tests on the working >

Re: [PATCH 7/8] config: add core.untrackedCache

On Thu, Dec 24, 2015 at 2:56 AM, Junio C Hamano wrote: > Junio C Hamano writes: > >> Duy Nguyen writes: >> >>> In that case we can just check config once in read_index_from and >>> destroy UNTR extension. Or the middle ground, we check

Re: [PATCH v3 04/11] update-index: add untracked cache notifications

On Thu, Dec 24, 2015 at 11:01 AM, Duy Nguyen <pclo...@gmail.com> wrote: > On Thu, Dec 24, 2015 at 4:03 AM, Christian Couder > <christian.cou...@gmail.com> wrote: >> @@ -1135,10 +1135,16 @@ int cmd_update_index(int argc, const char **argv

Re: [PATCH 7/8] config: add core.untrackedCache

On Thu, Dec 17, 2015 at 1:36 PM, Duy Nguyen wrote: > On Wed, Dec 16, 2015 at 4:53 AM, Ævar Arnfjörð Bjarmason > wrote: >> On Tue, Dec 15, 2015 at 8:40 PM, Junio C Hamano wrote: >>> Ævar Arnfjörð Bjarmason writes: >>> I

Fwd: [PATCH 7/8] config: add core.untrackedCache

(Sorry I sent this one privately to Duy by mistake too.) -- Forwarded message -- From: Christian Couder <christian.cou...@gmail.com> Date: Fri, Dec 18, 2015 at 11:35 PM Subject: Re: [PATCH 7/8] config: add core.untrackedCache To: Duy Nguyen <pclo...@gmail.com> On

Fwd: [PATCH 7/8] config: add core.untrackedCache

Sorry I sent this privately to Peff by mistake (once again). -- Forwarded message -- From: Christian Couder <christian.cou...@gmail.com> Date: Fri, Dec 18, 2015 at 11:09 PM Subject: Re: [PATCH 7/8] config: add core.untrackedCache To: Jeff King <p...@peff.net> On

[PATCH v3 01/11] dir: free untracked cache when removing it

Signed-off-by: Christian Couder <chrisc...@tuxfamily.org> --- builtin/update-index.c | 1 + 1 file changed, 1 insertion(+) diff --git a/builtin/update-index.c b/builtin/update-index.c index 7431938..a6fff87 100644 --- a/builtin/update-index.c +++ b/builtin/update-index.c @@ -1123,6 +

[PATCH v3 02/11] update-index: use enum for untracked cache options

Signed-off-by: Christian Couder <chrisc...@tuxfamily.org> --- builtin/update-index.c | 18 +- 1 file changed, 13 insertions(+), 5 deletions(-) diff --git a/builtin/update-index.c b/builtin/update-index.c index a6fff87..1e546a3 100644 --- a/builtin/update-index.c +++ b/b

[PATCH v3 03/11] update-index: add --test-untracked-cache

It is nice to just be able to test if untracked cache is supported without enabling it. Signed-off-by: Christian Couder <chrisc...@tuxfamily.org> --- Documentation/git-update-index.txt | 9 - builtin/update-index.c | 5 + 2 files changed, 13 insertions(+), 1 de

<    4   5   6   7   8   9   10   11   12   13   >