Re: What's cooking in git.git (Jan 2019, #01; Mon, 7)

2019-01-08 Thread Martin Ågren
On Tue, 8 Jan 2019 at 00:34, Junio C Hamano wrote: > * bc/sha-256 (2018-11-14) 12 commits > - hash: add an SHA-256 implementation using OpenSSL > - sha256: add an SHA-256 implementation using libgcrypt > - Add a base implementation of SHA-256 support > - commit-graph: convert to using the_hash

Re: [PATCH 3/3] t0006-date.sh: add `human` date format tests.

2019-01-08 Thread Johannes Sixt
Am 09.01.19 um 01:44 schrieb Stephen P. Smith: On Tuesday, January 8, 2019 2:27:22 PM MST Johannes Sixt wrote: Am 31.12.18 um 01:31 schrieb Stephen P. Smith: + +TODAY_REGEX='[A-Z][a-z][a-z] [012][0-9]:[0-6][0-9] .0200' The $...REGEX expansions must be put in double-quotes to protect them fro

Re: Regression: submodule worktrees can clobber core.worktree config

2019-01-08 Thread Tomasz Śniatowski
On Wed, 9 Jan 2019 at 00:23, Duy Nguyen wrote: > > On Wed, Jan 9, 2019 at 5:56 AM Tomasz Śniatowski wrote: > > > > After upgrading to 2.20.1 I noticed in some submodule+worktree scenarios git > > will break the submodule configuration. Reproducible with: > > git init a && (cd a; touch a; git

Re: git-lfs integration?

2019-01-08 Thread Harald Dunkel
Hi Ævar, thanx very much for your detailed response. Exactly what I was looking for. Regards Harri

HELLO'''''''''''

2019-01-08 Thread Mr Mike Murdock
Hello, Sorry for the delay, it was due to fire disaster in our Zonal Office but everything is okay now,I want to know if you still ready to work with us part time or Full time ? Kindly get back to me Via email asap for details. Email: humanresourcemanagermikemud...@gmail.com Mike Murdock

[PATCH v3 2/2] tree:: skip some trees even when collecting omits

2019-01-08 Thread Matthew DeVore
If a tree has already been recorded as omitted, we don't need to traverse it again just to collect its omits. Stop traversing trees a second time when collecting omits. Signed-off-by: Matthew DeVore --- list-objects-filter.c | 18 -- t/t6112-rev-list-filters-objects

[PATCH v3 1/2] list-objects-filter: teach tree:# how to handle >0

2019-01-08 Thread Matthew DeVore
Implement positive values for in the tree: filter. The exact semantics are described in Documentation/rev-list-options.txt. The long-term goal at the end of this is to allow a partial clone to eagerly fetch an entire directory of files by fetching a tree and specifying =1. This, for instance, wou

[PATCH v3 0/2] support for filtering trees and blobs based on depth

2019-01-08 Thread Matthew DeVore
This applies suggestions from Jonathan Tan and Junio. These are mostly stylistic and readability changes, although there is also an added test case in t/t6112-rev-list-filters-objects.sh which checks for the scenario when filtering which would exclude a blob, but the blob is given on the command li

Re: [PATCH v2 2/2] tree:: skip some trees even when collecting omits

2019-01-08 Thread MATTHEW DEVORE
> On January 8, 2019 at 3:22 PM Jonathan Tan wrote: > > > > > -static void filter_trees_update_omits( > > > +static int filter_trees_update_omits( > > > struct object *obj, > > > struct filter_trees_depth_data *filter_data, > > > int include_it) > > > { > > > if (!filter_data->omits)

Re: [PATCH v2 1/2] list-objects-filter: teach tree:# how to handle >0

2019-01-08 Thread MATTHEW DEVORE
> On January 8, 2019 at 3:39 PM Junio C Hamano wrote: > > Also, the asterisk sticks to the variable, not type, i.e. > > struct filter_trees_depth_data *d = filter_data; > Fixed. Thanks.

Re: [PATCH 3/3] t0006-date.sh: add `human` date format tests.

2019-01-08 Thread Stephen P. Smith
On Tuesday, January 8, 2019 2:27:22 PM MST Johannes Sixt wrote: > Am 31.12.18 um 01:31 schrieb Stephen P. Smith: > > + > > +TODAY_REGEX='[A-Z][a-z][a-z] [012][0-9]:[0-6][0-9] .0200' > The $...REGEX expansions must be put in double-quotes to protect them > from field splitting. But then the tests d

Re: [PATCH v2 2/2] tree:: skip some trees even when collecting omits

2019-01-08 Thread Matthew DeVore
Thank you for the review :) See below. On 2019/01/07 18:00, Jonathan Tan wrote: -static void filter_trees_update_omits( +static int filter_trees_update_omits( struct object *obj, struct filter_trees_depth_data *filter_data, int include_it) { if (!filter_data->om

Re: [PATCH v4 0/1] pack-redundant: remove unused functions

2019-01-08 Thread 16657101987
> 16657101...@163.com writes: > >> From: Sun Chao >> >> I'm particularly grateful to Junio and JiangXin for fixing the patches, >> and I noticed Junio send a new commit to remove more unused codes and >> suggest to SQUASH it. >> >> So I create this new version of patches to do this work, I also h

Re: [PATCH v2 1/2] list-objects-filter: teach tree:# how to handle >0

2019-01-08 Thread Jonathan Tan
> Jonathan Tan writes: > > >> For your reference, here is an interdiff for this particular patch after > >> applying your comments: > > > > The interdiff looks good, thanks. All my issues are resolved. > > Just to make sure. That's not "v2 is good", but "v2 plus that > proposed update, when ma

Re: [PATCH v2 1/2] list-objects-filter: teach tree:# how to handle >0

2019-01-08 Thread Junio C Hamano
Jonathan Tan writes: >> +static void filter_trees_free(void *filter_data) { >> +struct filter_trees_depth_data* d = filter_data; >> +oidmap_free(&d->seen_at_depth, 1); >> +free(d); >> +} > > Check for NULL-ness of filter_data too, to match the usual behavior of > free functions. Also

Re: [PATCH v2 1/2] list-objects-filter: teach tree:# how to handle >0

2019-01-08 Thread Junio C Hamano
Jonathan Tan writes: >> For your reference, here is an interdiff for this particular patch after >> applying your comments: > > The interdiff looks good, thanks. All my issues are resolved. Just to make sure. That's not "v2 is good", but "v2 plus that proposed update, when materializes, would

Re: [PATCH v2 1/2] Use packet_reader instead of packet_read_line

2019-01-08 Thread Masaya Suzuki
On Mon, Jan 7, 2019 at 2:33 PM Josh Steadmon wrote: > > On 2018.12.29 13:19, Masaya Suzuki wrote: > > By using and sharing a packet_reader while handling a Git pack protocol > > request, the same reader option is used throughout the code. This makes > > it easy to set a reader option to the reques

Re: [PATCH v2 2/2] tree:: skip some trees even when collecting omits

2019-01-08 Thread Jonathan Tan
> > -static void filter_trees_update_omits( > > +static int filter_trees_update_omits( > > struct object *obj, > > struct filter_trees_depth_data *filter_data, > > int include_it) > > { > > if (!filter_data->omits) > > - return; > > + return 1; > > > > if

Re: Regression: submodule worktrees can clobber core.worktree config

2019-01-08 Thread Duy Nguyen
On Wed, Jan 9, 2019 at 5:56 AM Tomasz Śniatowski wrote: > > After upgrading to 2.20.1 I noticed in some submodule+worktree scenarios git > will break the submodule configuration. Reproducible with: > git init a && (cd a; touch a; git add a; git commit -ma) > git init b && (cd b; git submod

Re: [PATCH v2 1/2] list-objects-filter: teach tree:# how to handle >0

2019-01-08 Thread Jonathan Tan
> > Any reason for moving "case LOFS_BLOB" (and "case LOFS_BEGIN_TREE" > > below) after LOFS_END_TREE? > > I put LOFS_BLOB and after LOFS_END_TREE since that is the order in all > the other filter logic functions. I put LOFS_BEGIN_TREE at the end > (which is different from the other filter logic

Re: [PATCH] log: add %S option (like --source) to log --format

2019-01-08 Thread Junio C Hamano
issac.tro...@gmail.com writes: > From: Issac Trotts Heh, I'll edit this line to match S-o-b: below. > > Make it possible to write for example > > git log --format="%H,%S" > > where the %S at the end is a new placeholder that prints out the ref > (tag/branch) for each commit. > > Using %

Regression: submodule worktrees can clobber core.worktree config

2019-01-08 Thread Tomasz Śniatowski
After upgrading to 2.20.1 I noticed in some submodule+worktree scenarios git will break the submodule configuration. Reproducible with: git init a && (cd a; touch a; git add a; git commit -ma) git init b && (cd b; git submodule add ../a; git commit -mb) git -C b worktree add ../b2 g

[PATCH v3 6/8] checkout: factor out mark_cache_entry_for_checkout function

2019-01-08 Thread Thomas Gummerer
Factor out the code that marks a cache entry as matched for checkout into a separate function. We are going to introduce a new mode in 'git checkout' in a subsequent commit, that is going to have a slightly different logic. This would make this code unnecessarily complex. Moving that complexity

[PATCH v3 8/8] checkout: introduce checkout.overlayMode config

2019-01-08 Thread Thomas Gummerer
In the previous patch we introduced a new no-overlay mode for git checkout. Some users (such as the author of this commit) may want to have this mode turned on by default as it matches their mental model more closely. Make that possible by introducing a new config option to that extend. Signed-o

[PATCH v3 4/8] read-cache: add invalidate parameter to remove_marked_cache_entries

2019-01-08 Thread Thomas Gummerer
When marking cache entries for removal, and later removing them all at once using 'remove_marked_cache_entries()', cache entries currently have to be invalidated manually in the cache tree and in the untracked cache. Add an invalidate flag to the function. With the flag set, the function will tak

[PATCH v3 2/8] entry: factor out unlink_entry function

2019-01-08 Thread Thomas Gummerer
Factor out the 'unlink_entry()' function from unpack-trees.c to entry.c. It will be used in other places as well in subsequent steps. As it's no longer a static function, also move the documentation to the header file to make it more discoverable. Signed-off-by: Thomas Gummerer --- cache.h

[PATCH v3 5/8] checkout: clarify comment

2019-01-08 Thread Thomas Gummerer
The key point for the if statement is that read_tree_some did not update the entry, because either it doesn't exist in tree-ish or doesn't match the pathspec. Clarify that. Suggested-by: Nguyễn Thái Ngọc Duy Signed-off-by: Thomas Gummerer --- builtin/checkout.c | 8 1 file changed, 4

[PATCH v3 3/8] entry: support CE_WT_REMOVE flag in checkout_entry

2019-01-08 Thread Thomas Gummerer
'checkout_entry()' currently only supports creating new entries in the working tree, but not deleting them. Add the ability to remove entries at the same time if the entry is marked with the CE_WT_REMOVE flag. Currently this doesn't have any effect, as the CE_WT_REMOVE flag is only used in unpack

[PATCH v3 7/8] checkout: introduce --{,no-}overlay option

2019-01-08 Thread Thomas Gummerer
Currently 'git checkout' is defined as an overlay operation, which means that if in 'git checkout -- []' we have an entry in the index that matches , but that doesn't exist in , that entry will not be removed from the index or the working tree. Introduce a new --{,no-}overlay option, which allows

[PATCH v3 1/8] move worktree tests to t24*

2019-01-08 Thread Thomas Gummerer
The 'git worktree' command used to be just another mode in 'git checkout', namely 'git checkout --to'. When the tests for the latter were retrofitted for the former, the test name was adjusted, but the test number was kept, even though the test is testing a different command now. t/README states:

[PATCH v3 0/8] introduce no-overlay mode in git checkout

2019-01-08 Thread Thomas Gummerer
Previous rounds are at <20181209200449.16342-1-t.gumme...@gmail.com> and <20181220134820.21810-1-t.gumme...@gmail.com>. Thanks Duy, Eric and Junio for comments on the previous round. This round fixes some inconsistencies and improves the grammar in the docs. Range-diff below: 1: fa450cda7c = 1

Re: sb/more-repo-in-api, was Re: What's cooking in git.git (Jan 2019, #01; Mon, 7)

2019-01-08 Thread Junio C Hamano
Jonathan Tan writes: >> The in-core repository instances are passed through more codepaths. > > I think this is ready to be considered for merging to next. This series looks > good both to Stolee [1] and to me (I replied to a previous version with > comments on patch 18 [2] which Stefan has addr

Re: [PATCH 3/3] t0006-date.sh: add `human` date format tests.

2019-01-08 Thread Johannes Sixt
Am 31.12.18 um 01:31 schrieb Stephen P. Smith: +check_human_date () { + time=$1 + expect=$2 + test_expect_success "check date ($format:$time)" ' + echo "$time -> $expect" >expect && + TZ=${zone:-$TZ} test-tool date show:"$format" "$time" >actual && +

Re: git rebase: retain original head?

2019-01-08 Thread Jacob Keller
On Tue, Jan 8, 2019 at 12:47 PM Markus Wiederkehr wrote: > > On Tue, Jan 8, 2019 at 6:43 PM Andreas Schwab wrote: > > > > On Jan 08 2019, Markus Wiederkehr wrote: > > > > > Would it be possible to retain this information? > > > > You could use the reflog of the current branch, where it is the se

sb/more-repo-in-api, was Re: What's cooking in git.git (Jan 2019, #01; Mon, 7)

2019-01-08 Thread Jonathan Tan
> * sb/more-repo-in-api (2018-12-28) 23 commits > - t/helper/test-repository: celebrate independence from the_repository > - path.h: make REPO_GIT_PATH_FUNC repository agnostic > - commit: prepare free_commit_buffer and release_commit_memory for any repo > - commit-graph: convert remaining func

Re: [PATCH 0/11] jk/loose-object-cache sha1/object_id fixups

2019-01-08 Thread Jeff King
On Tue, Jan 08, 2019 at 10:52:19AM -0800, Junio C Hamano wrote: > Junio C Hamano writes: > > > Jeff King writes: > > > >> Yeah, they should. I think one of them will need René's patch, which > >> changes the body of quick_has_loose(). I can roll it as a separate topic > >> if that's easier (or

Re: [PATCH v3 4/4] fetch-pack: support protocol version 2

2019-01-08 Thread Jeff King
On Tue, Jan 08, 2019 at 12:38:26PM -0800, Jonathan Tan wrote: > > Ævar Arnfjörð Bjarmason writes: > > > > > From: Jonathan Tan > > > > > > I was looking at the topics in 'pu' and noticed that I had v2 of > > this series, wanted to update to v3, but major part of it was > > superseded by anot

Re: git rebase: retain original head?

2019-01-08 Thread Andreas Schwab
On Jan 08 2019, Markus Wiederkehr wrote: > On Tue, Jan 8, 2019 at 6:43 PM Andreas Schwab wrote: >> >> On Jan 08 2019, Markus Wiederkehr wrote: >> >> > Would it be possible to retain this information? >> >> You could use the reflog of the current branch, where it is the second >> entry. > > It i

Re: git rebase: retain original head?

2019-01-08 Thread Markus Wiederkehr
On Tue, Jan 8, 2019 at 6:43 PM Andreas Schwab wrote: > > On Jan 08 2019, Markus Wiederkehr wrote: > > > Would it be possible to retain this information? > > You could use the reflog of the current branch, where it is the second > entry. It is not, depending on what happens in the rebase it could

Re: [PATCH v3 4/4] fetch-pack: support protocol version 2

2019-01-08 Thread Jonathan Tan
> Ævar Arnfjörð Bjarmason writes: > > > From: Jonathan Tan > > > I was looking at the topics in 'pu' and noticed that I had v2 of > this series, wanted to update to v3, but major part of it was > superseded by another topic (jk/proto-v2-hidden-refs-fix). That > leaves only this patch in the

Re: [PATCH v4 0/1] pack-redundant: remove unused functions

2019-01-08 Thread Junio C Hamano
16657101...@163.com writes: > From: Sun Chao > > I'm particularly grateful to Junio and JiangXin for fixing the patches, > and I noticed Junio send a new commit to remove more unused codes and > suggest to SQUASH it. > > So I create this new version of patches to do this work, I also have > check

Re: [PATCH v3 4/4] fetch-pack: support protocol version 2

2019-01-08 Thread Junio C Hamano
Ævar Arnfjörð Bjarmason writes: > From: Jonathan Tan I was looking at the topics in 'pu' and noticed that I had v2 of this series, wanted to update to v3, but major part of it was superseded by another topic (jk/proto-v2-hidden-refs-fix). That leaves only this patch in the v3 of this series.

RE: error: Use of uninitialized value $hash in chomp

2019-01-08 Thread Andrew Shearer
Hi Thanks for reply, but sorry I don't know how to do that - I don't have the git source code or know how to debug it. Is there another way I can capture logging/debugging information while running "git svn clone" and send it to you? Thanks Andrew Shearer / Web Developer DDI 021 469 888 / and

Re: [PATCH v2 1/2] list-objects-filter: teach tree:# how to handle >0

2019-01-08 Thread Matthew DeVore
On 2019/01/07 17:56, Jonathan Tan wrote: case LOFS_END_TREE: assert(obj->type == OBJ_TREE); + filter_data->current_depth--; return LOFR_ZERO; + case LOFS_BLOB: + filter_trees_update_omits(obj, filter_data, include_it); +

Re: [PATCH 0/11] jk/loose-object-cache sha1/object_id fixups

2019-01-08 Thread Junio C Hamano
Junio C Hamano writes: > Jeff King writes: > >> Yeah, they should. I think one of them will need René's patch, which >> changes the body of quick_has_loose(). I can roll it as a separate topic >> if that's easier (or just wait a week or so until René's cleanups >> graduate). > > Nah, what I got

Re: [PATCH 1/1] git-gc.txt: fix typo about gc.writeCommitGraph

2019-01-08 Thread Junio C Hamano
"Derrick Stolee via GitGitGadget" writes: > From: Derrick Stolee > > Reported-by: Stefan Haller > Signed-off-by: Derrick Stolee > --- Thanks. > Documentation/git-gc.txt | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/Documentation/git-gc.txt b/Documentation/git-gc.t

Re: Feature request: --preserve-merges to add "exec git merge ..." instead of "pick ..."

2019-01-08 Thread Junio C Hamano
Andreas Hennings writes: > I tried this option after upgrading my git. > Unfortunately, no matter which variation I use, it still attempts to > rebase most or all of the feature branches before merging them. > Possibly depending on their ancestry. Yes, I know that. But what I am hoping is that

Re: [PATCH v2 0/9] diff --color-moved-ws fixes and enhancment

2019-01-08 Thread Junio C Hamano
Phillip Wood writes: > I just wanted to check that these patches are on your radar as they > haven't made it into pu yet. Sorry, but they were not on my radar. I was waiting for comments to come in on them before doing anything, and now it is more than a month ago X-<.

Re: [PATCH 0/11] jk/loose-object-cache sha1/object_id fixups

2019-01-08 Thread Derrick Stolee
On 1/8/2019 1:07 PM, Junio C Hamano wrote: Jeff King writes: Yeah, they should. I think one of them will need René's patch, which changes the body of quick_has_loose(). I can roll it as a separate topic if that's easier (or just wait a week or so until René's cleanups graduate). Nah, what I g

Re: [PATCH] blame: add the ability to ignore commits

2019-01-08 Thread Junio C Hamano
Barret Rhoden writes: >> A policy decision like the above two shouldn't be hardcoded in the >> feature like this, but should be done as a separate option. By >> default, these shouldn't be marked with '*', as the same tools you >> said you are afraid of breaking would be expecting a word with on

Re: Feature request: --preserve-merges to add "exec git merge ..." instead of "pick ..."

2019-01-08 Thread Andreas Hennings
I tried this option after upgrading my git. Unfortunately, no matter which variation I use, it still attempts to rebase most or all of the feature branches before merging them. Possibly depending on their ancestry. On Mon, 7 Jan 2019 at 22:12, Andreas Hennings wrote: > > It sounds good! > I was

Re: [PATCH 0/11] jk/loose-object-cache sha1/object_id fixups

2019-01-08 Thread Junio C Hamano
Jeff King writes: > Yeah, they should. I think one of them will need René's patch, which > changes the body of quick_has_loose(). I can roll it as a separate topic > if that's easier (or just wait a week or so until René's cleanups > graduate). Nah, what I got is already good to work with. Both

Re: [PATCH 0/11] jk/loose-object-cache sha1/object_id fixups

2019-01-08 Thread Jeff King
On Tue, Jan 08, 2019 at 09:39:48AM -0800, Junio C Hamano wrote: > > I skimmed them; they look good to me. 6 and 8 are particularly > > satisfying; getting rid of hash copy operations just feels nice. :) > > > > Junio only took 1 to 5 into pu; 6, 7 and its sidekick 8, 10 and 11 > > conflict with s

Re: [PATCH] blame: add the ability to ignore commits

2019-01-08 Thread Barret Rhoden
On 2019-01-08 at 11:41 Barret Rhoden wrote: > Would you be OK with > also changing fsck to take a committish instead of a full SHA-1? Actually, in retrospect, I can keep the unabbreviated SHA-1 for the file inputs and use get_oid_committish() for the one-off --skip-rev= cases. Thanks, Barret

Re: tg/checkout-no-overlay, was Re: What's cooking in git.git (Jan 2019, #01; Mon, 7)

2019-01-08 Thread Junio C Hamano
Thomas Gummerer writes: > On Mon, Jan 7, 2019 at 11:34 PM Junio C Hamano wrote: >> * tg/checkout-no-overlay (2019-01-02) 8 commits >> - checkout: introduce checkout.overlayMode config >> - checkout: introduce --{,no-}overlay option >> - checkout: factor out mark_cache_entry_for_checkout funct

Git Test Coverage Report (Tues, Jan 8)

2019-01-08 Thread Derrick Stolee
Here is today's test coverage report. Thanks, -Stolee [1] https://git.visualstudio.com/git/_build/results?buildId=290 --- pu: a5fd499f842ac17440e29bd610058ccdd3cf24a1 jch: 2ccddf26bf6308d5ea65fa26ffd7372d083bfc16 next: d81d796ee0c49eae40e3f85467a8e6b18022bcea master: ecbdaf0899161c067986e9d9d5

Re: git rebase: retain original head?

2019-01-08 Thread Andreas Schwab
On Jan 08 2019, Markus Wiederkehr wrote: > During the rebase operation the original head seems to get stored in > 'rebase-merge/orig-head'. Unfortunately this references gets removed > after the rebase operation completes. > > Would it be possible to retain this information? You could use the re

Re: [PATCH 0/11] jk/loose-object-cache sha1/object_id fixups

2019-01-08 Thread Junio C Hamano
René Scharfe writes: > Am 07.01.2019 um 09:31 schrieb Jeff King: >> I also cleaned up my sha1/object_id patch and rebased it on top of what >> you have here. Though as I worked on it, it expanded in scope a bit. >> Possibly it should be a separate series entirely, but that would create >> some an

ag/sequencer-reduce-rewriting-todo Re: What's cooking in git.git (Jan 2019, #01; Mon, 7)

2019-01-08 Thread Alban Gruin
Hi Junio, Le 08/01/2019 à 00:34, Junio C Hamano a écrit : > * ag/sequencer-reduce-rewriting-todo (2018-11-12) 16 commits > . rebase--interactive: move transform_todo_file() to rebase--interactive.c > . sequencer: fix a call to error() in transform_todo_file() > . sequencer: use edit_todo_list()

[PATCH v4 0/1] pack-redundant: remove unused functions

2019-01-08 Thread 16657101987
From: Sun Chao I'm particularly grateful to Junio and JiangXin for fixing the patches, and I noticed Junio send a new commit to remove more unused codes and suggest to SQUASH it. So I create this new version of patches to do this work, I also have checked the left codes and remove a unused struc

[PATCH v4 0/1] pack-redundant: remove unused functions

2019-01-08 Thread 16657101987
From: Sun Chao I'm particularly grateful to Junio and JiangXin for fixing the patches, and I noticed Junio send a new commit to remove more unused codes and suggest to SQUASH it. So I create this new version of patches to do this work, I also have checked the left codes and remove a unused struc

[PATCH 0/1] git-gc.txt: fix typo about gc.writeCommitGraph

2019-01-08 Thread Derrick Stolee via GitGitGadget
Thanks to Stefan Haller for sending me a private message about this typo. Derrick Stolee (1): git-gc.txt: fix typo about gc.writeCommitGraph Documentation/git-gc.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) base-commit: c7e8ce6d1dd02f6569ea785eebc8692e8e2edf72 Published-As: htt

[PATCH 1/1] git-gc.txt: fix typo about gc.writeCommitGraph

2019-01-08 Thread Derrick Stolee via GitGitGadget
From: Derrick Stolee Reported-by: Stefan Haller Signed-off-by: Derrick Stolee --- Documentation/git-gc.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Documentation/git-gc.txt b/Documentation/git-gc.txt index c20ee6c789..a7442499f6 100644 --- a/Documentation/git-gc.txt

[PATCH v4 1/1] pack-redundant: remove unused functions

2019-01-08 Thread 16657101987
From: Sun Chao Remove unused functions to find `min` packs, such as `get_permutations`, `pll_free`, etc. Signed-off-by: Sun Chao Signed-off-by: Jiang Xin Signed-off-by: Junio C Hamano --- builtin/pack-redundant.c | 86 1 file changed, 86 delet

Re: [PATCH] blame: add the ability to ignore commits

2019-01-08 Thread Barret Rhoden
On 2019-01-08 at 14:12 Ævar Arnfjörð Bjarmason wrote: > On Mon, Jan 07 2019, Barret Rhoden wrote: > > > +static int handle_ignore_file(const char *path, struct string_list > > *ignores) > > +{ > > + FILE *fp = fopen(path, "r"); > > + struct strbuf sb = STRBUF_INIT; > > + > > + if (!fp) > >

Re: [PATCH 0/11] jk/loose-object-cache sha1/object_id fixups

2019-01-08 Thread René Scharfe
Am 07.01.2019 um 09:31 schrieb Jeff King: > I also cleaned up my sha1/object_id patch and rebased it on top of what > you have here. Though as I worked on it, it expanded in scope a bit. > Possibly it should be a separate series entirely, but that would create > some annoying textual conflicts on m

Re: [PATCH] blame: add the ability to ignore commits

2019-01-08 Thread Barret Rhoden
On 2019-01-07 at 15:13 Junio C Hamano wrote: > If I read it correctly, this gives a very limited form of -S, in the > sense that anything this can do can be expressed by using -S but the > reverse is not true, but is designed to be easier to use, in the > sense that unlike -S, this does not have t

Re: [PATCH v2 0/9] diff --color-moved-ws fixes and enhancment

2019-01-08 Thread Phillip Wood
Hi Junio I just wanted to check that these patches are on your radar as they haven't made it into pu yet. Best Wishes for the New Year Phillip On 23/11/2018 11:16, Phillip Wood wrote: From: Phillip Wood Thanks to Stefan for his feedback on v1. I've updated patches 2 & 8 in response to tho

Re: git-lfs integration?

2019-01-08 Thread Ævar Arnfjörð Bjarmason
On Tue, Jan 08 2019, Harald Dunkel wrote: > I wonder why git-lfs is needed to efficiently handle large files > in git. Would it be reasonable to integrate this functionality > into the native git? > > Please excuse me asking. I read some pretty scary articles about > rewriting history, asking ev

git-lfs integration?

2019-01-08 Thread Harald Dunkel
Hi folks, I wonder why git-lfs is needed to efficiently handle large files in git. Would it be reasonable to integrate this functionality into the native git? Please excuse me asking. I read some pretty scary articles about rewriting history, asking everybody to clone existing repositories again

git rebase: retain original head?

2019-01-08 Thread Markus Wiederkehr
Hello, I frequently run an interactive rebase to change the order of recent commits, apply fixups, etc. When merge conflicts occur I often want to compare the result with the original head prior to starting the rebase. In the past I used "git diff ORIG_HEAD". This used to work as long as I did no

Re: Recovering from a "detached from" HEAD

2019-01-08 Thread Junio C Hamano
Alyssa Ross writes: >> The commit the message shows is meant to indicate where your unnamed >> branch diverged at named branches. Immediately after moving to the >> unnamed branch by detaching the HEAD, the message says "at"; the >> HEAD is pointing directly at the tip of the then-current branch

Re: Recovering from a "detached from" HEAD

2019-01-08 Thread Alyssa Ross
> The commit the message shows is meant to indicate where your unnamed > branch diverged at named branches. Immediately after moving to the > unnamed branch by detaching the HEAD, the message says "at"; the > HEAD is pointing directly at the tip of the then-current branch and > that is where the t

Re: Translation of git manpages

2019-01-08 Thread Ævar Arnfjörð Bjarmason
On Mon, Jan 07 2019, Jean-Noël AVILA wrote: > Dear fellow translators, > > I'm trying to put up a longstanding project of providing translated manual > pages for Git. After several experiments, the best choice seemed to be the use > of po4a[1] to convert the asciidoc[2] sources of git manpages i

[PATCH] log: add %S option (like --source) to log --format

2019-01-08 Thread issac . trotts
From: Issac Trotts Make it possible to write for example git log --format="%H,%S" where the %S at the end is a new placeholder that prints out the ref (tag/branch) for each commit. Using %d might seem like an alternative but it only shows the ref for the last commit in the branch. Sig

Re: [PATCH] log: add %S option (like --source) to log --format

2019-01-08 Thread Issac Trotts
On Thu, Jan 3, 2019 at 6:33 AM Junio C Hamano wrote: > > issac.tro...@gmail.com writes: > > > From: Issac Trotts > > I think you want to have > > From: Issac Trotts > > instead, so that the authorship actually matches your sign-off. Makes sense. I'll do that. > > > - if (source) {

Re: [PATCH] blame: add the ability to ignore commits

2019-01-08 Thread Ævar Arnfjörð Bjarmason
On Mon, Jan 07 2019, Barret Rhoden wrote: > +static int handle_ignore_file(const char *path, struct string_list *ignores) > +{ > + FILE *fp = fopen(path, "r"); > + struct strbuf sb = STRBUF_INIT; > + > + if (!fp) > + return -1; > + while (!strbuf_getline(&sb, fp)) { >

Re: `git reset` for delete + intent-to-add doesn't reset

2019-01-08 Thread Duy Nguyen
On Tue, Jan 8, 2019 at 4:44 PM Duy Nguyen wrote: > > On Tue, Jan 8, 2019 at 2:28 PM Anthony Sottile wrote: > > > > ``` > > git --version > > rm -rf t > > git init t > > cd t > > touch a > > git add a > > git commit -m "add a" > > git rm a > > touch a > > git add --intent-to-add a > > git status -

tg/checkout-no-overlay, was Re: What's cooking in git.git (Jan 2019, #01; Mon, 7)

2019-01-08 Thread Thomas Gummerer
On Mon, Jan 7, 2019 at 11:34 PM Junio C Hamano wrote: > * tg/checkout-no-overlay (2019-01-02) 8 commits > - checkout: introduce checkout.overlayMode config > - checkout: introduce --{,no-}overlay option > - checkout: factor out mark_cache_entry_for_checkout function > - checkout: clarify comme

Re: `git reset` for delete + intent-to-add doesn't reset

2019-01-08 Thread Duy Nguyen
On Tue, Jan 8, 2019 at 2:28 PM Anthony Sottile wrote: > > ``` > git --version > rm -rf t > git init t > cd t > touch a > git add a > git commit -m "add a" > git rm a > touch a > git add --intent-to-add a > git status --short > git reset -- a "git reset" without "-- a" does remove intent-to-add st