Re: [PATCH 1/1] Mark messages for translations

2018-02-05 Thread Jeff King
On Tue, Feb 06, 2018 at 02:32:42AM -0500, Eric Sunshine wrote: > > diff --git a/t/t0002-gitfile.sh b/t/t0002-gitfile.sh > > @@ -31,7 +31,7 @@ test_expect_success 'bad setup: invalid .git file format' > > ' > > echo "git rev-parse accepted an invalid .git file" > >

Re: [PATCH 1/1] Mark messages for translations

2018-02-05 Thread Eric Sunshine
On Tue, Feb 6, 2018 at 1:15 AM, Alexander Shopov wrote: > Small changes in messages to fit the style and typography of rest > Reuse already translated messages if possible > Do not translate messages aimed at developers of git > Fix unit tests depending on the original string

Re: [GSoC][PATCH] commit: add a commit.signOff config variable

2018-02-05 Thread Chen Jingpiao
On 02/05 01:50, Stefan Beller wrote: > On Sat, Feb 3, 2018 at 6:03 PM, Chen Jingpiao wrote: > > Add the commit.signOff configuration variable to use the -s or --signoff > > option of git commit by default. > > > > Signed-off-by: Chen Jingpiao > >

[PATCH 1/1] Mark messages for translations

2018-02-05 Thread Alexander Shopov
Small changes in messages to fit the style and typography of rest Reuse already translated messages if possible Do not translate messages aimed at developers of git Fix unit tests depending on the original string Signed-off-by: Alexander Shopov --- git.c | 38

[PATCH 0/1] Mark messages for translations

2018-02-05 Thread Alexander Shopov
*** BLURB HERE *** Hi all, Again I am marking messages for translations in gui.c and setup.c based on the feedback I got last time. Now I am fixing unit tests as well, hopefully no regressions. Here is a recap: Johannes Sixt: 1. Lower-case letters at the beginning of error messages 2. Past

Re: [PATCH v2 01/41] parse-options: support --git-completion-helper

2018-02-05 Thread Eric Sunshine
On Mon, Feb 5, 2018 at 4:56 AM, Duy Nguyen wrote: > On Thu, Feb 01, 2018 at 02:16:46PM -0500, Eric Sunshine wrote: >> On Thu, Feb 1, 2018 at 5:21 AM, Duy Nguyen wrote: >> > On Thu, Feb 1, 2018 at 4:54 PM, Eric Sunshine >> > wrote:

Re: [PATCH 042/194] object-store: move alternates API to new alternates.h

2018-02-05 Thread Eric Sunshine
On Mon, Feb 5, 2018 at 6:55 PM, Stefan Beller wrote: > This should make these functions easier to find and object-store.h > less overwhelming to read. I think you mean: s/object-store.h/cache.h/ > Signed-off-by: Stefan Beller > Signed-off-by: Jonathan

Re: [RFH/PATCH] blame: tighten command line parser

2018-02-05 Thread Eric Sunshine
On Mon, Feb 5, 2018 at 6:37 PM, Junio C Hamano wrote: > The command line parser of "git blame" is prepared to take an > ancient odd argument order "blame " in addition to the > usual "blame [] ". It has at least two negative > ramifications: > > - In order to tell these two

Re: [GSoC][PATCH v2] commit: add a commit.signOff config variable

2018-02-05 Thread Chen Jingpiao
On 02/05 11:22, Junio C Hamano wrote: > Chen Jingpiao writes: > > > Add the commit.signOff configuration variable to use the -s or --signoff > > option of git commit by default. > > This is a rather old topic. Here is one from 2006: > > >

Re: [PATCH 050/194] replace-object: check_replace_refs is safe in multi repo environment

2018-02-05 Thread Eric Sunshine
On Mon, Feb 5, 2018 at 6:55 PM, Stefan Beller wrote: > In ecef23 (inline lookup_replace_object() calls, 2011-05-15) a shortcut > for checking the object replacement was added by setting check_replace_refs > to 0 once the replacements were evaluated to not exist. This works

Re: [PATCH 059/194] refs: store the main ref store inside the repository struct

