* What's cooking in git.git (Oct 2019, #01; Thu, 3)
@ 2019-10-03 5:04 Junio C Hamano
2019-10-03 18:28 ` Emily Shaffer
` (2 more replies)
0 siblings, 3 replies; 9+ messages in thread
From: Junio C Hamano @ 2019-10-03 5:04 UTC (permalink / raw)
To: git
Here are the topics that have been cooking. Commits prefixed with
'-' are only in 'pu' (proposed updates) while commits prefixed with
'+' are in 'next'. The ones marked with '.' do not appear in any of
the integration branches, but I am still holding onto them.
You can find the changes described here in the integration branches
of the repositories listed at
http://git-blame.blogspot.com/p/git-public-repositories.html
--------------------------------------------------
[New Topics]
* ag/sequencer-todo-updates (2019-10-02) 5 commits
- sequencer: directly call pick_commits() from complete_action()
- rebase: fill `squash_onto' in get_replay_opts()
- sequencer: move the code writing total_nr on the disk to a new function
- sequencer: update `done_nr' when skipping commands in a todo list
- sequencer: update `total_nr' when adding an item to a todo list
Expecting a reroll.
cf. <nycvar.QRO.7.76.6.1910021018360.46@tvgsbejvaqbjf.bet> to clarify log messages.
there may be others.
* ah/cleanups (2019-10-03) 4 commits
- git_mkstemps_mode(): replace magic numbers with computed value
- wrapper: use a loop instead of repetitive statements
- diffcore-break: use a goto instead of a redundant if statement
- commit-graph: remove a duplicate assignment
Miscellaneous code clean-ups.
Will merge to 'next'.
* as/shallow-slab-use-fix (2019-10-02) 1 commit
- shallow.c: don't free unallocated slabs
Correct code that tried to reference all entries in a sparse array
of pointers by mistake.
Will merge to 'next'.
* js/fetch-jobs (2019-10-02) 1 commit
- fetch: let --jobs=<n> parallelize --multiple, too
"git fetch --jobs=<n>" allowed <n> parallel jobs when fetching
submodules, but this did not apply to "git fetch --multiple" that
fetches from multiple remote repositories. It now does.
Will merge to 'next'.
* js/mingw-spawn-with-spaces-in-path (2019-10-02) 1 commit
- t0061: fix test for argv[0] with spaces (MINGW only)
Test fix.
Will merge to 'next'.
* gs/commit-graph-trace-with-cmd (2019-10-02) 1 commit
- commit-graph: emit trace2 cmd_mode for each sub-command
Dev support.
Will merge to 'next'.
* js/trace2-fetch-push (2019-10-03) 2 commits
- push: add trace2 instrumentation
- fetch: add trace2 instrumentation
Dev support.
Will merge to 'next'.
* js/range-diff-noprefix (2019-10-03) 1 commit
- range-diff: internally force `diff.noprefix=true`
"git range-diff" segfaulted when diff.noprefix configuration was
used, as it blindly expected the patch it internally generates to
have the standard a/ and b/ prefixes. The command now forces the
internal patch to be built without any prefix, not to be affected
by any end-user configuration.
Will merge to 'next'.
--------------------------------------------------
[Graduated to "master"]
* ar/mingw-run-external-with-non-ascii-path (2019-08-26) 1 commit
(merged to 'next' on 2019-09-09 at 1973826628)
+ mingw: fix launching of externals from Unicode paths
Windows update.
* bc/hash-independent-tests-part-5 (2019-08-26) 14 commits
(merged to 'next' on 2019-09-09 at 49defc07af)
+ t4009: make hash size independent
+ t4002: make hash independent
+ t4000: make hash size independent
+ t3903: abstract away SHA-1-specific constants
+ t3800: make hash-size independent
+ t3600: make hash size independent
+ t3506: make hash independent
+ t3430: avoid hard-coded object IDs
+ t3404: abstract away SHA-1-specific constants
+ t3306: abstract away SHA-1-specific constants
+ t3305: make hash size independent
+ t3301: abstract away SHA-1-specific constants
+ t3206: abstract away hash size constants
+ t3201: abstract away SHA-1-specific constants
Preparation for SHA-256 upgrade continues in the test department.
* bw/rebase-autostash-keep-current-branch (2019-09-07) 2 commits
(merged to 'next' on 2019-09-18 at 6d8ed846ed)
+ builtin/rebase.c: Remove pointless message
+ builtin/rebase.c: make sure the active branch isn't moved when autostashing
"git rebase --autostash <upstream> <branch>", when <branch> is
different from the current branch, incorrectly moved the tip of the
current branch, which has been corrected.
* dl/compat-cleanup (2019-09-05) 2 commits
(merged to 'next' on 2019-09-09 at e33c896e16)
+ compat/*.[ch]: remove extern from function declarations using spatch
+ mingw: apply array.cocci rule
(this branch is used by dl/cocci-everywhere.)
Code cleanup.
* dl/complete-cherry-pick-revert-skip (2019-08-27) 3 commits
(merged to 'next' on 2019-09-18 at 7dd173f824)
+ status: mention --skip for revert and cherry-pick
+ completion: add --skip for cherry-pick and revert
+ completion: merge options for cherry-pick and revert
The command line completion support (in contrib/) learned about the
"--skip" option of "git revert" and "git cherry-pick".
* dl/format-patch-doc-test-cleanup (2019-09-05) 13 commits
(merged to 'next' on 2019-09-09 at b3022b4002)
+ config/format.txt: specify default value of format.coverLetter
+ Doc: add more detail for git-format-patch
+ t4014: stop losing return codes of git commands
+ t4014: remove confusing pipe in check_threading()
+ t4014: use test_line_count() where possible
+ t4014: let sed open its own files
+ t4014: drop redirections to /dev/null
+ t4014: use indentable here-docs
+ t4014: remove spaces after redirect operators
+ t4014: use sq for test case names
+ t4014: move closing sq onto its own line
+ t4014: s/expected/expect/
+ t4014: drop unnecessary blank lines from test cases
(this branch is used by dl/format-patch-cover-letter-subject.)
The documentation and tests for "git format-patch" have been
cleaned up.
* dl/rebase-i-keep-base (2019-08-27) 9 commits
(merged to 'next' on 2019-09-18 at f94f486724)
+ rebase: teach rebase --keep-base
+ rebase tests: test linear branch topology
+ rebase: fast-forward --fork-point in more cases
+ rebase: fast-forward --onto in more cases
+ rebase: refactor can_fast_forward into goto tower
+ t3432: test for --no-ff's interaction with fast-forward
+ t3432: distinguish "noop-same" v.s. "work-same" in "same head" tests
+ t3432: test rebase fast-forward behavior
+ t3431: add rebase --fork-point tests
"git rebase --keep-base <upstream>" tries to find the original base
of the topic being rebased and rebase on top of that same base,
which is useful when running the "git rebase -i" (and its limited
variant "git rebase -x").
The command also has learned to fast-forward in more cases where it
can instead of replaying to recreate identical commits.
* dl/use-sq-from-test-lib (2019-09-06) 1 commit
(merged to 'next' on 2019-09-09 at 9a6619ddab)
+ t: use common $SQ variable
Code cleanup.
* ds/commit-graph-on-fetch (2019-09-03) 1 commit
(merged to 'next' on 2019-09-18 at 10702dd989)
+ fetch: add fetch.writeCommitGraph config setting
A configuration variable tells "git fetch" to write the commit
graph after finishing.
* ds/include-exclude (2019-09-05) 5 commits
(merged to 'next' on 2019-09-18 at 1a3655a089)
+ unpack-trees: rename 'is_excluded_from_list()'
+ treewide: rename 'exclude' methods to 'pattern'
+ treewide: rename 'EXCL_FLAG_' to 'PATTERN_FLAG_'
+ treewide: rename 'struct exclude_list' to 'struct pattern_list'
+ treewide: rename 'struct exclude' to 'struct path_pattern'
The internal code originally invented for ".gitignore" processing
got reshuffled and renamed to make it less tied to "excluding" and
stress more that it is about "matching", as it has been reused for
things like sparse checkout specification that want to check if a
path is "included".
* dt/remote-helper-doc-re-lock-option (2019-08-30) 1 commit
(merged to 'next' on 2019-09-09 at 970488992a)
+ clarify documentation for remote helpers
Doc update.
* en/filter-branch-deprecation (2019-09-05) 3 commits
(merged to 'next' on 2019-09-09 at 311ed09ba8)
+ t9902: use a non-deprecated command for testing
+ Recommend git-filter-repo instead of git-filter-branch
+ t6006: simplify, fix, and optimize empty message test
Start discouraging the use of "git filter-branch".
* en/merge-options-ff-and-friends (2019-09-01) 1 commit
(merged to 'next' on 2019-09-09 at b3bcc2cbc7)
+ merge-options.txt: clarify meaning of various ff-related options
Doc update.
* jc/test-cleanup (2019-09-05) 2 commits
(merged to 'next' on 2019-09-09 at 41ced23598)
+ t3005: remove unused variable
+ t: use LF variable defined in the test harness
Code cleanup.
* jh/trace2-pretty-output (2019-08-09) 7 commits
(merged to 'next' on 2019-09-18 at 503974aa46)
+ trace2: cleanup whitespace in perf format
+ trace2: cleanup whitespace in normal format
+ quote: add sq_append_quote_argv_pretty()
+ trace2: trim trailing whitespace in normal format error message
+ trace2: remove dead code in maybe_add_string_va()
+ trace2: trim whitespace in region messages in perf target format
+ trace2: cleanup column alignment in perf target format
Output from trace2 subsystem is formatted more prettily now.
* jk/fast-import-history-bugfix (2019-08-27) 2 commits
(merged to 'next' on 2019-09-09 at eeef864077)
+ fast-import: duplicate into history rather than passing ownership
+ fast-import: duplicate parsed encoding string
The memory ownership model of the "git fast-import" got
straightened out.
* jk/misc-uninitialized-fixes (2019-09-06) 6 commits
(merged to 'next' on 2019-09-09 at 39ccd21475)
+ pack-objects: drop packlist index_pos optimization
+ test-read-cache: drop namelen variable
+ diff-delta: set size out-parameter to 0 for NULL delta
+ bulk-checkin: zero-initialize hashfile_checkpoint
+ pack-objects: use object_id in packlist_alloc()
+ git-am: handle missing "author" when parsing commit
Various fixes to codepaths gcc 9 had trouble following dataflow.
* js/gitdir-at-unc-root (2019-08-26) 3 commits
(merged to 'next' on 2019-09-09 at 70ccc59b4b)
+ setup_git_directory(): handle UNC root paths correctly
+ Fix .git/ discovery at the root of UNC shares
+ setup_git_directory(): handle UNC paths correctly
On Windows, the root level of UNC share is now allowed to be used
just like any other directory.
* js/visual-studio (2019-09-05) 1 commit
(merged to 'next' on 2019-09-09 at 5493ee9474)
+ .gitignore: stop ignoring `.manifest` files
Adjust .gitignore to unignore a path that we started to track.
* mh/http-urlmatch-cleanup (2019-08-26) 1 commit
(merged to 'next' on 2019-09-09 at 9971b626f1)
+ http: don't leak urlmatch_config.vars
Leakfix.
* mh/notes-duplicate-entries (2019-08-26) 2 commits
(merged to 'next' on 2019-09-09 at 6c44967c97)
+ notes: avoid potential use-after-free during insertion
+ notes: avoid leaking duplicate entries
A few implementation fixes in the notes API.
* mh/release-commit-memory-fix (2019-08-26) 1 commit
(merged to 'next' on 2019-09-09 at 9fb7485032)
+ commit: free the right buffer in release_commit_memory
Leakfix.
* nd/switch-and-restore (2019-08-30) 1 commit
(merged to 'next' on 2019-09-09 at 97e7e06801)
+ checkout: add simple check for 'git checkout -b'
Resurrect a performance hack.
* rs/get-tagged-oid (2019-09-05) 2 commits
(merged to 'next' on 2019-09-09 at 0e9489a835)
+ use get_tagged_oid()
+ tag: factor out get_tagged_oid()
Code cleanup.
* rs/help-unknown-ref-does-not-return (2019-08-30) 1 commit
(merged to 'next' on 2019-09-09 at 4b00ea1ac9)
+ help: make help_unknown_ref() NORETURN
Code cleanup.
* rs/parse-tree-indirect (2019-09-03) 1 commit
(merged to 'next' on 2019-09-09 at 157e836dba)
+ tree: simplify parse_tree_indirect()
Code cleanup.
* rs/strbuf-detach (2019-08-26) 2 commits
(merged to 'next' on 2019-09-09 at f6bba7d52f)
+ grep: use return value of strbuf_detach()
+ log-tree: always use return value of strbuf_detach()
Straighten out the use of strbuf_detach() API function.
* rs/trace2-dst-warning (2019-08-26) 1 commit
(merged to 'next' on 2019-09-09 at 13893cdf8c)
+ trace2: use warning() directly in tr2_dst_malformed_warning()
Code cleanup.
* sg/clean-nested-repo-with-ignored (2019-08-26) 1 commit
(merged to 'next' on 2019-09-18 at 0da1269f07)
+ t7300-clean: demonstrate deleting nested repo with an ignored file breakage
(this branch is used by en/clean-nested-with-ignored.)
A bug documentation.
* sg/git-test-boolean (2019-09-06) 2 commits
(merged to 'next' on 2019-09-09 at 90406d6375)
+ ci: restore running httpd tests
+ t/lib-git-svn.sh: check GIT_TEST_SVN_HTTPD when running SVN HTTP tests
Fix an earlier regression in the test suite, which mistakenly
stopped running HTTPD tests.
* tb/banned-vsprintf-namefix (2019-08-26) 1 commit
(merged to 'next' on 2019-09-09 at e0f6d18a10)
+ banned.h: fix vsprintf()'s ban message
Error message fix.
* tb/file-url-to-unc-path (2019-08-26) 1 commit
(merged to 'next' on 2019-09-09 at 13d9c4000e)
+ mingw: support UNC in git clone file://server/share/repo
Windows update.
* tg/push-all-in-mirror-forbidden (2019-09-03) 1 commit
(merged to 'next' on 2019-09-09 at 9db61400c8)
+ push: disallow --all and refspecs when remote.<name>.mirror is set
Fix an earlier regression to "git push --all" which should have
been forbidden when the target remote repository is set to be a
mirror.
--------------------------------------------------
[Stalled]
* cb/pcre2-chartables-leakfix (2019-08-06) 3 commits
- grep: avoid leak of chartables in PCRE2
- grep: make PCRE2 aware of custom allocator
- grep: make PCRE1 aware of custom allocator
WIP (this is v3 which is rather old, where the last message in the
discussion talks about v6 that has never been sent to the list).
cf. <CAPUEspjJNSrJQT7xV2fsdp2t5odW5fzzPdDxuar_5x_JPUtf6Q@mail.gmail.com>
* es/walken-tutorial (2019-08-07) 1 commit
- documentation: add tutorial for revision walking
A tutorial on object enumeration.
Perhaps take the thing with as little tweaks as possible, retitling
it to "my first object enumeration" or something?
cf. <20190814183328.GA40797@google.com>
* dl/format-patch-cover-letter-subject (2019-09-05) 1 commit
- format-patch: learn --infer-cover-subject option
"git format-patch --cover-letter" learned to optionally use the
first paragraph (typically a single-liner) of branch.*.description
as the subject of the cover letter.
Reroll with a redesign with less emphasis on "subject" coming?
* jn/unknown-index-extensions (2018-11-21) 2 commits
- index: offer advice for unknown index extensions
- index: do not warn about unrecognized extensions
A bit too alarming warning given when unknown index extensions
exist is getting revamped.
Expecting a reroll.
* jc/format-patch-delay-message-id (2019-04-05) 1 commit
- format-patch: move message-id and related headers to the end
The location "git format-patch --thread" adds the Message-Id:
header in the series of header fields has been moved down, which
may help working around a suspected bug in GMail MSA, reported at
<CAHk-=whP1stFZNAaJiMi5eZ9rj0MRt20Y_yHVczZPH+O01d+sA@mail.gmail.com>
Waiting for feedback to see if it truly helps.
Needs tests.
* jt/fetch-cdn-offload (2019-03-12) 9 commits
- SQUASH???
- upload-pack: send part of packfile response as uri
- fetch-pack: support more than one pack lockfile
- upload-pack: refactor reading of pack-objects out
- Documentation: add Packfile URIs design doc
- Documentation: order protocol v2 sections
- http-fetch: support fetching packfiles by URL
- http: improve documentation of http_pack_request
- http: use --stdin when getting dumb HTTP pack
WIP for allowing a response to "git fetch" to instruct the bulk of
the pack contents to be instead taken from elsewhere (aka CDN).
* js/protocol-advertise-multi (2018-12-28) 1 commit
- protocol: advertise multiple supported versions
The transport layer has been updated so that the protocol version
used can be negotiated between the parties, by the initiator
listing the protocol versions it is willing to talk, and the other
side choosing from one of them.
Expecting a reroll.
cf. <CANq=j3u-zdb_FvNJGPCmygNMScseav63GhVvBX3NcVS4f7TejA@mail.gmail.com>
* mk/use-size-t-in-zlib (2018-10-15) 1 commit
- zlib.c: use size_t for size
The wrapper to call into zlib followed our long tradition to use
"unsigned long" for sizes of regions in memory, which have been
updated to use "size_t".
--------------------------------------------------
[Cooking]
* mt/threaded-grep-in-object-store (2019-10-02) 11 commits
- grep: move driver pre-load out of critical section
- grep: re-enable threads in non-worktree case
- grep: protect packed_git [re-]initialization
- grep: allow submodule functions to run in parallel
- submodule-config: add skip_if_read option to repo_read_gitmodules()
- grep: replace grep_read_mutex by internal obj read lock
- object-store: allow threaded access to object reading
- replace-object: make replace operations thread-safe
- grep: fix racy calls in grep_objects()
- grep: fix race conditions at grep_submodule()
- grep: fix race conditions on userdiff calls
Traditionally, we avoided threaded grep while searching in objects
(as opposed to files in the working tree) as accesses to the object
layer is not thread-safe. This limitation is getting lifted.
* jt/merge-recursive-symlink-is-not-a-dir-in-way (2019-09-20) 1 commit
(merged to 'next' on 2019-09-30 at a5d6788e2e)
+ merge-recursive: symlink's descendants not in way
A bug in merge-recursive code that triggers when a branch with a
symbolic link is merged with a branch that replaces it with a
directory has been fixed.
Will merge to 'master'.
* ps/my-first-contribution-alphasort (2019-09-28) 1 commit
(merged to 'next' on 2019-09-30 at 729e6dc708)
+ doc: MyFirstContribution: fix cmd placement instructions
Docfix.
Will merge to 'master'.
* rs/alias-use-copy-array (2019-09-20) 1 commit
(merged to 'next' on 2019-09-30 at 4d90f4ba93)
+ git: use COPY_ARRAY and MOVE_ARRAY in handle_alias()
Code cleanup.
Will merge to 'master'.
* sg/t-helper-gitignore (2019-09-20) 1 commit
(merged to 'next' on 2019-09-30 at 8e319a2eae)
+ t/helper: ignore only executable files
Update the way build artifacts in t/helper/ directory are ignored.
Will merge to 'master'.
* sg/travis-help-debug (2019-09-28) 1 commit
(merged to 'next' on 2019-09-30 at 054a66bb75)
+ travis-ci: do not skip successfully tested trees in debug mode
Dev support update.
Will merge to 'master'.
* am/mailmap-andrey-mazo (2019-09-20) 1 commit
(merged to 'next' on 2019-09-30 at 5e373b3cba)
+ .mailmap: update email address of Andrey Mazo
Will merge to 'master'.
* am/t0028-utf16-tests (2019-09-28) 2 commits
- t0028: add more tests
- t0028: fix test for UTF-16-LE-BOM
Test fixes.
* am/visual-studio-config-fix (2019-09-28) 1 commit
- contrib/buildsystems: fix Visual Studio Debug configuration
Dev support.
Will merge to 'next'.
* cb/do-not-use-test-cmp-with-a (2019-09-28) 1 commit
(merged to 'next' on 2019-09-30 at 273ff0e62d)
+ t4038: Remove non-portable '-a' option passed to test_cmp
Test portability fix.
Will merge to 'master'.
* cc/multi-promisor (2019-10-02) 2 commits
(merged to 'next' on 2019-10-03 at a0a8ea56a9)
+ promisor-remote: skip move_to_tail when no-op
(merged to 'next' on 2019-09-30 at c4826f5a08)
+ promisor-remote.h: drop extern from function declaration
Cleanup.
Will merge to 'master'.
* dl/honor-cflags-in-hdr-check (2019-10-03) 5 commits
(merged to 'next' on 2019-10-03 at a346b79a78)
+ ci: run `hdr-check` as part of the `Static Analysis` job
(merged to 'next' on 2019-09-30 at 708fb8988e)
+ Makefile: emulate compile in $(HCO) target better
+ pack-bitmap.h: remove magic number
+ promisor-remote.h: include missing header
+ apply.h: include missing header
Dev support.
Will merge to 'master'.
* en/fast-imexport-nested-tags (2019-10-02) 8 commits
- fast-export: handle nested tags
- t9350: add tests for tags of things other than a commit
- fast-export: allow user to request tags be marked with --mark-tags
- fast-export: add support for --import-marks-if-exists
- fast-import: add support for new 'alias' command
- fast-import: allow tags to be identified by mark labels
- fast-import: fix handling of deleted tags
- fast-export: fix exporting a tag and nothing else
Updates to fast-import/export.
Will merge to 'next'.
* js/diff-rename-force-stable-sort (2019-10-02) 2 commits
- diffcore_rename(): use a stable sort
- Move git_sort(), a stable sort, into into libgit.a
The rename detection logic sorts a list of rename source candidates
by similarity to pick the best candidate, which means that a tie
between sources with the same similarity is broken by the original
location in the original canidate list (which is sorted by path).
Force the sorting by similarity done with a stable sort, which is
not promised by system supplied qsort(3), to ensure consistent
results across platforms.
Will merge to 'next'.
* ma/user-manual-markup-update (2019-09-28) 4 commits
(merged to 'next' on 2019-09-30 at 822fa2ed99)
+ user-manual.txt: render ASCII art correctly under Asciidoctor
+ asciidoctor-extensions.rb: handle "book" doctype in linkgit
+ user-manual.txt: change header notation
+ user-manual.txt: add missing section label
The markup used in user-manual has been updated to work better with
asciidoctor.
Will merge to 'master'.
* sg/name-rev-cutoff-underflow-fix (2019-09-28) 1 commit
- name-rev: avoid cutoff timestamp underflow
Integer arith fix.
Will merge to 'next'.
* tk/git-svn-trim-author-name (2019-09-28) 1 commit
- git-svn: trim leading and trailing whitespaces in author name
The author names taken from SVN repositories may have extra leading
or trailing whitespaces, which are now munged away.
Will merge to 'next'.
* bw/submodule-helper-usage-fix (2019-09-29) 1 commit
(merged to 'next' on 2019-09-30 at 78d2f28ef7)
+ builtin/submodule--helper: fix usage string for 'update-clone'
Typofix.
Will merge to 'master'.
* bc/object-id-part17 (2019-08-19) 26 commits
- midx: switch to using the_hash_algo
- builtin/show-index: replace sha1_to_hex
- rerere: replace sha1_to_hex
- builtin/receive-pack: replace sha1_to_hex
- builtin/index-pack: replace sha1_to_hex
- packfile: replace sha1_to_hex
- wt-status: convert struct wt_status to object_id
- cache: remove null_sha1
- builtin/worktree: switch null_sha1 to null_oid
- builtin/repack: write object IDs of the proper length
- pack-write: use hash_to_hex when writing checksums
- sequencer: convert to use the_hash_algo
- bisect: switch to using the_hash_algo
- sha1-lookup: switch hard-coded constants to the_hash_algo
- config: use the_hash_algo in abbrev comparison
- combine-diff: replace GIT_SHA1_HEXSZ with the_hash_algo
- bundle: switch to use the_hash_algo
- connected: switch GIT_SHA1_HEXSZ to the_hash_algo
- show-index: switch hard-coded constants to the_hash_algo
- blame: remove needless comparison with GIT_SHA1_HEXSZ
- builtin/rev-parse: switch to use the_hash_algo
- builtin/blame: switch uses of GIT_SHA1_HEXSZ to the_hash_algo
- builtin/receive-pack: switch to use the_hash_algo
- fetch-pack: use parse_oid_hex
- patch-id: convert to use the_hash_algo
- builtin/replace: make hash size independent
Preparation for SHA-256 upgrade continues.
Will merge to 'next'.
* cb/skip-utf8-check-with-pcre1 (2019-09-09) 1 commit
(merged to 'next' on 2019-09-30 at f6113b33cf)
+ grep: skip UTF8 checks explicitly
Make sure the grep machinery does not abort when seeing a payload
that is not UTF-8 even when JIT is not in use with PCRE1.
Will merge to 'master'.
* jk/commit-graph-cleanup (2019-09-09) 2 commits
(merged to 'next' on 2019-09-30 at f78e24d14b)
+ commit-graph: turn off save_commit_buffer
+ commit-graph: don't show progress percentages while expanding reachable commits
A pair of small fixups to "git commit-graph" have been applied.
Will merge to 'master'.
* ma/asciidoctor-more-fixes (2019-09-09) 7 commits
(merged to 'next' on 2019-09-30 at 4937cac46f)
+ gitweb.conf.txt: switch pluses to backticks to help Asciidoctor
+ git-merge-index.txt: wrap shell listing in "----"
+ git-receive-pack.txt: wrap shell [script] listing in "----"
+ git-ls-remote.txt: wrap shell listing in "----"
+ Documentation: wrap config listings in "----"
+ git-merge-base.txt: render indentations correctly under Asciidoctor
+ Documentation: wrap blocks with "--"
Doc formatting updates.
Will merge to 'master'.
* rs/simplify-by-deco-with-deco-refs-exclude (2019-09-09) 2 commits
(merged to 'next' on 2019-09-30 at 3c155bbd24)
+ log-tree: call load_ref_decorations() in get_name_decoration()
+ log: test --decorate-refs-exclude with --simplify-by-decoration
"git log --decorate-refs-exclude=<pattern>" was incorrectly
overruled when the "--simplify-by-decoration" option is used, which
has been corrected.
Will merge to 'master'.
* ss/get-time-cleanup (2019-09-18) 2 commits
(merged to 'next' on 2019-09-30 at 21a0dced8f)
+ test_date.c: remove reference to GIT_TEST_DATE_NOW
+ Quit passing 'now' to date code
Code simplification.
Will merge to 'master'.
* tb/commit-graph-harden (2019-09-09) 3 commits
(merged to 'next' on 2019-09-30 at b9350a562d)
+ commit-graph.c: handle corrupt/missing trees
+ commit-graph.c: handle commit parsing errors
+ t/t5318: introduce failing 'git commit-graph write' tests
The code to parse and use the commit-graph file has been made more
robust against corrupted input.
Will merge to 'master'.
* cs/pretty-formats-doc-typofix (2019-09-12) 1 commit
(merged to 'next' on 2019-09-30 at a29a425208)
+ doc: minor formatting fix
Doc fix.
Will merge to 'master'.
* dl/cocci-everywhere (2019-09-17) 4 commits
(merged to 'next' on 2019-09-30 at 1bb1c0846f)
+ Makefile: run coccicheck on more source files
+ Makefile: strip leading ./ in $(FIND_SOURCE_FILES)
+ Makefile: define THIRD_PARTY_SOURCES
+ Makefile: strip leading ./ in $(LIB_H)
Coccinelle checks are done on more source files than before now.
Will merge to 'master'.
* dl/complete-rebase-and-archive (2019-09-12) 2 commits
(merged to 'next' on 2019-09-30 at ddeb81ff41)
+ completion: teach archive to use __gitcomp_builtin
+ completion: teach rebase to use __gitcomp_builtin
The command line completion for "git archive" and "git rebase" are
now made less prone to go out of sync with the binary.
Will merge to 'master'.
* gs/commit-graph-progress (2019-09-18) 1 commit
(merged to 'next' on 2019-09-30 at 7c03eac969)
+ commit-graph: add --[no-]progress to write and verify
Will merge to 'master'.
* jk/disable-commit-graph-during-upload-pack (2019-09-12) 2 commits
(merged to 'next' on 2019-09-30 at 87dae76fb7)
+ upload-pack: disable commit graph more gently for shallow traversal
+ commit-graph: bump DIE_ON_LOAD check to actual load-time
The "upload-pack" (the counterpart of "git fetch") needs to disable
commit-graph when responding to a shallow clone/fetch request, but
the way this was done made Git panic, which has been corrected.
Will merge to 'master'.
* jk/list-objects-optim-wo-trees (2019-09-12) 1 commit
(merged to 'next' on 2019-09-30 at 9ef098d07f)
+ list-objects: don't queue root trees unless revs->tree_objects is set
The object traversal machinery has been optimized not to load tree
objects when we are only interested in commit history.
Will merge to 'master'.
* mr/complete-more-for-log-etc (2019-09-12) 1 commit
(merged to 'next' on 2019-09-30 at b2507b21cb)
+ completion: add missing completions for log, diff, show
Completion updates.
Will merge to 'master'.
* ah/doc-submodule-ignore-submodules (2019-09-18) 1 commit
(merged to 'next' on 2019-09-30 at 9eb3de8d1b)
+ doc: fix reference to --ignore-submodules
Docfix.
Will merge to 'master'.
* bc/doc-use-docbook-5 (2019-09-16) 1 commit
(merged to 'next' on 2019-09-30 at 31c4cf70ae)
+ Documentation: fix build with Asciidoctor 2
Start using DocBook 5 (instead of DocBook 4.5) as Asciidoctor 2.0
no longer works with the older one.
Will merge to 'master'.
* cc/svn-fe-py-shebang (2019-09-18) 1 commit
(merged to 'next' on 2019-09-30 at 27c8457202)
+ contrib/svn-fe: fix shebang for svnrdump_sim.py
Will merge to 'master'.
* dl/submodule-set-branch (2019-09-16) 1 commit
(merged to 'next' on 2019-09-30 at c66b398cc1)
+ git-submodule.txt: fix AsciiDoc formatting error
Docfix.
Will merge to 'master'.
* en/clean-nested-with-ignored (2019-10-02) 13 commits
(merged to 'next' on 2019-10-03 at 969ec06cc7)
+ dir: special case check for the possibility that pathspec is NULL
(merged to 'next' on 2019-09-30 at 778cc31eac)
+ clean: fix theoretical path corruption
+ clean: rewrap overly long line
+ clean: avoid removing untracked files in a nested git repository
+ clean: disambiguate the definition of -d
+ git-clean.txt: do not claim we will delete files with -n/--dry-run
+ dir: add commentary explaining match_pathspec_item's return value
+ dir: if our pathspec might match files under a dir, recurse into it
+ dir: make the DO_MATCH_SUBMODULE code reusable for a non-submodule case
+ dir: also check directories for matching pathspecs
+ dir: fix off-by-one error in match_pathspec_item
+ dir: fix typo in comment
+ t7300: add testcases showing failure to clean specified pathspecs
"git clean" fixes.
Will merge to 'master'.
* hb/hg-to-git-py3 (2019-09-18) 1 commit
(merged to 'next' on 2019-09-30 at 28f7e9bae3)
+ hg-to-git: make it compatible with both python3 and python2
The hg-to-git script (in contrib/) has been updated to work with
Python 3.
Will merge to 'master'.
* jk/packfile-reuse-cleanup (2019-09-13) 10 commits
(merged to 'next' on 2019-09-30 at dc60b31833)
+ pack-objects: improve partial packfile reuse
+ builtin/pack-objects: introduce obj_is_packed()
+ pack-objects: introduce pack.allowPackReuse
+ csum-file: introduce hashfile_total()
+ pack-bitmap: introduce bitmap_walk_contains()
+ pack-bitmap: don't rely on bitmap_git->reuse_objects
+ ewah/bitmap: always allocate 2 more words
+ ewah/bitmap: introduce bitmap_word_alloc()
+ packfile: expose get_delta_base()
+ builtin/pack-objects: report reused packfile objects
The way "git pack-objects" reuses objects stored in existing pack
to generate its result has been improved.
On hold until we give it a more thorough review.
cf. <20191002155721.GD6116@sigill.intra.peff.net>
* jk/partial-clone-sparse-blob (2019-09-16) 4 commits
(merged to 'next' on 2019-09-30 at 44b33488ea)
+ list-objects-filter: use empty string instead of NULL for sparse "base"
+ list-objects-filter: give a more specific error sparse parsing error
+ list-objects-filter: delay parsing of sparse oid
+ t5616: test cloning/fetching with sparse:oid=<oid> filter
The name of the blob object that stores the filter specification
for sparse cloning/fetching was interpreted in a wrong place in the
code, causing Git to abort.
Will merge to 'master'.
* rs/commit-graph-use-list-count (2019-09-16) 1 commit
(merged to 'next' on 2019-09-30 at 8986e5537f)
+ commit-graph: use commit_list_count()
Code cleanup.
Will merge to 'master'.
* js/doc-patch-text (2019-09-17) 2 commits
(merged to 'next' on 2019-09-30 at f9b391a66b)
+ diff, log doc: small grammer, format, and language fixes
+ diff, log doc: say "patch text" instead of "patches"
Docfix.
Will merge to 'master'.
* ms/fetch-follow-tag-optim (2019-09-16) 1 commit
(merged to 'next' on 2019-09-30 at 97ec83d2a2)
+ fetch: use oidset to keep the want OIDs for faster lookup
The code used in following tags in "git fetch" has been optimized.
Will merge to 'master'.
* rs/nth-parent-parse (2019-09-16) 2 commits
(merged to 'next' on 2019-09-30 at 5bdfeacdff)
+ sha1-name: check for overflow of N in "foo^N" and "foo~N"
+ rev-parse: demonstrate overflow of N for "foo^N" and "foo~N"
The object name parser for "Nth parent" syntax has been made more
robust against integer overflows.
Will merge to 'master'.
* rs/nth-switch-code-simplification (2019-09-18) 1 commit
(merged to 'next' on 2019-09-30 at 4233f54a72)
+ sha1_name: simplify strbuf handling in interpret_nth_prior_checkout()
Code simplification.
Will merge to 'master'.
* sg/progress-fix (2019-09-17) 2 commits
(merged to 'next' on 2019-09-30 at d352332810)
+ Test the progress display
+ Revert "progress: use term_clear_line()"
Regression fix for progress output.
Will merge to 'master'.
* jc/git-gui-has-maintainer (2019-09-18) 1 commit
(merged to 'next' on 2019-09-30 at dfe61e947c)
+ SubmittingPatches: git-gui has a new maintainer
Will merge to 'master'.
* cb/pcre1-cleanup (2019-08-26) 2 commits
- grep: refactor and simplify PCRE1 support
- grep: make sure NO_LIBPCRE1_JIT disable JIT in PCRE1
(this branch uses ab/pcre-jit-fixes.)
PCRE fixes.
Will merge to 'next'.
* ew/hashmap (2019-09-28) 19 commits
- hashmap: remove type arg from hashmap_{get,put,remove}_entry
- OFFSETOF_VAR macro to simplify hashmap iterators
- hashmap: introduce hashmap_free_entries
- hashmap: hashmap_{put,remove} return hashmap_entry *
- hashmap: use *_entry APIs for iteration
- hashmap_cmp_fn takes hashmap_entry params
- hashmap_get{,_from_hash} return "struct hashmap_entry *"
- hashmap: use *_entry APIs to wrap container_of
- hashmap_get_next returns "struct hashmap_entry *"
- introduce container_of macro
- hashmap_put takes "struct hashmap_entry *"
- hashmap_remove takes "const struct hashmap_entry *"
- hashmap_get takes "const struct hashmap_entry *"
- hashmap_add takes "struct hashmap_entry *"
- hashmap_get_next takes "const struct hashmap_entry *"
- hashmap_entry_init takes "struct hashmap_entry *"
- packfile: use hashmap_entry in delta_base_cache_entry
- coccicheck: detect hashmap_entry.hash assignment
- diff: use hashmap_entry_init on moved_entry.ent
Code clean-up of the hashmap API, both users and implementation.
clang seems to be unhappy, even though gcc seems to be fine.
cf. <xmqqy2y7ihih.fsf@gitster-ct.c.googlers.com>
* jt/cache-tree-avoid-lazy-fetch-during-merge (2019-09-09) 1 commit
(merged to 'next' on 2019-09-30 at 5d8ffc2e0f)
+ cache-tree: do not lazy-fetch tentative tree
The cache-tree code has been taught to be less aggressive in
attempting to see if a tree object it computed already exists in
the repository.
Will merge to 'master'.
* ma/asciidoctor-refmiscinfo (2019-09-16) 3 commits
(merged to 'next' on 2019-09-30 at e5d13aceb8)
+ doc-diff: replace --cut-header-footer with --cut-footer
+ asciidoctor-extensions: provide `<refmiscinfo/>`
+ Doc/Makefile: give mansource/-version/-manual attributes
Update support for Asciidoctor documentation toolchain.
Will merge to 'master'.
* tg/stash-refresh-index (2019-09-20) 3 commits
(merged to 'next' on 2019-09-30 at de7759ad1d)
+ stash: make sure to write refreshed cache
+ merge: use refresh_and_write_cache
+ factor out refresh_and_write_cache function
"git stash" learned to write refreshed index back to disk.
Will merge to 'master'.
* js/builtin-add-i (2019-08-30) 11 commits
. built-in add -i: implement the `help` command
. built-in add -i: use color in the main loop
. built-in add -i: support `?` (prompt help)
. built-in add -i: show unique prefixes of the commands
. Add a function to determine unique prefixes for a list of strings
. built-in add -i: implement the main loop
. built-in add -i: color the header in the `status` command
. built-in add -i: refresh the index before running `status`
. built-in add -i: implement the `status` command
. diff: export diffstat interface
. Start to implement a built-in version of `git add --interactive`
The beginning of rewriting "git add -i" in C.
On hold, waiting for tg/stash-refresh-index to work well with this.
* en/merge-recursive-cleanup (2019-10-02) 25 commits
(merged to 'next' on 2019-10-03 at 7b5a32726b)
+ merge-recursive: fix the diff3 common ancestor label for virtual commits
(merged to 'next' on 2019-09-18 at f52cb08a14)
+ merge-recursive: alphabetize include list
+ merge-recursive: add sanity checks for relevant merge_options
+ merge-recursive: rename MERGE_RECURSIVE_* to MERGE_VARIANT_*
+ merge-recursive: split internal fields into a separate struct
+ merge-recursive: avoid losing output and leaking memory holding that output
+ merge-recursive: comment and reorder the merge_options fields
+ merge-recursive: consolidate unnecessary fields in merge_options
+ merge-recursive: move some definitions around to clean up the header
+ merge-recursive: rename merge_options argument to opt in header
+ merge-recursive: rename 'mrtree' to 'result_tree', for clarity
+ merge-recursive: use common name for ancestors/common/base_list
+ merge-recursive: fix some overly long lines
+ cache-tree: share code between functions writing an index as a tree
+ merge-recursive: don't force external callers to do our logging
+ merge-recursive: remove useless parameter in merge_trees()
+ merge-recursive: exit early if index != head
+ Ensure index matches head before invoking merge machinery, round N
+ merge-recursive: remove another implicit dependency on the_repository
+ merge-recursive: future-proof update_file_flags() against memory leaks
+ merge-recursive: introduce an enum for detect_directory_renames values
+ merge-recursive: provide a better label for diff3 common ancestor
+ merge-recursive: enforce opt->ancestor != NULL when calling merge_trees()
+ checkout: provide better conflict hunk description with detached HEAD
+ merge-recursive: be consistent with assert
The merge-recursive machiery is one of the most complex parts of
the system that accumulated cruft over time. This large series
cleans up the implementation quite a bit.
Will merge to 'master'.
* pw/rebase-i-show-HEAD-to-reword (2019-08-19) 3 commits
- sequencer: simplify root commit creation
- rebase -i: check for updated todo after squash and reword
- rebase -i: always update HEAD before rewording
(this branch is used by ra/rebase-i-more-options.)
"git rebase -i" showed a wrong HEAD while "reword" open the editor.
Will merge to 'next'.
* vn/reset-deleted-ita (2019-07-26) 1 commit
- reset: unstage empty deleted ita files
"git reset HEAD [<pathspec>]" did not reset an empty file that was
added with the intent-to-add bit.
Expecting a reroll.
* ra/rebase-i-more-options (2019-09-09) 6 commits
- rebase: add --reset-author-date
- rebase -i: support --ignore-date
- sequencer: rename amend_author to author_to_rename
- rebase -i: support --committer-date-is-author-date
- sequencer: allow callers of read_author_script() to ignore fields
- rebase -i: add --ignore-whitespace flag
(this branch uses pw/rebase-i-show-HEAD-to-reword.)
"git rebase -i" learned a few options that are known by "git
rebase" proper.
Is this ready for 'next'.
* ab/pcre-jit-fixes (2019-08-19) 18 commits
- grep: under --debug, show whether PCRE JIT is enabled
- grep: do not enter PCRE2_UTF mode on fixed matching
- grep: stess test PCRE v2 on invalid UTF-8 data
- grep: create a "is_fixed" member in "grep_pat"
- grep: consistently use "p->fixed" in compile_regexp()
- grep: stop using a custom JIT stack with PCRE v1
- grep: stop "using" a custom JIT stack with PCRE v2
- grep: remove overly paranoid BUG(...) code
- grep: use PCRE v2 for optimized fixed-string search
- grep: remove the kwset optimization
- grep: drop support for \0 in --fixed-strings <pattern>
- grep: make the behavior for NUL-byte in patterns sane
- grep tests: move binary pattern tests into their own file
- grep tests: move "grep binary" alongside the rest
- grep: inline the return value of a function call used only once
- t4210: skip more command-line encoding tests on MinGW
- grep: don't use PCRE2?_UTF8 with "log --encoding=<non-utf8>"
- log tests: test regex backends in "--encode=<enc>" tests
(this branch is used by cb/pcre1-cleanup.)
A few simplification and bugfixes to PCRE interface.
Will merge to 'next'.
* jc/format-patch-noclobber (2019-02-22) 1 commit
- format-patch: --no-clobber refrains from overwriting output files
"git format-patch" used to overwrite an existing patch/cover-letter
file. A new "--no-clobber" option stops it.
Will discard.
--------------------------------------------------
[Discarded]
* dl/remote-save-to-push (2018-12-11) 1 commit
. remote: add --save-to-push option to git remote set-url
"git remote set-url" learned a new option that moves existing value
of the URL field to pushURL field of the remote before replacing
the URL field with a new value.
Anybody who wants to champion this topic?
I am personally not yet quite convinced if this is worth pursuing.
* js/trace2-json-schema (2019-07-25) 3 commits
. ci: run trace2 schema validation in the CI suite
. trace2: add a schema validator for trace2 events
. trace2: add a JSON schema for trace2 events
The JSON output produced by "trace2" subsystem now has JSON schema
defined on it, to allow us validate the output and catch deviation.
Abandoned, at least for now.
cf. <20190910182305.GA11186@google.com>
* pd/fetch-jobs (2019-08-13) 5 commits
. fetch: make --jobs control submodules and remotes
. fetch: add the --submodule-fetch-jobs option
. fetch: add the fetch.jobs config key
. fetch: add the "--fetch-jobs" option
. fetch: rename max_children to max_children_for_submodules
"git fetch --jobs" is getting taught to also run fetch jobs in
parallel when fetching from multiple remote repositories.
cf. <nycvar.QRO.7.76.6.1909111359150.5377@tvgsbejvaqbjf.bet>
* js/honor-cflags-in-hdr-check (2019-08-26) 1 commit
(merged to 'next' on 2019-09-09 at fcd9ee9f1b)
+ hdr-check: make it work on Windows
Build fix to make sure hdr-check is run with the same preprocessor
macros predefined by the $(MAKE) procedure.
Superseded by dl/honor-cflags-in-hdr-check series.
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: What's cooking in git.git (Oct 2019, #01; Thu, 3)
2019-10-03 5:04 What's cooking in git.git (Oct 2019, #01; Thu, 3) Junio C Hamano
@ 2019-10-03 18:28 ` Emily Shaffer
2019-10-03 22:31 ` Junio C Hamano
2019-10-03 20:28 ` Elijah Newren
2019-10-04 9:44 ` Phillip Wood
2 siblings, 1 reply; 9+ messages in thread
From: Emily Shaffer @ 2019-10-03 18:28 UTC (permalink / raw)
To: Junio C Hamano; +Cc: git
On Thu, Oct 03, 2019 at 02:04:39PM +0900, 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 integration branches, but I am still holding onto them.
>
> You can find the changes described here in the integration branches
> of the repositories listed at
>
> http://git-blame.blogspot.com/p/git-public-repositories.html
>
Hi Junio,
How do you feel about
https://public-inbox.org/git/20190930220355.108394-1-emilyshaffer@google.com/
which fixes a user-facing SIGABRT during partial clone operations? It's
reviewed by Peff and Christian.
- Emily
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: What's cooking in git.git (Oct 2019, #01; Thu, 3)
2019-10-03 5:04 What's cooking in git.git (Oct 2019, #01; Thu, 3) Junio C Hamano
2019-10-03 18:28 ` Emily Shaffer
@ 2019-10-03 20:28 ` Elijah Newren
2019-10-03 22:31 ` Junio C Hamano
2019-10-04 9:44 ` Phillip Wood
2 siblings, 1 reply; 9+ messages in thread
From: Elijah Newren @ 2019-10-03 20:28 UTC (permalink / raw)
To: Junio C Hamano; +Cc: Git Mailing List
On Wed, Oct 2, 2019 at 10:22 PM Junio C Hamano <gitster@pobox.com> wrote:
>
> * en/fast-imexport-nested-tags (2019-10-02) 8 commits
> - fast-export: handle nested tags
> - t9350: add tests for tags of things other than a commit
> - fast-export: allow user to request tags be marked with --mark-tags
> - fast-export: add support for --import-marks-if-exists
> - fast-import: add support for new 'alias' command
> - fast-import: allow tags to be identified by mark labels
> - fast-import: fix handling of deleted tags
> - fast-export: fix exporting a tag and nothing else
>
> Updates to fast-import/export.
>
> Will merge to 'next'.
Actually, René posted a code cleanup suggestion for patch 2/8, so I
sent a V3 re-roll[1]. Could you pick up V3 instead of merging V2 down
to next?
[1] https://public-inbox.org/git/20191003202709.26279-1-newren@gmail.com/
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: What's cooking in git.git (Oct 2019, #01; Thu, 3)
2019-10-03 18:28 ` Emily Shaffer
@ 2019-10-03 22:31 ` Junio C Hamano
0 siblings, 0 replies; 9+ messages in thread
From: Junio C Hamano @ 2019-10-03 22:31 UTC (permalink / raw)
To: Emily Shaffer; +Cc: git
Emily Shaffer <emilyshaffer@google.com> writes:
> On Thu, Oct 03, 2019 at 02:04:39PM +0900, 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 integration branches, but I am still holding onto them.
>>
>> You can find the changes described here in the integration branches
>> of the repositories listed at
>>
>> http://git-blame.blogspot.com/p/git-public-repositories.html
>>
>
> Hi Junio,
>
> How do you feel about
> https://public-inbox.org/git/20190930220355.108394-1-emilyshaffer@google.com/
> which fixes a user-facing SIGABRT during partial clone operations? It's
> reviewed by Peff and Christian.
Already picked up on the tip of cc/multi-promisor and pushed out, I
think, as part of yesterday's integration cycle.
Thanks for fixing.
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: What's cooking in git.git (Oct 2019, #01; Thu, 3)
2019-10-03 20:28 ` Elijah Newren
@ 2019-10-03 22:31 ` Junio C Hamano
0 siblings, 0 replies; 9+ messages in thread
From: Junio C Hamano @ 2019-10-03 22:31 UTC (permalink / raw)
To: Elijah Newren; +Cc: Git Mailing List
Elijah Newren <newren@gmail.com> writes:
> Actually, René posted a code cleanup suggestion for patch 2/8, so I
> sent a V3 re-roll[1]. Could you pick up V3 instead of merging V2 down
> to next?
>
> [1] https://public-inbox.org/git/20191003202709.26279-1-newren@gmail.com/
Thanks for stopping me.
Will take a look.
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: What's cooking in git.git (Oct 2019, #01; Thu, 3)
2019-10-03 5:04 What's cooking in git.git (Oct 2019, #01; Thu, 3) Junio C Hamano
2019-10-03 18:28 ` Emily Shaffer
2019-10-03 20:28 ` Elijah Newren
@ 2019-10-04 9:44 ` Phillip Wood
2019-10-05 0:40 ` Elijah Newren
2019-10-05 23:36 ` Junio C Hamano
2 siblings, 2 replies; 9+ messages in thread
From: Phillip Wood @ 2019-10-04 9:44 UTC (permalink / raw)
To: Junio C Hamano, git
Hi Junio
On 03/10/2019 06:04, 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 integration branches, but I am still holding onto them.
> [...]
>
>
> * pw/rebase-i-show-HEAD-to-reword (2019-08-19) 3 commits
> - sequencer: simplify root commit creation
> - rebase -i: check for updated todo after squash and reword
> - rebase -i: always update HEAD before rewording
> (this branch is used by ra/rebase-i-more-options.)
>
> "git rebase -i" showed a wrong HEAD while "reword" open the editor.
>
> Will merge to 'next'.
That's great, thanks
>
> * ra/rebase-i-more-options (2019-09-09) 6 commits
> - rebase: add --reset-author-date
> - rebase -i: support --ignore-date
> - sequencer: rename amend_author to author_to_rename
> - rebase -i: support --committer-date-is-author-date
> - sequencer: allow callers of read_author_script() to ignore fields
> - rebase -i: add --ignore-whitespace flag
> (this branch uses pw/rebase-i-show-HEAD-to-reword.)
>
> "git rebase -i" learned a few options that are known by "git
> rebase" proper.
>
> Is this ready for 'next'.
Nearly, but not quite I think cf [1]. Also I'm still not convinced that
having different behaviors for --ignore-whitespace depending on the
backend is going to be helpful but maybe they are close enough not to
matter too much in practice [2].
[1]
https://public-inbox.org/git/20190806173638.17510-1-rohit.ashiwal265@gmail.com/T/#m965ce1f09d1d1b8010c04db0eabd4b19ce99fe82
[2]
https://public-inbox.org/git/20190806173638.17510-1-rohit.ashiwal265@gmail.com/T/#m94e059c18b7bbcc721d12b190bd4aaecc5e8d591
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: What's cooking in git.git (Oct 2019, #01; Thu, 3)
2019-10-04 9:44 ` Phillip Wood
@ 2019-10-05 0:40 ` Elijah Newren
2019-10-05 19:46 ` Phillip Wood
2019-10-05 23:36 ` Junio C Hamano
1 sibling, 1 reply; 9+ messages in thread
From: Elijah Newren @ 2019-10-05 0:40 UTC (permalink / raw)
To: Phillip Wood; +Cc: Junio C Hamano, Git Mailing List
On Fri, Oct 4, 2019 at 4:49 AM Phillip Wood <phillip.wood123@gmail.com> wrote:
>
> Hi Junio
>
> On 03/10/2019 06:04, 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 integration branches, but I am still holding onto them.
> > [...]
> >
> >
> > * pw/rebase-i-show-HEAD-to-reword (2019-08-19) 3 commits
> > - sequencer: simplify root commit creation
> > - rebase -i: check for updated todo after squash and reword
> > - rebase -i: always update HEAD before rewording
> > (this branch is used by ra/rebase-i-more-options.)
> >
> > "git rebase -i" showed a wrong HEAD while "reword" open the editor.
> >
> > Will merge to 'next'.
>
> That's great, thanks
>
> >
> > * ra/rebase-i-more-options (2019-09-09) 6 commits
> > - rebase: add --reset-author-date
> > - rebase -i: support --ignore-date
> > - sequencer: rename amend_author to author_to_rename
> > - rebase -i: support --committer-date-is-author-date
> > - sequencer: allow callers of read_author_script() to ignore fields
> > - rebase -i: add --ignore-whitespace flag
> > (this branch uses pw/rebase-i-show-HEAD-to-reword.)
> >
> > "git rebase -i" learned a few options that are known by "git
> > rebase" proper.
> >
> > Is this ready for 'next'.
>
> Nearly, but not quite I think cf [1]. Also I'm still not convinced that
> having different behaviors for --ignore-whitespace depending on the
> backend is going to be helpful but maybe they are close enough not to
> matter too much in practice [2].
Sorry I should have chimed in sooner; I can speak to the second point.
I would say that in practice it doesn't matter a lot; in most cases
the two overlap. Both am's --ignore-whitespace and merge's
-Xignore-space-change are buggy (in different ways) and should be
fixed, but I'd consider them both to be buggy in edge cases. I
recommended earlier this summer that Rohit submit the patches without
first attempting to fix apply or xdiff, and kept in my TODO list
that'd I'd go in and fix xdiff later if Rohit didn't have extra time
for it. I did a little digging back then to find out the differences
and suggested some text to use to explain them and to argue that they
shouldn't block this feature:
"""
am's --ignore-space-change (an alias for am's --ignore-whitespace; see
git-apply's description of those two flags) not only share the same
name with diff's --ignore-space-change and merge's
-Xignore-space-change, but the similarity in naming appears to have
been intentional with am's --ignore-space-change and merge's
-Xignore-space-change being designed to have the same functionality
(see e.g. the commit messages for f008cef4abb2 ("Merge branch
'jc/apply-ignore-whitespace'", 2014-06-03) and 4e5dd044c62f
("merge-recursive: options to ignore whitespace changes",
2010-08-26)). For the most part, these options do provide the same
behavior. However, there are some edge cases where both apply's
--ignore-space-change and merge's -Xignore-space-change fall short of
optimal behavior, and in different ways. In particular,
--ignore-space-change for apply will handle whitespace changes in the
context region but not in the region the other side modified, and
-Xignore-space-change will delete whitespace changes even when the
other side had no changes (thus treating both sides as unmodified).
Fixing these differences in edge cases is left for future work; this
patch simply wires interactive rebase to also understand
--ignore-whitespace by translating it to -Xignore-space-change.
"""
I've got another email with even more detail if folks need it.
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: What's cooking in git.git (Oct 2019, #01; Thu, 3)
2019-10-05 0:40 ` Elijah Newren
@ 2019-10-05 19:46 ` Phillip Wood
0 siblings, 0 replies; 9+ messages in thread
From: Phillip Wood @ 2019-10-05 19:46 UTC (permalink / raw)
To: Elijah Newren, Phillip Wood; +Cc: Junio C Hamano, Git Mailing List
Hi Elijah
On 05/10/2019 01:40, Elijah Newren wrote:
> On Fri, Oct 4, 2019 at 4:49 AM Phillip Wood <phillip.wood123@gmail.com> wrote:
>>
>> Hi Junio
>>
>> On 03/10/2019 06:04, 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 integration branches, but I am still holding onto them.
>>> [...]
>>>
>>>
>>> * pw/rebase-i-show-HEAD-to-reword (2019-08-19) 3 commits
>>> - sequencer: simplify root commit creation
>>> - rebase -i: check for updated todo after squash and reword
>>> - rebase -i: always update HEAD before rewording
>>> (this branch is used by ra/rebase-i-more-options.)
>>>
>>> "git rebase -i" showed a wrong HEAD while "reword" open the editor.
>>>
>>> Will merge to 'next'.
>>
>> That's great, thanks
>>
>>>
>>> * ra/rebase-i-more-options (2019-09-09) 6 commits
>>> - rebase: add --reset-author-date
>>> - rebase -i: support --ignore-date
>>> - sequencer: rename amend_author to author_to_rename
>>> - rebase -i: support --committer-date-is-author-date
>>> - sequencer: allow callers of read_author_script() to ignore fields
>>> - rebase -i: add --ignore-whitespace flag
>>> (this branch uses pw/rebase-i-show-HEAD-to-reword.)
>>>
>>> "git rebase -i" learned a few options that are known by "git
>>> rebase" proper.
>>>
>>> Is this ready for 'next'.
>>
>> Nearly, but not quite I think cf [1]. Also I'm still not convinced that
>> having different behaviors for --ignore-whitespace depending on the
>> backend is going to be helpful but maybe they are close enough not to
>> matter too much in practice [2].
>
> Sorry I should have chimed in sooner; I can speak to the second point.
> I would say that in practice it doesn't matter a lot; in most cases
> the two overlap. Both am's --ignore-whitespace and merge's
> -Xignore-space-change are buggy (in different ways) and should be
> fixed, but I'd consider them both to be buggy in edge cases. I
> recommended earlier this summer that Rohit submit the patches without
> first attempting to fix apply or xdiff, and kept in my TODO list
> that'd I'd go in and fix xdiff later if Rohit didn't have extra time
> for it. I did a little digging back then to find out the differences
> and suggested some text to use to explain them and to argue that they
> shouldn't block this feature:
>
> """
> am's --ignore-space-change (an alias for am's --ignore-whitespace; see
> git-apply's description of those two flags) not only share the same
> name with diff's --ignore-space-change and merge's
> -Xignore-space-change, but the similarity in naming appears to have
> been intentional with am's --ignore-space-change and merge's
> -Xignore-space-change being designed to have the same functionality
> (see e.g. the commit messages for f008cef4abb2 ("Merge branch
> 'jc/apply-ignore-whitespace'", 2014-06-03) and 4e5dd044c62f
> ("merge-recursive: options to ignore whitespace changes",
> 2010-08-26)). For the most part, these options do provide the same
> behavior. However, there are some edge cases where both apply's
> --ignore-space-change and merge's -Xignore-space-change fall short of
> optimal behavior, and in different ways. In particular,
> --ignore-space-change for apply will handle whitespace changes in the
> context region but not in the region the other side modified, and
> -Xignore-space-change will delete whitespace changes even when the
> other side had no changes (thus treating both sides as unmodified).
> Fixing these differences in edge cases is left for future work; this
> patch simply wires interactive rebase to also understand
> --ignore-whitespace by translating it to -Xignore-space-change.
> """
>
> I've got another email with even more detail if folks need it.
Thanks for clarifying that, it sounds like it shouldn't be a problem in
practice then (especially if you're planning some improvements)
Best Wishes
Phillip
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: What's cooking in git.git (Oct 2019, #01; Thu, 3)
2019-10-04 9:44 ` Phillip Wood
2019-10-05 0:40 ` Elijah Newren
@ 2019-10-05 23:36 ` Junio C Hamano
1 sibling, 0 replies; 9+ messages in thread
From: Junio C Hamano @ 2019-10-05 23:36 UTC (permalink / raw)
To: Phillip Wood; +Cc: git
Phillip Wood <phillip.wood123@gmail.com> writes:
>> * ra/rebase-i-more-options (2019-09-09) 6 commits
>> ...
>> Is this ready for 'next'.
>
> Nearly, but not quite I think cf [1]. Also I'm still not convinced
> that having different behaviors for --ignore-whitespace depending on
> the backend is going to be helpful but maybe they are close enough not
> to matter too much in practice [2].
>
> [1]
> https://public-inbox.org/git/20190806173638.17510-1-rohit.ashiwal265@gmail.com/T/#m965ce1f09d1d1b8010c04db0eabd4b19ce99fe82
OK, so I'll leave a note that this needs to consider existing
GIT_COMMITTER_DATE environment variable, etc, with a reference to
<5adde732-173b-d24d-d23f-bb4d043076d7@gmail.com>.
A URL to public-inbox.org is good, but that is because those who do
not use public-inbox.org (or after the site goes out of service) can
still identify exactly which message you are referring to if you use
the https://public-inbox.org/git/$message_id_without_brackets
format.
Unlike that form, the format you used only identifies the thread
starter, and without public-inbox.org in service, a reader cannot
tell which message in the thread you are referring to, even if the
reader had full mailing list archive.
IOW, I would have appreciated if the above reference were done like
so:
https://public-inbox.org/git/5adde732-173b-d24d-d23f-bb4d043076d7@gmail.com/
Thanks.
^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2019-10-05 23:36 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-10-03 5:04 What's cooking in git.git (Oct 2019, #01; Thu, 3) Junio C Hamano
2019-10-03 18:28 ` Emily Shaffer
2019-10-03 22:31 ` Junio C Hamano
2019-10-03 20:28 ` Elijah Newren
2019-10-03 22:31 ` Junio C Hamano
2019-10-04 9:44 ` Phillip Wood
2019-10-05 0:40 ` Elijah Newren
2019-10-05 19:46 ` Phillip Wood
2019-10-05 23:36 ` Junio C Hamano
Code repositories for project(s) associated with this public inbox
https://80x24.org/mirrors/git.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).