Hi.
git-fast-export would work too, but it creates an additional step. I don't
commit to the model repo during tests, but I do commit when I want to modify
the tests.
So far, I configured core.compression=0 and gc.auto=0, created the
.gitattributes inside the bare repo dir containing one line: *
> On Feb 21, 2018, at 9:37 PM, Jonathan Nieder wrote:
>
> Thanks for writing it.
>
> Do you mind if we forge your sign-off? (See Documentation/SubmittingPatches
> item '(5) Certify your work' for details about what this means.)
Sure, or I can just re-paste:
Signed-off-by: Dorian Taylor
---
Stefan Beller wrote:
> Signed-off-by: Stefan Beller
> Signed-off-by: Jonathan Nieder
> ---
> object-store.h | 3 +--
> sha1_file.c| 5 +++--
> 2 files changed, 4 insertions(+), 4 deletions(-)
Thanks: this is short and simple, making my life as a reviewer very
easy.
Reviewed-by: Jonathan N
Hi,
Stefan Beller wrote:
> Add a repository argument to allow the sha1_loose_object_info caller
> to be more specific about which repository to act on. This is a small
> mechanical change; it doesn't change the implementation to handle
> repositories other than the_repository yet.
>
> As with the
Hi,
Stefan Beller wrote:
> Add a repository argument to allow the stat_sha1_file caller to be
> more specific about which repository to act on. This is a small
> mechanical change; it doesn't change the implementation to handle
> repositories other than the_repository yet.
>
> As with the previou
Hi,
Stefan Beller wrote:
> See previous patch for explanation.
>
> While at it, move the declaration to object-store.h,
> where it should be easier to find.
Which declaration? It looks like prepare_alt_odb is already in
object-store.h.
[...]
> --- a/builtin/fsck.c
> +++ b/builtin/fsck.c
> @@ -
This is a repost of the two patches from:
https://public-inbox.org/git/20180209185710.ga23...@sigill.intra.peff.net/
(now just one patch, since sg/test-i18ngrep graduated and we can do it
all in one step). The idea got positive feedback, but nobody commented
on patches and I didn't see them in
Hi,
Stefan Beller wrote:
> In a process with multiple repositories open, packfile accessors
> should be associated to a single repository and not shared globally.
> Move packed_git and packed_git_mru into the_repository and adjust
> callers to reflect this.
>
> Patch generated by
>
> 1. Moving t
Hi,
Stefan Beller wrote:
> The raw object store field will contain any objects needed for
> access to objects in a given repository.
>
> This patch introduces the raw object store and populates it with the
> `objectdir`, which used to be part of the repository struct.
>
> As the struct gains memb
(+cc: bmwill@, who is working on protocol v2)
Hi,
Dorian Taylor wrote:
> On Feb 21, 2018, at 2:15 PM, Jeff King wrote:
>> Thanks, I agree the document is buggy. Do you want to submit a patch?
>
> Will this do?
Thanks for writing it.
Do you mind if we forge your sign-off? (See Documentation/Su
2018-02-21 20:41 GMT+03:00 Junio C Hamano :
> Olga Telezhnaya writes:
>
>> Get rid of goto command in ref-filter for better readability.
>>
>> Signed-off-by: Olga Telezhnaia
>> Mentored-by: Christian Couder
>> Mentored by: Jeff King
>> ---
>> ref-filter.c | 6 +++---
>> 1 file changed, 3 inser
+git-for-windows
Hi,
Raining Chain wrote:
> On Windows 10, git version 2.16.2.windows.1, running the command
>
> git status
>
> will trigger a file change event to file C:\myPath\.git "Attributes changed."
>
> This causes problems when using scripts that detect file changes such
> as tsc -w (Typ
On Windows 10, git version 2.16.2.windows.1, running the command
git status
will trigger a file change event to file C:\myPath\.git "Attributes changed."
This causes problems when using scripts that detect file changes such
as tsc -w (Typescript compiler) and using softwares that regularly
call
On Wed, Feb 21, 2018 at 06:58:34PM -0500, Randall S. Becker wrote:
> May I suggest storing the date/time in UTC+0 in all cases. I can see
> potential issues a couple of times a year where holes exist. I cannot even
> fathom what would happen on a merge or edit of history.
I consider storing the ti
On 2/21/2018 6:13 PM, Jeff King wrote:
On Wed, Feb 21, 2018 at 02:17:11PM -0500, Derrick Stolee wrote:
The get_cached_commit_buffer() method provides access to the buffer
loaded for a struct commit, if it was ever loadead and was not freed.
Two places use this to inform how to output informati
On 02/21, Junio C Hamano wrote:
> * bw/c-plus-plus (2018-02-14) 38 commits
> - fixup! diff: rename 'this' variables
> - replace: rename 'new' variables
> - trailer: rename 'template' variables
> - tempfile: rename 'template' variables
> - wrapper: rename 'template' variables
> - environment:
Junio C Hamano wrote:
>> Compared to v2:
>> - the potential loss of errno before it's printed out in builtin/am.c
>> is fixed.
>> - keep update_ref() in sequencer.c non-fatal like this rest
>> - rename ORIG_COMMIT to REBASE_HEAD
>> Interdiff:
> This round hasn't seen any comments. Is everyb
On 02/20, Stefan Beller wrote:
> Add a repository argument to allow sha1_file_name callers 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 previ
On 02/20, Stefan Beller wrote:
> Actually this also allows read_info_alternates and link_alt_odb_entry to
> handle arbitrary repositories, but link_alt_odb_entries is the most
> interesting function in this set of functions, hence the commit subject.
>
> These functions span a strongly connected c
On 02/20, Stefan Beller wrote:
> The approximate_object_count() function maintains a rough count of
> objects in a repository to estimate how long object name abbreviates
> should be. Object names are scoped to a repository and the
> appropriate length may differ by repository, so the object count
On Tue, 20 Feb 2018 17:54:27 -0800
Stefan Beller wrote:
> Signed-off-by: Stefan Beller
> Signed-off-by: Jonathan Nieder
Reviewed-by: Jonathan Tan
On Tue, 20 Feb 2018 17:54:25 -0800
Stefan Beller wrote:
> Signed-off-by: Stefan Beller
> Signed-off-by: Jonathan Nieder
Reviewed-by: Jonathan Tan
> -void sha1_file_name_the_repository(struct strbuf *buf, const unsigned char
> *sha1)
> +void sha1_file_name(struct repository *r, struct strbuf
On 02/20, Stefan Beller wrote:
> Signed-off-by: Stefan Beller
Straight forward change, looks good.
> ---
> builtin/am.c | 2 +-
> builtin/clone.c| 2 +-
> builtin/fetch.c| 2 +-
> builtin/merge.c| 2 +-
> builtin/receive-pack.c | 2 +-
> object.c
On 02/20, Stefan Beller wrote:
> Free the memory and reset alt_odb_{list, tail} to NULL.
Good to see memory leaks being avoided (well they will be on other
repository objects)
>
> Signed-off-by: Stefan Beller
> ---
> object.c | 17 +
> 1 file changed, 17 insertions(+)
>
> diff
On 02/20, Stefan Beller wrote:
> In a process with multiple repositories open, alternates should be
> associated to a single repository and not shared globally. Move
> alt_odb_list and alt_odb_tail into the_repository and adjust callers
> to reflect this.
>
> Now that the alternative object data b
On 02/20, Stefan Beller wrote:
> Migrate the struct alternate_object_database and all its related
> functions to the object store as these functions are easier found in
> that header. The migration is just a verbatim copy, no need to
> include the object store header at any C file, because cache.h
On Tue, 20 Feb 2018 17:54:22 -0800
Stefan Beller wrote:
> Add a repository argument to allow the map_sha1_file_1 caller to be
> more specific about which repository to act on. This is a small
> mechanical change; it doesn't change the implementation to handle
> repositories other than the_reposit
On Tue, 20 Feb 2018 17:54:18 -0800
Stefan Beller wrote:
> -void prepare_alt_odb_the_repository(void)
> +void prepare_alt_odb(struct repository *r)
> {
> - const char *alt;
> -
> - if (the_repository->objects.alt_odb_tail)
> + if (r->objects.alt_odb_tail)
> return;
>
>
Here are the topics that have been cooking. Commits prefixed with
'-' are only in 'pu' (proposed updates) while commits prefixed with
'+' are in 'next'. The ones marked with '.' do not appear in any of
the integration branches, but I am still holding onto them.
You can find the changes described
On Tue, 20 Feb 2018 17:54:12 -0800
Stefan Beller wrote:
> Add a raw_object_store to alt_odb_usable to be more specific about which
> repository to act on. The choice of the repository is delegated to its
> only caller link_alt_odb_entry.
>
> Signed-off-by: Stefan Beller
I checked that alt_odb_
On Tue, Feb 20, 2018 at 5:54 PM, Stefan Beller wrote:
> v3:
> * reverted back to use the repository for most of the functions
> (including unduplicating 'ignore_env')
> * rebased on master again (I lost that state when doing v2, as
> I did both rebase as well as conversion to object store in o
On Wed, Feb 21, 2018 at 10:47:19AM -0800, Junio C Hamano wrote:
> "brian m. carlson" writes:
>
> > This is the twelfth in a series of patches to convert from unsigned char
> > [20] to struct object_id. This series is based on next.
> >
> > Included in this series are conversions for find_unique_
Ævar Arnfjörð Bjarmason writes:
> Here's a v5 (correct subject line this time!). Many thanks to Eric for
> a thorough review.
We haven't seen any comments on this round. Is everybody happy?
I do not have a strong opinion on the new feature, either for or
against. I didn't find anything major
Nguyễn Thái Ngọc Duy writes:
> Compared to v2:
>
> - the potential loss of errno before it's printed out in builtin/am.c
> is fixed.
> - keep update_ref() in sequencer.c non-fatal like this rest
> - rename ORIG_COMMIT to REBASE_HEAD
>
> Interdiff:
This round hasn't seen any comments. Is ever
On Tue, 20 Feb 2018 17:54:10 -0800
Stefan Beller wrote:
> Each repository's object store can be initialized independently, so
> they must not share a run_once variable.
>
> Signed-off-by: Stefan Beller
> Signed-off-by: Jonathan Nieder
Reviewed-by: Jonathan Tan
> -#define RAW_OBJECT_STORE_IN
On Tue, 6 Feb 2018 17:13:12 -0800
Brandon Williams wrote:
> +test_expect_success 'push with http:// and a config of v2 does not request
> v2' '
> + # Till v2 for push is designed, make sure that if a client has
> + # protocol.version configured to use v2, that the client instead falls
>
On Tue, 6 Feb 2018 17:13:09 -0800
Brandon Williams wrote:
> @@ -172,6 +172,8 @@ struct http_get_options {
>* for details.
>*/
> struct strbuf *base_url;
> +
> + struct string_list *extra_headers;
Document this? For example:
If not NULL, additional HTTP headers to be
On Tue, 6 Feb 2018 17:13:07 -0800
Brandon Williams wrote:
> Make a copy of the service name being requested instead of relying on
> the buffer pointed to by the passed in 'const char *' to remain
> unchanged.
>
> Signed-off-by: Brandon Williams
Probably worth mentioning in the commit message:
On Tue, 6 Feb 2018 17:13:05 -0800
Brandon Williams wrote:
> Introduce the transport-helper capability 'stateless-connect'. This
> capability indicates that the transport-helper can be requested to run
> the 'stateless-connect' command which should attempt to make a
> stateless connection with a
On February 21, 2018 6:13 PM, Peter Backes wrote:
> On Wed, Feb 21, 2018 at 11:44:13PM +0100, Ævar Arnfjörð Bjarmason wrote:
> > If it were added as a first-level feature to git it would present a
> > lot of UX confusion. E.g. you run "git add" and it'll be showing the
> > mtime somehow, or you get
When using git log, boundary commits (ie, those commits added by
specifying --boundary) do not respect the order (e.g., --date-order,
--topo-order). Consider the following commit history, where number
indicates the order of the commit timestamps:
0125 <--A
\ \
TYPO IN EXPECTED OUTPUT.
To avoid inevitable confusion, creating new thread "Bug: git log:
boundary commits do not respect order (e.g. date-order, topo-order)
2".
DON'T REPLY TO THIS MESSAGE. Instead reply to the new message
On Wed, Feb 21, 2018 at 6:28 PM, Josh Tepper wrote:
> When using git
On Tue, 6 Feb 2018 17:13:01 -0800
Brandon Williams wrote:
> Instead of having each builtin transport asking for which protocol
> version the user has configured in 'protocol.version' by calling
> `get_protocol_version_config()` multiple times, factor this logic out
> so there is just a single ca
> On Feb 21, 2018, at 2:15 PM, Jeff King wrote:
>
> Thanks, I agree the document is buggy. Do you want to submit a patch?
Will this do?
Note I am not sure what the story is behind that `version 1` element, whether
it's supposed to go before or after the null packet or if there should be
anot
On Tue, 6 Feb 2018 17:12:57 -0800
Brandon Williams wrote:
> +want
> + Indicates to the server an object which the client wants to
> + retrieve.
Mention that the client can "want" anything even if not advertised by
the server (like uploadpack.allowanysha1inwant).
> +output = *s
On Wed, Feb 21, 2018 at 3:27 PM, Jeff King wrote:
>
> We'll skip the usual deprecation period because this was
> explicitly a debugging aid that was never documented.
Ack. I don't think I've used it since, and probably nobody else ever used it.
Linus
Jonathan Nieder writes:
> For defense in depth, it would be comforting if the git wrapper had
> some understanding of "don't support --help in handle_builtin when
> invoked as a dashed command". That is, I don't expect that anyone has
> been relying on
>
> git-add --help
>
> acting like
>
On Wed, Feb 21, 2018 at 02:19:17PM -0500, Derrick Stolee wrote:
> > These behaviors are undocumented, untested, and unlikely to be
> > expected by users or other software attempting to parse this output.
> >
> > Helped-by: Jeff King
> > Signed-off-by: Derrick Stolee
>
> This would be a good ti
On Wed, Feb 21, 2018 at 03:22:02PM -0800, Stefan Beller wrote:
> > Subject: [PATCH] commit: drop uses of get_cached_commit_buffer()
> > ---
> > builtin/rev-list.c | 2 +-
> > log-tree.c | 3 ---
> > 2 files changed, 1 insertion(+), 4 deletions(-)
>
> Now if we'd get around to rewrite pre
When using git log, boundary commits (ie, those commits added by
specifying --boundary) do not respect the order (e.g., --date-order,
--topo-order). Consider the following commit history, where number
indicates the order of the commit timestamps:
0125 <--A
\ \
On Wed, Feb 21, 2018 at 03:03:18PM -0800, Junio C Hamano wrote:
> > So what I'm wondering is whether we should consider just ripping it out
> > (but I'm OK with keeping it, as once the commit-buffer stuff is fixed,
> > it's probably not hurting anybody).
>
> I see no problem in removing it. With
> Subject: [PATCH] commit: drop uses of get_cached_commit_buffer()
> ---
> builtin/rev-list.c | 2 +-
> log-tree.c | 3 ---
> 2 files changed, 1 insertion(+), 4 deletions(-)
Now if we'd get around to rewrite pretty.c as well, we could make it static,
giving a stronger reason of not using
On Wed, Feb 21, 2018 at 11:44:13PM +0100, Ævar Arnfjörð Bjarmason wrote:
> If it were added as a first-level feature to git it would present a lot
> of UX confusion. E.g. you run "git add" and it'll be showing the mtime
> somehow, or you get a formatted patch over E-Mail and it doesn't only
> inclu
On Wed, Feb 21, 2018 at 02:17:11PM -0500, Derrick Stolee wrote:
> The get_cached_commit_buffer() method provides access to the buffer
> loaded for a struct commit, if it was ever loadead and was not freed.
>
> Two places use this to inform how to output information about commits.
>
> log-tree.c
Hello,
I have a business proposal i would like to discuss with you, reply if you are
interested for more info.
Thank You.
Ms. Melisa Mehmet
My Skype ID: ms.melisameh...@hotmail.com
WhatsApp": +90 534 716 2603
Jeff King writes:
> Out of curiosity, do you actually use --show-all for anything?
Absolutely not. I'd actually love it if I could say "not anymore"
instead, but I haven't had an opportunity to debug the revision
traversal code for quite some time so I do not even remember when
was the last tim
On Tue, 6 Feb 2018 17:12:53 -0800
Brandon Williams wrote:
> -const struct ref *transport_get_remote_refs(struct transport *transport)
> +const struct ref *transport_get_remote_refs(struct transport *transport,
> + const struct argv_array
> *ref_patterns)
On Tue, 6 Feb 2018 17:12:52 -0800
Brandon Williams wrote:
> @@ -21,7 +22,8 @@ struct transport_vtable {
>* the ref without a huge amount of effort, it should store it
>* in the ref's old_sha1 field; otherwise it should be all 0.
>**/
> - struct ref *(*get_refs_list)(s
Hello
Greetings to you and everyone around you please did you get my previous email
regarding my proposal ?
please let me know if we can work together on this.
Best Reagrds
Hello
Greetings to you and everyone around you please did you get my previous email
regarding my proposal ?
please let me know if we can work together on this.
Best Reagrds
On Tue, 6 Feb 2018 17:12:51 -0800
Brandon Williams wrote:
> +extern struct ref **get_remote_refs(int fd_out, struct packet_reader *reader,
> + struct ref **list, int for_push,
> + const struct argv_array *ref_patterns);
I haven't l
On Wed, Feb 21 2018, Basin Ilya jotted:
> Hi.
> I want to the test-repo-git under
> https://github.com/apache/maven-wagon/tree/master/wagon-providers/wagon-scm/src/test/resources/
> just like test-repo-cvs and test-repo-svn
>
> Which configuation options would suit that?
> I think core.compressi
On Wed, Feb 21, 2018 at 02:22:05PM -0800, Junio C Hamano wrote:
> > I think that repeating the oid is intentional; the point is to dump how
> > the traversal code is hitting the endpoints, even if we do so multiple
> > times.
> >
> > The --oneline behavior just looks like a bug. I think --format i
On Tue, 6 Feb 2018 17:12:49 -0800
Brandon Williams wrote:
> .gitignore | 1 +
> Documentation/technical/protocol-v2.txt | 114 +++
> Makefile| 2 +
> builtin.h | 1 +
> builtin/serve.c
On Wed, Feb 21 2018, Peter Backes jotted:
> On Wed, Feb 21, 2018 at 10:33:05PM +0100, Ævar Arnfjörð Bjarmason wrote:
>> This sounds like a sensible job for a git import tool, i.e. import a
>> target directory into git, and instead of 'git add'-ing the whole thing
>> it would look at the mtimes, s
Hello
Greetings to you and everyone around you please did you get my previous email
regarding my proposal ?
please let me know if we can work together on this.
Best Reagrds
Thanks. The entire thing looked reasonable.
Will replace.
Hello
Greetings to you and everyone around you please did you get my previous email
regarding my proposal ?
please let me know if we can work together on this.
Best Reagrds
Hello
Greetings to you and everyone around you please did you get my previous email
regarding my proposal ?
please let me know if we can work together on this.
Best Reagrds
On Mon, Feb 19, 2018 at 10:53 AM, Derrick Stolee wrote:
>
> Teach git-commit-graph to inspect the objects only in a certain list
> of pack-indexes within the given pack directory. This allows updating
> the commit graph iteratively, since we add all commits stored in a
> previous commit graph.
>
>
Jeff King writes:
> I think that repeating the oid is intentional; the point is to dump how
> the traversal code is hitting the endpoints, even if we do so multiple
> times.
>
> The --oneline behavior just looks like a bug. I think --format is broken
> with --show-all, too (it does not show anyth
On Wed, Feb 21, 2018 at 10:29:35AM -0800, Dorian Taylor wrote:
> I didn’t get an insight until I ran with GIT_TRACE_PACKET=true on a
> known-good remote (i.e. GitHub), that the null packet-line `` has to
> follow the service line. This is not reflected in the example here:
>
> https://githu
On Wed, Feb 21, 2018 at 10:33:05PM +0100, Ævar Arnfjörð Bjarmason wrote:
> This sounds like a sensible job for a git import tool, i.e. import a
> target directory into git, and instead of 'git add'-ing the whole thing
> it would look at the mtimes, sort files by mtime, then add them in order
> and
On Tue, 6 Feb 2018 17:12:45 -0800
Brandon Williams wrote:
> - get_remote_heads(fd[0], NULL, 0, &ref, 0, NULL, &shallow);
> +
> + packet_reader_init(&reader, fd[0], NULL, 0,
> +PACKET_READ_CHOMP_NEWLINE |
> +PACKET_READ_GENTLE_ON_EOF);
> +
>
On 2/21/2018 2:17 PM, Derrick Stolee wrote:
The get_cached_commit_buffer() method provides access to the buffer
loaded for a struct commit, if it was ever loadead and was not freed.
Two places use this to inform how to output information about commits.
log-tree.c uses this method to short-circu
Stefan Beller writes:
> +
> +/*
> + * The mru list_head is supposed to be initialized using
> + * the LIST_HEAD macro, assigning prev/next to itself.
> + * However this doesn't work in this case as some compilers dislike
> + * that macro on member variables. Use NULL instead as that is defined
>
Brandon Williams wrote:
> On 01/03, Stefan Beller wrote:
> > On Tue, Jan 2, 2018 at 4:18 PM, Brandon Williams wrote:
>>> In order to allow for code sharing with the server-side of fetch in
>>> protocol-v2 convert upload-pack to be a builtin.
>>
>> What is the security aspect of this patch?
>>
>>
On Tue, 6 Feb 2018 17:12:41 -0800
Brandon Williams wrote:
> 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
As Stefan mentioned in [1], also mention in the commit message that this
means
My Dear friend,
I am Mr. Micheal Pascal Anderson I got your contact on my personal search
of the person.. I willed the only funds left in my account to you.
If you want to know why I have willed the only funds left
in my account to you please contact the bank manager whose
name and address
On Mon, Feb 19, 2018 at 10:53 AM, Derrick Stolee wrote:
> graph_name = write_commit_graph(opts.obj_dir);
>
> if (graph_name) {
> if (opts.set_latest)
> set_latest_file(opts.obj_dir, graph_name);
>
> + if (opts.delete_expired)
>
On Wed, Feb 21 2018, Derek Fawcus jotted:
> On Mon, Feb 19, 2018 at 10:22:36PM +0100, Peter Backes wrote:
>>
>> It is pretty annoying that git cannot, even if I know what I am doing,
>> and explicitly want it to, preserve the modification time.
>
> The use case I've come across where it would be
On 20/02/18 22:48, Peter Backes wrote:
On Tue, Feb 20, 2018 at 11:32:23PM +0100, Johannes Schindelin wrote:
Hi Peter,
On Tue, 20 Feb 2018, Peter Backes wrote:
On Tue, Feb 20, 2018 at 11:46:38AM +0100, Johannes Schindelin wrote:
I would probably invent a file format (``)
I'm stuck there b
On Mon, Feb 19, 2018 at 10:22:36PM +0100, Peter Backes wrote:
>
> It is pretty annoying that git cannot, even if I know what I am doing,
> and explicitly want it to, preserve the modification time.
The use case I've come across where it would be of value is for code
archeology, either importing
On Wed, Feb 21, 2018 at 08:53:16AM -0800, Junio C Hamano wrote:
> Even though keeping track of list of known-leaky tests may not be so
> interesting, we can still salvage useful pieces from the discussion
> and make them available to developers, e.g. something like
>
> prove --dry --state=fa
This is similar to ff1e72483 (tag: change default of `pager.tag` to
"on", 2017-08-02) and is safe now that we do not consider `pager.config`
at all when we are not listing or getting configuration. This change
will help with listing large configurations, but will not hurt users of
`git config --edi
On Wed, Feb 21, 2018 at 09:13:22AM -0500, Derrick Stolee wrote:
> > So there it is. It does show commits multiple times, but suppresses the
> > verbose header after the first showing. If we do something like this:
> >
> >git rev-list --show-all --pretty --boundary c93150cfb0^-
> >
> > you'll
On Wed, Feb 21, 2018 at 01:48:11PM -0500, Jeff King wrote:
> > What confuses me about this behavior is that the OID is still shown on the
> > repeat (and in the case of `git log --oneline` will not actually have a line
> > break between two short-OIDs). I don't believe this behavior is something t
Derrick Stolee writes:
> +static int graph_write(int argc, const char **argv)
> +{
> + ...
> + graph_name = write_commit_graph(opts.obj_dir);
> +
> + if (graph_name) {
> + printf("%s\n", graph_name);
> + FREE_AND_NULL(graph_name);
> + }
> +
> + return 0;
>
> +[3]
> https://public-inbox.org/git/20170907094718.b6kuzp2uhvkmw...@sigill.intra.peff.net/t/#m7a2ea7b355aeda962e6b86404bcbadc648abfbba
> +More discussion about generation numbers and not storing them inside
> +commit objects. A valuable quote:
Unlike the other public inbox links this li
Hello list,
I had been banging my head all morning trying to figure out why I couldn’t get
a little HTTP implementation to clone/push via the smart protocol (just
wrapping git-receive-pack/git-upload-pack). I kept getting the following
(likely familiar to some) error:
```
fatal: Could not read
>>
>> [ so in small repos, where there are fewer than 256 objects,
>> F[i] == F[i+1], for all i'th where there is no object starting with i
>> byte]
>
>
> Correct. I'm not sure this additional information is valuable for the
> document, though.
It is not, I was just making sure I'd understand corr
Brandon Williams writes:
> Update the documentation for the 'submodule.recurse' config to identify
> that the clone command does not respect it.
>
> Signed-off-by: Brandon Williams
> ---
> Documentation/config.txt | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
Thanks.
> diff --git
Derrick Stolee writes:
> +'read'::
> +
> +Read a graph file given by the graph-head file and output basic
> +details about the graph file.
> ++
> +With `--file=` option, consider the graph stored in the file at
> +the path /info/.
> +
A sample reader confusion after reading the above twice:
The next couple of commits will change how `git config` handles
`pager.config`, similar to how de121ffe5 (tag: respect `pager.tag` in
list-mode only, 2017-08-02) and ff1e72483 (tag: change default of
`pager.tag` to "on", 2017-08-02) changed `git tag`. Similar work has
also been done to `git branch`
On 2/21/2018 2:23 PM, Stefan Beller wrote:
On Mon, Feb 19, 2018 at 10:53 AM, Derrick Stolee wrote:
+In order to allow extensions that add extra data to the graph, we organize
+the body into "chunks" and provide a binary lookup table at the beginning
+of the body. The header includes certain val
This is v2 of my series to teach `git config` to only respect
`pager.config` when listing configuration, then changing the default to
"on". Thanks to Duy and Junio for feedback on the first version.
Based on Duy's feeback, I've changed the approach to more carefully
divide the various getters into
On 02/06, Brandon Williams wrote:
> Changes in v3:
> * There were some comments about how the protocol should be designed
>stateless first. I've made this change and instead of having to
>supply the `stateless-rpc=true` capability to force stateless
>behavior, the protocol just requir
On Mon, Feb 19, 2018 at 10:53 AM, Derrick Stolee wrote:
> Add document specifying the binary format for commit graphs. This
> format allows for:
>
> * New versions.
> * New hash functions and hash lengths.
> * Optional extensions.
>
> Basic header information is followed by a binary table of conte
The get_cached_commit_buffer() method provides access to the buffer
loaded for a struct commit, if it was ever loadead and was not freed.
Two places use this to inform how to output information about commits.
log-tree.c uses this method to short-circuit the output of commit
information when the b
Junio C Hamano writes:
> Derrick Stolee writes:
>
>> +int cmd_commit_graph(int argc, const char **argv, const char *prefix)
>> +{
>> +static struct option builtin_commit_graph_options[] = {
>> +{ OPTION_STRING, 'p', "object-dir", &opts.obj_dir,
>> +N_("dir"),
1 - 100 of 119 matches
Mail list logo