Re: [PATCH v5 3/5] rebase: fast-forward --onto in more cases

2019-04-15 Thread Junio C Hamano
Denton Liu writes: > Before, when we had the following graph, > > A---B---C (master) > \ >D (side) This is minor, but comparing the above with below > F---G topic > / > A---B---C---D---E master you'll notice that branches g

Re: [BUG] GIT_SSH_COMMAND is not being decomposed

2019-04-15 Thread Junio C Hamano
"Randall S. Becker" writes: > As a suggestion, with people who know how to escape stuff properly > (or not), perhaps we can select the alternate behaviour explicitly > using a core.sshIgnoreEscape=true/false option. Thoughts on that? The semantics of prepare_shell_cmd() is, regardless of any "fu

Re: [PATCH 4/7] docs: exclude documentation for commands that have been excluded

2019-04-15 Thread Junio C Hamano
Jeff King writes: > What about command-line options that influence the outcome? It sounds > like this is the same problem we have in lots of other places (like say, > compiler flags being updated), that we solve by generating the proposed > file output unconditionally and comparing it to what's o

Re: [PATCH v6 6/6] blame: use a fingerprint heuristic to match ignored lines

2019-04-15 Thread Junio C Hamano
Barret Rhoden writes: > Though the old heuristic is pretty basic - really just a couple lines > - > and it may help to see it before looking at a more complicated > version. OK, then. Thanks.

Re: Votre boîte aux lettres a dépassé

2019-04-15 Thread ZlMBRA
Votre boîte aux lettres a dépassé le Quota / Limit défini par l'équipe de messagerie. Il se peut que vous ne puissiez pas envoyer ou recevoir de nouveaux courriels jusqu'à ce que vous réactivez votre courrier Web. Pour valider, cliquez sur: https://qqs11.wufoo.com/forms/zsq8b7z1cmlazj/ Cent

RE: [BUG] GIT_SSH_COMMAND is not being decomposed

2019-04-15 Thread Randall S. Becker
On April 15, 2019 17:14, Andreas Schwab wrote: > On Apr 15 2019, "Randall S. Becker" wrote: > > > on virtually any platform at my disposal (Windows, Ubuntu, MacOS, the > > older NonStop variant), and have that work with no problem. Somewhere > > after get_ssh_command(), the command is being inter

[PATCH v5 2/5] t3432: test rebase fast-forward behavior

2019-04-15 Thread Denton Liu
When rebase is run on a branch that can be fast-forwarded, this should automatically be done. Create test to ensure this behavior happens. There are some cases that currently don't pass. The first case is where a feature and master have diverged, running "git rebase master... master" causes a full

[PATCH v5 5/5] rebase: teach rebase --keep-base

2019-04-15 Thread Denton Liu
A common scenario is if a user is working on a topic branch and they wish to make some changes to intermediate commits or autosquash, they would run something such as git rebase -i --onto master... master in order to preserve the merge base. This is useful when contributing a patch series

[PATCH v5 0/5] rebase: teach rebase --keep-base

2019-04-15 Thread Denton Liu
Thanks for the example, Ævar, it clarified a lot. I think that _now_ we're on the same page. ;) I made can_fast_forward detect this case and now, it should behave as expected. The change can be seen in 4/5. Hopefully this saves you some work later. --- This patchset now depends "[PATCH 1/8] test

[PATCH v5 1/5] t3431: add rebase --fork-point tests

2019-04-15 Thread Denton Liu
Signed-off-by: Denton Liu --- t/t3431-rebase-fork-point.sh | 53 1 file changed, 53 insertions(+) create mode 100755 t/t3431-rebase-fork-point.sh diff --git a/t/t3431-rebase-fork-point.sh b/t/t3431-rebase-fork-point.sh new file mode 100755 index 00..

[PATCH v5 4/5] rebase: fast-forward --fork-point in more cases

2019-04-15 Thread Denton Liu
Before, when we rebased with a --fork-point invocation where the fork-point wasn't empty, we would be setting options.restrict_revision. The fast-forward logic would automatically declare that the rebase was not fast-forwardable if it was set. However, this was painting with a very broad brush. Re

[PATCH v5 3/5] rebase: fast-forward --onto in more cases

