[PATCH v1] travis-ci: printf $STATUS as string

2017-04-26 Thread Lars Schneider
If the $STATUS variable contains a "%" character then printf will interpret that as invalid format string. Fix this by formatting $STATUS as string. Signed-off-by: Lars Schneider <larsxschnei...@gmail.com> --- Notes: Base Ref: master Web-Diff: https://github.com/larsxschn

[PATCH v2 0/4] travis-ci: build docs with asciidoctor

2017-04-26 Thread Lars Schneider
Hi, changes since v1: * check Asciidoctor stderr output (Brian) http://public-inbox.org/git/20170418104411.hdkzh3psvej63...@genre.crustytoothpaste.net/ * fix make style nit (Junio) http://public-inbox.org/git/xmqq37dcorr7@gitster.mtv.corp.google.com/ Thanks, Lars Base Ref: master

[PATCH v2 4/4] travis-ci: unset compiler for jobs that do not need one

2017-04-26 Thread Lars Schneider
should clear this, too. Signed-off-by: Lars Schneider <larsxschnei...@gmail.com> --- .travis.yml | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/.travis.yml b/.travis.yml index c26f3bf789..e5e7c3edd7 100644 --- a/.travis.yml +++ b/.travis.yml @@ -41,6 +41,7 @@ matrix: i

[PATCH v2 3/4] travis-ci: check AsciiDoc/AsciiDoctor stderr output

2017-04-26 Thread Lars Schneider
uses `bash`. This shouldn't be a problem as the script is only executed on TravisCI. Signed-off-by: Lars Schneider <larsxschnei...@gmail.com> --- ci/test-documentation.sh | 9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/ci/test-documentation.sh b/ci/test-documentat

[PATCH v2 1/4] travis-ci: build documentation with AsciiDoc and Asciidoctor

2017-04-26 Thread Lars Schneider
ec3366e introduced a knob to enable the use of Asciidoctor in addition to AsciiDoc. Build the documentation on TravisCI with this knob to reduce the likeliness of breaking Asciidoctor support in the future. Signed-off-by: Lars Schneider <larsxschnei...@gmail.com> --- .trav

[PATCH v2 2/4] travis-ci: parallelize documentation build

2017-04-26 Thread Lars Schneider
The documentation job without parallelization takes ~10min on TravisCI. With parallelization ("--jobs=2") it takes ~6min. Signed-off-by: Lars Schneider <larsxschnei...@gmail.com> --- ci/test-documentation.sh | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff

[PATCH v1] travis-ci: set DEVELOPER knob for Linux32 build

2017-04-26 Thread Lars Schneider
The Linux32 build was not build with our strict compiler settings (e.g. warnings as errors). Fix this by passing the DEVELOPER environment variable to the docker container. Signed-off-by: Lars Schneider <larsxschnei...@gmail.com> --- Notes: Base Ref: master Web-Diff: https://gith

Re: t0025 flaky on OSX

2017-04-26 Thread Lars Schneider
> On 26 Apr 2017, at 06:12, Torsten Bögershausen wrote: > > >>> >>> So all in all it seams as if there is a very old race condition here, >>> which we "never" have seen yet. >>> Moving the file to a different inode number fixes the test case, >>> Git doesn't treat it as

Re: t0025 flaky on OSX

2017-04-25 Thread Lars Schneider
> On 24 Apr 2017, at 21:48, Torsten Bögershausen <tbo...@web.de> wrote: > > On 2017-04-24 19:00, Torsten Bögershausen wrote: >> On 2017-04-24 18:45, Lars Schneider wrote: >>> Hi, >>> >>> "t0025.3 - crlf=true causes a CRLF file to be no

Re: [BUG] test suite broken with GETTEXT_POISON=YesPlease

2017-04-25 Thread Lars Schneider
> On 24 Apr 2017, at 22:37, Jeff King wrote: > > On Mon, Apr 24, 2017 at 08:22:36PM +0700, Duy Nguyen wrote: > >> Off topic, is it possible to receive mail notifications from Travis >> when a fault is found in either 'pu', 'next' or 'master'? I know how >> to do it in Jenkins,

Re: [BUG] test suite broken with GETTEXT_POISON=YesPlease

2017-04-28 Thread Lars Schneider
> On 25 Apr 2017, at 12:01, Jeff King <p...@peff.net> wrote: > > On Tue, Apr 25, 2017 at 10:51:20AM +0200, Lars Schneider wrote: > >>>> Off topic, is it possible to receive mail notifications from Travis >>>> when a fault is found in either 'pu',

[PATCH v1] travis-ci: handle Git for Windows CI status "failed" explicitly

2017-04-29 Thread Lars Schneider
Git for Windows CI returns "completed: failed" if a build or test failure happened. This case was processed as "Unhandled status". Handle the case explicitly. Signed-off-by: Lars Schneider <larsxschnei...@gmail.com> --- Notes: Base Ref: next Web-Diff: https:/

[PATCH v1] travis-ci: retry if Git for Windows CI returns HTTP error 502 or 503

