From: Christian Couder
The callers of the fetch_object() and fetch_objects() might
be interested in knowing if these functions succeeded or not.
Signed-off-by: Christian Couder
Signed-off-by: Junio C Hamano
---
fetch-object.c | 13 -
fetch-object.h | 4 ++--
sha1-file.c| 4
From: Christian Couder
The promisor-remote.{c,h} files will contain functions to
manage many promisor remotes.
We expect that there will not be a lot of promisor remotes,
so it is ok to use a simple linked list to manage them.
Helped-by: Jeff King
Signed-off-by: Christian Couder
-inbox.org/git/20180802061505.2983-1-chrisc...@tuxfamily.org/
V3: https://public-inbox.org/git/20180713174959.16748-1-chrisc...@tuxfamily.org/
V2: https://public-inbox.org/git/20180630083542.20347-1-chrisc...@tuxfamily.org/
V1: https://public-inbox.org/git/20180623121846.19750-1-chrisc...@tuxfamily.
From: Christian Couder
We will need to reinitialize the promisor remote configuration
as we will make some changes to it in a later commit.
Signed-off-by: Christian Couder
---
promisor-remote.c | 14 --
promisor-remote.h | 1 +
2 files changed, 13 insertions(+), 2 deletions
A remote specified using the extensions.partialClone config
option should be considered a promisor remote too.
This remote should be at the end of the promisor remote list,
so that it is used only if objects have not been found in other
remotes.
Signed-off-by: Christian Couder
---
promisor
As the infrastructure for more than one promisor remote
has been introduced in previous patches, we can remove
code that forbids the registration of more than one
promisor remote.
Signed-off-by: Christian Couder
---
builtin/fetch.c | 20 +---
1 file changed, 5 insertions(+), 15
when cloning or fetching using a partial clone filter,
remote.origin.promisor will be set to "true" instead of
setting extensions.partialClone to "origin". This makes it
possible to use many promisor remote just by fetching from
them.
Signed-off-by: Christian Couder
This makes it possible to specify a different partial clone
filter for each promisor remote.
Signed-off-by: Christian Couder
---
builtin/fetch.c | 2 +-
list-objects-filter-options.c | 27 +++
list-objects-filter-options.h | 3 ++-
promisor-remote.c
From: Christian Couder
This is implemented for now by calling fetch_objects(). It fetches
from all the promisor remotes.
Signed-off-by: Christian Couder
---
promisor-remote.c | 17 +
promisor-remote.h | 1 +
2 files changed, 18 insertions(+)
diff --git a/promisor-remote.c b
From: Christian Couder
This shows that it is now possible to fetch objects from more
than one promisor remote, and that fetching from a new
promisor remote can configure it as one.
Signed-off-by: Christian Couder
---
t/t0410-partial-clone.sh | 47 +++-
1
Signed-off-by: Christian Couder
---
Documentation/config/remote.txt | 8
1 file changed, 8 insertions(+)
diff --git a/Documentation/config/remote.txt b/Documentation/config/remote.txt
index 6c4cad83a2..a8e6437a90 100644
--- a/Documentation/config/remote.txt
+++ b/Documentation/config
While at it, let's remove a reference to ODB effort as the ODB
effort has been replaced by directly enhancing partial clone
and promisor remote features.
Signed-off-by: Christian Couder
---
Documentation/technical/partial-clone.txt | 83 ---
1 file changed, 58 inser
Hi,
On Fri, Mar 15, 2019 at 2:55 AM jonathan chang wrote:
>
> On Mon, Mar 11, 2019 at 1:59 AM Thomas Gummerer wrote:
> >
> > On 03/10, Jonathan Chang wrote:
> > > Also I found that there is no such function as test_char_count,
> > > is it worthwile to add such function? Here are some stat:
> >
On Thu, Feb 21, 2019 at 9:24 AM Jeff King wrote:
>
> On Wed, Feb 20, 2019 at 11:59:04PM -0500, Jeff King wrote:
>
> > On Wed, Feb 20, 2019 at 02:03:40PM +0100, Ævar Arnfjörð Bjarmason wrote:
> >
> > > > I wanted to point a colleague of mine to one of the talks.
> > > > Unfortunately I could not fi
Hi everyone!
A draft of a new Git Rev News edition is available here:
https://github.com/git/git.github.io/blob/master/rev_news/drafts/edition-49.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
On Mon, Mar 18, 2019 at 1:52 PM Duy Nguyen wrote:
>
> On Mon, Feb 4, 2019 at 4:17 PM Christian Couder
> wrote:
> > Anyway feel free to comment and suggest improvements on those pages,
> > especially the micro-projects and ideas one. Pull requests on
> > https://github
Hi everyone,
The 49th edition of Git Rev News is now published:
https://git.github.io/rev_news/2019/03/20/edition-49/
Thanks a lot to the contributor: Luca Milanesio!
Enjoy,
Christian, Jakub, Markus and Gabriel.
On Fri, Mar 22, 2019 at 5:08 PM Kapil Jain wrote:
>
> Signed-off-by: Kapil Jain
> ---
>
> The test written does not pass, imo there's some problem with the regex part.
> please let me know about the fault.
To save some work by people who could help you, it might be a good
idea to show the output
Hi Rohit,
On Fri, Mar 22, 2019 at 4:12 PM Rohit Ashiwal
wrote:
>
> Hey People
>
> I am Rohit Ashiwal and here my first draft of the proposal for the project
> titled: `Improve consistency of sequencer commands' this summer. I need your
> feedback and more than that I need help to improve the time
On Sun, Mar 24, 2019 at 9:04 AM Kapil Jain wrote:
>
> On Sun, Mar 24, 2019 at 1:34 AM Christian Couder
> wrote:
> >
> > To save some work by people who could help you, it might be a good
> > idea to show the output of the failing test, for example the output of
>
On Wed, Mar 27, 2019 at 11:24 AM Andreas Schwab wrote:
>
> When running `git replace --graft A B' where B is a non-commit (eg. a
> tag) it displays an error,
Yeah, it seems that when A is a commit and B a tag I get:
"error: object A is a tag, not a commit"
which is wrong as A is a commit.
> bu
On Wed, Mar 27, 2019 at 2:11 PM Christian Couder
wrote:
>
> On Wed, Mar 27, 2019 at 11:24 AM Andreas Schwab wrote:
> >
> > When running `git replace --graft A B' where B is a non-commit (eg. a
> > tag) it displays an error,
>
> Yeah, it seems that when A is a c
On Wed, Mar 27, 2019 at 2:21 PM Christian Couder
wrote:
>
> On Wed, Mar 27, 2019 at 2:11 PM Christian Couder
> wrote:
> >
> > On Wed, Mar 27, 2019 at 11:24 AM Andreas Schwab wrote:
> > >
> > > When running `git replace --graft A B' where B is a non-c
On Sat, Jul 21, 2018 at 12:23 AM, Junio C Hamano wrote:
> Stefan Beller writes:
>
>> Signed-off-by: Stefan Beller
>> ---
>>
>> Maybe we rather want to refer to the options that are described further
>> down in the document?
>
> I have no strong preference either way.
>
> The patch looks reasonab
ta islands in "builtin/pack-objects.c".
Signed-off-by: Jeff King
Signed-off-by: Christian Couder
---
Documentation/config.txt | 4 +
Makefile | 1 +
delta-islands.c | 490 +++
delta-islands.h | 11 +
pack-object
From: Jeff King
Signed-off-by: Jeff King
Signed-off-by: Christian Couder
---
t/t9930-delta-islands.sh | 143 +++
1 file changed, 143 insertions(+)
create mode 100755 t/t9930-delta-islands.sh
diff --git a/t/t9930-delta-islands.sh b/t/t9930-delta-islands.sh
From: Jeff King
As get_delta_base() will be used outside 'packfile.c' in
a following commit, let's make it non static and let's
declare it in 'packfile.h'.
Signed-off-by: Jeff King
Signed-off-by: Christian Couder
---
packfile.c | 10 +-
packfile.
This patch series is upstreaming work made by GitHub and available in:
https://github.com/peff/git/commits/jk/delta-islands
The patch in the above branch has been split into 5 patches with their
own new commit message, but no other change has been made.
I kept Peff as the author and took the lib
From: Jeff King
Implement support for delta islands in git pack-objects
and document how delta islands work in
"Documentation/git-pack-objects.txt".
Signed-off-by: Jeff King
Signed-off-by: Christian Couder
---
Documentation/git-pack-objects.txt | 88 +++
bu
From: Jeff King
Implement simple support for --delta-islands option and
repack.useDeltaIslands config variable in git repack.
Signed-off-by: Jeff King
Signed-off-by: Christian Couder
---
Documentation/config.txt | 4
Documentation/git-repack.txt | 5 +
builtin/repack.c
On Sun, Jul 22, 2018 at 10:50 AM, Duy Nguyen wrote:
> On Sun, Jul 22, 2018 at 7:51 AM Christian Couder
> wrote:
>> +pack.island::
>> + A regular expression configuring a set of delta islands. See
>> + "DELTA ISLANDS" in linkgit:git-pack-objects[1]
Hi,
On Wed, Jul 25, 2018 at 12:03 AM, Beat Bolli wrote:
>
> On 24.07.18 23:59, Jonathan Nieder wrote:
>>
>> Beat Bolli wrote:
>>> -inline void remote_odb_reinit(void)
>>> +void remote_odb_reinit(void)
>>
>> This looks like an oversight in
>> https://public-inbox.org/git/20180713174959.16748-6-ch
On Sat, Jul 28, 2018 at 11:00 AM, Jeff King wrote:
>
> On Fri, Jul 27, 2018 at 10:22:09AM -0700, Stefan Beller wrote:
>
> > > That would solve the problem that fetching torvalds/linux from GitHub
> > > yields a bigger pack than fetching it from kernel.org. But again, it's
> > > making that root fo
From: Christian Couder
Signed-off-by: Christian Couder
Signed-off-by: Junio C Hamano
---
t/t0410-partial-clone.sh | 24 +++-
1 file changed, 23 insertions(+), 1 deletion(-)
diff --git a/t/t0410-partial-clone.sh b/t/t0410-partial-clone.sh
index 71a9b9a3e8..9d513ebf57
From: Christian Couder
We will need to reinitialize the remote odb configuration
as we will make some changes to it in a later commit when
we will detect that a remote is also a remote odb.
Signed-off-by: Christian Couder
Signed-off-by: Junio C Hamano
---
remote-odb.c | 14
From: Christian Couder
Let's make the partial clone filter specific to one odb
instead of general to all the odbs.
This makes it possible to have different partial clone
filters for different odbs.
Signed-off-by: Christian Couder
Signed-off-by: Junio C Hamano
---
builtin/fe
From: Christian Couder
Instead of using the repository_format_partial_clone global
and fetch_object() directly, let's use has_remote_odb() and
remote_odb_get_direct().
Signed-off-by: Christian Couder
Signed-off-by: Junio C Hamano
---
builtin/cat-file.c| 5 +++--
builtin/fe
From: Christian Couder
The callers of the fetch_object() and fetch_objects() might
be interested in knowing if these functions succeeded or not.
Signed-off-by: Christian Couder
Signed-off-by: Junio C Hamano
---
fetch-object.c | 15 +--
fetch-object.h | 6 +++---
sha1-file.c
From: Christian Couder
The remote-odb.{c,h} files will contain the functions
that are called by the rest of Git mostly from
"sha1-file.c" to access the objects managed by the
remote odbs.
The odb-helper.{c,h} files will contain the functions to
actually implement communication with
From: Christian Couder
This is implemented only in the promisor remote mode
for now by calling fetch_object().
Signed-off-by: Christian Couder
Signed-off-by: Junio C Hamano
---
odb-helper.c | 14 ++
odb-helper.h | 3 ++-
remote-odb.c | 17 +
remote-odb.h | 1
From: Christian Couder
This function will be used to get many objects from a promisor
remote.
Signed-off-by: Christian Couder
Signed-off-by: Junio C Hamano
---
odb-helper.c | 15 +++
odb-helper.h | 3 +++
remote-odb.c | 17 +
remote-odb.h | 1 +
4 files changed
romisor remotes and external ODB support" series on
GitHub:
https://github.com/chriscool/git/commits/gl-small-promisor-external-odb12
https://github.com/chriscool/git/commits/gl-small-promisor-external-odb71
Christian Couder (9):
fetch-object: make functions return an error code
Add initi
From: Christian Couder
Signed-off-by: Junio C Hamano
---
Documentation/config.txt | 5 +
1 file changed, 5 insertions(+)
diff --git a/Documentation/config.txt b/Documentation/config.txt
index 43b2de7b5f..2d048d47f2 100644
--- a/Documentation/config.txt
+++ b/Documentation/config.txt
On Thu, Jul 26, 2018 at 6:57 PM, Junio C Hamano wrote:
> Оля Тележная writes:
>
>> 2018-07-26 1:13 GMT+03:00 Junio C Hamano :
>>>
>>> * ot/ref-filter-object-info (2018-07-17) 5 commits
>>> - ref-filter: use oid_object_info() to get object
>>> - ref-filter: merge get_obj and get_object
>>> - r
se of it.
A few new fields are added in 'struct object_entry'
in "pack-objects.h" though.
The documentation will follow in a patch that actually
uses delta islands in "builtin/pack-objects.c".
Signed-off-by: Jeff King
Signed-off-by:
From: Jeff King
As get_delta_base() will be used outside 'packfile.c' in
a following commit, let's make it non static and let's
declare it in 'packfile.h'.
Signed-off-by: Jeff King
Signed-off-by: Christian Couder
---
packfile.c | 10 +-
packfile.h |
intensive.
Signed-off-by: Jeff King
Signed-off-by: Christian Couder
---
Documentation/config.txt | 4
Documentation/git-repack.txt | 5 +
builtin/repack.c | 9 +
3 files changed, 18 insertions(+)
diff --git a/Documentation/config.txt b/Documentation/config.txt
index
From: Jeff King
Signed-off-by: Jeff King
Signed-off-by: Christian Couder
---
t/t5319-delta-islands.sh | 143 +++
1 file changed, 143 insertions(+)
create mode 100755 t/t5319-delta-islands.sh
diff --git a/t/t5319-delta-islands.sh b/t/t5319-delta-islands.sh
also available on GitHub in:
https://github.com/chriscool/git/commits/delta-islands
The previous version is available there:
https://github.com/chriscool/git/commits/delta-islands6
https://public-inbox.org/git/20180722054836.28935-1-chrisc...@tuxfamily.org/
Christian Couder (1):
pack-objects: m
loning and fetching
is still quite fast and not much more CPU intensive.
Signed-off-by: Jeff King
Signed-off-by: Christian Couder
---
Documentation/config.txt | 15
Documentation/git-pack-objects.txt | 97 +
builtin/pack-objects.c
This reduces the size of 'struct object_entry' and therefore
makes packing objects more efficient.
This also renames cmp_tree_depth() into tree_depth_compare(),
as it is more modern to have the name of the compare functions
end with "compare".
Signed-off-by: Christian Coude
On Sun, Jul 22, 2018 at 10:55 AM, Duy Nguyen wrote:
> On Sun, Jul 22, 2018 at 7:52 AM Christian Couder
> wrote:
>>
>> - /*
>> -* And then all remaining commits and tags.
>> -*/
>> - for (i = last_untagged; i < to_pack.nr_objects
On Tue, Jul 24, 2018 at 7:11 PM, Junio C Hamano wrote:
>
> Another thing I noticed from 2/5 is that you can have up to 7 such
> capture groups. I do not have any opinion if 7 is too few or too
> many, but we would want the number to be documented, and end-user
> input diagnosed when it requires m
On Sun, Jul 22, 2018 at 10:50 AM, Duy Nguyen wrote:
> On Sun, Jul 22, 2018 at 7:51 AM Christian Couder
> wrote:
>> diff --git a/pack-objects.h b/pack-objects.h
>> index edf74dabdd..8eecd67991 100644
>> --- a/pack-objects.h
>> +++ b/pack-objects.h
>> @@ -
On Sun, Aug 5, 2018 at 7:40 PM, Christian Couder
wrote:
> On Tue, Jul 24, 2018 at 7:11 PM, Junio C Hamano wrote:
> This is the new documentation:
>
> -> Refs are grouped into islands based on their "names", and two regexes
> -> that produce the same name a
On Mon, Aug 6, 2018 at 5:53 PM, Duy Nguyen wrote:
> On Sun, Aug 5, 2018 at 8:53 PM Christian Couder
> wrote:
>>
>> As you can see the patch 6/6 (in the v2 of this patch series that I
>> just sent) moves `unsigned int tree_depth` from 'struct object_entry'
>
Hi Phillip,
On Tue, Aug 7, 2018 at 3:57 PM, Phillip Wood wrote:
>
> On 31/07/18 18:59, Alban Gruin wrote:
>>
>> +
>> + ret = fputs(buf.buf, todo);
>
> It is not worth changing the patch just for this but strbuf_write()
> might be clearer (you use it in a later patch)
>
>> + if (ret < 0)
>
On Tue, Aug 7, 2018 at 6:15 PM, Phillip Wood wrote:
> On 07/08/18 16:25, Christian Couder wrote:
>>
>> I agree about checking the return value from fputs(), but it seems to
>> me that we don't usually check the value of fclose().
>
> A quick grep shows you'
ly.org/
V1: https://public-inbox.org/git/20180722054836.28935-1-chrisc...@tuxfamily.org/
Christian Couder (3):
pack-objects: refactor code into compute_layer_order()
pack-objects: move tree_depth into 'struct packing_data'
pack-objects: move 'layer' into 'struct packing_data
From: Jeff King
As get_delta_base() will be used outside 'packfile.c' in
a following commit, let's make it non static and let's
declare it in 'packfile.h'.
Signed-off-by: Jeff King
Signed-off-by: Christian Couder
---
packfile.c | 10 +-
packfile.h |
loning and fetching
is still quite fast and not much more CPU intensive.
Signed-off-by: Jeff King
Signed-off-by: Christian Couder
---
Documentation/config.txt | 15 +
Documentation/git-pack-objects.txt | 97 ++
builtin/pack-objects.c
will make it easier to see and understand what the
following changes are doing.
Helped-by: Duy Nguyen
Signed-off-by: Christian Couder
---
builtin/pack-objects.c | 90 +++---
1 file changed, 50 insertions(+), 40 deletions(-)
diff --git a/builtin/pack-object
From: Jeff King
Signed-off-by: Jeff King
Signed-off-by: Christian Couder
---
t/t5319-delta-islands.sh | 143 +++
1 file changed, 143 insertions(+)
create mode 100755 t/t5319-delta-islands.sh
diff --git a/t/t5319-delta-islands.sh b/t/t5319-delta-islands.sh
intensive.
Signed-off-by: Jeff King
Signed-off-by: Christian Couder
---
Documentation/config.txt | 4
Documentation/git-repack.txt | 5 +
builtin/repack.c | 9 +
3 files changed, 18 insertions(+)
diff --git a/Documentation/config.txt b/Documentation/config.txt
index
: Duy Nguyen
Signed-off-by: Christian Couder
---
builtin/pack-objects.c | 4 ++--
delta-islands.c| 27 ++-
pack-objects.c | 6 ++
pack-objects.h | 21 -
4 files changed, 46 insertions(+), 12 deletions(-)
diff --git a/buil
Duy Nguyen
Signed-off-by: Christian Couder
---
builtin/pack-objects.c | 4 ++--
delta-islands.c| 4 ++--
pack-objects.c | 6 ++
pack-objects.h | 20 ++--
4 files changed, 28 insertions(+), 6 deletions(-)
diff --git a/builtin/pack-objects.c b/bu
se of it.
A few new fields are added in 'struct object_entry'
in "pack-objects.h" though.
The documentation will follow in a patch that actually
uses delta islands in "builtin/pack-objects.c".
Signed-off-by: Jeff King
Signed-off-by:
On Thu, Aug 9, 2018 at 5:55 PM, Christian Couder
wrote:
> The following changes have been made since the previous iteration:
>
> * suggested by Duy: move the code computing the write order for a
> layer to a new separate function called compute_layer_order() in
> builtin/pa
On Sat, Aug 11, 2018 at 11:04 AM, SZEDER Gábor wrote:
>> diff --git a/delta-islands.c b/delta-islands.c
>> new file mode 100644
>> index 00..448ddcbbe4
>> --- /dev/null
>> +++ b/delta-islands.c
>
>> +static void deduplicate_islands(void)
>> +{
>> + struct remote_island *island, *core =
Hi Ramsay and Peff,
On Sun, Aug 12, 2018 at 3:16 AM, Jeff King wrote:
> On Sun, Aug 12, 2018 at 01:30:02AM +0100, Ramsay Jones wrote:
>
>> My static-check.pl script has pinged me about the get_delta_base()
>> symbol from packfile.[co]. The first patch from your 'cc/delta-islands'
>> branch export
On Sat, Aug 11, 2018 at 4:12 PM, Jeff King wrote:
> On Sat, Aug 11, 2018 at 12:32:32PM +0200, Christian Couder wrote:
>
>> Ok, I have made the following changes in the branch I will send next.
>>
>> diff --git a/delta-islands.c b/delta-islands.c
>> index 92137f2e
will make it easier to see and understand what the
following changes are doing.
Helped-by: Duy Nguyen
Signed-off-by: Christian Couder
---
builtin/pack-objects.c | 90 +++---
1 file changed, 50 insertions(+), 40 deletions(-)
diff --git a/builtin/pack-object
intensive.
Signed-off-by: Jeff King
Signed-off-by: Christian Couder
---
Documentation/config.txt | 4
Documentation/git-repack.txt | 5 +
builtin/repack.c | 9 +
3 files changed, 18 insertions(+)
diff --git a/Documentation/config.txt b/Documentation/config.txt
index
55532.26151-1-chrisc...@tuxfamily.org/
V2: https://public-inbox.org/git/20180805172525.15278-1-chrisc...@tuxfamily.org/
V1: https://public-inbox.org/git/20180722054836.28935-1-chrisc...@tuxfamily.org/
Christian Couder (3):
pack-objects: refactor code into compute_layer_order()
pack-objects: move tre
loning and fetching
is still quite fast and not much more CPU intensive.
Signed-off-by: Jeff King
Signed-off-by: Christian Couder
---
Documentation/config.txt | 15 +
Documentation/git-pack-objects.txt | 97 ++
builtin/pack-objects.c
se of it.
A few new fields are added in 'struct object_entry'
in "pack-objects.h" though.
The documentation will follow in a patch that actually
uses delta islands in "builtin/pack-objects.c".
Signed-off-by: Jeff King
Signed-off-by:
: Duy Nguyen
Signed-off-by: Christian Couder
---
builtin/pack-objects.c | 4 ++--
delta-islands.c| 27 ++-
pack-objects.c | 6 ++
pack-objects.h | 21 -
4 files changed, 46 insertions(+), 12 deletions(-)
diff --git a/buil
Duy Nguyen
Signed-off-by: Christian Couder
---
builtin/pack-objects.c | 4 ++--
delta-islands.c| 4 ++--
pack-objects.c | 6 ++
pack-objects.h | 20 ++--
4 files changed, 28 insertions(+), 6 deletions(-)
diff --git a/builtin/pack-objects.c b/bu
From: Jeff King
Signed-off-by: Jeff King
Signed-off-by: Christian Couder
---
t/t5319-delta-islands.sh | 143 +++
1 file changed, 143 insertions(+)
create mode 100755 t/t5319-delta-islands.sh
diff --git a/t/t5319-delta-islands.sh b/t/t5319-delta-islands.sh
On Mon, Aug 13, 2018 at 3:14 AM, Ramsay Jones
wrote:
> On 12/08/18 06:11, Christian Couder wrote:
>> Because the inefficiency primarily arises when an
>> object is delitified against another object that does
>
> s/delitified/deltified/ ?
Ok, this will be in the next reroll
On Mon, Aug 13, 2018 at 7:46 PM, Stefan Beller wrote:
> On Mon, Aug 13, 2018 at 9:31 AM Jeff King wrote:
>>
>> For the past several years, we've held a Git Contributor Summit as part
>> of the Git Merge conference. I'd like to get opinions from the community
>> to help plan future installments. A
Hi,
When cloning with --mirror, the clone gets its HEAD initialized with
the value HEAD has in its origin remote. After that if HEAD changes in
origin there is no simple way to sync HEAD at the same time as the
refs are synced.
It looks like the simplest way to sync HEAD is:
1) git remote show o
On Tue, Aug 14, 2018 at 11:47 PM, Jeff King wrote:
> On Tue, Aug 14, 2018 at 05:06:16PM -0400, Jeff King wrote:
>
>> On Tue, Aug 14, 2018 at 10:09:37PM +0200, Christian Couder wrote:
>>
>> > When cloning with --mirror, the clone gets its HEAD initialized with
>> &
On Mon, Aug 13, 2018 at 8:11 PM, Jeff King wrote:
> On Mon, Aug 13, 2018 at 01:17:18PM +0100, Ramsay Jones wrote:
>
>> >>> +struct island_bitmap {
>> >>> + uint32_t refcount;
>> >>> + uint32_t bits[];
>> >>
>> >> Use FLEX_ARRAY here? We are slowly moving toward requiring
>> >> certain C99
On Mon, Aug 13, 2018 at 9:00 PM, Jeff King wrote:
> On Mon, Aug 13, 2018 at 05:33:59AM +0200, Christian Couder wrote:
>
>> >> + memcpy(&sha_core, oid->hash, sizeof(uint64_t));
>> >> + rl->hash += sha_core;
>> >
>> > Hmm, s
51151.6425-1-chrisc...@tuxfamily.org/
V3: https://public-inbox.org/git/20180809155532.26151-1-chrisc...@tuxfamily.org/
V2: https://public-inbox.org/git/20180805172525.15278-1-chrisc...@tuxfamily.org/
V1: https://public-inbox.org/git/20180722054836.28935-1-chrisc...@tuxfamily.org/
Christian Couder
intensive.
Signed-off-by: Jeff King
Signed-off-by: Christian Couder
---
Documentation/config.txt | 4
Documentation/git-repack.txt | 5 +
builtin/repack.c | 9 +
3 files changed, 18 insertions(+)
diff --git a/Documentation/config.txt b/Documentation/config.txt
index
will make it easier to see and understand what the
following changes are doing.
Helped-by: Duy Nguyen
Signed-off-by: Christian Couder
---
builtin/pack-objects.c | 90 +++---
1 file changed, 50 insertions(+), 40 deletions(-)
diff --git a/builtin/pack-object
loning and fetching
is still quite fast and not much more CPU intensive.
Signed-off-by: Jeff King
Signed-off-by: Christian Couder
---
Documentation/config.txt | 15 +
Documentation/git-pack-objects.txt | 97 ++
builtin/pack-objects.c
Duy Nguyen
Signed-off-by: Christian Couder
---
builtin/pack-objects.c | 4 ++--
delta-islands.c| 4 ++--
pack-objects.c | 6 ++
pack-objects.h | 20 ++--
4 files changed, 28 insertions(+), 6 deletions(-)
diff --git a/builtin/pack-objects.c b/bu
se of it.
A few new fields are added in 'struct object_entry'
in "pack-objects.h" though.
The documentation will follow in a patch that actually
uses delta islands in "builtin/pack-objects.c".
Signed-off-by: Jeff King
Signed-off-by:
: Duy Nguyen
Signed-off-by: Christian Couder
---
builtin/pack-objects.c | 4 ++--
delta-islands.c| 27 ++-
pack-objects.c | 6 ++
pack-objects.h | 21 -
4 files changed, 46 insertions(+), 12 deletions(-)
diff --git a/buil
From: Jeff King
Signed-off-by: Jeff King
Signed-off-by: Christian Couder
---
t/t5319-delta-islands.sh | 143 +++
1 file changed, 143 insertions(+)
create mode 100755 t/t5319-delta-islands.sh
diff --git a/t/t5319-delta-islands.sh b/t/t5319-delta-islands.sh
Sorry, I made a copy paste error in the subject there should not be
"[PATCH v4 0/7]" in it.
On Thu, Aug 16, 2018 at 8:13 AM, Christian Couder
wrote:
> This patch series is upstreaming work made by GitHub and available in:
[...]
On Sat, Aug 18, 2018 at 12:44 AM Junio C Hamano wrote:
> * cc/delta-islands (2018-08-16) 7 commits
> - pack-objects: move 'layer' into 'struct packing_data'
> - pack-objects: move tree_depth into 'struct packing_data'
> - t5320: tests for delta islands
> - repack: add delta-islands support
>
On Sat, Aug 18, 2018 at 1:34 PM Ævar Arnfjörð Bjarmason
wrote:
> On Sat, Aug 18 2018, Christian Couder wrote:
> > FYI this has been requested from GitLab by Drupal (as well as others)
> > see
> > https://www.drupal.org/drupalorg/blog/developer-tools-initiative-part
Hi,
A draft of a new Git Rev News edition is available here:
https://github.com/git/git.github.io/blob/master/rev_news/drafts/edition-42.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 is
On Mon, Aug 20, 2018 at 10:51 AM, Ævar Arnfjörð Bjarmason
wrote:
> On Mon, Aug 20 2018, Christian Couder wrote:
>> Jakub, Markus, Gabriel and me plan to publish this edition on
>> Wednesday August 22nd.
>
> Let's mention that we've picked SHA-256 as NewHash, a
On Fri, Aug 17, 2018 at 12:41 AM, Jeff King wrote:
> So here are the nominations I came up with. If you'd like to nominate
> somebody else (or yourself!), please do. If you have opinions, let me
> know (public or private, as you prefer).
>
> - Christian Couder
> -
Hi everyone,
The 42nd edition of Git Rev News is now published:
https://git.github.io/rev_news/2018/08/22/edition-42/
Thanks a lot to the contributors: Derrick Stolee and Ævar Arnfjörð Bjarmason!
Enjoy,
Christian, Jakub, Markus and Gabriel.
1601 - 1700 of 3241 matches
Mail list logo