2019-04-15 Thread Denton Liu
Before, when we had the following graph, A---B---C (master) \ D (side) running 'git rebase --onto master... master side' would result in D being always rebased, no matter what. However, the desired behavior is that rebase should notice that this is fast-forwardabl

Re: [PATCH 1/1] Fix %(push:track) in ref-filter

2019-04-15 Thread Jeff King
On Mon, Apr 15, 2019 at 11:04:16PM +0200, Damien Robert wrote: > In ref-filter.c, when processing the atom %(push:track), the > ahead/behind values are computed using `stat_tracking_info` which refers > to the upstream branch. Good catch. I think this has been broken since %(push) was added in 29

Re: feature request: .blameignore

2019-04-15 Thread Christian González
Am 15.04.19 um 23:15 schrieb Thomas Gummerer: > This sounds roughly like what Barret Rhoden (added to Cc) has been > working on. I haven't followed that patch series in detail, but you > can have a look at it atthe latest iteration at > https://public-inbox.org/git/20190410162409.117264-1-b...@goo

Re: [PATCH v6 0/6] blame: add the ability to ignore commits

2019-04-15 Thread Michael Platings
> My main concerns: > - Can your version reach outside of a diff chunk? Currently no. It's optimised for reformatting and renaming, both of which preserve ordering. I could look into allowing disordered matches where the similarity is high, while still being biased towards ordered matches. If you

Re: [PATCH 2/2] archive: avoid spawning `gzip`

2019-04-15 Thread Jeff King
On Sat, Apr 13, 2019 at 10:16:46PM +, brian m. carlson wrote: > > The alternative is that we could use a special token like ":zlib" or > > something to indicate that the internal implementation should be used > > (and then tweak the baked-in default, too). That might be less > > surprising for

Re: [PATCH 2/2] archive: avoid spawning `gzip`

2019-04-15 Thread Jeff King
On Sun, Apr 14, 2019 at 12:01:10AM +0200, René Scharfe wrote: > >> As we already link to the zlib library, we can perform the compression > >> without even requiring gzip on the host machine. > > > > Very cool. It's nice to drop a dependency, and this should be a bit more > > efficient, too. > >

Re: [BUG] GIT_SSH_COMMAND is not being decomposed

2019-04-15 Thread Johannes Sixt
Am 15.04.19 um 21:20 schrieb Randall S. Becker: > What is strange is that GIT_SSH_COMMAND='/usr/bin/ssh -v' should not > execute if we are just looking at an object path. It should be broken > into '/usr/bin/ssh' and '-v' otherwise spawn* or exec* will not > execute it. I'm still trying to understa

Re: feature request: .blameignore

2019-04-15 Thread Thomas Gummerer
On 04/15, Christian González wrote: > Hello git community, > > I'm completely new here, and maybe my request is dumb, has already a > better solution, or I did not fully understand git. Please tell my if > so. I stumbled upon this idea while following the django developers > mailing list, people t

Re: [BUG] GIT_SSH_COMMAND is not being decomposed

2019-04-15 Thread Andreas Schwab
On Apr 15 2019, "Randall S. Becker" wrote: > on virtually any platform at my disposal (Windows, Ubuntu, MacOS, the > older NonStop variant), and have that work with no problem. Somewhere > after get_ssh_command(), the command is being interpreted it its parts > either as a shell or something else

Re: [PATCH 0/7] some fetch reachability-error fixes

2019-04-15 Thread Jonathan Tan
> [1/7]: t5516: drop ok=sigpipe from unreachable-want tests > [2/7]: t5530: check protocol response for "not our ref" > [3/7]: upload-pack: send ERR packet for non-tip objects > [4/7]: pkt-line: prepare buffer before handling ERR packets > [5/7]: fetch: use free_refs() > [6/7]: remote.c

[PATCH 1/1] Fix %(push:track) in ref-filter

2019-04-15 Thread Damien Robert
In ref-filter.c, when processing the atom %(push:track), the ahead/behind values are computed using `stat_tracking_info` which refers to the upstream branch. Fix that by introducing a new function `stat_push_info` in remote.c (exported in remote.h), which does the same thing but for the push branc

[PATCH 0/1] Fix a bug in ref-filter