2017-04-29 Thread Lars Schneider
The Git for Windows CI web app sometimes returns HTTP errors of "502 bad gateway" or "503 service unavailable" [1]. Wait a little and retry the request if this happens. [1] https://docs.microsoft.com/en-in/azure/app-service-web/app-service-web-troubleshoot-http-502-http-503

Re: [git-for-windows] [ANNOUNCE] Git for Windows 2.14.0

2017-08-06 Thread Lars Schneider
> On 06 Aug 2017, at 01:00, Johannes Schindelin > wrote: > > Dear Git users, > > It is my pleasure to announce that Git for Windows 2.14.0 is available from: Thank you :-) > * It is now possible to switch between Secure Channel and OpenSSL for >Git's HTTPS

Re: [PATCH for NEXT v3 2/2] sub-process: refactor handshake to common function

2017-08-06 Thread Lars Schneider
> On 26 Jul 2017, at 20:17, Jonathan Tan wrote: > > Refactor, into a common function, the version and capability negotiation > done when invoking a long-running process as a clean or smudge filter. > This will be useful for other Git code that needs to interact

Re: [PATCH for NEXT v3 2/2] sub-process: refactor handshake to common function

2017-08-07 Thread Lars Schneider
> On 07 Aug 2017, at 19:21, Jonathan Tan <jonathanta...@google.com> wrote: > > On Sun, 6 Aug 2017 21:58:24 +0200 > Lars Schneider <larsxschnei...@gmail.com> wrote: > >>> + struct cmd2process *entry = (struct cmd2process *)subprocess; >>> + return

Re: Submodule regression in 2.14?

2017-08-17 Thread Lars Schneider
> On 16 Aug 2017, at 20:51, Stefan Beller <sbel...@google.com> wrote: > > On Wed, Aug 16, 2017 at 11:35 AM, Lars Schneider > <larsxschnei...@gmail.com> wrote: >> Hi, >> >> I think we discovered a regression in Git 2.14.1 today. >> It looks lik

Re: [PATCH] sub-process: print the cmd when a capability is unsupported

2017-08-17 Thread Lars Schneider
> On 17 Aug 2017, at 23:01, Junio C Hamano wrote: > > Christian Couder writes: > >> ... but I think we should then emphasize more in our test >> scripts (maybe by giving a good example) and perhaps also in the doc >> that the

Submodule regression in 2.14?

2017-08-16 Thread Lars Schneider
Hi, I think we discovered a regression in Git 2.14.1 today. It looks like as if "git submodule update --init --recursive" removes the "skip submodules" config. Consider the following steps: git clone https://server/repo.git cd repo git config --local submodule.some/other/repo.update

Re: Submodule regression in 2.14?

2017-08-19 Thread Lars Schneider
> On 18 Aug 2017, at 19:16, Stefan Beller wrote: > >> In the past "submodule..update=none" was an easy way >> to selectively disable certain Submodules. >> >> How would I do this with Git 2.14? > >submodule..active = false That's what I thought after your first

Re: [PATCH] pull: respect submodule update configuration

2017-08-22 Thread Lars Schneider
> On 21 Aug 2017, at 20:21, Brandon Williams <bmw...@google.com> wrote: > > On 08/21, Stefan Beller wrote: >> On Mon, Aug 21, 2017 at 10:20 AM, Lars Schneider >> <larsxschnei...@gmail.com> wrote: >>> >>>> On 21 Aug 2017, at 18:55, Stefan Bel

[PATCH v1] convert: display progress for filtered objects that have been delayed

2017-08-20 Thread Lars Schneider
d below the stop_progress() call in unpack-trees.c to ensure that the "Checking out files" progress is properly stopped before the "Filtering content" progress starts in finish_delayed_checkout(). Signed-off-by: Lars Schneider <larsxschnei...@gmail.com> Suggested-by: Taylor B

Re: [PATCH/RFC] push: anonymize URL in error output

