Hi,
I am looking into different options to cache Git repositories on build
machines. The two most promising ways seem to be git-worktree [1] and
git-alternates [2].
I wonder if you see an advantage of one over the other?
My impression is that git-worktree supersedes git-alternates. Would
that b
On Wed, May 16 2018, Lars Schneider wrote:
> I am looking into different options to cache Git repositories on build
> machines. The two most promising ways seem to be git-worktree [1] and
> git-alternates [2].
>
> I wonder if you see an advantage of one over the other?
>
> My impression is that g
On Wed, 16 May 2018, Ævar Arnfjörð Bjarmason wrote:
>
> On Wed, May 16 2018, Lars Schneider wrote:
>
> > I am looking into different options to cache Git repositories on build
> > machines. The two most promising ways seem to be git-worktree [1] and
> > git-alternates [2].
> >
> > I wonder if you
> On 16 May 2018, at 11:29, Ævar Arnfjörð Bjarmason wrote:
>
>
> On Wed, May 16 2018, Lars Schneider wrote:
>
>> I am looking into different options to cache Git repositories on build
>> machines. The two most promising ways seem to be git-worktree [1] and
>> git-alternates [2].
>>
>> I wonde
On Wed, May 16 2018, Lars Schneider wrote:
>> On 16 May 2018, at 11:29, Ævar Arnfjörð Bjarmason wrote:
>>
>>
>> On Wed, May 16 2018, Lars Schneider wrote:
>>
>>> I am looking into different options to cache Git repositories on build
>>> machines. The two most promising ways seem to be git-worktr
On Wed, May 16 2018, Robert P. J. Day wrote:
> On Wed, 16 May 2018, Ævar Arnfjörð Bjarmason wrote:
>
>>
>> On Wed, May 16 2018, Lars Schneider wrote:
>>
>> > I am looking into different options to cache Git repositories on build
>> > machines. The two most promising ways seem to be git-worktree [
Hi everyone,
The 39th edition of Git Rev News is now published:
https://git.github.io/rev_news/2018/05/16/edition-39/
Thanks a lot to all the contributors: Johannes Schindelin, Elijah
Newren and Luca Milanesio!
Enjoy,
Christian, Jakub, Markus and Gabriel.
On 5/16/2018 6:33 AM, Ævar Arnfjörð Bjarmason wrote:
[big snip]
And here's where this isn't at all like "worktree", each of those 100
will have their own "master" branch, and they can all create 100
different branches called "topic" that can be different.
This is the biggest difference. You ca
Hi Stefan,
On Tue, 15 May 2018, Stefan Beller wrote:
> in a context unrelated to Git, I found https://github.com/grosser/wwtd
> ("What would travis do?"), which may be interesting for those who
> tweak the travis file.
While that is a nice project, I have to admit that it would not help in
most
Hi Martin,
On Sat, Apr 28, 2018 at 4:32 AM, Martin Ågren wrote:
> As you noted elsewhere [1], Ben is also working in this area. I'd be
> perfectly happy to sit on these patches until both of your contributions
> come through to master.
>
> [1]
> https://public-inbox.org/git/CABPp-BFh=gl6rnbst2bg
On 05/16/18 09:02, Derrick Stolee wrote:
> This is the biggest difference. You cannot have the same ref checked out
> in multiple worktrees, as they both may edit that ref. The alternates
> allow you to share data in a "read only" fashion. If you have one repo
> that is the "base" repo that manages
On Tue, May 15, 2018 at 6:44 PM, Stefan Beller wrote:
> On Tue, May 15, 2018 at 6:13 AM, Duy Nguyen wrote:
>> On Tue, May 15, 2018 at 3:04 AM, Stefan Beller wrote:
>>> Any other caller of 'repo_read_index' dies upon a negative return of
>>> it, so grep should, too.
>>>
>>> Signed-off-by: Stefan
On Wed, May 16 2018, Konstantin Ryabitsev wrote:
> On 05/16/18 09:02, Derrick Stolee wrote:
>> This is the biggest difference. You cannot have the same ref checked out
>> in multiple worktrees, as they both may edit that ref. The alternates
>> allow you to share data in a "read only" fashion. If
On Wed, May 16, 2018 at 05:34:34PM +0200, Ævar Arnfjörð Bjarmason wrote:
I may have missed some edge case, but I believe this entire workaround
isn't needed if you guarantee that the parent repo doesn't contain any
objects that will get un-referenced.
You can't guarantee that, because the paren
Hi Elijah
On 16 May 2018 at 16:32, Elijah Newren wrote:
> On Sat, Apr 28, 2018 at 4:32 AM, Martin Ågren wrote:
>> As you noted elsewhere [1], Ben is also working in this area. I'd be
>> perfectly happy to sit on these patches until both of your contributions
>> come through to master.
>>
>> [1]
After we initialize the various fields in `opts` but before we actually
use them, we might return early. Move the initialization further down,
to immediately before we use `opts`.
This limits the scope of `opts` and will help a later commit fix a
memory leak without having to worry about those ear
The strings allocated in `setup_unpack_trees_porcelain()` are never
freed. Provide a function `clear_unpack_trees_porcelain()` to do so and
call it where we use `setup_unpack_trees_porcelain()`. The only
non-trivial user is `unpack_trees_start()`, where we should place the
new call in `unpack_trees
From: Elijah Newren
Rename `git_merge_trees()` to `unpack_trees_start()` and extract the
call to `discard_index()` into a new function `unpack_trees_finish()`.
As a result, these are called early resp. late in `merge_trees()`,
making the resource handling clearer. The next commit will expand on
t
+ Jonathan Tan for a side discussion on anchoring.
On Wed, May 16, 2018 at 9:30 AM, Martin Ågren wrote:
>
> This patch is best viewed using something like this (note the tab!):
> --color-moved --anchored=" trees[nr_trees] = parse_tree_indirect"
Heh! Having a "is best viewed" paragraph is th
Hi brian,
On Tue, May 15, 2018 at 6:56 PM, brian m. carlson
wrote:
> Since this is a core test that tests basic functionality, annotate the
> assertions that have dependencies on SHA-1 with the appropriate
> prerequisite.
>
> Signed-off-by: brian m. carlson
> ---
> t/t1007-hash-object.sh | 16 +
On Wednesday, May 16, 2018 10:58:19 AM Konstantin Ryabitsev
wrote:
>
> 1. Find every repo mentioning the parent repository in
> their alternates 2. Repack them without the -l switch
> (which copies all the borrowed objects into those repos)
> 3. Once all child repos have been repacked this way, p
On Tue, May 15, 2018 at 6:56 PM, brian m. carlson
wrote:
> Strip out the index lines in the diff before comparing them, as these
> will differ between hash algorithms. This leads to a smaller, simpler
> change than editing the index line.
>
> Signed-off-by: brian m. carlson
> ---
> t/t3702-add-
On 05/16/18 13:14, Martin Fick wrote:
> On Wednesday, May 16, 2018 10:58:19 AM Konstantin Ryabitsev
> wrote:
>>
>> 1. Find every repo mentioning the parent repository in
>> their alternates 2. Repack them without the -l switch
>> (which copies all the borrowed objects into those repos)
>> 3. Once
On Wed, May 16 2018, Konstantin Ryabitsev wrote:
> On Wed, May 16, 2018 at 05:34:34PM +0200, Ævar Arnfjörð Bjarmason wrote:
>>I may have missed some edge case, but I believe this entire workaround
>>isn't needed if you guarantee that the parent repo doesn't contain any
>>objects that will get un-
On Wed, May 16 2018, Konstantin Ryabitsev wrote:
> Maybe git-repack can be told to only borrow parent objects if they are
> in packs. Anything not in packs should be hardlinked into the child
> repo. That's my wishful think for the day. :)
Can you elaborate on how this would help?
We're just go
On 05/16/18 14:02, Ævar Arnfjörð Bjarmason wrote:
>
> On Wed, May 16 2018, Konstantin Ryabitsev wrote:
>
>> Maybe git-repack can be told to only borrow parent objects if they are
>> in packs. Anything not in packs should be hardlinked into the child
>> repo. That's my wishful think for the day. :
Git Version: Version: 2.14.2
When reversing a range in git log, it does not start from the expected commit:
$ git show 8e11b4a41ec21e47fb0bf8b76e1edba739f57a9b
commit 8e11b4a41ec21e47fb0bf8b76e1edba739f57a9b
Author: Some Name
Date: Mon Nov 3 19:01:53 2014 +
.
.
.
$ git show
Author: Some O
Hi Mendi,
On 5/16/2018 2:19 PM, Mehdi Zeinali wrote:
Git Version: Version: 2.14.2
When reversing a range in git log, it does not start from the expected commit:
$ git show 8e11b4a41ec21e47fb0bf8b76e1edba739f57a9b
commit 8e11b4a41ec21e47fb0bf8b76e1edba739f57a9b
Author: Some Name
Date: Mon No
On Wednesday, May 16, 2018 02:12:24 PM Konstantin Ryabitsev
wrote:
> The loose objects I'm thinking of are those that are
> generated when we do "git repack -Ad" -- this takes all
> unreachable objects and loosens them (see man git-repack
> for more info). Normally, these would be pruned after a
>
On 05/16/18 14:26, Martin Fick wrote:
> If you are going to keep the unreferenced objects around
> forever, it might be better to keep them around in packed
> form?
I'm undecided about that. On the one hand this does create lots of small
files and inevitably causes (some) performance degradation
On Wednesday, May 16, 2018 03:01:13 PM Konstantin Ryabitsev
wrote:
> On 05/16/18 14:26, Martin Fick wrote:
> > If you are going to keep the unreferenced objects around
> > forever, it might be better to keep them around in
> > packed
> > form?
>
> I'm undecided about that. On the one hand this do
On 05/16/18 15:03, Martin Fick wrote:
>> I'm undecided about that. On the one hand this does create
>> lots of small files and inevitably causes (some)
>> performance degradation. On the other hand, I don't want
>> to keep useless objects in the pack, because that would
>> also cause performance de
On Wed, May 16, 2018 at 10:58:19AM -0400, Konstantin Ryabitsev wrote:
> The parent repo is not keeping track of any other repositories that may
> be using it for alternates, which is why you basically:
>
> 1. never run auto-gc in the parent repo
> 2. repack it manually using -Ad to keep loose obj
On Wednesday, May 16, 2018 03:11:47 PM Konstantin Ryabitsev
wrote:
> On 05/16/18 15:03, Martin Fick wrote:
> >> I'm undecided about that. On the one hand this does
> >> create lots of small files and inevitably causes
> >> (some) performance degradation. On the other hand, I
> >> don't want to kee
On Wed, May 16, 2018 at 03:01:13PM -0400, Konstantin Ryabitsev wrote:
> On 05/16/18 14:26, Martin Fick wrote:
> > If you are going to keep the unreferenced objects around
> > forever, it might be better to keep them around in packed
> > form?
>
> I'm undecided about that. On the one hand this d
On 16 May 2018 at 18:41, Stefan Beller wrote:
> On Wed, May 16, 2018 at 9:30 AM, Martin Ågren wrote:
>>
>> This patch is best viewed using something like this (note the tab!):
>> --color-moved --anchored=" trees[nr_trees] = parse_tree_indirect"
>
> Heh! Having a "is best viewed" paragraph is
On 05/16/18 15:23, Jeff King wrote:
> I implemented "repack -k", which keeps all objects and just rolls them
> into the new pack (along with any currently-loose unreachable objects).
> Aside from corner cases (e.g., where somebody accidentally added a 20GB
> file to an otherwise 100MB-repo and then
On Wed, May 16, 2018 at 03:29:42PM -0400, Konstantin Ryabitsev wrote:
> On 05/16/18 15:23, Jeff King wrote:
> > I implemented "repack -k", which keeps all objects and just rolls them
> > into the new pack (along with any currently-loose unreachable objects).
> > Aside from corner cases (e.g., wher
On Wednesday, May 16, 2018 12:37:45 PM Jeff King wrote:
> On Wed, May 16, 2018 at 03:29:42PM -0400, Konstantin
Ryabitsev wrote:
> Yes, that's pretty close to what we do at GitHub. Before
> doing any repacking in the mother repo, we actually do
> the equivalent of:
>
> git fetch --prune ../$id.g
On 05/16/18 15:37, Jeff King wrote:
> Yes, that's pretty close to what we do at GitHub. Before doing any
> repacking in the mother repo, we actually do the equivalent of:
>
> git fetch --prune ../$id.git +refs/*:refs/remotes/$id/*
> git repack -Adl
>
> from each child to pick up any new objec
On Wed, May 16, 2018 at 01:40:56PM -0600, Martin Fick wrote:
> > In theory the fetch means that it's safe to actually prune
> > in the mother repo, but in practice there are still
> > races. They don't come up often, but if you have enough
> > repositories, they do eventually. :)
>
> Peff,
>
> I
On Wed, May 16, 2018 at 04:02:53PM -0400, Konstantin Ryabitsev wrote:
> On 05/16/18 15:37, Jeff King wrote:
> > Yes, that's pretty close to what we do at GitHub. Before doing any
> > repacking in the mother repo, we actually do the equivalent of:
> >
> > git fetch --prune ../$id.git +refs/*:ref
On Wednesday, May 16, 2018 01:06:59 PM Jeff King wrote:
> On Wed, May 16, 2018 at 01:40:56PM -0600, Martin Fick
wrote:
> > > In theory the fetch means that it's safe to actually
> > > prune in the mother repo, but in practice there are
> > > still races. They don't come up often, but if you
> > >
On 13 May 2018 at 14:52, Merland Romain wrote:
> Hello Luke,
>
> Very interseting
> This is indeed an option we are waiting since the introduction of option
> --shelve for git p4 submit
> What I like most in your approach is the preservation of link to p4/master
> inducing small changes of git-p
Am 16.05.2018 um 20:19 schrieb Mehdi Zeinali:
Git Version: Version: 2.14.2
When reversing a range in git log, it does not start from the expected commit:
--reverse does not change the meaning A..B to B..A or something. For a
particular A..B specification, the set of commits selected when
--r
>
> You can also do periodic maintenance like:
>
> 1. Copy each ref in the forked repositories into the parent repository
> (e.g., giving each child that borrows from the parent its own
> hierarchy in refs/remotes//*).
Can you just copy? I assume the mother repo doesn't know about
all
On Wed, May 16, 2018 at 9:41 AM, Stefan Beller wrote:
> + Jonathan Tan for a side discussion on anchoring.
>
> On Wed, May 16, 2018 at 9:30 AM, Martin Ågren wrote:
>>
>> This patch is best viewed using something like this (note the tab!):
>> --color-moved --anchored=" trees[nr_trees] = parse
On Wed, May 16, 2018 at 12:29 PM, Martin Ågren wrote:
> On 16 May 2018 at 18:41, Stefan Beller wrote:
>> On Wed, May 16, 2018 at 9:30 AM, Martin Ågren wrote:
>>>
>>> This patch is best viewed using something like this (note the tab!):
>>> --color-moved --anchored=" trees[nr_trees] = parse_t
Signed-off-by: Ramsay Jones
---
Hi Brandon,
If you need to re-roll your 'bw/refspec-api' branch, could you please
squash this, or the equivalent change before the 'struct refname' to
'struct refname_item' name change, into the relevant patch. (which
would be patch #1, commit 8999381ed).
This p
On 05/16, Ramsay Jones wrote:
>
> Signed-off-by: Ramsay Jones
> ---
>
> Hi Brandon,
>
> If you need to re-roll your 'bw/refspec-api' branch, could you please
> squash this, or the equivalent change before the 'struct refname' to
> 'struct refname_item' name change, into the relevant patch. (whi
On Wed, May 16, 2018 at 2:42 PM, Brandon Williams wrote:
> On 05/16, Ramsay Jones wrote:
>>
>> Signed-off-by: Ramsay Jones
>> ---
>>
>> Hi Brandon,
>>
>> If you need to re-roll your 'bw/refspec-api' branch, could you please
>> squash this, or the equivalent change before the 'struct refname' to
>
On Wed, 16 May 2018, Stefan Beller wrote:
> On Wed, May 16, 2018 at 2:42 PM, Brandon Williams wrote:
>
> > Though now I'm confused, I thought we were going towards eliminating
> > using the extern keyword? ...of course I guess it means something
> > _slightly_ different when using with a variable
Hi Martin,
On Wed, May 16, 2018 at 9:30 AM, Martin Ågren wrote:
> On 16 May 2018 at 16:32, Elijah Newren wrote:
>> On Sat, Apr 28, 2018 at 4:32 AM, Martin Ågren wrote:
>>> As you noted elsewhere [1], Ben is also working in this area. I'd be
>>> perfectly happy to sit on these patches until both
Signed-off-by: Ramsay Jones
---
Hi Stefan,
If you need to re-roll your 'sb/object-store-grafts' branch, could
you please squash this into the relevant patch (whichever one that
would be)! ;-)
I have not looked to see which patch needs to change (just being
lazy, sorry!), but this variable was
Hi Ramsay,
> That commit seems to rename the 'shallow_stat' symbol to the
> 'the_repository_shallow_stat' symbol, but at the same time adds an
> 'shallow_stat' field to the parsed_object_pool struct, so ... :(
Thanks for catching this! it shows again, how rebase can be a
dangerous tool if not u
Any other caller of 'repo_read_index' dies upon a negative return of
it, so grep should, too.
Signed-off-by: Stefan Beller
Signed-off-by: Junio C Hamano
---
builtin/grep.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/builtin/grep.c b/builtin/grep.c
index 6e7bc76785a..69
Despite ls-files being a plumbing command, which promises to not change its
output ever, and to be easy on machines (e.g. non-localized output),
it may make sense to localize the error message for a corrupt index
nevertheless:
1. that is more consistent with the rest of Git.
2. Searching for "ls-t
Signed-off-by: Stefan Beller
---
builtin/checkout-index.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/builtin/checkout-index.c b/builtin/checkout-index.c
index a730f6a1aa4..aaba99d36c0 100644
--- a/builtin/checkout-index.c
+++ b/builtin/checkout-index.c
@@ -10,6 +10,7
By switching to repo_read_index_or_die, we'll get a slightly different
error message ("index file corrupt") as well as localization of it.
Signed-off-by: Stefan Beller
---
rerere.c | 10 --
1 file changed, 4 insertions(+), 6 deletions(-)
diff --git a/rerere.c b/rerere.c
index 18cae2d11c
This replaces all patterns of "if (read_cached() < 0) die(some-msg);"
with repo_read_index_or_die; this changes the output of the error message.
However as all error messages before were translated, these are for human
consumption, so a change in error message is not bad; in fact it makes Git
more
Signed-off-by: Stefan Beller
---
blame.c | 5 +++--
builtin/am.c | 3 ++-
builtin/diff.c| 3 ++-
builtin/fsck.c| 3 ++-
builtin/merge-index.c | 3 ++-
check-racy.c | 2 +-
diff.c| 5 +++--
merge-recursive.c | 3 ++-
revision.c
> If you have time, yes translate them all. I don't see how any of these
> strings are meant for script. If not, just _() the new string you
> added is fine.
> With a majority of call sites dying like this though, I wonder if we
> should just add repo_read_index_or_die() with die() inside. Then th
Signed-off-by: Stefan Beller
---
builtin/check-attr.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/builtin/check-attr.c b/builtin/check-attr.c
index 91444dc0448..bf05e7e93ca 100644
--- a/builtin/check-attr.c
+++ b/builtin/check-attr.c
@@ -4,6 +4,7 @@
#include "attr.h"
Signed-off-by: Stefan Beller
---
t/helper/test-dump-cache-tree.c | 5 ++---
t/helper/test-dump-untracked-cache.c | 4 ++--
t/helper/test-lazy-init-name-hash.c | 11 ++-
t/helper/test-read-cache.c | 3 ++-
t/helper/test-scrap-cache-tree.c | 4 ++--
t/helper/test-wri
A common pattern with the repo_read_index function is to die if the return
of repo_read_index is negative. Move this pattern into a function.
This patch replaces the calls of repo_read_index with its _or_die version,
if the error message is exactly the same.
Signed-off-by: Stefan Beller
---
bu
grep is a porcelain command, so translating its error message is a good
idea.
Signed-off-by: Stefan Beller
---
builtin/grep.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/builtin/grep.c b/builtin/grep.c
index 69f0743619f..2c2d6cc6bca 100644
--- a/builtin/grep.c
+++ b/bui
The code is used by the fetch command, which is a main porcelain command,
so we should localize its error messages.
Signed-off-by: Stefan Beller
---
submodule.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/submodule.c b/submodule.c
index 74d35b25779..71c042e1371 100644
-
On 05/16, Stefan Beller wrote:
> Signed-off-by: Stefan Beller
> ---
> blame.c | 5 +++--
> builtin/am.c | 3 ++-
> builtin/diff.c| 3 ++-
> builtin/fsck.c| 3 ++-
> builtin/merge-index.c | 3 ++-
> check-racy.c | 2 +-
> diff.c| 5 ++
Convert 'valid_fetch_refspec()' to use the new 'parse_refspec()'
function to only parse a single refspec and eliminate an allocation.
Signed-off-by: Brandon Williams
---
refspec.c | 17 -
refspec.h | 3 ++-
2 files changed, 10 insertions(+), 10 deletions(-)
diff --git a/refspec
Convert 'push_check()' to use 'struct refspec'.
Signed-off-by: Brandon Williams
---
builtin/submodule--helper.c | 13 +++--
1 file changed, 7 insertions(+), 6 deletions(-)
diff --git a/builtin/submodule--helper.c b/builtin/submodule--helper.c
index c0c4db007..88a149a2c 100644
--- a/buil
Changes in v2:
* added missing extern keyword in the first patch
* reordered patch 2 and 3 and updated some comments to be clearer.
* fixed some memory leaks
* squashed in some changes recommended by Aevar
Brandon Williams (36):
refspec: move refspec parsing logic into its own file
refspe
Introduce 'struct refspec', an abstraction around a collection of
'struct refspec_item's much like how 'struct pathspec' holds a
collection of 'struct pathspec_item's.
A refspec struct also contains an array of the original refspec strings
which will be used to facilitate the migration to using th
Convert 'get_tracking_branch()' to use 'refspec_item_init()' instead of
the old 'parse_fetch_refspec()' function.
Signed-off-by: Brandon Williams
---
builtin/pull.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/builtin/pull.c b/builtin/pull.c
index 5a79deae5..09575f
Factor out the logic which parses a single refspec into its own
function. This makes it easier to reuse this logic in a future patch.
Signed-off-by: Brandon Williams
---
refspec.c | 195 +-
1 file changed, 104 insertions(+), 91 deletions(-)
d
In preparation for performing a refactor on refspec related code, move
the refspec parsing logic into its own file.
Signed-off-by: Brandon Williams
---
Makefile| 1 +
branch.c| 1 +
builtin/clone.c | 1 +
builtin/fast-export.c | 1
Convert 'match_push_refs()' to use struct refspec.
Signed-off-by: Brandon Williams
---
remote.c | 13 -
1 file changed, 8 insertions(+), 5 deletions(-)
diff --git a/remote.c b/remote.c
index 191855118..bce6e7ce4 100644
--- a/remote.c
+++ b/remote.c
@@ -1312,7 +1312,7 @@ int check_pu
In preparation for introducing an abstraction around a collection of
refspecs (much like how a 'struct pathspec' is a collection of 'struct
pathspec_item's) rename the existing 'struct refspec' to 'struct
refspec_item'.
Signed-off-by: Brandon Williams
---
branch.c| 6 ++---
Convert 'cmd_clone()' to use 'refspec_item_init()' instead of relying on
the old 'parse_fetch_refspec()' to initialize a single refspec item.
Signed-off-by: Brandon Williams
---
builtin/clone.c | 10 --
1 file changed, 4 insertions(+), 6 deletions(-)
diff --git a/builtin/clone.c b/built
Convert the set of push refspecs stored in 'struct remote' to use
'struct refspec'.
Signed-off-by: Brandon Williams
---
builtin/push.c | 10 +-
builtin/remote.c | 14 +++---
remote.c | 35 ++-
remote.h | 6 ++
4 files changed
Convert the refmap in builtin/fetch.c to be stored in a
'struct refspec'.
Signed-off-by: Brandon Williams
---
builtin/fetch.c | 17 +++--
1 file changed, 7 insertions(+), 10 deletions(-)
diff --git a/builtin/fetch.c b/builtin/fetch.c
index 18704c6cd..6b909cd5b 100644
--- a/builtin/f
Convert the refspecs in transport-helper.c to be stored in a
'struct refspec'.
Signed-off-by: Brandon Williams
---
transport-helper.c | 38 --
1 file changed, 12 insertions(+), 26 deletions(-)
diff --git a/transport-helper.c b/transport-helper.c
index b156a37
Convert fast-export to use 'struct refspec' instead of using a list of
refspec_item's.
Signed-off-by: Brandon Williams
---
builtin/fast-export.c | 21 +++--
1 file changed, 7 insertions(+), 14 deletions(-)
diff --git a/builtin/fast-export.c b/builtin/fast-export.c
index 6f105dc7
Convert the set of fetch refspecs stored in 'struct remote' to use
'struct refspec'.
Signed-off-by: Brandon Williams
---
builtin/fetch.c | 20 ++--
builtin/remote.c | 18 +-
remote.c | 38 --
remote.h | 5 +
Convert 'do_fetch()' to take a 'struct refspec' as a parameter instead
of a list of 'struct refspec_item'.
Signed-off-by: Brandon Williams
---
builtin/fetch.c | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/builtin/fetch.c b/builtin/fetch.c
index 6b909cd5b..ec54b1d
Convert 'check_push_refs()' to use 'struct refspec'.
Signed-off-by: Brandon Williams
---
remote.c | 10 ++
1 file changed, 6 insertions(+), 4 deletions(-)
diff --git a/remote.c b/remote.c
index 89820c476..191855118 100644
--- a/remote.c
+++ b/remote.c
@@ -1282,12 +1282,14 @@ static void
Now that there are no callers of 'parse_push_refspec()',
'parse_fetch_refspec()', and 'free_refspec()', remove these
functions.
Signed-off-by: Brandon Williams
---
refspec.c | 49 -
refspec.h | 5 -
2 files changed, 54 deletions(-)
diff --git
Convert 'fetch_one()' to use 'struct refspec'.
Signed-off-by: Brandon Williams
---
builtin/fetch.c | 46 +++---
1 file changed, 19 insertions(+), 27 deletions(-)
diff --git a/builtin/fetch.c b/builtin/fetch.c
index 7a1637d35..18704c6cd 100644
--- a/builti
Move the error checking for using the "--mirror", "--all", and "--tags"
options earlier and explicitly check for the presence of the flags
instead of checking for a side-effect of the flag.
Signed-off-by: Brandon Williams
---
builtin/push.c | 31 ++-
1 file changed, 1
Remove 'add_prune_tags_to_fetch_refspec()' function and instead have the
only caller directly add the tag refspec using 'refspec_append()'.
Signed-off-by: Brandon Williams
---
builtin/fetch.c | 2 +-
remote.c| 5 -
remote.h| 2 --
3 files changed, 1 insertion(+), 8 deletions(
Convert 'match_explicit_refs()' to take a 'struct refspec' as a
parameter instead of a list of 'struct refspec_item'.
Signed-off-by: Brandon Williams
---
remote.c | 9 -
1 file changed, 4 insertions(+), 5 deletions(-)
diff --git a/remote.c b/remote.c
index 9eb79ea19..84dda3fd0 100644
--
Convert 'get_ref_match()' to take a 'struct refspec' as a parameter
instead of a list of 'struct refspec_item'.
Signed-off-by: Brandon Williams
---
remote.c | 26 ++
1 file changed, 14 insertions(+), 12 deletions(-)
diff --git a/remote.c b/remote.c
index dd68e6b22..9eb79
Convert 'apply_refspecs()' to take a 'struct refspec' as a parameter instead
of a list of 'struct refspec_item'.
Signed-off-by: Brandon Williams
---
builtin/fast-export.c | 2 +-
remote.c | 15 ++-
remote.h | 3 +--
transport-helper.c| 6 +++---
4 fil
Convert 'get_ref_map()' to take a 'struct refspec' as a parameter
instead of a list of 'struct refspec_item'.
Signed-off-by: Brandon Williams
---
builtin/fetch.c | 43 ---
1 file changed, 20 insertions(+), 23 deletions(-)
diff --git a/builtin/fetch.c b/bu
Convert 'query_refspecs()' to take a 'struct refspec' as a parameter instead
of a list of 'struct refspec_item'.
Signed-off-by: Brandon Williams
---
builtin/push.c | 3 +--
remote.c | 10 +-
remote.h | 2 +-
3 files changed, 7 insertions(+), 8 deletions(-)
diff --git a/bui
Convert 'check_push_refs()' to take a 'struct refspec' as a parameter
instead of an array of 'const char *'.
Signed-off-by: Brandon Williams
---
remote.c| 14 +-
remote.h| 2 +-
transport.c | 2 +-
3 files changed, 7 insertions(+), 11 deletions(-)
diff --git a/remote.c b/r
Convert 'match_push_refs()' to take a 'struct refspec' as a parameter
instead of an array of 'const char *'.
Signed-off-by: Brandon Williams
---
builtin/remote.c| 3 +--
builtin/send-pack.c | 2 +-
http-push.c | 3 +--
remote.c| 21 -
remote.h
Convert http-push.c to store refspecs in a 'struct refspec' instead of
in an array of 'const char *'.
Signed-off-by: Brandon Williams
---
http-push.c | 17 -
1 file changed, 8 insertions(+), 9 deletions(-)
diff --git a/http-push.c b/http-push.c
index f308ce019..a724ef03f 100644
Remove 'transprot_verify_remote_names()' because all callers have
migrated to using 'struct refspec' which performs the same checks in
'parse_refspec()'.
Signed-off-by: Brandon Williams
---
builtin/send-pack.c | 2 --
transport.c | 24
transport.h | 2 -
Convert 'push_unpushed_submodules()' to take a 'struct refspec' as a
parameter instead of an array of 'const char *'.
Signed-off-by: Brandon Williams
---
submodule.c | 19 +--
submodule.h | 3 ++-
transport.c | 2 +-
3 files changed, 12 insertions(+), 12 deletions(-)
diff --gi
Convert 'transport_push()' to take a 'struct refspec' as a
parameter instead of an array of strings which represent
refspecs.
Signed-off-by: Brandon Williams
---
builtin/push.c | 3 +--
transport.c| 17 +++--
transport.h| 2 +-
3 files changed, 9 insertions(+), 13 deletions
1 - 100 of 123 matches
Mail list logo