2018-02-05 Thread Eric Sunshine
On Mon, Feb 5, 2018 at 6:55 PM, Stefan Beller wrote: > diff --git a/refs.c b/refs.c > @@ -1609,9 +1609,6 @@ static struct ref_store_hash_entry > *alloc_ref_store_hash_entry( > -/* A pointer to the ref_store for the main repository: */ > -static struct ref_store

Re: [PATCH 075/194] fetch, push: do not use submodule as alternate in has_commits check

2018-02-05 Thread Eric Sunshine
On Mon, Feb 5, 2018 at 6:55 PM, Stefan Beller wrote: > Both fetch and push still use alternates to access submodules in some > other code paths, but this is progress towards eliminating the alternates > hack that conflates access to the_repository and other repositories. > >

Re: [PATCH 092/194] object: move grafts to object parser

2018-02-05 Thread Eric Sunshine
On Mon, Feb 5, 2018 at 6:55 PM, Stefan Beller wrote: > Grafts are only meaningful in the context of a single repository. > Therefore they cannot be global. > > Signed-off-by: Stefan Beller > Signed-off-by: Jonathan Nieder > --- > diff

Re: [PATCH 136/194] alloc: allow arbitrary repositories for alloc functions

2018-02-05 Thread Eric Sunshine
On Mon, Feb 5, 2018 at 7:16 PM, Stefan Beller wrote: > We have to convert them all at once, because alloc_report uses funky a s/funky a/a funky/ > macro for reporting. It is better for the sake of mechanical conversion > to convert multiple functions at once rather than

Re: [RFC PATCH 000/194] Moving global state into the repository object

2018-02-05 Thread brian m. carlson
On Mon, Feb 05, 2018 at 03:51:54PM -0800, Stefan Beller wrote: > This series moves a lot of global state into the repository struct. > It applies on top of 2512f15446149235156528dafbe75930c712b29e (2.16.0) > It can be found at https://github.com/stefanbeller/git/tree/object-store > > Motivation

Re: [PATCH 076/194] push: add test showing bad interaction of replace refs and submodules

2018-02-05 Thread brian m. carlson
On Mon, Feb 05, 2018 at 03:55:37PM -0800, Stefan Beller wrote: > The ref subsystem has not been migrated yet to access the object store > via passed in repository objects. As a result replace when the object store > tries to access replace refs in a repository other than the_repository > it

Re: [PATCH 3/7] worktree move: new command

2018-02-05 Thread Jeff King
On Mon, Feb 05, 2018 at 08:28:10PM +0700, Duy Nguyen wrote: > >> This is still leaking 'worktrees'[1]. You probably want > >> free_worktrees() immediately after the find_worktree() invocation. > > > > Sorry, free_worktrees() after the last use of 'wt' since you still > > need to access its

Re: [PATCH 065/194] object-store: add repository argument to sha1_object_info

2018-02-05 Thread brian m. carlson
On Mon, Feb 05, 2018 at 03:55:26PM -0800, Stefan Beller wrote: > Add a repository argument to allow the callers of sha1_object_info > to be more specific about which repository to handle. This is a small > mechanical change; it doesn't change the implementation to handle > repositories other than

Re: [PATCH v1] dir.c: don't flag the index as dirty for changes to the untracked cache

2018-02-05 Thread Ben Peart
On 2/5/2018 4:58 PM, Brandon Williams wrote: On 02/05, Ben Peart wrote: The untracked cache saves its current state in the UNTR index extension. Currently, _any_ change to that state causes the index to be flagged as dirty and written out to disk. Unfortunately, the cost to write out the

Re: [PATCH 042/194] object-store: move alternates API to new alternates.h

2018-02-05 Thread brian m. carlson
On Mon, Feb 05, 2018 at 03:55:03PM -0800, Stefan Beller wrote: > From: Jonathan Nieder > > This should make these functions easier to find and object-store.h > less overwhelming to read. > > Signed-off-by: Stefan Beller > Signed-off-by: Jonathan Nieder

Re: [PATCH 025/194] object-store: allow prepare_alt_odb to handle arbitrary repositories

2018-02-05 Thread brian m. carlson
On Mon, Feb 05, 2018 at 03:54:46PM -0800, Stefan Beller wrote: > @@ -434,12 +433,12 @@ static int link_alt_odb_entry_the_repository(const char > *entry, > ent = alloc_alt_odb(pathbuf.buf); > > /* add the alternate entry */ > - *the_repository->objects.alt_odb_tail = ent; > -

Re: [RFC PATCH 000/194] Moving global state into the repository object

2018-02-05 Thread Stefan Beller
On Mon, Feb 5, 2018 at 3:51 PM, Stefan Beller wrote: > > Any suggestions welcome! > I wouldn't say sending out this many patches is a smooth experience: * After trying it out internally, it stopped at patch ~80 and the rate limiter kicked in. * Okay fine, I'll rate limit

Re: Missing git options

2018-02-05 Thread brian m. carlson
On Mon, Feb 05, 2018 at 10:12:05AM +0100, Martin Häcker wrote: > Hi there, > > I just recently learned that not all command line switches seem to > automatically correlate to options in the git configuration. > > This seems something that should be relatively easy to fix. > > What I’m most

Re: [PATCH v2 00/12] object_id part 11 (the_hash_algo)

2018-02-05 Thread brian m. carlson
On Fri, Feb 02, 2018 at 11:46:03AM -0800, Junio C Hamano wrote: > Thanks for working on this. All changes looked sensible (even > though I spotted one nit in the original, which was moved as-is, > which does not count as a "change" ;-)). I forgot to ask: do you want a reroll which fixes this, or

[PATCH 194/194] submodule: use submodule repos for object lookup

2018-02-05 Thread Stefan Beller
This converts the 'show_submodule_header' function to use the repository API properly, such that the submodule objects are not added to the main object store. When using the inline diff of submodules, the object store is not polluted with objects from the submodules. When using the "log" summary,

[PATCH 193/194] submodule: Reorder open_submodule function

2018-02-05 Thread Stefan Beller
In the next patch we'll need open_submodule in another function, so move it up. Signed-off-by: Stefan Beller --- submodule.c | 54 +++--- 1 file changed, 27 insertions(+), 27 deletions(-) diff --git a/submodule.c b/submodule.c

[PATCH 192/194] submodule: add repository argument to print_submodule_summary

2018-02-05 Thread Stefan Beller
Signed-off-by: Stefan Beller --- submodule.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/submodule.c b/submodule.c index 6125bfbdde..022e366940 100644 --- a/submodule.c +++ b/submodule.c @@ -441,7 +441,8 @@ static int

[PATCH 189/194] commit: allow in_merge_bases_many to handle arbitrary repositories

2018-02-05 Thread Stefan Beller
Signed-off-by: Stefan Beller --- commit.c | 8 commit.h | 3 +-- 2 files changed, 5 insertions(+), 6 deletions(-) diff --git a/commit.c b/commit.c index 900ecf84b8..a14933b208 100644 --- a/commit.c +++ b/commit.c @@ -1049,18 +1049,18 @@ int is_descendant_of(struct

[PATCH 191/194] pretty: allow format_commit_message to handle arbitrary repositories

2018-02-05 Thread Stefan Beller
Signed-off-by: Stefan Beller --- pretty.c | 6 +++--- pretty.h | 4 +--- 2 files changed, 4 insertions(+), 6 deletions(-) diff --git a/pretty.c b/pretty.c index 6dce4bb3c0..ed1b778c92 100644 --- a/pretty.c +++ b/pretty.c @@ -1503,7 +1503,7 @@ void

[PATCH 190/194] commit: allow in_merge_bases to handle arbitrary repositories

2018-02-05 Thread Stefan Beller
Signed-off-by: Stefan Beller --- commit.c | 4 ++-- commit.h | 3 +-- 2 files changed, 3 insertions(+), 4 deletions(-) diff --git a/commit.c b/commit.c index a14933b208..cdee8d3917 100644 --- a/commit.c +++ b/commit.c @@ -1072,9 +1072,9 @@ int in_merge_bases_many(struct

[PATCH 188/194] commit: add repository argument to in_merge_bases

2018-02-05 Thread Stefan Beller
Signed-off-by: Stefan Beller --- builtin/branch.c | 4 ++-- builtin/fetch.c| 2 +- builtin/log.c | 2 +- builtin/merge-base.c | 2 +- builtin/receive-pack.c

[PATCH 187/194] commit: add repository argument to in_merge_bases_many

2018-02-05 Thread Stefan Beller
Signed-off-by: Stefan Beller --- commit.c | 4 ++-- commit.h | 3 ++- remote.c | 2 +- shallow.c | 5 +++-- 4 files changed, 8 insertions(+), 6 deletions(-) diff --git a/commit.c b/commit.c index b74086d43d..98e04b949f 100644 --- a/commit.c +++ b/commit.c @@ -1049,7

[PATCH 186/194] pretty: allow logmsg_reencode to handle arbitrary repositories

2018-02-05 Thread Stefan Beller
Signed-off-by: Stefan Beller --- commit.h | 3 +-- pretty.c | 8 2 files changed, 5 insertions(+), 6 deletions(-) diff --git a/commit.h b/commit.h index 77e131d15b..7bdc808018 100644 --- a/commit.h +++ b/commit.h @@ -129,8 +129,7 @@ void free_commit_list(struct

[PATCH 184/194] pretty: add repository argument to format_commit_message

2018-02-05 Thread Stefan Beller
Signed-off-by: Stefan Beller --- archive.c | 3 ++- builtin/commit.c | 17 +++-- builtin/fmt-merge-msg.c| 2 +- builtin/notes.c| 3 ++- builtin/shortlog.c

[PATCH 185/194] commit: allow get_commit_buffer to handle arbitrary repositories

2018-02-05 Thread Stefan Beller
Signed-off-by: Stefan Beller --- commit.c | 6 +++--- commit.h | 3 +-- 2 files changed, 4 insertions(+), 5 deletions(-) diff --git a/commit.c b/commit.c index b241cc7979..b74086d43d 100644 --- a/commit.c +++ b/commit.c @@ -277,13 +277,13 @@ const void

[PATCH 183/194] commit: add repository argument to logmsg_reencode

2018-02-05 Thread Stefan Beller
Signed-off-by: Stefan Beller --- builtin/am.c | 3 ++- builtin/blame.c| 2 +- builtin/commit.c | 2 +- builtin/reset.c| 3 ++- commit.h

[PATCH 182/194] commit: add repository argument to get_commit_buffer

2018-02-05 Thread Stefan Beller
Signed-off-by: Stefan Beller --- builtin/fast-export.c | 2 +- builtin/fmt-merge-msg.c| 2 +- builtin/log.c | 3 ++- builtin/replace.c | 2 +- commit.c

[PATCH 179/194] tag: allow deref_tag to handle arbitrary repositories

2018-02-05 Thread Stefan Beller
Signed-off-by: Stefan Beller --- tag.c | 5 ++--- tag.h | 3 +-- 2 files changed, 3 insertions(+), 5 deletions(-) diff --git a/tag.c b/tag.c index 96757f37dc..d2b2bbb79c 100644 --- a/tag.c +++ b/tag.c @@ -64,12 +64,11 @@ int gpg_verify_tag(const struct object_id *oid, const

[PATCH 180/194] commit: allow lookup_commit_reference_gently to handle arbitrary repositories

2018-02-05 Thread Stefan Beller
Signed-off-by: Stefan Beller --- commit.c | 8 +++- commit.h | 4 +--- 2 files changed, 4 insertions(+), 8 deletions(-) diff --git a/commit.c b/commit.c index 33f5442dc1..426cbc9b55 100644 --- a/commit.c +++ b/commit.c @@ -21,16 +21,14 @@ int save_commit_buffer = 1;

[PATCH 181/194] commit: allow lookup_commit_reference to handle arbitrary repositories

2018-02-05 Thread Stefan Beller
Signed-off-by: Stefan Beller --- commit.c | 4 ++-- commit.h | 4 +--- 2 files changed, 3 insertions(+), 5 deletions(-) diff --git a/commit.c b/commit.c index 426cbc9b55..ac65c1dbd7 100644 --- a/commit.c +++ b/commit.c @@ -31,9 +31,9 @@ struct commit

[PATCH 178/194] object: allow parse_object to handle arbitrary repositories

2018-02-05 Thread Stefan Beller
Signed-off-by: Stefan Beller --- object.c | 20 ++-- object.h | 3 +-- 2 files changed, 11 insertions(+), 12 deletions(-) diff --git a/object.c b/object.c index 2a05613c64..2fe5fac3ce 100644 --- a/object.c +++ b/object.c @@ -244,39 +244,39 @@ struct object

[PATCH 177/194] objects: allow check_sha1_signature to handle arbitrary repositories

2018-02-05 Thread Stefan Beller
Signed-off-by: Stefan Beller --- object-store.h | 3 +-- sha1_file.c| 4 ++-- 2 files changed, 3 insertions(+), 4 deletions(-) diff --git a/object-store.h b/object-store.h index 47c38170e3..62763b8412 100644 --- a/object-store.h +++ b/object-store.h @@ -108,8 +108,7 @@

[PATCH 175/194] tag: allow parse_tag_buffer to handle arbitrary repositories

2018-02-05 Thread Stefan Beller
Signed-off-by: Stefan Beller --- tag.c | 10 +- tag.h | 3 +-- 2 files changed, 6 insertions(+), 7 deletions(-) diff --git a/tag.c b/tag.c index 52009bf454..96757f37dc 100644 --- a/tag.c +++ b/tag.c @@ -118,7 +118,7 @@ static timestamp_t parse_tag_date(const char

[PATCH 176/194] object: allow parse_object_buffer to handle arbitrary repositories

2018-02-05 Thread Stefan Beller
Signed-off-by: Stefan Beller --- object.c | 18 +- object.h | 3 +-- 2 files changed, 10 insertions(+), 11 deletions(-) diff --git a/object.c b/object.c index c2268990dd..2a05613c64 100644 --- a/object.c +++ b/object.c @@ -184,21 +184,21 @@ struct object

[PATCH 174/194] tag: allow lookup_tag to handle arbitrary repositories

2018-02-05 Thread Stefan Beller
Signed-off-by: Stefan Beller --- tag.c | 10 +- tag.h | 3 +-- 2 files changed, 6 insertions(+), 7 deletions(-) diff --git a/tag.c b/tag.c index e0b695cb60..52009bf454 100644 --- a/tag.c +++ b/tag.c @@ -92,13 +92,13 @@ struct object *deref_tag_noverify(struct object

[PATCH 171/194] commit: add repository argument to get_merge_bases

2018-02-05 Thread Stefan Beller
Signed-off-by: Stefan Beller --- builtin/log.c | 2 +- builtin/merge.c | 4 ++-- builtin/rev-parse.c | 2 +- commit.c| 4 ++-- commit.h| 3 ++- merge-recursive.c | 2 +- notes-merge.c | 2 +- revision.c | 4 ++-- sha1_name.c

[PATCH 173/194] blob: allow lookup_blob to handle arbitrary repositories

2018-02-05 Thread Stefan Beller
Signed-off-by: Stefan Beller --- blob.c | 10 +- blob.h | 3 +-- 2 files changed, 6 insertions(+), 7 deletions(-) diff --git a/blob.c b/blob.c index 7c661f178a..5dcaf63697 100644 --- a/blob.c +++ b/blob.c @@ -4,13 +4,13 @@ const char *blob_type = "blob";

[PATCH 172/194] commit: allow get_merge_bases to handle arbitrary repositories

2018-02-05 Thread Stefan Beller
Signed-off-by: Stefan Beller --- commit.c | 4 ++-- commit.h | 3 +-- 2 files changed, 3 insertions(+), 4 deletions(-) diff --git a/commit.c b/commit.c index 8b1e35d2a1..33f5442dc1 100644 --- a/commit.c +++ b/commit.c @@ -1025,9 +1025,9 @@ struct commit_list

[PATCH 170/194] commit: allow get_merge_bases_many_0 to handle arbitrary repositories

2018-02-05 Thread Stefan Beller
Signed-off-by: Stefan Beller --- commit.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/commit.c b/commit.c index a8f964c6f9..db03550d70 100644 --- a/commit.c +++ b/commit.c @@ -969,8 +969,8 @@ static int remove_redundant(struct repository *r,

[PATCH 168/194] commit: add repository argument to remove_redundant

2018-02-05 Thread Stefan Beller
Signed-off-by: Stefan Beller --- commit.c | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/commit.c b/commit.c index b01352f54a..3ed46e7372 100644 --- a/commit.c +++ b/commit.c @@ -913,7 +913,8 @@ struct commit_list *get_octopus_merge_bases(struct

[PATCH 169/194] commit: allow remove_redundant to handle arbitrary repositories

2018-02-05 Thread Stefan Beller
Signed-off-by: Stefan Beller --- commit.c | 7 +++ 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/commit.c b/commit.c index 3ed46e7372..a8f964c6f9 100644 --- a/commit.c +++ b/commit.c @@ -913,8 +913,7 @@ struct commit_list *get_octopus_merge_bases(struct

[PATCH 165/194] commit: allow parse_commit to handle arbitrary repositories

2018-02-05 Thread Stefan Beller
Signed-off-by: Stefan Beller --- commit.h | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/commit.h b/commit.h index 153f13600b..b60c287e7d 100644 --- a/commit.h +++ b/commit.h @@ -66,10 +66,9 @@ struct commit *lookup_commit_or_die(const struct

[PATCH 167/194] commit: allow merge_bases_many to handle arbitrary repositories

2018-02-05 Thread Stefan Beller
Signed-off-by: Stefan Beller --- commit.c | 9 - 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/commit.c b/commit.c index 5454bb5e11..b01352f54a 100644 --- a/commit.c +++ b/commit.c @@ -854,8 +854,7 @@ static struct commit_list

[PATCH 166/194] commit: allow paint_down_to_common to handle arbitrary repositories

2018-02-05 Thread Stefan Beller
Signed-off-by: Stefan Beller --- commit.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/commit.c b/commit.c index 3886266785..5454bb5e11 100644 --- a/commit.c +++ b/commit.c @@ -805,8 +805,7 @@ static int queue_has_nonstale(struct prio_queue *queue)

[PATCH 163/194] commit: add repository argument to merge_bases_many

2018-02-05 Thread Stefan Beller
Signed-off-by: Stefan Beller --- commit.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/commit.c b/commit.c index a4f35cd9fc..ed7349bbd7 100644 --- a/commit.c +++ b/commit.c @@ -854,7 +854,8 @@ static struct commit_list *paint_down_to_common(struct

[PATCH 159/194] commit: allow lookup_commit_graft to handle arbitrary repositories

2018-02-05 Thread Stefan Beller
Signed-off-by: Stefan Beller --- commit.c | 8 commit.h | 3 +-- 2 files changed, 5 insertions(+), 6 deletions(-) diff --git a/commit.c b/commit.c index fcda65bee2..a9660f7401 100644 --- a/commit.c +++ b/commit.c @@ -210,14 +210,14 @@ static void

[PATCH 162/194] commit: add repository argument to get_merge_bases_many_0

2018-02-05 Thread Stefan Beller
Signed-off-by: Stefan Beller --- commit.c | 9 + 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/commit.c b/commit.c index bd9049a9df..a4f35cd9fc 100644 --- a/commit.c +++ b/commit.c @@ -969,7 +969,8 @@ static int remove_redundant(struct commit **array,

[PATCH 164/194] commit: add repository argument to paint_down_to_common

2018-02-05 Thread Stefan Beller
Signed-off-by: Stefan Beller --- commit.c | 9 + 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/commit.c b/commit.c index ed7349bbd7..3886266785 100644 --- a/commit.c +++ b/commit.c @@ -805,7 +805,8 @@ static int queue_has_nonstale(struct prio_queue

[PATCH 160/194] commit: allow arbitrary repository argument for parse_commit_buffer

2018-02-05 Thread Stefan Beller
Signed-off-by: Stefan Beller --- commit.c | 10 +- commit.h | 3 +-- 2 files changed, 6 insertions(+), 7 deletions(-) diff --git a/commit.c b/commit.c index a9660f7401..67d329e13c 100644 --- a/commit.c +++ b/commit.c @@ -334,7 +334,7 @@ const void

[PATCH 161/194] commit: allow parse_commit_gently to handle arbitrary repository

2018-02-05 Thread Stefan Beller
Signed-off-by: Stefan Beller --- commit.c | 8 commit.h | 3 +-- 2 files changed, 5 insertions(+), 6 deletions(-) diff --git a/commit.c b/commit.c index 67d329e13c..bd9049a9df 100644 --- a/commit.c +++ b/commit.c @@ -393,7 +393,7 @@ int parse_commit_buffer(struct

[PATCH 156/194] migrate cached path to use the_repository

2018-02-05 Thread Stefan Beller
Signed-off-by: Stefan Beller --- blame.c | 8 --- branch.c | 14 ++-- builtin/commit.c | 38 +++ builtin/fetch.c | 4 ++--

[PATCH 158/194] commit: allow prepare_commit_graft to handle arbitrary repositories

2018-02-05 Thread Stefan Beller
Signed-off-by: Stefan Beller --- commit.c | 14 ++ object.h | 3 +++ 2 files changed, 9 insertions(+), 8 deletions(-) diff --git a/commit.c b/commit.c index 63dac3e601..fcda65bee2 100644 --- a/commit.c +++ b/commit.c @@ -197,19 +197,17 @@ static int

[PATCH 157/194] shallow: migrate shallow information into the object parser

2018-02-05 Thread Stefan Beller
Signed-off-by: Stefan Beller --- commit.h | 11 --- object.h | 6 ++ repository.c | 4 shallow.c| 52 +--- 4 files changed, 39 insertions(+), 34 deletions(-) diff --git a/commit.h b/commit.h index

[PATCH 153/194] shallow: add repository argument to register_shallow

2018-02-05 Thread Stefan Beller
Signed-off-by: Stefan Beller --- builtin/pack-objects.c | 2 +- builtin/receive-pack.c | 2 +- commit.h | 3 ++- fetch-pack.c | 2 +- shallow.c | 4 ++-- upload-pack.c | 7 --- 6 files changed, 11 insertions(+), 9 deletions(-)

[PATCH 155/194] shallow: add repository argument to is_repository_shallow

2018-02-05 Thread Stefan Beller
Signed-off-by: Stefan Beller --- builtin/fetch.c | 2 +- builtin/pack-objects.c| 4 ++-- builtin/prune.c | 2 +- builtin/rev-parse.c | 3 ++- commit.c | 2 +- commit.h

[PATCH 154/194] shallow: add repository argument to check_shallow_file_for_update

2018-02-05 Thread Stefan Beller
Signed-off-by: Stefan Beller --- shallow.c | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/shallow.c b/shallow.c index 67feeb3a4f..2a7685fa7c 100644 --- a/shallow.c +++ b/shallow.c @@ -219,7 +219,8 @@ struct commit_list

[PATCH 149/194] commit: allow commit buffer functions to handle arbitrary repositories

2018-02-05 Thread Stefan Beller
Signed-off-by: Stefan Beller --- builtin/index-pack.c | 2 +- commit.c | 33 ++--- commit.h | 17 - object.h | 6 +- repository.c | 6 ++ 5 files changed, 42 insertions(+), 22

[PATCH 152/194] shallow: add repository argument to set_alternate_shallow_file

2018-02-05 Thread Stefan Beller
Signed-off-by: Stefan Beller --- commit.h | 3 ++- environment.c | 2 +- git.c | 2 +- shallow.c | 2 +- 4 files changed, 5 insertions(+), 4 deletions(-) diff --git a/commit.h b/commit.h index 0d3ccd7279..7ef21a0fbf 100644 --- a/commit.h +++ b/commit.h @@

[PATCH 151/194] cache: convert get_graft_file to handle arbitrary repositories

2018-02-05 Thread Stefan Beller
Signed-off-by: Stefan Beller --- cache.h | 2 +- commit.c | 2 +- environment.c | 6 +++--- 3 files changed, 5 insertions(+), 5 deletions(-) diff --git a/cache.h b/cache.h index ee01daf130..a74a10fe1f 100644 --- a/cache.h +++ b/cache.h @@ -489,7 +489,7 @@ extern

[PATCH 150/194] tree: allow lookup_tree to handle arbitrary repositories

2018-02-05 Thread Stefan Beller
Signed-off-by: Stefan Beller --- tree.c | 10 +- tree.h | 3 +-- 2 files changed, 6 insertions(+), 7 deletions(-) diff --git a/tree.c b/tree.c index 1635a01794..e8150ba54b 100644 --- a/tree.c +++ b/tree.c @@ -194,13 +194,13 @@ int read_tree(struct tree *tree, int

[PATCH 148/194] commit: add repository argument to free_commit_buffer

2018-02-05 Thread Stefan Beller
Add a repository argument to allow callers of free_commit_buffer to be more specific about which repository to handle. This is a small mechanical change; it doesn't change the implementation to handle repositories other than the_repository yet. As with the previous commits, use a macro to catch

[PATCH 147/194] commit: add repository argument to unuse_commit_buffer

2018-02-05 Thread Stefan Beller
Add a repository argument to allow callers of unuse_commit_buffer to be more specific about which repository to handle. This is a small mechanical change; it doesn't change the implementation to handle repositories other than the_repository yet. As with the previous commits, use a macro to catch

[PATCH 141/194] commit: convert register_commit_graft to handle arbitrary repositories

2018-02-05 Thread Stefan Beller
From: Brandon Williams Signed-off-by: Brandon Williams Signed-off-by: Stefan Beller --- commit.c | 29 +++-- commit.h | 3 +-- 2 files changed, 16 insertions(+), 16 deletions(-) diff --git a/commit.c

[PATCH 143/194] object: add repository argument to parse_commit_gently

2018-02-05 Thread Stefan Beller
Add a repository argument to allow callers of parse_commit_gently to be more specific about which repository to handle. This is a small mechanical change; it doesn't change the implementation to handle repositories other than the_repository yet. As with the previous commits, use a macro to catch

[PATCH 144/194] commit: add repository argument to parse_commit

2018-02-05 Thread Stefan Beller
Add a repository argument to allow callers of parse_commit to be more specific about which repository to handle. This is a small mechanical change; it doesn't change the implementation to handle repositories other than the_repository yet. As with the previous commits, use a macro to catch callers

[PATCH 146/194] commit: add repository argument to get_cached_commit_buffer

2018-02-05 Thread Stefan Beller
Add a repository argument to allow callers of get_cached_commit_buffer to be more specific about which repository to handle. This is a small mechanical change; it doesn't change the implementation to handle repositories other than the_repository yet. As with the previous commits, use a macro to

[PATCH 142/194] commit: convert read_graft_file to handle arbitrary repositories

2018-02-05 Thread Stefan Beller
From: Brandon Williams Signed-off-by: Brandon Williams Signed-off-by: Stefan Beller --- commit.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/commit.c b/commit.c index 43ed3298cc..3a990b7078 100644 ---

[PATCH 145/194] commit: add repository argument to set_commit_buffer

2018-02-05 Thread Stefan Beller
Add a repository argument to allow callers of set_commit_buffer to be more specific about which repository to handle. This is a small mechanical change; it doesn't change the implementation to handle repositories other than the_repository yet. As with the previous commits, use a macro to catch

[PATCH 140/194] commit: convert commit_graft_pos() to handle arbitrary repositories

2018-02-05 Thread Stefan Beller
From: Brandon Williams Signed-off-by: Brandon Williams Signed-off-by: Stefan Beller --- commit.c | 7 +++ 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/commit.c b/commit.c index 435735ed2c..0d006ab1bb 100644 ---

[PATCH 139/194] sha1_file: allow add_to_alternates_file to handle arbitrary repositories

2018-02-05 Thread Stefan Beller
From: Brandon Williams Signed-off-by: Brandon Williams Signed-off-by: Stefan Beller --- alternates.h | 3 +-- sha1_file.c | 8 2 files changed, 5 insertions(+), 6 deletions(-) diff --git a/alternates.h b/alternates.h index

[PATCH 138/194] commit: allow lookup_commit to handle arbitrary repositories

2018-02-05 Thread Stefan Beller
Signed-off-by: Stefan Beller --- commit.c | 10 +- commit.h | 3 +-- 2 files changed, 6 insertions(+), 7 deletions(-) diff --git a/commit.c b/commit.c index 17bdd13e0d..435735ed2c 100644 --- a/commit.c +++ b/commit.c @@ -50,13 +50,13 @@ struct commit

[PATCH 137/194] object: allow object_as_type to handle arbitrary repositories

2018-02-05 Thread Stefan Beller
Signed-off-by: Stefan Beller --- object.c | 4 ++-- object.h | 3 +-- 2 files changed, 3 insertions(+), 4 deletions(-) diff --git a/object.c b/object.c index e84447f229..db15de6623 100644 --- a/object.c +++ b/object.c @@ -155,13 +155,13 @@ void *create_object(struct

[PATCH 136/194] alloc: allow arbitrary repositories for alloc functions

2018-02-05 Thread Stefan Beller
We have to convert them all at once, because alloc_report uses funky a macro for reporting. It is better for the sake of mechanical conversion to convert multiple functions at once rather than changing the structure of the reporting function. Signed-off-by: Stefan Beller ---

[PATCH 135/194] alloc: add repository argument to alloc_commit_index

2018-02-05 Thread Stefan Beller
Signed-off-by: Stefan Beller --- alloc.c | 4 ++-- cache.h | 3 ++- object.c | 2 +- 3 files changed, 5 insertions(+), 4 deletions(-) diff --git a/alloc.c b/alloc.c index 28b85b2214..277dadd221 100644 --- a/alloc.c +++ b/alloc.c @@ -82,7 +82,7 @@ void

[PATCH 129/194] alloc: add repository argument to alloc_blob_node

2018-02-05 Thread Stefan Beller
Signed-off-by: Stefan Beller --- alloc.c | 2 +- blob.c | 2 +- cache.h | 3 ++- 3 files changed, 4 insertions(+), 3 deletions(-) diff --git a/alloc.c b/alloc.c index 12afadfacd..6c5c376a25 100644 --- a/alloc.c +++ b/alloc.c @@ -49,7 +49,7 @@ static inline void

[PATCH 131/194] alloc: add repository argument to alloc_commit_node

2018-02-05 Thread Stefan Beller
Signed-off-by: Stefan Beller --- alloc.c | 2 +- blame.c | 2 +- cache.h | 3 ++- commit.c | 2 +- merge-recursive.c | 2 +- 5 files changed, 6 insertions(+), 5 deletions(-) diff --git a/alloc.c b/alloc.c index 2c8d143075..9e2b897ec1

[PATCH 133/194] alloc: add repository argument to alloc_object_node

2018-02-05 Thread Stefan Beller
Signed-off-by: Stefan Beller --- alloc.c | 2 +- cache.h | 3 ++- object.c | 2 +- 3 files changed, 4 insertions(+), 3 deletions(-) diff --git a/alloc.c b/alloc.c index 290250e359..f031ce422d 100644 --- a/alloc.c +++ b/alloc.c @@ -73,7 +73,7 @@ void

[PATCH 132/194] alloc: add repository argument to alloc_tag_node

2018-02-05 Thread Stefan Beller
Signed-off-by: Stefan Beller --- alloc.c | 2 +- cache.h | 3 ++- tag.c | 2 +- 3 files changed, 4 insertions(+), 3 deletions(-) diff --git a/alloc.c b/alloc.c index 9e2b897ec1..290250e359 100644 --- a/alloc.c +++ b/alloc.c @@ -65,7 +65,7 @@ void

[PATCH 134/194] alloc: add repository argument to alloc_report

2018-02-05 Thread Stefan Beller
Signed-off-by: Stefan Beller --- alloc.c | 2 +- cache.h | 3 ++- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/alloc.c b/alloc.c index f031ce422d..28b85b2214 100644 --- a/alloc.c +++ b/alloc.c @@ -105,7 +105,7 @@ static void report(const char *name, unsigned

[PATCH 130/194] alloc: add repository argument to alloc_tree_node

2018-02-05 Thread Stefan Beller
Signed-off-by: Stefan Beller --- alloc.c | 2 +- cache.h | 3 ++- tree.c | 2 +- 3 files changed, 4 insertions(+), 3 deletions(-) diff --git a/alloc.c b/alloc.c index 6c5c376a25..2c8d143075 100644 --- a/alloc.c +++ b/alloc.c @@ -57,7 +57,7 @@ void

[PATCH 127/194] alternates: convert add_to_alternates_memory to handle arbitrary repos

2018-02-05 Thread Stefan Beller
Signed-off-by: Stefan Beller --- alternates.h | 3 +-- sha1_file.c | 7 +++ tmp-objdir.c | 1 + 3 files changed, 5 insertions(+), 6 deletions(-) diff --git a/alternates.h b/alternates.h index e4520ae5c7..b3d8d57ba4 100644 --- a/alternates.h +++ b/alternates.h @@ -56,8

[PATCH 128/194] object: add repository argument to object_as_type

2018-02-05 Thread Stefan Beller
Signed-off-by: Stefan Beller --- blob.c | 2 +- builtin/fsck.c | 2 +- commit.c | 4 ++-- object.c | 2 +- object.h | 3 ++- refs.c | 2 +- tag.c | 2 +- tree.c | 2 +- 8 files changed, 10 insertions(+), 9 deletions(-) diff

[PATCH 126/194] streaming: allow open_istream to handle arbitrary repositories

2018-02-05 Thread Stefan Beller
Signed-off-by: Jonathan Nieder Signed-off-by: Stefan Beller --- streaming.c | 11 ++- streaming.h | 3 +-- 2 files changed, 7 insertions(+), 7 deletions(-) diff --git a/streaming.c b/streaming.c index cd52f517eb..2373395afe 100644 ---

[PATCH 124/194] streaming: allow open_istream_pack_non_delta to handle arbitrary repositories

2018-02-05 Thread Stefan Beller
Signed-off-by: Jonathan Nieder Signed-off-by: Stefan Beller --- streaming.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/streaming.c b/streaming.c index 94651ffbf0..21bd069315 100644 --- a/streaming.c +++ b/streaming.c @@ -440,9 +440,6 @@ static

[PATCH 125/194] streaming: allow open_istream_loose to handle arbitrary repositories

2018-02-05 Thread Stefan Beller
Signed-off-by: Jonathan Nieder Signed-off-by: Stefan Beller --- streaming.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/streaming.c b/streaming.c index 21bd069315..cd52f517eb 100644 --- a/streaming.c +++ b/streaming.c @@

[PATCH 123/194] streaming: allow open_istream_incore to handle arbitrary repositories

2018-02-05 Thread Stefan Beller
Signed-off-by: Jonathan Nieder Signed-off-by: Stefan Beller --- streaming.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/streaming.c b/streaming.c index cb7f8b2042..94651ffbf0 100644 --- a/streaming.c +++ b/streaming.c @@

[PATCH 122/194] object-store.h: allow read_sha1_file{_extended} to handle arbitrary repositories

2018-02-05 Thread Stefan Beller
Signed-off-by: Stefan Beller --- object-store.h | 10 +- sha1_file.c| 11 ++- 2 files changed, 11 insertions(+), 10 deletions(-) diff --git a/object-store.h b/object-store.h index 974f2de78d..47c38170e3 100644 --- a/object-store.h +++ b/object-store.h @@

[PATCH 121/194] sha1_file: allow read_object to handle arbitrary repositories

2018-02-05 Thread Stefan Beller
Signed-off-by: Stefan Beller --- sha1_file.c | 7 +++ 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/sha1_file.c b/sha1_file.c index a5ad67143d..6164ba29c5 100644 --- a/sha1_file.c +++ b/sha1_file.c @@ -1319,9 +1319,8 @@ int sha1_object_info(struct

[PATCH 120/194] streaming: allow istream_source to handle arbitrary repositories

2018-02-05 Thread Stefan Beller
Signed-off-by: Jonathan Nieder Signed-off-by: Stefan Beller --- streaming.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/streaming.c b/streaming.c index 5d2c24798e..cb7f8b2042 100644 --- a/streaming.c +++ b/streaming.c @@

[PATCH 116/194] packfile: allow has_packed_and_bad to handle arbitrary repositories

2018-02-05 Thread Stefan Beller
Signed-off-by: Jonathan Nieder Signed-off-by: Stefan Beller --- packfile.c | 4 ++-- packfile.h | 3 +-- 2 files changed, 3 insertions(+), 4 deletions(-) diff --git a/packfile.c b/packfile.c index 28453739b2..21ca15b1ce 100644 --- a/packfile.c +++

  1   2   3   4   >