2017-08-23 Thread Lars Schneider
> On 23 Aug 2017, at 11:49, Ivan Vyshnevskyi wrote: > > Commits 47abd85 (fetch: Strip usernames from url's before storing them, > 2009-04-17) and later 882d49c (push: anonymize URL in status output, > 2016-07-14) made fetch and push strip the authentication part of the >

Re: Submodule regression in 2.14?

2017-08-18 Thread Lars Schneider
> On 17 Aug 2017, at 23:55, Stefan Beller <sbel...@google.com> wrote: > > On Thu, Aug 17, 2017 at 2:21 PM, Lars Schneider > <larsxschnei...@gmail.com> wrote: >> >>> Oh, wait. >>> $ git log --oneline v2.13.0..v2.14.1 -- builtin/pull.c >&

[PATCH v2] travis-ci: retry if Git for Windows CI returns HTTP error 502 or 503

2017-05-03 Thread Lars Schneider
nd retry the request if this happens. [1] https://docs.microsoft.com/en-in/azure/app-service-web/app-service-web-troubleshoot-http-502-http-503 Signed-off-by: Lars Schneider <larsxschnei...@gmail.com> --- Hi Junio, I can't really test this as my TravisCI account does not have the extend

Re: What's cooking in git.git (May 2017, #03; Wed, 10)

2017-05-11 Thread Lars Schneider
> On 11 May 2017, at 14:09, Johannes Schindelin > wrote: > > Hi, > > [making sure Lars reads this] > > On Wed, 10 May 2017, Junio C Hamano wrote: > >> * ls/filter-process-delayed (2017-03-06) 1 commit >> - convert: add "status=delayed" to filter process protocol

Re: [PATCH v2] travis-ci: retry if Git for Windows CI returns HTTP error 502 or 503

2017-05-09 Thread Lars Schneider
> On 09 May 2017, at 07:31, Junio C Hamano <gits...@pobox.com> wrote: > > Lars Schneider <larsxschnei...@gmail.com> writes: > >> The Git for Windows CI web app sometimes returns HTTP errors of >> "502 bad gateway" or "503 service unavaila

Re: Transform log message during migration svn -> git (using git-svn)

2017-06-20 Thread Lars Schneider
> On 20 Jun 2017, at 09:32, paul.mat...@s4m.com wrote: > > Hi there, > > this is actually not really a bug report, but much more a feature request > (if I did not oversee an already existing feature like this): > > We want to migrate our SVN repository to GIT and will be using git-svn for >

Re: Transform log message during migration svn -> git (using git-svn)

2017-06-20 Thread Lars Schneider
: +49 221 28555-210 > E-Mail: paul.mat...@s4m.com > www.s4m.arvato-systems.com > > > -Ursprüngliche Nachricht- > Von: Lars Schneider [mailto:larsxschnei...@gmail.com] > Gesendet: Dienstag, 20. Juni 2017 11:32 > An: Mattke, Paul, NMM-BPDD <paul.mat...@s4m.co

Re: [PATCH] git-p4: changelist template with p4 -G change -o

2017-06-24 Thread Lars Schneider
> On 24 Jun 2017, at 13:49, Luke Diamand wrote: > > On 22 June 2017 at 18:32, Junio C Hamano wrote: >> Miguel Torroja writes: >> >>> The option -G of p4 (python marshal output) gives more context about the >>> data being output.

Re: [PATCH v5 5/5] convert: add "status=delayed" to filter process protocol

2017-06-24 Thread Lars Schneider
> On 24 Jun 2017, at 16:19, Jeff King <p...@peff.net> wrote: > > On Thu, Jun 01, 2017 at 10:22:03AM +0200, Lars Schneider wrote: > >> Some `clean` / `smudge` filters might require a significant amount of >> time to process a single blob (e.g. the Git LFS smudge f

Re: [PATCH v6 3/6] t0021: write "OUT" only on success

2017-06-26 Thread Lars Schneider
> On 26 Jun 2017, at 19:50, Junio C Hamano <gits...@pobox.com> wrote: > > Lars Schneider <larsxschnei...@gmail.com> writes: > >>> On 26 Jun 2017, at 00:17, Junio C Hamano <gits...@pobox.com> wrote: >>> >>> Lars Schneider <larsxs

Re: [PATCH v6 6/6] convert: add "status=delayed" to filter process protocol

2017-06-26 Thread Lars Schneider
> On 26 Jun 2017, at 21:02, Junio C Hamano <gits...@pobox.com> wrote: > > Lars Schneider <larsxschnei...@gmail.com> writes: > >> Some `clean` / `smudge` filters might require a significant amount of >> time to process a single blob (e.g. the Git LFS sm

[PATCH v6 0/6] convert: add "status=delayed" to filter process protocol

2017-06-25 Thread Lars Schneider
e->istate, path->string, strlen(path->string), 0); - dco->state = CE_RETRY; + assert(dco->state == CE_RETRY); errs |= (ce ? checkout_entry(ce, state,

[PATCH v6 2/6] t0021: make debug log file name configurable

2017-06-25 Thread Lars Schneider
uent patch 'convert: add "status=delayed" to filter process protocol'. Signed-off-by: Lars Schneider <larsxschnei...@gmail.com> --- t/t0021-conversion.sh | 44 ++-- t/t0021/rot13-filter.pl | 8 +--- 2 files changed, 27 insertions(+),

[PATCH v6 3/6] t0021: write "OUT" only on success

2017-06-25 Thread Lars Schneider
delayed" to filter process protocol'. Signed-off-by: Lars Schneider <larsxschnei...@gmail.com> --- t/t0021-conversion.sh | 6 +++--- t/t0021/rot13-filter.pl | 6 +++--- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/t/t0021-conversion.sh b/t/t0021-conversion.sh ind

[PATCH v6 1/6] t0021: keep filter log files on comparison

2017-06-25 Thread Lars Schneider
The filter log files are modified on comparison. Write the modified log files to temp files for comparison to fix this. This is useful for the subsequent patch 'convert: add "status=delayed" to filter process protocol'. Signed-off-by: Lars Schneider <larsxschnei...@gmail.com&

[PATCH v6 6/6] convert: add "status=delayed" to filter process protocol

2017-06-25 Thread Lars Schneider
ees.c) and `checkout` operations for now. The optimization is most effective in these code paths as all files of the tree are processed. Signed-off-by: Lars Schneider <larsxschnei...@gmail.com> --- Documentation/gitattributes.txt | 65 +- builtin/checkout.c

[PATCH v6 5/6] convert: move multiple file filter error handling to separate function

2017-06-25 Thread Lars Schneider
Refactoring the filter error handling is useful for the subsequent patch 'convert: add "status=delayed" to filter process protocol'. In addition, replace the parentheses around the empty "if" block with a single semicolon to adhere to the Git style guide. Signed-of

[PATCH v6 4/6] convert: put the flags field before the flag itself for consistent style

2017-06-25 Thread Lars Schneider
Suggested-by: Jeff King <p...@peff.net> Signed-off-by: Lars Schneider <larsxschnei...@gmail.com> --- convert.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/convert.c b/convert.c index f1e168bc30..9907e3b9ba 100644 --- a/convert.c +++ b/convert.

Re: [PATCH v6 1/6] t0021: keep filter log files on comparison

2017-06-26 Thread Lars Schneider
> On 26 Jun 2017, at 00:12, Junio C Hamano <gits...@pobox.com> wrote: > > Lars Schneider <larsxschnei...@gmail.com> writes: > >> The filter log files are modified on comparison. Write the modified log files >> to temp files for comparison to fix this. &

Re: What's cooking in git.git (Jun 2017, #07; Sat, 24)

2017-06-26 Thread Lars Schneider
> On 25 Jun 2017, at 01:25, Junio C Hamano wrote: > ... > * ab/sha1dc (2017-06-07) 2 commits > - sha1collisiondetection: automatically enable when submodule is populated > - sha1dc: optionally use sha1collisiondetection as a submodule > > The "collission-detecting"

Re: What's cooking in git.git (Jun 2017, #07; Sat, 24)

2017-06-26 Thread Lars Schneider
> On 26 Jun 2017, at 11:44, Ævar Arnfjörð Bjarmason <ava...@gmail.com> wrote: > > > On Mon, Jun 26 2017, Lars Schneider jotted: > >>> On 25 Jun 2017, at 01:25, Junio C Hamano <gits...@pobox.com> wrote: >> >>> ... >> >>&g

Re: [PATCH v6 3/6] t0021: write "OUT" only on success

2017-06-26 Thread Lars Schneider
> On 26 Jun 2017, at 00:17, Junio C Hamano <gits...@pobox.com> wrote: > > Lars Schneider <larsxschnei...@gmail.com> writes: > >> "rot13-filter.pl" used to write "OUT " to the debug log even in case of >> an abort or error. Fix this by w

Re: [PATCH v5 4/5] convert: move multiple file filter error handling to separate function

2017-06-19 Thread Lars Schneider
> On 19 Jun 2017, at 19:18, Torsten Bögershausen <tbo...@web.de> wrote: > > On 2017-06-18 13:47, Lars Schneider wrote: >> >>> On 18 Jun 2017, at 09:20, Torsten Bögershausen <tbo...@web.de> wrote: >>> >>> >>> On 2017-06-01 10:2

Re: What's cooking in git.git (Jun 2017, #06; Thu, 22)

2017-06-23 Thread Lars Schneider
> On 23 Jun 2017, at 00:35, Junio C Hamano wrote: > > 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

Re: [PATCH v4 4/4] convert: add "status=delayed" to filter process protocol

2017-05-22 Thread Lars Schneider
> On 22 May 2017, at 21:58, Torsten Bögershausen <tbo...@web.de> wrote: > > On 2017-05-22 15:50, Lars Schneider wrote: >> + >> +int async_query_available_blobs(const char *cmd, struct string_list >> *delayed_paths) >> +{ >> +int err; >>

Re: [PATCH v4 4/4] convert: add "status=delayed" to filter process protocol

2017-05-22 Thread Lars Schneider
> On 22 May 2017, at 21:52, Torsten Bögershausen <tbo...@web.de> wrote: > > On 2017-05-22 15:50, Lars Schneider wrote: >> +After Git received the pathnames, it will request the corresponding >> +blobs again. These requests contain a pathname and an empty cont

Re: [PATCH v4 4/4] convert: add "status=delayed" to filter process protocol

2017-05-23 Thread Lars Schneider
> On 23 May 2017, at 07:22, Junio C Hamano <gits...@pobox.com> wrote: > > Lars Schneider <larsxschnei...@gmail.com> writes: > >>>> + sigchain_pop(SIGPIPE); >>>> + >>>> + if (err || errno == EPIPE) { >>> >

[PATCH v4 3/4] t0021: write "OUT" only on success

2017-05-22 Thread Lars Schneider
delayed" to filter process protocol'. Signed-off-by: Lars Schneider <larsxschnei...@gmail.com> --- t/t0021-conversion.sh | 6 +++--- t/t0021/rot13-filter.pl | 6 +++--- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/t/t0021-conversion.sh b/t/t0021-conversion.sh ind

[PATCH v4 1/4] t0021: keep filter log files on comparison

2017-05-22 Thread Lars Schneider
The filter log files are modified on comparison. Write the modified log files to temp files for comparison to fix this. This is useful for the subsequent patch 'convert: add "status=delayed" to filter process protocol'. Signed-off-by: Lars Schneider <larsxschnei...@gmail.com&

[PATCH v4 4/4] convert: add "status=delayed" to filter process protocol

2017-05-22 Thread Lars Schneider
ees.c) and `checkout` operations. Signed-off-by: Lars Schneider <larsxschnei...@gmail.com> --- Documentation/gitattributes.txt | 65 +- builtin/checkout.c | 3 + cache.h | 6 +- convert.c | 131 +

[PATCH v4 2/4] t0021: make debug log file name configurable

2017-05-22 Thread Lars Schneider
uent patch 'convert: add "status=delayed" to filter process protocol'. Signed-off-by: Lars Schneider <larsxschnei...@gmail.com> --- t/t0021-conversion.sh | 44 ++-- t/t0021/rot13-filter.pl | 8 +--- 2 files changed, 27 insertions(+),

[PATCH v4 0/4] convert: add "status=delayed" to filter process protocol

2017-05-22 Thread Lars Schneider
rs RFC: http://public-inbox.org/git/d10f7c47-14e8-465b-8b7a-a09a1b28a...@gmail.com/ v1: http://public-inbox.org/git/20170108191736.47359-1-larsxschnei...@gmail.com/ v2: http://public-inbox.org/git/20170226184816.30010-1-larsxschnei...@gmail.com/ v3: http://public-inbox.org/git/20170409191107.20547-1-la

Re: [PATCH v4 4/4] convert: add "status=delayed" to filter process protocol

2017-05-29 Thread Lars Schneider
> On 23 May 2017, at 10:43, Lars Schneider <larsxschnei...@gmail.com> wrote: > > >> On 23 May 2017, at 07:22, Junio C Hamano <gits...@pobox.com> wrote: >> >> Lars Schneider <larsxschnei...@gmail.com> writes: >> >>>>>

Re: [PATCH v5 5/5] convert: add "status=delayed" to filter process protocol

2017-06-05 Thread Lars Schneider
> On 02 Jun 2017, at 04:21, Junio C Hamano <gits...@pobox.com> wrote: > > Lars Schneider <larsxschnei...@gmail.com> writes: > >> ... >> +After Git received the pathnames, it will request the corresponding >> +blobs again. These requests contain a

Re: [PATCH] test-lib: add ability to cap the runtime of tests

2017-06-05 Thread Lars Schneider
> On 04 Jun 2017, at 09:29, Ævar Arnfjörð Bjarmason wrote: > > On Sun, Jun 4, 2017 at 2:31 AM, Junio C Hamano wrote: >> Ævar Arnfjörð Bjarmason writes: >> >>> Speeding up the test suite by simply cataloging and skipping tests >>> that

Re: RFC: Would a config fetch.retryCount make sense?

2017-06-05 Thread Lars Schneider
> On 01 Jun 2017, at 15:33, Ben Peart <peart...@gmail.com> wrote: > > > > On 6/1/2017 8:48 AM, Lars Schneider wrote: >> Hi, >> we occasionally see "The remote end hung up unexpectedly" (pkt-line.c:265) >> on our `git fetch` calls (most noticeabl

t5313-pack-bounds-checks.sh flaky?

2017-06-05 Thread Lars Schneider
Hi Peff, t5313.7 failed recently on master [1]. Are you aware of any flaky parts in this test? Here is the output: expecting success: # We need two objects here, so we can plausibly require # an extended table (if the first object were larger than 2^31). do_pack

[PATCH v5 2/5] t0021: make debug log file name configurable

2017-06-01 Thread Lars Schneider
uent patch 'convert: add "status=delayed" to filter process protocol'. Signed-off-by: Lars Schneider <larsxschnei...@gmail.com> --- t/t0021-conversion.sh | 44 ++-- t/t0021/rot13-filter.pl | 8 +--- 2 files changed, 27 insertions(+),

[PATCH v5 1/5] t0021: keep filter log files on comparison

2017-06-01 Thread Lars Schneider
The filter log files are modified on comparison. Write the modified log files to temp files for comparison to fix this. This is useful for the subsequent patch 'convert: add "status=delayed" to filter process protocol'. Signed-off-by: Lars Schneider <larsxschnei...@gmail.com&

[PATCH v5 5/5] convert: add "status=delayed" to filter process protocol

2017-06-01 Thread Lars Schneider
ees.c) and `checkout` operations. Signed-off-by: Lars Schneider <larsxschnei...@gmail.com> --- Documentation/gitattributes.txt | 65 +- builtin/checkout.c | 3 + cache.h | 6 +- convert.c | 119 ++

[PATCH v5 0/5] convert: add "status=delayed" to filter process protocol

2017-06-01 Thread Lars Schneider
1107.20547-1-larsxschnei...@gmail.com/ v4: http://public-inbox.org/git/20170522135001.54506-1-larsxschnei...@gmail.com/ Base Ref: master Web-Diff: https://github.com/larsxschneider/git/commit/23001c0fe2 Checkout: git fetch https://github.com/larsxschneider/git filter-process/delay-v5 &&am

[PATCH v5 4/5] convert: move multiple file filter error handling to separate function

2017-06-01 Thread Lars Schneider
This is useful for the subsequent patch 'convert: add "status=delayed" to filter process protocol'. Signed-off-by: Lars Schneider <larsxschnei...@gmail.com> --- convert.c | 47 ++- 1 file changed, 26 insertions(+), 21 deletion

[PATCH v5 3/5] t0021: write "OUT" only on success

2017-06-01 Thread Lars Schneider
delayed" to filter process protocol'. Signed-off-by: Lars Schneider <larsxschnei...@gmail.com> --- t/t0021-conversion.sh | 6 +++--- t/t0021/rot13-filter.pl | 6 +++--- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/t/t0021-conversion.sh b/t/t0021-conversion.sh ind

Re: Unaligned accesses in sha1dc

2017-06-01 Thread Lars Schneider
> On 01 Jun 2017, at 10:28, Andreas Schwab wrote: > > The sh1dc implementation is making unaligned accesses, which will crash > on some architectures, others have to emulate them in software. Is SPARC an architecture that would run into this problem? I think there was a thread

RFC: Would a config fetch.retryCount make sense?

2017-06-01 Thread Lars Schneider
Hi, we occasionally see "The remote end hung up unexpectedly" (pkt-line.c:265) on our `git fetch` calls (most noticeably in our automations). I expect random network glitches to be the cause. In some places we added a basic retry mechanism and I was wondering if this could be a useful feature

"There are too many unreachable loose objects" - why don't we run 'git prune' automatically?

2017-06-08 Thread Lars Schneider
Hi, I recently ran into "There are too many unreachable loose objects; run 'git prune' to remove them." after a "Auto packing the repository in background for optimum performance." message. This was introduced with a087cc9 "git-gc --auto: protect ourselves from accumulated cruft" but I don't

Re: There are too many unreachable loose objects

2017-06-08 Thread Lars Schneider
> On 16 Feb 2017, at 23:57, Jeff King wrote: > > On Thu, Feb 16, 2017 at 02:36:10PM -0800, Jacob Keller wrote: > >>> Whenever I run "git push --force(-with-lease)" I get a variation of >>> >>> Counting objects: 187, done. >>> Delta compression using up to 12 threads. >>>

Re: Continous Integration (was: RE: Git v2.13.1 SHA1 very broken)

2017-06-06 Thread Lars Schneider
> On 06 Jun 2017, at 16:47, Jason Pyeron wrote: > > Do we have Jenkins (or something else) setup for Git? > > We would be happy to donate (slave) VMs for cygwin builds og Git. > > -Jason Pyeron > We use TravisCI for Linux, Mac, and (in a special way) Windows:

[PATCH v1] Configure Git contribution guidelines for github.com

2017-06-09 Thread Lars Schneider
the contribution guidelines for a project [3]. Let's make use of this! [1] https://github.com/git/git [2] https://github.com/git/git/pulls [3] https://help.github.com/articles/creating-a-pull-request-template-for-your-repository/ Signed-off-by: Lars Schneider <larsxschnei...@gmail.com> --- Hi,

Re: [PATCH v1] Configure Git contribution guidelines for github.com

2017-06-13 Thread Lars Schneider
> On 10 Jun 2017, at 03:51, Junio C Hamano <gits...@pobox.com> wrote: > > Jonathan Nieder <jrnie...@gmail.com> writes: > >> Lars Schneider wrote: >> >>> Many open source projects use github.com for their contribution process. >>> Although

[PATCH v2] Configure Git contribution guidelines for github.com

2017-06-13 Thread Lars Schneider
that is visible to every Pull Request creator can be configured with special files in a Git repository [2]. Let's make use of this! [1] https://github.com/git/git [2] https://help.github.com/articles/creating-a-pull-request-template-for-your-repository/ Signed-off-by: Lars Schneider <larsxsch

Re: "There are too many unreachable loose objects" - why don't we run 'git prune' automatically?

2017-06-09 Thread Lars Schneider
> On 09 Jun 2017, at 07:27, Jeff King <p...@peff.net> wrote: > > On Thu, Jun 08, 2017 at 02:45:48PM +0200, Lars Schneider wrote: > >> I recently ran into "There are too many unreachable loose objects; run >> 'git prune' to remove them." after a "

Re: There are too many unreachable loose objects

2017-06-09 Thread Lars Schneider
> On 09 Jun 2017, at 07:36, Jeff King <p...@peff.net> wrote: > > On Thu, Jun 08, 2017 at 02:52:42PM +0200, Lars Schneider wrote: > >>>> It may be that it's the server side that needs to git-prune, and not >>>> your local side? I'm not really certai

Re: [PATCH v1] Configure Git contribution guidelines for github.com

2017-06-09 Thread Lars Schneider
> On 09 Jun 2017, at 18:18, Ævar Arnfjörð Bjarmason <ava...@gmail.com> wrote: > > On Fri, Jun 9, 2017 at 4:21 PM, Lars Schneider <larsxschnei...@gmail.com> > wrote: >> Many open source projects use github.com for their contribution process. >> Althou

Re: What's cooking in git.git (Jun 2017, #04; Tue, 13)

2017-06-14 Thread Lars Schneider
> On 13 Jun 2017, at 23:40, Junio C Hamano wrote: > > > * ls/github (2017-06-13) 1 commit > - Configure Git contribution guidelines for github.com > > Help contributors that visit us at GitHub. > > Will merge to 'next'. I just pushed v3. If possible, please use it for

[PATCH v3] Configure Git contribution guidelines for github.com

2017-06-14 Thread Lars Schneider
that is visible to every Pull Request creator can be configured with special files in a Git repository [2]. Let's make use of this! [1] https://github.com/git/git [2] https://help.github.com/articles/creating-a-pull-request-template-for-your-repository/ Signed-off-by: Lars Schneider <larsxsch

Re: [PATCH v2] Configure Git contribution guidelines for github.com

2017-06-14 Thread Lars Schneider
> On 13 Jun 2017, at 11:04, Jeff King <p...@peff.net> wrote: > > On Tue, Jun 13, 2017 at 10:18:07AM +0200, Lars Schneider wrote: > >> changes since v1: >> * mention submitGit >> * link to mailing list address instead of mailing list archive > > You

Re: Restoring detached HEADs after Git operations

2017-06-19 Thread Lars Schneider
> On 19 Jun 2017, at 10:46, Patrick Lehmann wrote: > > Hello, > > I wrote a Bash script to recover branch names after Git operations have > create detached HEADs in a Git repository containing lots of Git submodules. > The script works recursively. I did run into

Re: [PATCH v2] Configure Git contribution guidelines for github.com

2017-06-14 Thread Lars Schneider
> On 14 Jun 2017, at 12:01, Junio C Hamano <gits...@pobox.com> wrote: > > Lars Schneider <larsxschnei...@gmail.com> writes: > >>> On 13 Jun 2017, at 11:04, Jeff King <p...@peff.net> wrote: >>> >>> On Tue, Jun 13, 2017 at 10:18:07A

Re: [PATCH] Fix KeyError "fileSize" in verbose mode

2017-06-13 Thread Lars Schneider
> On 12 Jun 2017, at 19:11, Junio C Hamano wrote: > > Sergey Yurzin writes: > >> Subject: Re: [PATCH] Fix KeyError "fileSize" in verbose mode > > ... > >> git-p4.py | 7 +-- >> 1 file changed, 5 insertions(+), 2 deletions(-) >> >> diff --git

Re: [PATCH v1] Configure Git contribution guidelines for github.com

2017-06-13 Thread Lars Schneider
> On 12 Jun 2017, at 17:52, Junio C Hamano <gits...@pobox.com> wrote: > > "Philip Oakley" <philipoak...@iee.org> writes: > >> From: "Lars Schneider" <larsxschnei...@gmail.com> >>> Many open source projects use github.com fo

Re: [PATCH v1] Configure Git contribution guidelines for github.com

2017-06-13 Thread Lars Schneider
> On 10 Jun 2017, at 14:48, Philip Oakley <philipoak...@iee.org> wrote: > > From: "Lars Schneider" <larsxschnei...@gmail.com> >> Many open source projects use github.com for their contribution process. >> Although we mirror the Git core repository t

Re: [PATCH v1] Configure Git contribution guidelines for github.com

2017-06-13 Thread Lars Schneider
> On 10 Jun 2017, at 09:35, Jeff King wrote: > > On Fri, Jun 09, 2017 at 10:03:57AM -0700, Jonathan Nieder wrote: > >> Would putting a PULL_REQUEST_TEMPLATE and CONTRIBUTING in the >> top-level directory work? If I'm reading >>

Re: "There are too many unreachable loose objects" - why don't we run 'git prune' automatically?

2017-06-18 Thread Lars Schneider
> On 10 Jun 2017, at 10:06, Jeff King <p...@peff.net> wrote: > > On Fri, Jun 09, 2017 at 02:03:18PM +0200, Lars Schneider wrote: > >>> I agree the existing message isn't great. There should probably be a big >>> advise() block explaining what's going on (a

Re: [PATCH v5 4/5] convert: move multiple file filter error handling to separate function

2017-06-18 Thread Lars Schneider
> On 18 Jun 2017, at 09:20, Torsten Bögershausen <tbo...@web.de> wrote: > > > On 2017-06-01 10:22, Lars Schneider wrote: >> This is useful for the subsequent patch 'convert: add "status=delayed" to >> filter process protocol'. > > May be &g

Re: Automating Coverity, was Re: [PATCH 00/26] Address a couple of issues identified by Coverity

2017-05-01 Thread Lars Schneider
> On 28 Apr 2017, at 22:29, Johannes Schindelin > wrote: > > Hi Stefan, > > On Fri, 28 Apr 2017, Stefan Beller wrote: > >> On Thu, Apr 27, 2017 at 3:50 PM, Johannes Schindelin >> wrote: >> >>> I still have to find the time to figure

Re: [PATCH v1] travis-ci: retry if Git for Windows CI returns HTTP error 502 or 503

2017-05-01 Thread Lars Schneider
> On 29 Apr 2017, at 20:48, Lars Schneider <larsxschnei...@gmail.com> wrote: > > The Git for Windows CI web app sometimes returns HTTP errors of > "502 bad gateway" or "503 service unavailable" [1]. Wait a little and > retry the request if this happens.

Re: [PATCH v1] travis-ci: retry if Git for Windows CI returns HTTP error 502 or 503

2017-05-02 Thread Lars Schneider
> On 02 May 2017, at 11:52, Junio C Hamano <gits...@pobox.com> wrote: > > On Mon, May 1, 2017 at 8:32 PM, Lars Schneider <larsxschnei...@gmail.com> > wrote: >>> >>> this should make the Git for Windows build a bit more stable. We saw >>> a few

[PATCH v1 0/2] run GETTEXT_POISON on TravisCI

2017-05-05 Thread Lars Schneider
status: 1 Base Ref: next Web-Diff: https://github.com/larsxschneider/git/commit/a835cd4775 Checkout: git fetch https://github.com/larsxschneider/git travisci/poison-v1 && git checkout a835cd4775 Lars Schneider (2): travis-ci: setup "prove cache" in "script"

[PATCH v1 1/2] travis-ci: setup "prove cache" in "script" step

2017-05-05 Thread Lars Schneider
quot;make test" command. Therefore, we should configure the "prove cache" in this step. This change is useful for a subsequent patch that adds a job which does not need the "before_install" step but wants to run the "script" step to execute the tests. Signed-off-by:

[PATCH v1 2/2] travis-ci: add job to run tests with GETTEXT_POISON

2017-05-05 Thread Lars Schneider
necessary dependencies) with an empty override. Signed-off-by: Lars Schneider <larsxschnei...@gmail.com> --- .travis.yml | 5 + 1 file changed, 5 insertions(+) diff --git a/.travis.yml b/.travis.yml index aa03f8eb82..278943d14a 100644 --- a/.travis.yml +++ b/.travis.yml @@ -39,6 +39,11 @

Re: git and the Clang Static Analyzer

2017-05-08 Thread Lars Schneider
> On 08 May 2017, at 12:12, Johannes Schindelin > wrote: > > Hi Dilyan, > > > On Sun, 7 May 2017, Дилян Палаузов wrote: > >> ... > > Also: to make it *really* useful for other developers, it would be a good > idea for you to try your hand at patching the

Script to rebase branches

2017-05-06 Thread Lars Schneider
Hi, I am about to write a bash/sh script that helps me to rebase a bunch of branches (e.g. select branches based on prefix, conflict resolution/ rerere support, ...). I wonder if anyone has such a script already and is willing to share it. Thanks, Lars

Re: [PATCH v3 4/4] convert: add "status=delayed" to filter process protocol

2017-05-21 Thread Lars Schneider
> On 19 Apr 2017, at 20:55, Torsten Bögershausen wrote: > > >>> (Back to the roots) >>> Which criteria do you have in mind: When should a filter process the blob >>> and return it immediately, and when would it respond "delayed" ? >> >> See above: it's up to the filter. In case

Re: git-scm.com status report

2017-05-18 Thread Lars Schneider
> On 17 May 2017, at 04:03, Jeff King wrote: > > On Tue, May 16, 2017 at 09:56:37PM -0400, Samuel Lijin wrote: > >> So I've finally found the time to get everything set up (in the >> process discovering that remote_genbook2 consistently induces a >> segfault in VirtualBox's

Re: [PATCH] t5313: make extended-table test more deterministic

2017-06-06 Thread Lars Schneider
t's make that very > explicit by just generating a second blob with a known sha1. Works for me! Thanks for the explanation! - Lars > > Reported-by: Lars Schneider <larsxschnei...@gmail.com> > Signed-off-by: Jeff King <p...@peff.net> > --- > t/t5313-pack-bounds-chec

[PATCH v7 6/6] convert: add "status=delayed" to filter process protocol

2017-06-27 Thread Lars Schneider
lob. Support delayed checkouts only in `clone` (in unpack-trees.c) and `checkout` operations for now. The optimization is most effective in these code paths as all files of the tree are processed. Signed-off-by: Lars Schneider <larsxschnei...@gmail.com> --- Documentation/gitattribut

[PATCH v7 2/6] t0021: make debug log file name configurable

2017-06-27 Thread Lars Schneider
uent patch 'convert: add "status=delayed" to filter process protocol'. Signed-off-by: Lars Schneider <larsxschnei...@gmail.com> --- t/t0021-conversion.sh | 44 ++-- t/t0021/rot13-filter.pl | 8 +--- 2 files changed, 27 insertions(+),

<    2   3   4   5   6   7   8   9   10   11   >