On Mon, May 1, 2017 at 7:35 AM, Junio C Hamano <gitster@xxxxxxxxx> wrote:
> * ab/clone-no-tags (2017-05-01) 3 commits
> (merged to 'next' on 2017-04-30 at 601649896a)
> + tests: rename a test having to do with shallow submodules
> + clone: add a --no-tags option to clone without tags
> + tests: change "cd ... && git fetch" to "cd &&\n\tgit fetch"
>
> "git clone" learned the "--no-tags" option not to fetch all tags
> initially, and also set up the tagopt not to follow any tags in
> subsequent fetches.
>
> Will cook in 'next'.
Thanks for trimming off the top 2 patches. I've dropped those myself,
if someone (Brandon || Stefan) more interested in working on
submodules wants to pick them up that would be neat, but I don't need
it myself & doing it differently than the existing submodule options
would take too much of my time.
> * ab/grep-pcre-v2 (2017-04-25) 20 commits
> - SQUASH???
> - Makefile & configure: make PCRE v2 the default PCRE implementation
> - grep: remove support for concurrent use of both PCRE v1 & v2
> - grep: add support for PCRE v2
> - grep: add support for the PCRE v1 JIT API
> - perf: add a performance comparison test of grep -E and -P
> - grep: change the internal PCRE code & header names to be PCRE1
> - grep: change the internal PCRE macro names to be PCRE1
> - test-lib: rename the LIBPCRE prerequisite to PCRE
> - grep: make grep.patternType=[pcre|pcre1] a synonym for "perl"
> - grep & rev-list doc: stop promising libpcre for --perl-regexp
> - log: add -P as a synonym for --perl-regexp
> - log: add exhaustive tests for pattern style options & config
> - grep: add a test for backreferences in PCRE patterns
> - Makefile & configure: reword outdated comment about PCRE
> - grep: remove redundant `regflags &= ~REG_EXTENDED` assignments
> - grep: remove redundant regflags assignment under PCRE
> - grep: submodule-related case statements should die if new fields are added
> - grep: add tests for grep pattern types being passed to submodules
> - grep: amend submodule recursion test in preparation for rx engine testing
>
> PCRE2, which has an API different from and incompatible with PCRE,
> can now be chosen to support "grep -P -e '<pattern>'" and friends.
That squash looks good to me.
> * ab/grep-threading-cleanup (2017-04-16) 8 commits
> - grep: given --threads with NO_PTHREADS=YesPlease, warn
> - pack-objects: fix buggy warning about threads under NO_PTHREADS=YesPlease
> - pack-object & index-pack: add test for --threads warning under NO_PTHREADS
> - tests: add a PTHREADS prerequisite
> - grep: skip pthreads overhead when using one thread
> - grep: don't redundantly compile throwaway patterns under threading
> - grep: add tests for --threads=N and grep.threads
> - grep: assert that threading is enabled when calling grep_{lock,unlock}
>
> Code cleanup.
>
> Needs review.
Between these two series there's 27 patches, and I understand it's a
bit of a PITA to review/get comments on it.
Anything I should be doing differently here other than just waiting
for 2.13 to come out so they can be cooked further & merged down to
next & then master if there's no objections?