On Fri, Dec 8, 2017 at 12:16 AM, Jeff King wrote:
> Commit 84ff053d47 (pretty.c: delimit "%(trailers)" arguments
> with ",", 2017-10-01) switched the syntax of the trailers
> placeholder, but forgot to update the documentation in
> pretty-formats.txt.
>
> There's need to mention the old syntax; it
On Fri, Dec 08, 2017 at 03:10:34AM -0500, Eric Sunshine wrote:
> On Fri, Dec 8, 2017 at 12:16 AM, Jeff King wrote:
> > Commit 84ff053d47 (pretty.c: delimit "%(trailers)" arguments
> > with ",", 2017-10-01) switched the syntax of the trailers
> > placeholder, but forgot to update the documentation
On Thu, Dec 07, 2017 at 10:47:43PM +0100, Johannes Schindelin wrote:
> > We could add that example to the test helper as then we have a good (tested)
> > example for that case, too.
>
> What we could *also* do, and what would probably make *even more* sense,
> is to simplify the example drastical
On Thu, Dec 07, 2017 at 02:31:38PM -0800, Junio C Hamano wrote:
> If this goes on top as a standalone patch, then the reason why it is
> separate from the other users of _default() is not because the way
> it uses the null return is special, but because it was written by a
> different author, I wo
On Thu, Dec 07, 2017 at 04:24:47PM -0800, Stefan Beller wrote:
> Sometimes users are given a hash of an object and they want to
> identify it further (ex.: Use verify-pack to find the largest blobs,
> but what are these? or [1])
>
> One might be tempted to extend git-describe to also work with bl
On 12/07/2017 05:31 PM, Junio C Hamano wrote:
Correct. fakeroot would report that the files that are actually
owned by the user who is running fakeroot are owned by root; the
cached stat information in the index would be "corrected" to say
that they are owned by root. So once the index is refr
On Thu, Dec 07, 2017 at 04:14:24PM -0800, Jonathan Tan wrote:
> Improve the names of the identifiers in decorate.h, document them, and
> add an example of how to use these functions.
>
> The example is compiled and run as part of the test suite.
>
> Signed-off-by: Jonathan Tan
> ---
> This patc
Commit 5b4efea666 (cvsimport: shell-quote variable used in
backticks, 2017-09-11) tried to shell-quote a variable, but
forgot to use the "/g" modifier to apply the quoting to the
whole variable. This means we'd miss any embedded
single-quotes after the first one.
Reported-by:
Signed-off-by: Jeff
On Thu, Dec 07, 2017 at 09:59:49AM +0100, SZEDER Gábor wrote:
> Since gitfiles were introduced in b44ebb19e (Add platform-independent
> .git "symlink", 2008-02-20) the order of checks during .git directory
> discovery is: gitfile, gitdir, bare repo. However, that commit did
> only partially updat
On Thu, Dec 07, 2017 at 01:47:14PM -0800, Junio C Hamano wrote:
> > diff --git a/builtin/fmt-merge-msg.c b/builtin/fmt-merge-msg.c
> > index 22034f87e7..8e8a15ea4a 100644
> > --- a/builtin/fmt-merge-msg.c
> > +++ b/builtin/fmt-merge-msg.c
> > @@ -377,7 +377,8 @@ static void shortlog(const char *na
You may want to run the test suite with a different shell
than you use to build Git. For instance, you may build with
SHELL_PATH=/bin/sh (because it's faster, or it's what you
expect to exist on systems where the build will be used) but
want to run the test suite with bash (e.g., since that
allows
The "-x" tracing option implies "--verbose". This is a
problem when running under a TAP harness like "prove", where
we need to use "--verbose-log" instead. Instead, let's
handle this the same way we do for --valgrind, including the
recent fix from 88c6e9d31c (test-lib: --valgrind should not
overrid
File descriptors 3 and 4 are special in our test suite, as
they link back to the test script's original stdout and
stderr. Normally this isn't something tests need to worry
about: they are free to clobber these descriptors for
sub-commands without affecting the overall script.
But there's one very
This series fixes some rough edges in the "-x" feature of the test
suite. With it, it should be possible to turn on tracing for CI runs.
This is mostly a repost of v1 at:
https://public-inbox.org/git/20171019210140.64lb52cqtgdh2...@sigill.intra.peff.net
which had some discussion, but wasn't p
When the test suite's "-x" option is used with bash, we end
up seeing cleanup cruft in the output:
$ bash t0001-init.sh -x
[...]
++ diff -u expected actual
+ test_eval_ret_=0
+ want_trace
+ test t = t
+ test t = t
+ set +x
ok 42 - re-init from a linked worktree
This ranges from
This patch fixes a regression in v2.14.0. It's actually fixed already in
v2.15.0 because all of the packed-ref code there was rewritten. So
there's no point in applying this on "master" or even "maint". But I
figured it was worth sharing here in case somebody else runs across it,
and in case we eve
Hi,
On Fri, 8 Dec 2017, Torsten Bögershausen wrote:
> > * tb/check-crlf-for-safe-crlf (2017-11-27) 1 commit
> > (merged to 'next' on 2017-12-05 at 7adaa1fe01)
> > + convert: tighten the safe autocrlf handling
> >
> > The "safe crlf" check incorrectly triggered for contents that does
> > not
Kindly Assist Me
In good faith from Mr.Sonami, actually could you please consider to
help me to relocate this sum of five million, three hundred thousand
dollars (US$5.3 m) to your country for establishing a medium industry
in your country? The said 5.3 million dollars was deposited in our
bank b
Create header for pretty.c to make formatting interface more structured.
This is a middle point, this file would be merged futher with other
files which contain formatting stuff.
Signed-off-by: Olga Telezhnaia
Mentored-by: Christian Couder
Mentored by: Jeff King
---
archive.c | 1
Write some docs for functions in pretty.h.
Take it as a first draft, they would be changed later.
Signed-off-by: Olga Telezhnaia
Mentored-by: Christian Couder
Mentored by: Jeff King
---
pretty.h | 44
1 file changed, 44 insertions(+)
diff --git a/p
Stefan Beller writes:
> diff --git a/diffcore-blobfind.c b/diffcore-blobfind.c
> new file mode 100644
> index 00..e65c7cad6e
> --- /dev/null
> +++ b/diffcore-blobfind.c
> @@ -0,0 +1,41 @@
> +/*
> + * Copyright (c) 2017 Google Inc.
> + */
> +#include "cache.h"
> +#include "diff.h"
> +#incl
Hi Peff,
the other three patches look good to me.
On Fri, 8 Dec 2017, Jeff King wrote:
> You may want to run the test suite with a different shell
> than you use to build Git. For instance, you may build with
> SHELL_PATH=/bin/sh (because it's faster, or it's what you
> expect to exist on system
Changed 'he' to 'them' to be more neutral in "gitworkflows.txt".
See discussion at:
https://public-inbox.org/git/xmqqvahieeqy@gitster.mtv.corp.google.com/
Signed-off-by: Matthieu Moy
Signed-off-by: Timothee Albertin
Signed-off-by: Nathan Payre
Signed-off-by: Daniel Bensoussan
---
Docume
From: Jeff Hostetler
This is V7 of part 2 of partial clone. This builds upon V6 of part 1.
This version squashes the fixup commits that I added to the V6p2 series.
The net result is identical.
Jonathan Tan (10):
extension.partialclone: introduce partial clone extension
fsck: introduce part
From: Jonathan Tan
Teach fsck to not treat refs referring to missing promisor objects as an
error when extensions.partialclone is set.
For the purposes of warning about no default refs, such refs are still
treated as legitimate refs.
Signed-off-by: Jonathan Tan
---
builtin/fsck.c |
From: Jonathan Tan
Introduce new repository extension option:
`extensions.partialclone`
See the update to Documentation/technical/repository-version.txt
in this patch for more information.
Signed-off-by: Jonathan Tan
---
Documentation/technical/repository-version.txt | 12
ca
From: Jonathan Tan
Introduce fetch-object, providing the ability to fetch one object from a
promisor remote.
This uses fetch-pack. To do this, the transport mechanism has been
updated with 2 flags, "from-promisor" to indicate that the resulting
pack comes from a promisor remote (and thus should
From: Jonathan Tan
Teach rev-list to support termination of an object traversal at any
object from a promisor remote (whether one that the local repo also has,
or one that the local repo knows about because it has another promisor
object that references it).
This will be used subsequently in gc
From: Jonathan Tan
In a subsequent commit, index-pack will be taught to write ".promisor"
files which are similar to the ".keep" files it knows how to write.
Refactor the writing of ".keep" files, so that the implementation of
writing ".promisor" files becomes easier.
Signed-off-by: Jonathan Tan
From: Jonathan Tan
Teach sha1_file to fetch objects from the remote configured in
extensions.partialclone whenever an object is requested but missing.
The fetching of objects can be suppressed through a global variable.
This is used by fsck and index-pack.
However, by default, such fetching is
From: Jonathan Tan
Teach gc to stop traversal at promisor objects, and to leave promisor
packfiles alone. This has the effect of only repacking non-promisor
packfiles, and preserves the distinction between promisor packfiles and
non-promisor packfiles.
Signed-off-by: Jonathan Tan
Signed-off-by:
From: Jonathan Tan
Currently, Git does not support repos with very large numbers of objects
or repos that wish to minimize manipulation of certain blobs (for
example, because they are very large) very well, even if the user
operates mostly on part of the repo, because Git is designed on the
assum
From: Jonathan Tan
Teach fsck to not treat missing promisor objects provided on the CLI as
an error when extensions.partialclone is set.
Signed-off-by: Jonathan Tan
---
builtin/fsck.c | 2 ++
t/t0410-partial-clone.sh | 13 +
2 files changed, 15 insertions(+)
diff --git
From: Jonathan Tan
Teach fsck to not treat missing promisor objects indirectly pointed to
by refs as an error when extensions.partialclone is set.
Signed-off-by: Jonathan Tan
---
builtin/fsck.c | 11 +++
t/t0410-partial-clone.sh | 23 +++
2 files changed,
On Thu, Dec 7, 2017 at 7:04 PM, Junio C Hamano wrote:
> * cc/skip-to-optional-val (2017-12-07) 7 commits
> - t4045: test 'diff --relative' for real
> - t4045: reindent to make helpers readable
> - diff: use skip-to-optional-val in parsing --relative
> - diff: use skip_to_optional_val_default
Elazar Leibovich writes:
> ignore unused information, such as commit
> 2cb45e95438c113871fbbea5b4f629f9463034e7
> which ignores st_dev, because it doesn't actually matter, or
I do not think it ignores because "it doesn't matter". st_dev is
known not to be stable (e.g. across reboots and reconne
Christian Couder writes:
> On Thu, Dec 7, 2017 at 7:04 PM, Junio C Hamano wrote:
>
>
>> * cc/skip-to-optional-val (2017-12-07) 7 commits
>> - t4045: test 'diff --relative' for real
>> - t4045: reindent to make helpers readable
>> - diff: use skip-to-optional-val in parsing --relative
>> - di
From: Jeff Hostetler
This is V7 of part 3 of partial clone. It builds upon V7 of part 2
(which builds upon V6 of part 1).
This version adds additional tests, fixes test errors on the MAC version,
and squashes some fixup commits.
It also restores functionality accidentally dropped from the V6 s
From: Jonathan Tan
Teach gc to stop traversal at promisor objects, and to leave promisor
packfiles alone. This has the effect of only repacking non-promisor
packfiles, and preserves the distinction between promisor packfiles and
non-promisor packfiles.
Signed-off-by: Jonathan Tan
Signed-off-by:
From: Jeff Hostetler
Teach fetch to support filters. This is only allowed for the remote
configured in extensions.partialcloneremote.
Signed-off-by: Jonathan Tan
---
builtin/fetch.c | 23 +--
connected.c | 2 ++
remote-curl.c | 6 ++
t/t5500-fe
From: Jonathan Tan
In fetch-pack, the global variable save_commit_buffer is set to 0, but
not restored to its original value after use.
In particular, if show_log() (in log-tree.c) is invoked after
fetch_pack() in the same process, show_log() will return before printing
out the commit message (b
From: Jonathan Tan
When running checkout, first prefetch all blobs that are to be updated
but are missing. This means that only one pack is downloaded during such
operations, instead of one per missing blob.
This operates only on the blob level - if a repository has a missing
tree, they are stil
From: Jonathan Tan
Signed-off-by: Jonathan Tan
Signed-off-by: Jeff Hostetler
---
builtin/clone.c | 22 --
t/t5601-clone.sh | 49 +
2 files changed, 69 insertions(+), 2 deletions(-)
diff --git a/builtin/clone.c b/builtin/clon
From: Jeff Hostetler
Teach (partial) fetch to inherit the filter-spec used by
the partial clone. Extend --no-filter to override this
inheritance.
Signed-off-by: Jeff Hostetler
---
builtin/fetch-pack.c | 2 +-
builtin/fetch.c | 56 +++
From: Jeff Hostetler
Add test to t5616 to bulk fetch missing objects following
a partial fetch. A technique like this could be used in
a pre-command hook for example.
Signed-off-by: Jeff Hostetler
---
t/t5616-partial-clone.sh | 32 +++-
1 file changed, 31 insertion
From: Jeff Hostetler
Create get and set routines for "partial clone" config settings.
These will be used in a future commit by clone and fetch to
remember the promisor remote and the default filter-spec.
Signed-off-by: Jeff Hostetler
---
cache.h | 1 +
config.c
From: Jeff Hostetler
Additional end-to-end tests for partial clone.
Signed-off-by: Jeff Hostetler
---
t/t5616-partial-clone.sh | 96
1 file changed, 96 insertions(+)
create mode 100755 t/t5616-partial-clone.sh
diff --git a/t/t5616-partial-clon
From: Jeff Hostetler
Fixup fetch-pack to accept --no-filter to be consistent with
rev-list and pack-objects.
Signed-off-by: Jeff Hostetler
---
builtin/fetch-pack.c | 4
1 file changed, 4 insertions(+)
diff --git a/builtin/fetch-pack.c b/builtin/fetch-pack.c
index 7957807..cbf5035 100644
From: Jonathan Tan
Separate out the calculation of remotes to be fetched from and the
actual fetching. This will allow us to include an additional step before
the actual fetching in a subsequent commit.
Signed-off-by: Jonathan Tan
---
builtin/fetch.c | 14 --
1 file changed, 8 inse
From: Jeff Hostetler
Signed-off-by: Jeff Hostetler
---
builtin/fetch-pack.c | 4
fetch-pack.c | 13 +
fetch-pack.h | 2 ++
transport-helper.c | 5 +
transport.c | 4
transport.h | 5 +
6 files changed, 33 insertions(+)
diff
From: Jonathan Tan
Created tests to verify fetch-pack and upload-pack support
for excluding large blobs using --filter=blobs:limit=
parameter.
Signed-off-by: Jonathan Tan
Signed-off-by: Jeff Hostetler
---
t/t5500-fetch-pack.sh | 27 +++
upload-pack.c | 13 +
From: Jonathan Tan
Teach sha1_file to fetch objects from the remote configured in
extensions.partialclone whenever an object is requested but missing.
The fetching of objects can be suppressed through a global variable.
This is used by fsck and index-pack.
However, by default, such fetching is
From: Jonathan Tan
Teach rev-list to support termination of an object traversal at any
object from a promisor remote (whether one that the local repo also has,
or one that the local repo knows about because it has another promisor
object that references it).
This will be used subsequently in gc
From: Jeff Hostetler
Teach upload-pack to negotiate object filtering over the protocol and
to send filter parameters to pack-objects. This is intended for partial
clone and fetch.
The idea to make upload-pack configurable using uploadpack.allowFilter
comes from Jonathan Tan's work in [1].
[1]
Igor Djordjevic writes:
> To get back on track, and regarding what`s already been said, would
> having something like this(1) feel useful?
>
> (1) git commit --onto
Are you asking me if _I_ find it useful? It is not a very useful
question to ask, as I've taken things that I do not find useful
On 08/12/17 09:34, Jeff King wrote:
> On Thu, Dec 07, 2017 at 04:24:47PM -0800, Stefan Beller wrote:
[snip]
>> Junio hinted at a different approach of solving this problem, which this
>> patch implements. Teach the diff machinery another flag for restricting
>> the information to what is shown. F
Jeff King writes:
> Commit 84ff053d47 (pretty.c: delimit "%(trailers)" arguments
> with ",", 2017-10-01) switched the syntax of the trailers
> placeholder, but forgot to update the documentation in
> pretty-formats.txt.
>
> There's need to mention the old syntax; it was never in a
> released vers
Jeff King writes:
> On Fri, Dec 08, 2017 at 03:10:34AM -0500, Eric Sunshine wrote:
>
>> On Fri, Dec 8, 2017 at 12:16 AM, Jeff King wrote:
>> > Commit 84ff053d47 (pretty.c: delimit "%(trailers)" arguments
>> > with ",", 2017-10-01) switched the syntax of the trailers
>> > placeholder, but forgot
In Git for Windows, we ask users to paste the output of said command
into their bug reports, with the idea that this frequently helps
identify where the problems are coming from.
There are some obvious missing bits of information in said output,
though, and this patch series tries to fill the gaps
In particular when local tags are used (or tags that are pushed to some
fork) to build Git, it is very hard to figure out from which particular
revision a particular Git executable was built.
Let's just report that in our build options.
We need to be careful, though, to report when the current co
From: Adric Norris
When asking for bug reports to include the output of `git version
--build-options`, the idea is that we get a better idea of the
environment where said bug occurs. In this context, it is useful to
distinguish between 32 and 64-bit builds.
We start by distinguishing between x86
Junio C Hamano writes:
> Stefan Beller writes:
> ...
>> @@ -2883,6 +2884,8 @@ int prepare_revision_walk(struct rev_info *revs)
>> simplify_merges(revs);
>> if (revs->children.name)
>> set_children(revs);
>> +if (revs->diffopt.blobfind)
>> +revs->sim
Hi,
Johannes Schindelin wrote:
> From: Adric Norris
>
> When asking for bug reports to include the output of `git version
> --build-options`, the idea is that we get a better idea of the
> environment where said bug occurs. In this context, it is useful to
> distinguish between 32 and 64-bit bui
Hi,
Johannes Schindelin wrote:
> In particular when local tags are used (or tags that are pushed to some
> fork) to build Git, it is very hard to figure out from which particular
> revision a particular Git executable was built.
Hm, can you say more about how this comes up in practice? I wonder
Am 08.12.2017 um 11:14 schrieb Jeff King:
> On Thu, Dec 07, 2017 at 01:47:14PM -0800, Junio C Hamano wrote:
>
>>> diff --git a/builtin/fmt-merge-msg.c b/builtin/fmt-merge-msg.c
>>> index 22034f87e7..8e8a15ea4a 100644
>>> --- a/builtin/fmt-merge-msg.c
>>> +++ b/builtin/fmt-merge-msg.c
>>> @@ -377,7
Am 07.12.2017 um 22:27 schrieb Jeff King:
> Grepping for "list_append.*detach" shows a few other possible cases in
> transport-helper.c, which I think are leaks.
-- >8 --
Subject: [PATCH] transport-helper: plug strbuf and string_list leaks
Transfer ownership of detached strbufs to string_lists of
On Friday 08 December 2017 04:44 AM, Junio C Hamano wrote:
Junio C Hamano writes:
Somehow this fell underneath my radar horizon. I see v4 and v5 of
4/4 but do not seem to find 1-3/4. Is this meant to be a standalone
patch, or am I expected to already have 1-3 that we already are
committed to
Olga Telezhnaya writes:
> -extern void get_commit_format(const char *arg, struct rev_info *);
> -extern const char *format_subject(struct strbuf *sb, const char *msg,
> - const char *line_separator);
> -extern void userformat_find_requirements(const char *fmt, struct
Jonathan Nieder writes:
>> @@ -390,6 +390,7 @@ const char *help_unknown_cmd(const char *cmd)
>>
>> int cmd_version(int argc, const char **argv, const char *prefix)
>> {
>> +static char build_platform[] = GIT_BUILD_PLATFORM;
>> int build_options = 0;
>> const char * const usage[]
From: Torsten Bögershausen
The new MIX tests don't pass under Windows, adapt them
to use the correct native line ending.
Signed-off-by: Torsten Bögershausen
---
Sorry for the breakage.
This needs to go on top of tb/check-crlf-for-safe-crlf
t/t0027-auto-crlf.sh | 17 +
1 fi
From: Torsten Bögershausen
Replace `git commit -m "comment" ""` with `git commit -m "comment"` to
remove the empty path spec.
Signed-off-by: Torsten Bögershausen
---
t/t0027-auto-crlf.sh | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/t/t0027-auto-crlf.sh b/t/t0027-auto-crl
Jonathan Nieder writes:
>> We need to be careful, though, to report when the current commit cannot be
>> determined, e.g. when building from a tarball without any associated Git
>> repository.
>
> This means that on Debian, it would always print
>
> built from commit: (unknown)
>
> Maybe I
Jeff Hostetler writes:
> From: Jeff Hostetler
>
> This is V7 of part 3 of partial clone. It builds upon V7 of part 2
> (which builds upon V6 of part 1).
Aren't the three patches at the bottom sort-of duplicate from the
part 2 series?
On Fri, Dec 8, 2017 at 10:18 AM, Daniel Bensoussan
wrote:
> doc: reword gitworflows for neutrality
s/gitworflows/gitworkflows/
> Changed 'he' to 'them' to be more neutral in "gitworkflows.txt".
>
> See discussion at:
> https://public-inbox.org/git/xmqqvahieeqy@gitster.mtv.corp.google.com/
>
On 12/07, Stefan Beller wrote:
> On Mon, Dec 4, 2017 at 3:58 PM, Brandon Williams wrote:
>
> > diff --git a/pkt-line.h b/pkt-line.h
> > index 3dad583e2..f1545929b 100644
> > --- a/pkt-line.h
> > +++ b/pkt-line.h
> > @@ -60,8 +60,16 @@ int write_packetized_from_buf(const char *src_in, size_t
> >
On 12/8/2017 12:58 PM, Junio C Hamano wrote:
Jeff Hostetler writes:
From: Jeff Hostetler
This is V7 of part 3 of partial clone. It builds upon V7 of part 2
(which builds upon V6 of part 1).
Aren't the three patches at the bottom sort-of duplicate from the
part 2 series?
oops. yes, y
On 12/07, Stefan Beller wrote:
> On Mon, Dec 4, 2017 at 3:58 PM, Brandon Williams wrote:
> > Sometimes it is advantageous to be able to peek the next packet line
> > without consuming it (e.g. to be able to determine the protocol version
> > a server is speaking). In order to do that introduce 's
On Fri, Dec 8, 2017 at 8:21 AM, Olga Telezhnaya
wrote:
> Create header for pretty.c to make formatting interface more structured.
> This is a middle point, this file would be merged futher with other
s/futher/further/
> files which contain formatting stuff.
>
> Signed-off-by: Olga Telezhnaia
>
tbo...@web.de writes:
> From: Torsten Bögershausen
>
> Replace `git commit -m "comment" ""` with `git commit -m "comment"` to
> remove the empty path spec.
>
> Signed-off-by: Torsten Bögershausen
> ---
> t/t0027-auto-crlf.sh | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
This looks a
Junio C Hamano writes:
> tbo...@web.de writes:
>
>> From: Torsten Bögershausen
>>
>> Replace `git commit -m "comment" ""` with `git commit -m "comment"` to
>> remove the empty path spec.
>>
>> Signed-off-by: Torsten Bögershausen
>> ---
>> t/t0027-auto-crlf.sh | 2 +-
>> 1 file changed, 1 inser
Jeff Hostetler writes:
> On 12/8/2017 12:58 PM, Junio C Hamano wrote:
>> Jeff Hostetler writes:
>>
>>> From: Jeff Hostetler
>>>
>>> This is V7 of part 3 of partial clone. It builds upon V7 of part 2
>>> (which builds upon V6 of part 1).
>>
>> Aren't the three patches at the bottom sort-of dupl
René Scharfe writes:
>> I'm not sure it's string-list's fault. Many callers (including this one)
>> ...
> The two modes (dup/nodup) make string_list code tricky. Not sure
> how far we'd get with something simpler (e.g. an array of char pointers),
> but having the caller do all string allocations
René Scharfe writes:
> Am 07.12.2017 um 22:27 schrieb Jeff King:
>> Grepping for "list_append.*detach" shows a few other possible cases in
>> transport-helper.c, which I think are leaks.
>
> -- >8 --
> Subject: [PATCH] transport-helper: plug strbuf and string_list leaks
>
> Transfer ownership of
On Fri, Dec 08, 2017 at 10:21:19AM -0800, Junio C Hamano wrote:
> Junio C Hamano writes:
>
> > tbo...@web.de writes:
> >
> >> From: Torsten Bögershausen
> >>
> >> Replace `git commit -m "comment" ""` with `git commit -m "comment"` to
> >> remove the empty path spec.
> >>
> >> Signed-off-by: Tors
On Fri, Dec 8, 2017 at 8:21 AM, Olga Telezhnaya
wrote:
> Write some docs for functions in pretty.h.
> Take it as a first draft, they would be changed later.
>
> Signed-off-by: Olga Telezhnaia
> Mentored-by: Christian Couder
> Mentored by: Jeff King
> ---
> diff --git a/pretty.h b/pretty.h
> @@
From: Jeff Hostetler
This patch contains a design document that Jonathan Tan and I have
been working on that describes the partial clone feature currently
under development.
Since edits to this document are independent of the code, I did not
include in the part 1,2,3 patch series.
Please let us
From: Jeff Hostetler
First draft of design document for partial clone feature.
Signed-off-by: Jeff Hostetler
Signed-off-by: Jonathan Tan
---
Documentation/technical/partial-clone.txt | 240 ++
1 file changed, 240 insertions(+)
create mode 100644 Documentation/tech
On 12/07, Stefan Beller wrote:
> On Mon, Dec 4, 2017 at 3:58 PM, Brandon Williams wrote:
> > One of the design goals of protocol-v2 is to improve the semantics of
> > flush packets. Currently in protocol-v1, flush packets are used both to
> > indicate a break in a list of packet lines as well as
Am 08.12.2017 um 19:44 schrieb Junio C Hamano:
> René Scharfe writes:
>
>> Am 07.12.2017 um 22:27 schrieb Jeff King:
>>> Grepping for "list_append.*detach" shows a few other possible cases in
>>> transport-helper.c, which I think are leaks.
>>
>> -- >8 --
>> Subject: [PATCH] transport-helper: plu
On 12/07, Junio C Hamano wrote:
> Brandon Williams writes:
>
> > While we could wrap the preamble into a function it sort of defeats the
> > purpose since you want to be able to call different functions based on
> > the protocol version you're speaking. That way you can have hard
> > separations
On 12/06, Junio C Hamano wrote:
> Brandon Williams writes:
>
> > In order to allow for code sharing with the server-side of fetch in
> > protocol-v2 convert upload-pack to be a builtin.
> >
> > Signed-off-by: Brandon Williams
> > ---
>
> This looks obvious and straight-forward to a cursory look
Jeff Hostetler writes:
> From: Jeff Hostetler
>
> First draft of design document for partial clone feature.
>
> Signed-off-by: Jeff Hostetler
> Signed-off-by: Jonathan Tan
> ---
Thanks.
> +Non-Goals
> +-
> +
> +Partial clone is independent of and not intended to conflict with
> +shal
On 12/06, Junio C Hamano wrote:
> Brandon Williams writes:
>
>
> EXPECTING_DONE sounded like we are expecting to see 'done' packet
> sent from the other side, but I was mistaken. It is the state
> where we are "done" expecting anything ;-).
>
> Having an (unconditional) assignment to 'state' i
On Fri, Dec 08, 2017 at 04:28:23PM +, Ramsay Jones wrote:
> On 08/12/17 09:34, Jeff King wrote:
> > On Thu, Dec 07, 2017 at 04:24:47PM -0800, Stefan Beller wrote:
> [snip]
> >> Junio hinted at a different approach of solving this problem, which this
> >> patch implements. Teach the diff machin
On 12/07, Junio C Hamano wrote:
> Brandon Williams writes:
>
> > +static struct protocol_capability *get_capability(const char *key)
> > +{
> > + int i;
> > +
> > + if (!key)
> > + return NULL;
> > +
> > + for (i = 0; i < ARRAY_SIZE(capabilities); i++) {
> > + struct pro
Johannes Schindelin writes:
> On Wed, 6 Dec 2017, Junio C Hamano wrote:
> ...
>> I vaguely recalled and feared that we on purpose kept this program
>> separate from the rest of the system for a reason, but my mailing
>> list search is coming up empty.
>
> I only recall that we kept it in the bin/
From: "Ulrich Windl"
Hi Philip!
I'm unsure what you are asking for...
Ulrich
Hi Ulrich,
I was doing a retrospective follow up (of the second kind [1]).
In your initial email
https://public-inbox.org/git/5a1d70fd02a100029...@gwsmtp1.uni-regensburg.de/
you said
"I wanted to delete the t
Olga Telezhnaya writes:
> archive.c | 1 +
> builtin/notes.c | 2 +-
> builtin/reset.c | 2 +-
> builtin/show-branch.c | 2 +-
> combine-diff.c| 1 +
> commit.c | 1 +
> commit.h | 80 --
>
On Fri, Dec 8, 2017 at 12:19 PM, Jeff King wrote:
> On Fri, Dec 08, 2017 at 04:28:23PM +, Ramsay Jones wrote:
>
>> On 08/12/17 09:34, Jeff King wrote:
>> > On Thu, Dec 07, 2017 at 04:24:47PM -0800, Stefan Beller wrote:
>> [snip]
>> >> Junio hinted at a different approach of solving this proble
1 - 100 of 134 matches
Mail list logo