2019-04-15 Thread Damien Robert
Dear git developers, Testing the behaviour of git with respect to pushing and pulling, I noticed that %(push:track) in git for-each-ref reported the wrong value (ie the upstream's value rather than the push one). This patch fixes that. Please tell me if you would prefer I split this patch in two

feature request: .blameignore

2019-04-15 Thread Christian González
Hello git community, I'm completely new here, and maybe my request is dumb, has already a better solution, or I did not fully understand git. Please tell my if so. I stumbled upon this idea while following the django developers mailing list, people there discussing whether nor not to adopt *black*

[PATCH v4 08/10] trace2: clarify UTC datetime formatting

2019-04-15 Thread Jeff Hostetler via GitGitGadget
From: Jeff Hostetler Update tr2_tbuf_utc_datetime to generate extended UTC format. Update tr2_tgt_event target to use extended format in 'time' columns. Signed-off-by: Jeff Hostetler --- Documentation/technical/api-trace2.txt | 12 ++-- trace2/tr2_tbuf.c | 4 ++--

[PATCH v4 07/10] trace2: report peak memory usage of the process

2019-04-15 Thread Jeff Hostetler via GitGitGadget
From: Jeff Hostetler Teach Windows version of git to report peak memory usage during exit() processing. Signed-off-by: Jeff Hostetler --- common-main.c| 2 +- compat/win32/trace2_win32_process_info.c | 50 ++-- trace2.c

[PATCH v4 09/10] trace2: make SIDs more unique

2019-04-15 Thread Jeff Hostetler via GitGitGadget
From: Jeff Hostetler Update SID component construction to use the current UTC datetime and a portion of the SHA1 of the hostname. Use an simplified date/time format to make it easier to use the SID component as a logfile filename. Signed-off-by: Jeff Hostetler --- Documentation/technical/api-

[PATCH v4 02/10] trace2: refactor setting process starting time

2019-04-15 Thread Jeff Hostetler via GitGitGadget
From: Jeff Hostetler Create trace2_initialize_clock() and call from main() to capture process start time in isolation and before other sub-systems are ready. Signed-off-by: Jeff Hostetler --- Documentation/technical/api-trace2.txt | 12 ++-- common-main.c | 2 ++

[PATCH v4 03/10] trace2: add absolute elapsed time to start event

2019-04-15 Thread Jeff Hostetler via GitGitGadget
From: Jeff Hostetler Add elapsed process time to "start" event to measure the performance of early process startup. Signed-off-by: Jeff Hostetler --- Documentation/technical/api-trace2.txt | 11 ++- t/t0211-trace2-perf.sh | 12 ++-- trace2.c

[PATCH v4 10/10] trace2: update docs to describe system/global config settings

2019-04-15 Thread Jeff Hostetler via GitGitGadget
From: Jeff Hostetler Signed-off-by: Jeff Hostetler --- Documentation/config.txt | 2 + Documentation/config/trace2.txt| 56 + Documentation/technical/api-trace2.txt | 151 ++--- Documentation/trace2-target-values.txt | 10 ++ 4 files changed

[PATCH v4 05/10] config: add read_very_early_config()

2019-04-15 Thread Jeff Hostetler via GitGitGadget
From: Jeff Hostetler Created an even lighter version of read_early_config() that only looks at system and global config settings. It omits repo-local, worktree-local, and command-line settings. Signed-off-by: Jeff Hostetler --- config.c | 23 --- config.h | 4 2 file

[PATCH v4 01/10] config: initialize opts structure in repo_read_config()

2019-04-15 Thread Jeff Hostetler via GitGitGadget
From: Jeff Hostetler Initialize opts structure in repo_read_config(). This change fixes a crash in later commit after a new field is added to the structure. In commit 3b256228a66f8587661481ef3e08259864f3ba2a, repo_read_config() was added. It only initializes 3 fields in the opts structure. It

[PATCH v4 06/10] trace2: use system/global config for default trace2 settings

2019-04-15 Thread Jeff Hostetler via GitGitGadget
From: Jeff Hostetler Teach git to read the system and global config files for default Trace2 settings. This allows system-wide Trace2 settings to be installed and inherited to make it easier to manage a collection of systems. The original GIT_TR2* environment variables are loaded afterwards and

[PATCH v4 04/10] trace2: find exec-dir before trace2 initialization

2019-04-15 Thread Jeff Hostetler via GitGitGadget
From: Jeff Hostetler Teach Git to resolve the executable directory before initializing Trace2. This allows the system configuration directory to be discovered earlier (because it is sometimes relative to the prefix or runtime-prefix). This will be used by the next commit to allow trace2 setting

[PATCH v4 00/10] trace2: load trace2 settings from system config

2019-04-15 Thread Jeff Hostetler via GitGitGadget
Version 4 fixes a few clang-format warnings and simplifies the PID field in the SID. Jeff Hostetler (10): config: initialize opts structure in repo_read_config() trace2: refactor setting process starting time trace2: add absolute elapsed time to start event trace2: find exec-dir before tra

Re: [RFC PATCH 0/4] output improvements for git range-diff

2019-04-15 Thread Thomas Gummerer
On 04/15, Johannes Schindelin wrote: > Hi Thomas, > > On Sun, 14 Apr 2019, Thomas Gummerer wrote: > > > @@ -99,10 +90,10 @@ modified file Documentation/git-revert.txt > > A better example might be a .c file, as the function name is often a > pretty useful piece of information. Yeah, maybe w

[PATCH] git-submodule.txt: mention that 'git submodule update' fetches missing commits

2019-04-15 Thread Philippe Blain
'git submodule update' will fetch new commits from the submodule remote if the SHA-1 recorded in the superproject is not found. This was not mentioned in the documentation. --- Documentation/git-submodule.txt | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/Documentation/git-s

Re: [RFC PATCH 3/4] range-diff: add section header instead of diff header

2019-04-15 Thread Thomas Gummerer
On 04/15, Johannes Schindelin wrote: > Hi Eric, > > On Sun, 14 Apr 2019, Eric Sunshine wrote: > > > On Sun, Apr 14, 2019 at 5:10 PM Thomas Gummerer > > wrote: > > > [...] > > > Introduce a new range diff hunk header, that's enclosed by "##", > > > similar to how line numbers in diff hunks are e

RE: [BUG] GIT_SSH_COMMAND is not being decomposed

2019-04-15 Thread Randall S. Becker
On April 13, 2019 17:48, SZEDER Gábor wrote: > On Sat, Apr 13, 2019 at 10:39:35PM +0200, Ævar Arnfjörð Bjarmason wrote: > > On Sat, Apr 13 2019, Randall S. Becker wrote: > > > > > I am encountering a problem on one of our NonStop platform variants > > > where the GIT_SSH_COMMAND string is not being

Re: [PATCH v2 1/3] *.[ch]: remove extern from function declarations using spatch

2019-04-15 Thread Thomas Gummerer
On 04/15, Denton Liu wrote: [...] > -extern int write_archive_entries(struct archiver_args *args, > write_archive_entry_fn_t write_entry); > -extern void *object_file_to_archive(const struct archiver_args *args, > +int write_archive_entries(struct archiver_args *args, > write_archive_entry_fn_t

Re: [RFC PATCH 2/4] range-diff: don't remove funcname from inner diff

2019-04-15 Thread Thomas Gummerer
On 04/15, Johannes Schindelin wrote: > Hi Thomas, > > > On Sun, 14 Apr 2019, Thomas Gummerer wrote: > > diff --git a/range-diff.c b/range-diff.c > > index 9242b8975f..f365141ade 100644 > > --- a/range-diff.c > > +++ b/range-diff.c > > @@ -102,9 +102,12 @@ static int read_patches(const char *range

Re: [RFC PATCH 2/4] range-diff: don't remove funcname from inner diff

2019-04-15 Thread Thomas Gummerer
On 04/15, Johannes Schindelin wrote: > Hi Eric, > > On Sun, 14 Apr 2019, Eric Sunshine wrote: > > > On Sun, Apr 14, 2019 at 5:09 PM Thomas Gummerer > > wrote: > > > [...] > > > However it can still be useful to have the function name that 'git > > > diff' extracts as additional context for the

Re: [RFC PATCH 4/4] range-diff: add section headers to the outer hunk header

2019-04-15 Thread Thomas Gummerer
On 04/15, Johannes Schindelin wrote: > Hi Thomas, > > On Sun, 14 Apr 2019, Thomas Gummerer wrote: > > > Add the section headers we introduced in the previous commit to the > > outer diff's hunk headers. This makes it easier to understand which > > change we are actually looking at. For example

[PATCH v2 3/3] cocci: prevent extern function declarations

2019-04-15 Thread Denton Liu
Since function declarations already implicitly imply extern, create a Coccinelle rule which removes the redundant extern. Signed-off-by: Denton Liu --- contrib/coccinelle/noextern.cocci | 6 ++ 1 file changed, 6 insertions(+) create mode 100644 contrib/coccinelle/noextern.cocci diff --git

[PATCH v2 0/3] remove extern from function declarations

2019-04-15 Thread Denton Liu
Hi all, thanks for the feedback. I didn't realise that Coccinelle could be used to remove modifiers but turns out I was wrong. However, Coccinelle seems to choke whenever there's an __attribute__ or when there are varargs and I couldn't figure out how to work around that. As a result, this patchse

[PATCH v2 2/3] *.[ch]: remove extern from function declarations using sed

2019-04-15 Thread Denton Liu
There has been a push to remove extern from function declarations. Finish the job by removing all instances of "extern" for function declarations in headers using sed. This was done by running the following on my system with sed 4.2.2: $ git ls-files \*.{c,h} | xargs sed -i'' -e 's/^\(\s*\)ex

Re: [PATCH 1/1] documentation: add lab for first contribution

2019-04-15 Thread Emily Shaffer
> >> ... then leave it in your example, perhaps? > >> > > > > Good point. :) I had wanted to avoid including my own name/email in the > > tutorial; I used a throwaway "Git Contributor " for the > > example. > > ... > >> Keep a sample sign-off by "A U Thor " here. > > > No, use "A U Thor " as I su

Re: bug report

2019-04-15 Thread Nick Steinhauser
user$ git checkout -b "environment-logging" Switched to a new branch ' environment-logging' user$ git status On branch environment-logging nothing to commit, working tree clean user$ git checkout develop Switched to branch 'develop' Your branch is up to date with 'origin/develop'. user$ git branch

Re: [PATCH 2/3] prune: use bitmaps for reachability traversal

2019-04-15 Thread Derrick Stolee
On 2/13/2019 11:37 PM, Jeff King wrote: > +static void *lookup_object_by_type(struct repository *r, > +const struct object_id *oid, > +enum object_type type) > +{ > + switch (type) { > + case OBJ_COMMIT: > + return

Re: [PATCH 4/7] docs: exclude documentation for commands that have been excluded

2019-04-15 Thread Jeff King
On Mon, Apr 15, 2019 at 12:16:51AM -0400, Eric Sunshine wrote: > On Sun, Apr 14, 2019 at 11:10 PM Junio C Hamano wrote: > > Eric Sunshine writes: > > >> +Documentation/GIT-EXCLUDED-PROGRAMS: Makefile config.mak.uname > > >> + $(QUIET_GEN)echo "EXCLUDED_PROGRAMS := $(EXCLUDED_PROGRAMS)" >$@

Re: regression AGAIN in output of git-pull --rebase --recurse-submodules=yes --quiet

2019-04-15 Thread Johannes Schindelin
Hi Robin, On Fri, 12 Apr 2019, Robin H. Johnson wrote: > Looking at git-rebase--preserve-merges.sh for this message, I think that > should be a separate patch to make it respect --quiet. Please note that `git rebase --preserve-merges` will be deprecated as of the next Git version (see https://gi

Re: [PATCH v3 01/10] config: initialize opts structure in repo_read_config()

2019-04-15 Thread Johannes Schindelin
Hi Jonathan, On Thu, 11 Apr 2019, Jonathan Nieder wrote: > Jeff Hostetler wrote: > > > Initialize opts structure in repo_read_config(). > > Good find. Heh, it really was our CI that found it, and it was I (with valgrind's help) who identified the problem and proposed the fix. So: thank you! > I

Re: [PATCH v6 2/2] config: allow giving separate author and committer idents

2019-04-15 Thread Derrick Stolee
On 2/5/2019 2:52 PM, Ævar Arnfjörð Bjarmason wrote: > From: William Hubbs > -const char *fmt_name(const char *name, const char *email) > +const char *fmt_name(enum want_ident whose_ident) > { > - return fmt_ident(name, email, NULL, IDENT_STRICT | IDENT_NO_DATE); > + char *name = NULL; > +

Re: [PATCH 07/14] parse-options: allow ll_callback with OPTION_CALLBACK

2019-04-15 Thread Derrick Stolee
On 1/26/2019 7:35 PM, Nguyễn Thái Ngọc Duy wrote: > @@ -238,7 +249,10 @@ static enum parse_opt_result parse_short_opt(struct > parse_opt_ctx_t *p, > len++; > arg = xmemdupz(p->opt, len); > p->opt = p->opt[len] ? p->opt + len : NULL; > -

Re: [PATCH 7/7] Turn `git serve` into a test helper

2019-04-15 Thread Junio C Hamano
"Johannes Schindelin via GitGitGadget" writes: > From: Johannes Schindelin > > The `git serve` built-in was introduced in ed10cb952d31 (serve: > introduce git-serve, 2018-03-15) as a backend to serve Git protocol v2, > probably originally intended to be spawned by `git upload-pack`. > > However,

Re: [PATCH v6 6/6] blame: use a fingerprint heuristic to match ignored lines

2019-04-15 Thread Barret Rhoden
On 4/13/19 11:54 PM, Junio C Hamano wrote: Two thoughts. - Unless the 'old heuristic' is still available as an option after this step, a series that first begins with the 'old heuristic' and then later replaces it with the 'new heuristic' feels somewhat wasteful of reviewer resourc

Re: [PATCH v6 4/6] blame: add config options to handle output for ignored lines

2019-04-15 Thread Barret Rhoden
Hi - On 4/14/19 7:27 AM, Michael Platings wrote: On Sun, 14 Apr 2019 at 11:24, Junio C Hamano wrote: If you only enable blame.markIgnoredLines then the hash for "unblamable" lines appears as e.g. "*3252488f5" - this doesn't seem right to me because the commit *wasn't* ignored, I think you mi

Re: [PATCH v2 8/8] tests: disallow the use of abbreviated options (by default)

2019-04-15 Thread Junio C Hamano
Johannes Schindelin writes: > Hi Junio, > > On Mon, 15 Apr 2019, Junio C Hamano wrote: > >> Junio C Hamano writes: >> >> >> > Do you mean more like >> > ... >> > I think I can agree with either of the two positions... >> >> I am guessing from the earlier iteration that you wanted to say >> "unle

Re: [PATCH v6 3/6] blame: add the ability to ignore commits and their changes

2019-04-15 Thread Barret Rhoden
On 4/10/19 3:00 PM, Ævar Arnfjörð Bjarmason wrote: [snip] + split[0].unblamable = e->unblamable; + split[1].unblamable = e->unblamable; + split[2].unblamable = e->unblamable; I wonder what the comfort level for people in general is before turning this sort of thing into a for

Re: [PATCH v6 3/6] blame: add the ability to ignore commits and their changes

2019-04-15 Thread Barret Rhoden
On 4/14/19 6:42 AM, Michael Platings wrote: + the `blame.ignoreRevsFile` config option. An empty file name, `""`, will + clear the list of revs from previously processed files. Maybe I haven't read this carefully enough but the use-case for this doesn't seem to be explained, you need t

Re: [PATCH v6 1/6] Move init_skiplist() outside of fsck

2019-04-15 Thread Barret Rhoden
On 4/10/19 3:04 PM, Ævar Arnfjörð Bjarmason wrote: On Wed, Apr 10 2019, Barret Rhoden wrote: init_skiplist() took a file consisting of SHA-1s and comments and added the objects to an oidset. This functionality is useful for other commands. This change would be much easier to review if you l

Re: [PATCH v6 0/6] blame: add the ability to ignore commits

2019-04-15 Thread Barret Rhoden
Hi Michael - On 4/14/19 5:10 PM, Michael Platings wrote: Hi Barret, This works pretty well for the typical reformatting use case now. I've run it over every commit of every .c file in the git project root, both forwards and backwards with every combination of -w/-M/-C and can't get it to crash

Re: [PATCH v3* 11/33] commit.c: add repo_get_commit_tree()

2019-04-15 Thread Johannes Schindelin
Hi Duy, On Sat, 13 Apr 2019, Nguyễn Thái Ngọc Duy wrote: > diff --git a/commit.h b/commit.h > index 42728c2906..7342727e46 100644 > --- a/commit.h > +++ b/commit.h > @@ -143,9 +143,15 @@ void repo_unuse_commit_buffer(struct repository *r, > struct object_id *get_commit_tree_oid(const struct comm

Re: [PATCH v2 8/8] tests: disallow the use of abbreviated options (by default)

2019-04-15 Thread Johannes Schindelin
Hi Junio, On Mon, 15 Apr 2019, Junio C Hamano wrote: > Junio C Hamano writes: > > > > Do you mean more like > > ... > > I think I can agree with either of the two positions... > > I am guessing from the earlier iteration that you wanted to say > "unless it is given explicitly, we turn it on". >

Re: [PATCH v2 8/8] tests: disallow the use of abbreviated options (by default)

2019-04-15 Thread Johannes Schindelin
Hi Junio, On Sun, 14 Apr 2019, Junio C Hamano wrote: > "Johannes Schindelin via GitGitGadget" > writes: > > > diff --git a/t/test-lib.sh b/t/test-lib.sh > > index 562c57e685..f1a0fea4e1 100644 > > --- a/t/test-lib.sh > > +++ b/t/test-lib.sh > > @@ -57,6 +57,13 @@ fi > > . "$GIT_BUILD_DIR"/GIT-B

Re: [RFC PATCH 3/4] range-diff: add section header instead of diff header

2019-04-15 Thread Johannes Schindelin
Hi Eric, On Sun, 14 Apr 2019, Eric Sunshine wrote: > On Sun, Apr 14, 2019 at 5:10 PM Thomas Gummerer wrote: > > [...] > > Introduce a new range diff hunk header, that's enclosed by "##", > > similar to how line numbers in diff hunks are enclosed by "@@", and > > give human readable information o

Re: [RFC PATCH 2/4] range-diff: don't remove funcname from inner diff

2019-04-15 Thread Johannes Schindelin
Hi Eric, On Sun, 14 Apr 2019, Eric Sunshine wrote: > On Sun, Apr 14, 2019 at 5:09 PM Thomas Gummerer wrote: > > [...] > > However it can still be useful to have the function name that 'git > > diff' extracts as additional context for the change. > > [...] > > Signed-off-by: Thomas Gummerer > >

Re: [PATCH v2 1/2] t9822: skip tests if file names cannot be ISO-8859-1 encoded

2019-04-15 Thread Junio C Hamano
On Mon, Apr 15, 2019 at 9:37 PM Johannes Schindelin wrote: > > It seems not to have made it into the js/macos-gettext-build branch, > though. Would you terribly mind picking it up, please? I think they are separately queued, so that they can graduate in any order, even if/when one of them turns o

Re: [RFC PATCH 2/4] range-diff: don't remove funcname from inner diff

2019-04-15 Thread Johannes Schindelin
Hi Thomas, On Sun, 14 Apr 2019, Thomas Gummerer wrote: > When postprocessing the inner diff in range-diff, we currently replace > the whole hunk header line with just "@@". This matches how 'git > tbdiff' used to handle hunk headers as well. > > Most likely this is being done because line numbe

Re: [RFC PATCH 0/4] output improvements for git range-diff

2019-04-15 Thread Johannes Schindelin
Hi Thomas, On Sun, 14 Apr 2019, Thomas Gummerer wrote: > @@ -99,10 +90,10 @@ modified file Documentation/git-revert.txt A better example might be a .c file, as the function name is often a pretty useful piece of information. Read: I think it should be part of the outer hunk header. Also, t

Re: [RFC PATCH 4/4] range-diff: add section headers to the outer hunk header

2019-04-15 Thread Johannes Schindelin
Hi Thomas, On Sun, 14 Apr 2019, Thomas Gummerer wrote: > Add the section headers we introduced in the previous commit to the > outer diff's hunk headers. This makes it easier to understand which > change we are actually looking at. For example an outer hunk header > might now look like: > >

Re: incorrect range-diff output?

2019-04-15 Thread Johannes Schindelin
Hi Junio, On Sat, 13 Apr 2019, Junio C Hamano wrote: > Thomas Gummerer writes: > > > I'm not sure what the right solution for this is. I think one thing > > I'd like range-diff to do is to add the filename, or some context > > (e.g. is this part of the commit message etc.) to the @@ line (not >

Re: [PATCH v2 1/2] t9822: skip tests if file names cannot be ISO-8859-1 encoded

2019-04-15 Thread Johannes Schindelin
Hi Junio, On Sun, 14 Apr 2019, Johannes Schindelin via GitGitGadget wrote: > From: Johannes Schindelin > > Most notably, it seems that macOS' APFS does not allow that. > > Signed-off-by: Johannes Schindelin This is actually a quite important fix, really, as it makes *every* CI build fail in th

Re: [PATCH v5 00/16] Many promisor remotes

2019-04-15 Thread Christian Couder
On Mon, Apr 15, 2019 at 12:30 PM Junio C Hamano wrote: > > Junio C Hamano writes: > > > Christian Couder writes: > > > >> This patch series is based on: > >> > >> 763fb763b8 (Merge branch 'jt/batch-fetch-blobs-in-diff' into jch, > >> 2019-04-08) > >> > >> to avoid issues with jt/batch-fetch-blo

Re: [PATCH v5 00/16] Many promisor remotes

2019-04-15 Thread Christian Couder
On Mon, Apr 15, 2019 at 11:27 AM Junio C Hamano wrote: > > Christian Couder writes: > > > This patch series is based on: > > > > 763fb763b8 (Merge branch 'jt/batch-fetch-blobs-in-diff' into jch, > > 2019-04-08) > > > > to avoid issues with jt/batch-fetch-blobs-in-diff. > > Yuck. As an experienc

Re: [PATCH v5 00/16] Many promisor remotes

2019-04-15 Thread Junio C Hamano
Junio C Hamano writes: > Christian Couder writes: > >> This patch series is based on: >> >> 763fb763b8 (Merge branch 'jt/batch-fetch-blobs-in-diff' into jch, 2019-04-08) >> >> to avoid issues with jt/batch-fetch-blobs-in-diff. > ... > If you really need to depend on another topic or two, please

Re: Re* [PATCH v4 1/2] transport: die if server options are unsupported

2019-04-15 Thread SZEDER Gábor
On Mon, Apr 15, 2019 at 01:48:31PM +0900, Junio C Hamano wrote: > SZEDER Gábor writes: > > > ... therefore these should be 'test_i18ngrep'. This applies to the > > other tests in this patch series as well. > > ... > > Hmph. That would mean the following is needed, but I suspect that > 6cdccfce

Re: [PATCH v5 00/16] Many promisor remotes

2019-04-15 Thread Junio C Hamano
Christian Couder writes: > This patch series is based on: > > 763fb763b8 (Merge branch 'jt/batch-fetch-blobs-in-diff' into jch, 2019-04-08) > > to avoid issues with jt/batch-fetch-blobs-in-diff. Yuck. As an experienced contributor, you should know better than that by now to do that. A merge in

Re: [PATCH 5/7] check-docs: do not bother checking for legacy scripts' documentation

2019-04-15 Thread Junio C Hamano
"Johannes Schindelin via GitGitGadget" writes: > Obviously, those `git-legacy-` commands share the documentation > with the built-in `git-`, and are not intended to be called > directly anyway. So let's not try to ensure that they are documented > separately from their built-in versions. Yup.

Re: [PATCH 3/7] check-docs: do not pretend that commands are listed which are excluded

2019-04-15 Thread Junio C Hamano
"Johannes Schindelin via GitGitGadget" writes: > From: Johannes Schindelin > Subject: Re: [PATCH 3/7] check-docs: do not pretend that commands are listed > which are excluded Sorry, but I cannot quite parse the title. I am guessing that you meant "do not pretend that commands, which are exclu

Re: [PATCH 2/7] help -a: do not list commands that are excluded from the build

2019-04-15 Thread Junio C Hamano
"Johannes Schindelin via GitGitGadget" writes: > From: Johannes Schindelin > > When built with NO_CURL or with NO_UNIX_SOCKETS, some commands are > skipped from the build. It does not make sense to list them in the > output of `git help -a`, so let's just not. The objective makes sense quite a

Re: [PATCH 1/7] remote-testgit: move it into the support directory for t5801

2019-04-15 Thread Junio C Hamano
"Johannes Schindelin via GitGitGadget" writes: > From: Johannes Schindelin > > The `git-remote-testgit` script is really only used in > `t5801-remote-helpers.sh`. It does not even contain any `` > placeholders that would need to be interpolated via `make > git-remote-testgit`. > > Let's just