threads / discuss / 48798

What's cooking in git.git (Jun 2018, #07; Thu, 28)

Subject: What's cooking in git.git (Jun 2018, #07; Thu, 28)

## tl;dr

24 messages between Jun 28, 2018 and Jul 9, 2018.

replies: 23people: 9as markdown or json

Junio C Hamano· Jun 28, 2018, 21:40 UTC · lore

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.

The tip of 'next' has been rewound and it currently has only 4 topics. Quite a many topics are cooking in 'pu' and need to be sifted into good bins (for 'next') and the remainder. Help is appreciated in that area ;-)

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

-------------------------------------------------- [Graduated to "master"]

* ab/refspec-init-fix (2018-06-11) 3 commits
  (merged to 'next' on 2018-06-13 at 91d71d8435)
 + refspec: initalize `refspec_item` in `valid_fetch_refspec()`
 + refspec: add back a refspec_item_init() function
 + refspec: s/refspec_item_init/&_or_die/g
 Make refspec parsing codepath more robust.
* as/safecrlf-quiet-fix (2018-06-11) 1 commit
  (merged to 'next' on 2018-06-13 at b163674843)
 + config.c: fix regression for core.safecrlf false
 Fix for 2.17-era regression around `core.safecrlf`.
* jc/clean-after-sanity-tests (2018-06-15) 1 commit
  (merged to 'next' on 2018-06-22 at 2df77b8af9)
 + tests: clean after SANITY tests
 test cleanup.
* jh/partial-clone (2018-06-12) 1 commit
  (merged to 'next' on 2018-06-13 at 818f864b0c)
 + list-objects: check if filter is NULL before using
 The recent addition of "partial clone" experimental feature kicked
 in when it shouldn't, namely, when there is no partial-clone filter
 defined even if extensions.partialclone is set.
* jk/fetch-all-peeled-fix (2018-06-13) 2 commits
  (merged to 'next' on 2018-06-13 at 1333bb9d90)
 + fetch-pack: test explicitly that --all can fetch tag references pointing to non-commits
 + fetch-pack: don't try to fetch peel values with --all
 "git fetch-pack --all" used to unnecessarily fail upon seeing an
 annotated tag that points at an object other than a commit.
* ms/send-pack-honor-config (2018-06-12) 1 commit
  (merged to 'next' on 2018-06-13 at e2cd933715)
 + builtin/send-pack: populate the default configs
 "git send-pack --signed" (hence "git push --signed" over the http
 transport) did not read user ident from the config mechanism to
 determine whom to sign the push certificate as, which has been
 corrected.
* nd/completion-negation (2018-06-11) 3 commits
  (merged to 'next' on 2018-06-19 at a3be59b450)
 + completion: collapse extra --no-.. options
 + completion: suppress some -no- options
 + parse-options: option to let --git-completion-helper show negative form
 Continuing with the idea to programmatically enumerate various
 pieces of data required for command line completion, the codebase
 has been taught to enumerate options prefixed with "--no-" to
 negate them.
* pw/add-p-recount (2018-06-11) 1 commit
  (merged to 'next' on 2018-06-19 at 1880ecc3f1)
 + add -p: fix counting empty context lines in edited patches
 When user edits the patch in "git add -p" and the user's editor is
 set to strip trailing whitespaces indiscriminately, an empty line
 that is unchanged in the patch would become completely empty
 (instead of a line with a sole SP on it).  The code introduced in
 Git 2.17 timeframe failed to parse such a patch, but now it learned
 to notice the situation and cope with it.
* sb/fix-fetching-moved-submodules (2018-06-14) 2 commits
  (merged to 'next' on 2018-06-22 at 16039dc62a)
 + t5526: test recursive submodules when fetching moved submodules
 + submodule: fix NULL correctness in renamed broken submodules
 The code to try seeing if a fetch is necessary in a submodule
 during a fetch with --recurse-submodules got confused when the path
 to the submodule was changed in the range of commits in the
 superproject, sometimes showing "(null)".  This has been corrected.
* sg/gpg-tests-fix (2018-06-11) 2 commits
  (merged to 'next' on 2018-06-13 at f3a05f1c41)
 + tests: make forging GPG signed commits and tags more robust
 + t7510-signed-commit: use 'test_must_fail'
 Some flaky tests have been fixed.
* tz/cred-netrc-cleanup (2018-06-18) 4 commits
  (merged to 'next' on 2018-06-22 at a471dd714c)
 + git-credential-netrc: make "all" default target of Makefile
 + git-credential-netrc: fix exit status when tests fail
 + git-credential-netrc: use in-tree Git.pm for tests
 + git-credential-netrc: minor whitespace cleanup in test script
 Build and test procedure for netrc credential helper (in contrib/)
 has been updated.

-------------------------------------------------- [New Topics]

* ao/config-from-gitmodules (2018-06-26) 6 commits
 - submodule-config: reuse config_from_gitmodules in repo_read_gitmodules
 - submodule-config: pass repository as argument to config_from_gitmodules
 - submodule-config: make 'config_from_gitmodules' private
 - submodule-config: add helper to get 'update-clone' config from .gitmodules
 - submodule-config: add helper function to get 'fetch' config from .gitmodules
 - config: move config_from_gitmodules to submodule-config.c
 Tighten the API to make it harder to misuse in-tree .gitmodules
 file, even though it shares the same syntax with configuration
 files, to read random configuration items from it.
 Will merge to 'next'.
* as/sequencer-customizable-comment-char (2018-06-28) 1 commit
 - sequencer: use configured comment character
 Honor core.commentchar when preparing the list of commits to replay
 in "rebase -i".
 
* dj/runtime-prefix (2018-06-26) 1 commit
 - Makefile: tweak sed invocation
 POSIX portability fix in Makefile to fix a glitch introduced a few
 releases ago.
 Will merge to 'next'.
* ds/commit-graph (2018-06-28) 1 commit
 - commit-graph: fix documentation inconsistencies
 Docfix.
 Will merge to 'next'.
* en/rebase-i-microfixes (2018-06-27) 3 commits
 - git-rebase--merge: modernize "git-$cmd" to "git $cmd"
 - Fix use of strategy options with interactive rebases
 - t3418: add testcase showing problems with rebase -i and strategy options
 Will merge to 'next'.
* js/enhanced-version-info (2018-06-28) 1 commit
 - Makefile: fix the "built from commit" code
 Build fix.
 Will merge to 'next'.
* js/rebase-recreate-merge (2018-06-27) 1 commit
 - rebase: fix documentation formatting
 Docfix.
 Will merge to 'next'.
* jt/connectivity-check-after-unshallow (2018-06-27) 1 commit
 - fetch: when deepening, check connectivity fully
 "git fetch" failed to correctly validate the set of objects it
 received when making a shallow history deeper, which has been
 corrected.
* jt/fetch-nego-tip (2018-06-27) 1 commit
 - fetch-pack: support negotiation tip whitelist
 (this branch uses jt/fetch-pack-negotiator.)
 "git fetch" learned a new option "--negotiation-tip" to limit the
 set of commits it tells the other end as "have", to reduce wasted
 bandwidth and cycles, which would be helpful when the receiving
 repository has a lot of refs that have little to do with the
 history at the remote it is fetching from.
* mb/filter-branch-optim (2018-06-26) 1 commit
 - filter-branch: skip commits present on --state-branch
 "git filter-branch" when used with the "--state-branch" option
 still attempted to rewrite the commits whose filtered result is
 known from the previous attempt (which is recorded on the state
 branch); the command has been corrected not to waste cycles doing
 so.
 Will merge to 'next'.
* ms/core-icase-doc (2018-06-28) 1 commit
 - Documentation: declare "core.ignoreCase" as internal variable
 Clarify that setting core.ignoreCase to deviate from reality would
 not turn a case-incapable filesystem into a case-capable one.
 Will merge to 'next'.
* rj/submodule-fsck-skip (2018-06-27) 1 commit
 - fsck: check skiplist for object in fsck_blob()
 "fsck.skipList" did not prevent a blob object listed there from
 being inspected for is contents (e.g. we recently started to
 inspect the contents of ".gitmodules" for certain malicious
 patterns), which has been corrected.
 Will merge to 'next'.
* tb/grep-only-matching (2018-06-27) 3 commits
 - SQUASH??? emerg compilation fix
 - grep.c: teach 'git grep --only-matching'
 - grep.c: extract show_line_header()
 (this branch uses tb/grep-column.)
 "git grep" learned the "--only-matching" option.
* tz/exclude-doc-smallfixes (2018-06-27) 2 commits
 - dir.c: fix typos in core.excludesfile comment
 - gitignore.txt: clarify default core.excludesfile path
 Doc updates.
 Will merge to 'next'.

-------------------------------------------------- [Stalled]

* ab/fetch-tags-noclobber (2018-05-16) 9 commits
 - fixup! push tests: assert re-pushing annotated tags
 - fetch: stop clobbering existing tags without --force
 - fetch tests: add a test clobbering tag behavior
 - fetch tests: correct a comment "remove it" -> "remove them"
 - push doc: correct lies about how push refspecs work
 - push tests: assert re-pushing annotated tags
 - push tests: add more testing for forced tag pushing
 - push tests: fix logic error in "push" test assertion
 - push tests: remove redundant 'git push' invocation
 Expecting a reboot of the discussion to take it to some conclusion
 and then a reroll.
 cf. <f3b891c3-381f-de42-51d8-24fdfbca91d2@gmail.com>
 cf. <xmqq603yn50l.fsf@gitster-ct.c.googlers.com>
 cf. <xmqqzi1alodz.fsf@gitster-ct.c.googlers.com>
 cf. <xmqqvabylnbi.fsf@gitster-ct.c.googlers.com>
* pw/add-p-select (2018-03-16) 3 commits
 - add -p: optimize line selection for short hunks
 - add -p: allow line selection to be inverted
 - add -p: select individual hunk lines
 "git add -p" interactive interface learned to let users choose
 individual added/removed lines to be used in the operation, instead
 of accepting or rejecting a whole hunk.
 Expecting a reroll to reignite the discussion.
 cf. <9895c7b7-eac4-28c1-90c6-443acd1131b7@talktalk.net>
* jh/json-writer (2018-06-13) 1 commit
 - json_writer: new routines to create JSON data
 Preparatory code to later add json output for unspecified telemetry
 data.
 We do not add random code that does not have real users to our
 codebase, so let's have it wait until such a real code materializes
 before too long.
* hn/bisect-first-parent (2018-04-21) 1 commit
 - bisect: create 'bisect_flags' parameter in find_bisection()
 Preliminary code update to allow passing more flags down the
 bisection codepath in the future.
 We do not add random code that does not have real users to our
 codebase, so let's have it wait until such a real code materializes
 before too long.
* av/fsmonitor-updates (2018-01-04) 6 commits
 - fsmonitor: use fsmonitor data in `git diff`
 - fsmonitor: remove debugging lines from t/t7519-status-fsmonitor.sh
 - fsmonitor: make output of test-dump-fsmonitor more concise
 - fsmonitor: update helper tool, now that flags are filled later
 - fsmonitor: stop inline'ing mark_fsmonitor_valid / _invalid
 - dir.c: update comments to match argument name
 Code clean-up on fsmonitor integration, plus optional utilization
 of the fsmonitor data in diff-files.
 Waiting for an update.
 cf. <alpine.DEB.2.21.1.1801042335130.32@MININT-6BKU6QN.europe.corp.microsoft.com>
* pb/bisect-helper-2 (2018-06-13) 8 commits
 - t6030: make various test to pass GETTEXT_POISON tests
 - bisect--helper: `bisect_start` shell function partially in C
 - bisect--helper: `get_terms` & `bisect_terms` shell function in C
 - bisect--helper: `bisect_next_check` shell function in C
 - bisect--helper: `check_and_set_terms` shell function in C
 - wrapper: move is_empty_file() and rename it as is_empty_or_missing_file()
 - bisect--helper: `bisect_write` shell function in C
 - bisect--helper: `bisect_reset` shell function in C
 Expecting a reroll.
 cf. <0102015f5e5ee171-f30f4868-886f-47a1-a4e4-b4936afc545d-000000@eu-west-1.amazonses.com>
 I just rebased the topic to a newer base as it did not build
 standalone with the base I originally queued the topic on, but
 otherwise there is no update to address any of the review comments
 in the thread above---we are still waiting for a reroll.
* mk/http-backend-content-length (2018-06-11) 3 commits
 - http-backend: respect CONTENT_LENGTH for receive-pack
 - http-backend: respect CONTENT_LENGTH as specified by rfc3875
 - http-backend: cleanup writing to child process
 The http-backend (used for smart-http transport) used to slurp the
 whole input until EOF, without paying attention to CONTENT_LENGTH
 that is supplied in the environment and instead expecting the Web
 server to close the input stream.  This has been fixed.
* jk/drop-ancient-curl (2017-08-09) 5 commits
 - http: #error on too-old curl
 - curl: remove ifdef'd code never used with curl >=7.19.4
 - http: drop support for curl < 7.19.4
 - http: drop support for curl < 7.16.0
 - http: drop support for curl < 7.11.1
 Some code in http.c that has bitrot is being removed.
 Expecting a reroll.
* mk/use-size-t-in-zlib (2017-08-10) 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".
 Needs resurrecting by making sure the fix is good and still applies
 (or adjusted to today's codebase).

-------------------------------------------------- [Cooking]

* ld/p423 (2018-06-19) 6 commits
 - git-p4: python3: fix octal constants
 - git-p4: python3: use print() function
 - git-p4: python3: basestring workaround
 - git-p4: python3: remove backticks
 - git-p4: python3: replace dict.has_key(k) with "k in dict"
 - git-p4: python3: replace <> with !=
 Code preparation to make "git p4" closer to be usable with Python 3.
 Will merge to 'next'.
* pw/rebase-i-keep-reword-after-conflict (2018-06-19) 1 commit
 - sequencer: do not squash 'reword' commits when we hit conflicts
 Bugfix for "rebase -i" corner case regression.
 Will merge to 'next'.
* xy/format-patch-prereq-patch-id-fix (2018-06-19) 1 commit
 - format-patch: clear UNINTERESTING flag before prepare_bases
 Recently added "--base" option to "git format-patch" command did
 not correctly generate prereq patch ids.
 Will merge to 'next'.
* bw/config-refer-to-gitsubmodules-doc (2018-06-21) 1 commit
 - docs: link to gitsubmodules
 Docfix.
 Will merge to 'next'.
* bw/ref-in-want (2018-06-28) 8 commits
 - fetch-pack: implement ref-in-want
 - fetch-pack: put shallow info in output parameter
 - fetch: refactor to make function args narrower
 - fetch: refactor fetch_refs into two functions
 - fetch: refactor the population of peer ref OIDs
 - upload-pack: test negotiation with changing repository
 - upload-pack: implement ref-in-want
 - test-pkt-line: add unpack-sideband subcommand
 Protocol v2 has been updated to allow slightly out-of-sync set of
 servers to work together to serve a single client, which would be
 useful with load-balanced servers that talk smart HTTP transport.
* en/rebase-consistency (2018-06-27) 9 commits
 - git-rebase: make --allow-empty-message the default
 - t3401: add directory rename testcases for rebase and am
 - git-rebase.txt: document behavioral differences between modes
 - directory-rename-detection.txt: technical docs on abilities and limitations
 - git-rebase.txt: address confusion between --no-ff vs --force-rebase
 - git-rebase: error out when incompatible options passed
 - t3422: new testcases for checking when incompatible options passed
 - git-rebase.sh: update help messages a bit
 - git-rebase.txt: document incompatible options
 "git rebase" behaved slightly differently depending on which one of
 the three backends gets used; this has been documented and an
 effort to make them more uniform has begun.
* jt/remove-pack-bitmap-global (2018-06-21) 2 commits
 - pack-bitmap: add free function
 - pack-bitmap: remove bitmap_git global variable
 The effort to move globals to per-repository in-core structure
 continues.
* sb/submodule-move-head-error-msg (2018-06-25) 1 commit
 - submodule.c: report the submodule that an error occurs in
 Needs a reroll.
 cf. <20180622081713.5360-1-szeder.dev@gmail.com>
* bw/protocol-v2 (2018-06-22) 1 commit
 - protocol-v2 doc: put HTTP headers after request
 Doc fix.
 Will merge to 'next'.
* jk/branch-l-1-repurpose (2018-06-22) 1 commit
 - branch: make "-l" a synonym for "--list"
 (this branch uses jk/branch-l-0-deprecation.)
 Updated plan to repurpose the "-l" option to "git branch".
 Will hold in 'pu' until jk/branch-l-0-deprecation progresses sufficiently.
* vs/typofixes (2018-06-22) 1 commit
 - Documentation: spelling and grammar fixes
 Doc fix.
 Will merge to 'next'.
* cc/remote-odb (2018-06-25) 9 commits
 . Documentation/config: add odb.<name>.promisorRemote
 . t0410: test fetching from many promisor remotes
 . Use odb.origin.partialclonefilter instead of core.partialclonefilter
 . Use remote_odb_get_direct() and has_remote_odb()
 . remote-odb: add remote_odb_reinit()
 . remote-odb: implement remote_odb_get_many_direct()
 . remote-odb: implement remote_odb_get_direct()
 . Add initial remote odb support
 . fetch-object: make functions return an error code
 Needs a reroll.
* ds/multi-pack-index (2018-06-25) 24 commits
 - midx: clear midx on repack
 - packfile: skip loading index if in multi-pack-index
 - midx: prevent duplicate packfile loads
 - midx: use midx in approximate_object_count
 - midx: use existing midx when writing new one
 - midx: use midx in abbreviation calculations
 - midx: read objects from multi-pack-index
 - midx: prepare midxed_git struct
 - config: create core.multiPackIndex setting
 - midx: write object offsets
 - midx: write object id fanout chunk
 - midx: write object ids in a chunk
 - midx: sort and deduplicate objects from packfiles
 - midx: read pack names into array
 - multi-pack-index: write pack names in chunk
 - multi-pack-index: read packfile list
 - packfile: generalize pack directory list
 - multi-pack-index: expand test data
 - multi-pack-index: load into memory
 - midx: write header information to lockfile
 - multi-pack-index: add 'write' verb
 - multi-pack-index: add builtin
 - multi-pack-index: add format details
 - multi-pack-index: add design document
 When there are too many packfiles in a repository (which is not
 recommended), looking up an object in these would require
 consulting many pack .idx files; a new mechanism to have a single
 file that consolidates all of these .idx files is introduced.
* nd/use-the-index-compat-less (2018-06-25) 13 commits
 - wt-status.c: stop using index compat macros
 - sha1-name.c: stop using index compat macros
 - sequencer.c: stop using index compat macros
 - revision.c: stop using index compat macros
 - rerere.c: stop using index compat macros
 - merge.c: stop using index compat macros
 - merge-recursive.c: stop using index compat macros
 - entry.c: stop using index compat macros
 - diff.c: stop using index compat macros
 - diff-lib.c: stop using index compat macros
 - check-racy.c: stop using index compat macros
 - blame.c: stop using index compat macros
 - apply.c: stop using index compat macros
 Use of many convenience functions that operate on the_index
 "primary in-core index instance" have been rewritten to explicitly
 call the coutnerpart functions that take arbitrary index_state and
 pass the_index.
 I'd say that alone is a useless code churn, but people seem to be
 excited about the change, so it is queued here.
* tb/grep-column (2018-06-22) 7 commits
 - contrib/git-jump/git-jump: jump to exact location
 - grep.c: add configuration variables to show matched option
 - builtin/grep.c: add '--column' option to 'git-grep(1)'
 - grep.c: display column number of first match
 - grep.[ch]: extend grep_opt to allow showing matched column
 - grep.c: expose {,inverted} match column in match_line()
 - Documentation/config.txt: camel-case lineNumber for consistency
 (this branch is used by tb/grep-only-matching.)
 "git grep" learned the "--column" option that gives not just the
 line number but the column number of the hit.
 Will merge to 'next'.
* jt/fetch-pack-negotiator (2018-06-15) 7 commits
 - fetch-pack: introduce negotiator API
 - fetch-pack: move common check and marking together
 - fetch-pack: make negotiation-related vars local
 - fetch-pack: use ref adv. to prune "have" sent
 - fetch-pack: directly end negotiation if ACK ready
 - fetch-pack: clear marks before re-marking
 - fetch-pack: split up everything_local()
 (this branch is used by jt/fetch-nego-tip.)
 Code restructuring and a small fix to transport protocol v2 during
 fetching.
 Is this ready for 'next'?
* ag/rebase-i-append-todo-help (2018-06-14) 2 commits
 - rebase--interactive: rewrite append_todo_help() in C
 - Merge branch 'ag/rebase-p' into ag/rebase-i-append-todo-help
 (this branch is used by ag/rebase-i-rewrite-todo.)
 Stepwise rewriting of the machinery of "rebase -i" into C continues.
 Needs a reroll.
 cf. <nycvar.QRO.7.76.6.1806261125330.21419@tvgsbejvaqbjf.bet>
* ag/rebase-i-rewrite-todo (2018-06-15) 3 commits
 - rebase--interactive: rewrite the edit-todo functionality in C
 - editor: add a function to launch the sequence editor
 - Merge branch 'bc/t3430-fixup' into ag/rebase-i-rewrite-todo
 (this branch uses ag/rebase-i-append-todo-help.)
 Stepwise rewriting of the machinery of "rebase -i" into C continues.
* sb/submodule-core-worktree (2018-06-19) 3 commits
  (merged to 'next' on 2018-06-28 at 96e1a8dbd1)
 + submodule deinit: unset core.worktree
 + submodule: ensure core.worktree is set after update
 + submodule: unset core.worktree if no working tree is present
 Originally merged to 'next' on 2018-06-22
 "git submodule" did not correctly adjust core.worktree setting that
 indicates whether/where a submodule repository has its associated
 working tree across various state transitions, which has been
 corrected.
 Will merge to 'master'.
* ds/ewah-cleanup (2018-06-21) 10 commits
  (merged to 'next' on 2018-06-28 at 9cd7c0d54a)
 + ewah: delete unused 'rlwit_discharge_empty()'
 + ewah: drop ewah_serialize_native function
 + ewah: drop ewah_deserialize function
 + ewah_io: delete unused 'ewah_serialize()'
 + ewah_bitmap: delete unused 'ewah_or()'
 + ewah_bitmap: delete unused 'ewah_not()'
 + ewah_bitmap: delete unused 'ewah_and_not()'
 + ewah_bitmap: delete unused 'ewah_and()'
 + ewah/bitmap.c: delete unused 'bitmap_each_bit()'
 + ewah/bitmap.c: delete unused 'bitmap_clear()'
 Originally merged to 'next' on 2018-06-22
 Remove unused function definitions and declarations from ewah
 bitmap subsystem.
 Will merge to 'master'.
* is/parsing-line-range (2018-06-15) 2 commits
 - log: prevent error if line range ends past end of file
 - blame: prevent error if range ends past end of file
 Parsing of -L[<N>][,[<M>]] parameters "git blame" and "git log"
 take has been tweaked.
* en/merge-recursive-cleanup (2018-06-12) 6 commits
  (merged to 'next' on 2018-06-28 at 1a3646eb7d)
 + merge-recursive: add pointer about unduly complex looking code
 + merge-recursive: rename conflict_rename_*() family of functions
 + merge-recursive: clarify the rename_dir/RENAME_DIR meaning
 + merge-recursive: align labels with their respective code blocks
 + merge-recursive: fix numerous argument alignment issues
 + merge-recursive: fix miscellaneous grammar error in comment
 Originally merged to 'next' on 2018-06-19
 Code cleanup.
 Will merge to 'master'.
* ab/checkout-default-remote (2018-06-11) 8 commits
 - checkout & worktree: introduce checkout.defaultRemote
 - checkout: add advice for ambiguous "checkout <branch>"
 - builtin/checkout.c: use "ret" variable for return
 - checkout: pass the "num_matches" up to callers
 - checkout.c: change "unique" member to "num_matches"
 - checkout.c: introduce an *_INIT macro
 - checkout.h: wrap the arguments to unique_tracking_name()
 - checkout tests: index should be clean after dwim checkout
* ds/commit-graph-fsck (2018-06-27) 22 commits
 - commit-graph: update design document
 - gc: automatically write commit-graph files
 - commit-graph: add '--reachable' option
 - commit-graph: use string-list API for input
 - fsck: verify commit-graph
 - commit-graph: verify contents match checksum
 - commit-graph: test for corrupted octopus edge
 - commit-graph: verify commit date
 - commit-graph: verify generation number
 - commit-graph: verify parent list
 - commit-graph: verify root tree OIDs
 - commit-graph: verify objects exist
 - commit-graph: verify corrupt OID fanout and lookup
 - commit-graph: verify required chunks are present
 - commit-graph: verify catches corrupt signature
 - commit-graph: add 'verify' subcommand
 - commit-graph: load a root tree from specific graph
 - commit: force commit to parse from object database
 - commit-graph: parse commit from chosen graph
 - commit-graph: fix GRAPH_MIN_SIZE
 - commit-graph: UNLEAK before die()
 - t5318-commit-graph.sh: use core.commitGraph
 "git fsck" learns to make sure the optional commit-graph file is in
 a sane state.
 Is this ready for 'next'?
* ma/wrapped-info (2018-05-28) 2 commits
 - usage: prefix all lines in `vreportf()`, not just the first
 - usage: extract `prefix_suffix_lines()` from `advise()`
 An attempt to help making multi-line messages fed to warning(),
 error(), and friends more easily translatable.
 Will discard and wait for a cleaned-up rewrite.
 cf. <20180529213957.GF7964@sigill.intra.peff.net>
* jm/cache-entry-from-mem-pool (2018-06-28) 8 commits
 - block alloc: add validations around cache_entry lifecyle
 - block alloc: allocate cache entries from mem-pool
 - mem-pool: fill out functionality
 - mem-pool: add life cycle management functions
 - mem-pool: only search head block for available space
 - block alloc: add lifecycle APIs for cache_entry structs
 - read-cache: make_cache_entry should take object_id struct
 - read-cache: teach refresh_cache_entry() to take istate
 For a large tree, the index needs to hold many cache entries
 allocated on heap.  These cache entries are now allocated out of a
 dedicated memory pool to amortize malloc(3) overhead.
* sb/object-store-grafts (2018-05-18) 19 commits
  (merged to 'next' on 2018-06-28 at 02f70d6302)
 + commit: allow lookup_commit_graft to handle arbitrary repositories
 + commit: allow prepare_commit_graft to handle arbitrary repositories
 + shallow: migrate shallow information into the object parser
 + path.c: migrate global git_path_* to take a repository argument
 + cache: convert get_graft_file to handle arbitrary repositories
 + commit: convert read_graft_file to handle arbitrary repositories
 + commit: convert register_commit_graft to handle arbitrary repositories
 + commit: convert commit_graft_pos() to handle arbitrary repositories
 + shallow: add repository argument to is_repository_shallow
 + shallow: add repository argument to check_shallow_file_for_update
 + shallow: add repository argument to register_shallow
 + shallow: add repository argument to set_alternate_shallow_file
 + commit: add repository argument to lookup_commit_graft
 + commit: add repository argument to prepare_commit_graft
 + commit: add repository argument to read_graft_file
 + commit: add repository argument to register_commit_graft
 + commit: add repository argument to commit_graft_pos
 + object: move grafts to object parser
 + object-store: move object access functions to object-store.h
 Originally merged to 'next' on 2018-06-22
 The conversion to pass "the_repository" and then "a_repository"
 throughout the object access API continues.
 Will merge to 'master'.
* js/branch-diff (2018-05-16) 19 commits
 - fixup! Add a function to solve least-cost assignment problems
 - completion: support branch-diff
 - branch-diff: add a man page
 - branch-diff --dual-color: work around bogus white-space warning
 - branch-diff: offer to dual-color the diffs
 - diff: add an internal option to dual-color diffs of diffs
 - color: provide inverted colors, too
 - branch-diff: use color for the commit pairs
 - branch-diff: add tests
 - branch-diff: do not show "function names" in hunk headers
 - branch-diff: adjust the output of the commit pairs
 - branch-diff: suppress the diff headers
 - branch-diff: indent the diffs just like tbdiff
 - branch-diff: right-trim commit messages
 - branch-diff: also show the diff between patches
 - branch-diff: improve the order of the shown commits
 - branch-diff: first rudimentary implementation
 - Add a new builtin: branch-diff
 - Add a function to solve least-cost assignment problems
 "git tbdiff" that lets us compare individual patches in two
 iterations of a topic has been rewritten and made into a built-in
 command.
 Expecting a reroll.
 cf. <nycvar.QRO.7.76.6.1805052351560.77@tvgsbejvaqbjf.bet>
* sb/diff-color-move-more (2018-06-25) 11 commits
 - diff: fix a sparse 'dubious one-bit signed bitfield' error
 - SQUASH??? t/4015 GETTEXT_POISON emergency fix
 - SQUASH????? Documentation breakage emergency fix
 - diff.c: add white space mode to move detection that allows indent changes
 - diff.c: factor advance_or_nullify out of mark_color_as_moved
 - diff.c: decouple white space treatment from move detection algorithm
 - diff.c: add a blocks mode for moved code detection
 - diff.c: adjust hash function signature to match hashmap expectation
 - diff.c: do not pass diff options as keydata to hashmap
 - xdiff/xdiffi.c: remove unneeded function declarations
 - xdiff/xdiff.h: remove unused flags
 "git diff --color-moved" feature has further been tweaked.
 Needs to be cleaned-up with various fix-up bits applied inline.
* jk/branch-l-0-deprecation (2018-06-22) 3 commits
 - branch: deprecate "-l" option
 - t: switch "branch -l" to "branch --create-reflog"
 - t3200: unset core.logallrefupdates when testing reflog creation
 (this branch is used by jk/branch-l-1-repurpose.)
 The "-l" option in "git branch -l" is an unfortunate short-hand for
 "--create-reflog", but many users, both old and new, somehow expect
 it to be something else, perhaps "--list".  This step warns when "-l"
 is used as a short-hand for "--create-reflog" and warns about the
 future repurposing of the it when it is used.
 Will merge to 'next'.
Stefan Beller· Jun 28, 2018, 22:42 UTC · re: Junio C Hamano · lore

Re: What's cooking in git.git (Jun 2018, #07; Thu, 28)

On Thu, Jun 28, 2018 at 2:40 PM Junio C Hamano <gitster@pobox.com> wrote:
> The tip of 'next' has been rewound and it currently has only 4
> topics.  Quite a many topics are cooking in 'pu' and need to be
> sifted into good bins (for 'next') and the remainder.  Help is
> appreciated in that area ;-)

Which branches are needing review most? Or should I start commenting on random stuff? ;-)

Show 15 quoted lines
>
> * bw/ref-in-want (2018-06-28) 8 commits
>  - fetch-pack: implement ref-in-want
>  - fetch-pack: put shallow info in output parameter
>  - fetch: refactor to make function args narrower
>  - fetch: refactor fetch_refs into two functions
>  - fetch: refactor the population of peer ref OIDs
>  - upload-pack: test negotiation with changing repository
>  - upload-pack: implement ref-in-want
>  - test-pkt-line: add unpack-sideband subcommand
>
>  Protocol v2 has been updated to allow slightly out-of-sync set of
>  servers to work together to serve a single client, which would be
>  useful with load-balanced servers that talk smart HTTP transport.
>

I think another selling point, which should be emphasized more is the ease of ACL checking on the server side. Even when the read permissions are per repository (e.g. githubs model AFAICT), the serving side still has to do a lookup "Can a wanted sha1 be reached from all refs?", which now can be optimized away ("Can I access the master branch?") and it is cheaper to run the server. Specifically if read permissions are per ref (Gerrits model), I'd expect some CPU savings on the serving side.

Show 6 quoted lines
> * jt/remove-pack-bitmap-global (2018-06-21) 2 commits
>  - pack-bitmap: add free function
>  - pack-bitmap: remove bitmap_git global variable
>
>  The effort to move globals to per-repository in-core structure
>  continues.

This is mostly done, though Peff seems to expect a reroll with clarification on how the series is structured? https://public-inbox.org/git/20180611211033.GB26235@sigill.intra.peff.net/

Show 5 quoted lines
> * sb/submodule-move-head-error-msg (2018-06-25) 1 commit
>  - submodule.c: report the submodule that an error occurs in
>
>  Needs a reroll.
>  cf. <20180622081713.5360-1-szeder.dev@gmail.com>
https://public-inbox.org/git/xmqqmuviq2n7.fsf@gitster-ct.c.googlers.com/
suggests that you applied that change and a reroll would not be needed.
Show 55 quoted lines
>
> * ds/multi-pack-index (2018-06-25) 24 commits
>  - midx: clear midx on repack
>  - packfile: skip loading index if in multi-pack-index
>  - midx: prevent duplicate packfile loads
>  - midx: use midx in approximate_object_count
>  - midx: use existing midx when writing new one
>  - midx: use midx in abbreviation calculations
>  - midx: read objects from multi-pack-index
>  - midx: prepare midxed_git struct
>  - config: create core.multiPackIndex setting
>  - midx: write object offsets
>  - midx: write object id fanout chunk
>  - midx: write object ids in a chunk
>  - midx: sort and deduplicate objects from packfiles
>  - midx: read pack names into array
>  - multi-pack-index: write pack names in chunk
>  - multi-pack-index: read packfile list
>  - packfile: generalize pack directory list
>  - multi-pack-index: expand test data
>  - multi-pack-index: load into memory
>  - midx: write header information to lockfile
>  - multi-pack-index: add 'write' verb
>  - multi-pack-index: add builtin
>  - multi-pack-index: add format details
>  - multi-pack-index: add design document
>
>  When there are too many packfiles in a repository (which is not
>  recommended), looking up an object in these would require
>  consulting many pack .idx files; a new mechanism to have a single
>  file that consolidates all of these .idx files is introduced.
>
>
> * nd/use-the-index-compat-less (2018-06-25) 13 commits
>  - wt-status.c: stop using index compat macros
>  - sha1-name.c: stop using index compat macros
>  - sequencer.c: stop using index compat macros
>  - revision.c: stop using index compat macros
>  - rerere.c: stop using index compat macros
>  - merge.c: stop using index compat macros
>  - merge-recursive.c: stop using index compat macros
>  - entry.c: stop using index compat macros
>  - diff.c: stop using index compat macros
>  - diff-lib.c: stop using index compat macros
>  - check-racy.c: stop using index compat macros
>  - blame.c: stop using index compat macros
>  - apply.c: stop using index compat macros
>
>  Use of many convenience functions that operate on the_index
>  "primary in-core index instance" have been rewritten to explicitly
>  call the coutnerpart functions that take arbitrary index_state and
>  pass the_index.
>
>  I'd say that alone is a useless code churn, but people seem to be
>  excited about the change, so it is queued here.

As someone who also does lots of refactoring lately, I am excited to see the code health moving in the right direction.

It is easy to quantify how often we are bitten by code churn (that you call useless here); and very hard to quantify bugs introduced or features not written/upstreamed due to clunky API (as we don't see those or do not attribute a bug to a bad API). So we are naturally biased towards seeing code churn as bad, as the signals favor one over the other.

Show 15 quoted lines
>
> * jt/fetch-pack-negotiator (2018-06-15) 7 commits
>  - fetch-pack: introduce negotiator API
>  - fetch-pack: move common check and marking together
>  - fetch-pack: make negotiation-related vars local
>  - fetch-pack: use ref adv. to prune "have" sent
>  - fetch-pack: directly end negotiation if ACK ready
>  - fetch-pack: clear marks before re-marking
>  - fetch-pack: split up everything_local()
>  (this branch is used by jt/fetch-nego-tip.)
>
>  Code restructuring and a small fix to transport protocol v2 during
>  fetching.
>
>  Is this ready for 'next'?
That is a good topic to review for me, I'll jump on it.
> * ds/commit-graph-fsck (2018-06-27) 22 commits
[...]
Show 5 quoted lines
>
>  "git fsck" learns to make sure the optional commit-graph file is in
>  a sane state.
>
>  Is this ready for 'next'?

I hope so, as I plan to reroll the next object store series based on it. I'll also review that.

Show 14 quoted lines
>
> * jm/cache-entry-from-mem-pool (2018-06-28) 8 commits
>  - block alloc: add validations around cache_entry lifecyle
>  - block alloc: allocate cache entries from mem-pool
>  - mem-pool: fill out functionality
>  - mem-pool: add life cycle management functions
>  - mem-pool: only search head block for available space
>  - block alloc: add lifecycle APIs for cache_entry structs
>  - read-cache: make_cache_entry should take object_id struct
>  - read-cache: teach refresh_cache_entry() to take istate
>
>  For a large tree, the index needs to hold many cache entries
>  allocated on heap.  These cache entries are now allocated out of a
>  dedicated memory pool to amortize malloc(3) overhead.
This is also on my review todo list, still.
Show 5 quoted lines
>
> * sb/diff-color-move-more (2018-06-25) 11 commits
>  - diff: fix a sparse 'dubious one-bit signed bitfield' error
>  - SQUASH??? t/4015 GETTEXT_POISON emergency fix
>  - SQUASH????? Documentation breakage emergency fix
[...]
>
>  "git diff --color-moved" feature has further been tweaked.
>
>  Needs to be cleaned-up with various fix-up bits applied inline.

I'll resend with those squashes and another (test-)fix SZEDER mentioned soon.

Thanks, Stefan

Derrick Stolee· Jun 29, 2018, 13:55 UTC · re: Stefan Beller · lore

Re: What's cooking in git.git (Jun 2018, #07; Thu, 28)

On 6/28/2018 6:42 PM, Stefan Beller wrote:
Show 10 quoted lines
> On Thu, Jun 28, 2018 at 2:40 PM Junio C Hamano <gitster@pobox.com> wrote:
>
>> * ds/commit-graph-fsck (2018-06-27) 22 commits
> [...]
>>   "git fsck" learns to make sure the optional commit-graph file is in
>>   a sane state.
>>
>>   Is this ready for 'next'?
> I hope so, as I plan to reroll the next object store series based
> on it. I'll also review that.

I think the series is ready. The only new feedback in a while is your style comments on "commit-graph: add '--reachable' option", which can hopefully be squashed in. I'm also open to sending a new series on top for new feedback.

Thanks,
-Stolee
Junio C Hamano· Jun 29, 2018, 19:51 UTC · re: Stefan Beller · lore

Re: What's cooking in git.git (Jun 2018, #07; Thu, 28)

Stefan Beller <sbeller@google.com> writes:
Show 10 quoted lines
>> * jt/remove-pack-bitmap-global (2018-06-21) 2 commits
>>  - pack-bitmap: add free function
>>  - pack-bitmap: remove bitmap_git global variable
>>
>>  The effort to move globals to per-repository in-core structure
>>  continues.
>
> This is mostly done, though Peff seems to expect a reroll with
> clarification on how the series is structured?
> https://public-inbox.org/git/20180611211033.GB26235@sigill.intra.peff.net/

That one has been resolved by squashing the updated log message in, I think, so we should be able to merge it down.

Show 9 quoted lines
>> * sb/submodule-move-head-error-msg (2018-06-25) 1 commit
>>  - submodule.c: report the submodule that an error occurs in
>>
>>  Needs a reroll.
>>  cf. <20180622081713.5360-1-szeder.dev@gmail.com>
>
> https://public-inbox.org/git/xmqqmuviq2n7.fsf@gitster-ct.c.googlers.com/
>
> suggests that you applied that change and a reroll would not be needed.
Yup, I forgot about that one.  Thanks.
> It is easy to quantify how often we are bitten by code churn
> (that you call useless here); and very hard to quantify bugs
By definition, churn is useless.  Useful ones are refactoring ;-).

And when you do want to operate on _the_ single in-core instance, not having to say &the_index in the argument and use $foo_cache() function does *not* become a source of "bugs". It just saves typing, and turning it to $foo_index(&the_index,...) does *not* make it less error prone.

Show 12 quoted lines
>> * sb/diff-color-move-more (2018-06-25) 11 commits
>>  - diff: fix a sparse 'dubious one-bit signed bitfield' error
>>  - SQUASH??? t/4015 GETTEXT_POISON emergency fix
>>  - SQUASH????? Documentation breakage emergency fix
> [...]
>>
>>  "git diff --color-moved" feature has further been tweaked.
>>
>>  Needs to be cleaned-up with various fix-up bits applied inline.
>
> I'll resend with those squashes and another (test-)fix SZEDER
> mentioned soon.
The interdiff of the topic looked alright.  Thanks.
Alban Gruin· Jun 29, 2018, 11:10 UTC · re: Junio C Hamano · lore

ag/rebase-i-append-todo-help, was Re: What's cooking in git.git (Jun 2018, #07; Thu, 28)

Hi Junio,
Le 28/06/2018 à 23:40, Junio C Hamano a écrit :
Show 11 quoted lines
> * ag/rebase-i-append-todo-help (2018-06-14) 2 commits
>  - rebase--interactive: rewrite append_todo_help() in C
>  - Merge branch 'ag/rebase-p' into ag/rebase-i-append-todo-help
>  (this branch is used by ag/rebase-i-rewrite-todo.)
> 
>  Stepwise rewriting of the machinery of "rebase -i" into C continues.
> 
>  Needs a reroll.
>  cf. <nycvar.QRO.7.76.6.1806261125330.21419@tvgsbejvaqbjf.bet>
> 
> 

As I moved append_todo_help() to `rebase-interactive.c`, it should be because I did not changed `msg = _(…)` to `msg = N_(…)`.

It was pointed out on IRC that it was not necessary, after all[0]. Basically, N_() is needed for static variables, not for "dynamic" variables like `msg` in append_todo_help().

[0] http://colabti.org/irclogger/irclogger_log/git-devel?date=2018-06-26#l44

Cheers, Alban

Ævar Arnfjörð Bjarmason· Jun 29, 2018, 21:04 UTC · re: Junio C Hamano · lore

Re: What's cooking in git.git (Jun 2018, #07; Thu, 28)

On Thu, Jun 28 2018, Junio C Hamano wrote:
> The tip of 'next' has been rewound and it currently has only 4
> topics.  Quite a many topics are cooking in 'pu' and need to be
> sifted into good bins (for 'next') and the remainder.  Help is
> appreciated in that area ;-)

Per my https://public-inbox.org/git/CACBZZX4yG5h5kk4NFQz_NzAweMa+Nh3H-39OHtcH4XWsA6FGpg@mail.gmail.com/ (seems to not have been seen) I'd like to suggest that:

[...]
> * ab/fetch-tags-noclobber (2018-05-16) 9 commits
This be ejected for now.
[...]
> * ab/checkout-default-remote (2018-06-11) 8 commits
And this be merged down to "next".
Johannes Schindelin· Jul 3, 2018, 11:44 UTC · re: Junio C Hamano · lore

as/safecrlf-quiet-fix, was Re: What's cooking in git.git (Jun 2018, #07; Thu, 28)

Hi Junio,
On Thu, 28 Jun 2018, Junio C Hamano wrote:
Show 5 quoted lines
> * as/safecrlf-quiet-fix (2018-06-11) 1 commit
>   (merged to 'next' on 2018-06-13 at b163674843)
>  + config.c: fix regression for core.safecrlf false
> 
>  Fix for 2.17-era regression around `core.safecrlf`.
Is this `maint` material?

Ciao, Dscho

Junio C Hamano· Jul 3, 2018, 17:59 UTC · re: Johannes Schindelin · lore

Re: as/safecrlf-quiet-fix, was Re: What's cooking in git.git (Jun 2018, #07; Thu, 28)

Johannes Schindelin <Johannes.Schindelin@gmx.de> writes:
Show 11 quoted lines
> Hi Junio,
>
> On Thu, 28 Jun 2018, Junio C Hamano wrote:
>
>> * as/safecrlf-quiet-fix (2018-06-11) 1 commit
>>   (merged to 'next' on 2018-06-13 at b163674843)
>>  + config.c: fix regression for core.safecrlf false
>> 
>>  Fix for 2.17-era regression around `core.safecrlf`.
>
> Is this `maint` material?
Possibly.  Why do you ask?
Johannes Schindelin· Jul 4, 2018, 09:25 UTC · re: Junio C Hamano · lore

Re: as/safecrlf-quiet-fix, was Re: What's cooking in git.git (Jun 2018, #07; Thu, 28)

Hi Junio,
On Tue, 3 Jul 2018, Junio C Hamano wrote:
Show 13 quoted lines
> Johannes Schindelin <Johannes.Schindelin@gmx.de> writes:
> 
> > On Thu, 28 Jun 2018, Junio C Hamano wrote:
> >
> >> * as/safecrlf-quiet-fix (2018-06-11) 1 commit
> >>   (merged to 'next' on 2018-06-13 at b163674843)
> >>  + config.c: fix regression for core.safecrlf false
> >> 
> >>  Fix for 2.17-era regression around `core.safecrlf`.
> >
> > Is this `maint` material?
> 
> Possibly.  Why do you ask?

This is my feeble attempt at helping you triage. My question was more like a suggestion.

Ciao, Dscho

Junio C Hamano· Jul 6, 2018, 15:38 UTC · re: Johannes Schindelin · lore

Re: as/safecrlf-quiet-fix, was Re: What's cooking in git.git (Jun 2018, #07; Thu, 28)

Johannes Schindelin <Johannes.Schindelin@gmx.de> writes:
Show 18 quoted lines
> On Tue, 3 Jul 2018, Junio C Hamano wrote:
>
>> Johannes Schindelin <Johannes.Schindelin@gmx.de> writes:
>> 
>> > On Thu, 28 Jun 2018, Junio C Hamano wrote:
>> >
>> >> * as/safecrlf-quiet-fix (2018-06-11) 1 commit
>> >>   (merged to 'next' on 2018-06-13 at b163674843)
>> >>  + config.c: fix regression for core.safecrlf false
>> >> 
>> >>  Fix for 2.17-era regression around `core.safecrlf`.
>> >
>> > Is this `maint` material?
>> 
>> Possibly.  Why do you ask?
>
> This is my feeble attempt at helping you triage. My question was more like
> a suggestion.

Ah, thanks. That line of suggestion does help, and it would even be more helpful when given before the topic hits 'next' or after 'master' from the workflow's point of view.

While the topic is in 'pu' I can change where the topic forks from, and to minimize temptation to stupidly merge an inappropriate topic (e.g. new developments and somewhat dubious fixes to commonly used part of Git) down to 'maint' during the pre-release freeze (i.e. when the tree is otherwise hopefully sleepily quiet and boring), I try to base the (beginning of) good 'maint' material on 'maint' or older, and everything else on the tip of 'master' (or the most recent feature release), when each topic is originally queued in my tree. If I missed a good topic and based it on 'master', it is easier to correct such a mistake before it hits 'next', and a suggestion like above would help me tremendously.

After the topic hits 'next', its course is pretty much set, until it graduates to 'master'. About a week or two after such event, I'd go through "merge ... later to maint" entries in the draft release notes and if the topic still looked relevant to the maintenance branch, merge them to 'maint'. Before that happens is another chance to stop me from making a stupid mistake, or remind me of an urgent one that I've been letting wait in 'master' before merging to 'maint'.

Thanks.
Johannes Schindelin· Jul 3, 2018, 11:46 UTC · re: Junio C Hamano · lore

jh/partial-clone, was Re: What's cooking in git.git (Jun 2018, #07; Thu, 28)

Hi Junio,
On Thu, 28 Jun 2018, Junio C Hamano wrote:
Show 7 quoted lines
> * jh/partial-clone (2018-06-12) 1 commit
>   (merged to 'next' on 2018-06-13 at 818f864b0c)
>  + list-objects: check if filter is NULL before using
> 
>  The recent addition of "partial clone" experimental feature kicked
>  in when it shouldn't, namely, when there is no partial-clone filter
>  defined even if extensions.partialclone is set.
Is this `maint` material?

Ciao, Dscho

Johannes Schindelin· Jul 3, 2018, 11:54 UTC · re: Junio C Hamano · lore

as/sequencer-customizable-comment-char, was Re: What's cooking in git.git (Jun 2018, #07; Thu, 28)

Hi Junio & Aaron,
On Thu, 28 Jun 2018, Junio C Hamano wrote:
Show 5 quoted lines
> * as/sequencer-customizable-comment-char (2018-06-28) 1 commit
>  - sequencer: use configured comment character
> 
>  Honor core.commentchar when preparing the list of commits to replay
>  in "rebase -i".

As per https://public-inbox.org/git/nycvar.QRO.7.76.6.1806291607501.74@tvgsbejvaqbjf.bet/ I am inclined to ask for a "v2" (i.e. second iteration of the patch, see e.g. https://github.com/git-for-windows/git/blob/master/CONTRIBUTING.md#submit-the-patch where it talks about submitting an "nth version") that adds an explanation to the commit message why the "auto" value is handled correctly by this patch, as this is not really obvious from reading the diff alone.

After that change, I think this is ready for `next`.

Ciao, Dscho

Johannes Schindelin· Jul 3, 2018, 12:52 UTC · re: Junio C Hamano · lore

ag/rebase-i-rewrite-todo, was Re: What's cooking in git.git (Jun 2018, #07; Thu, 28)

Hi Junio,
On Thu, 28 Jun 2018, Junio C Hamano wrote:
Show 7 quoted lines
> * ag/rebase-i-rewrite-todo (2018-06-15) 3 commits
>  - rebase--interactive: rewrite the edit-todo functionality in C
>  - editor: add a function to launch the sequence editor
>  - Merge branch 'bc/t3430-fixup' into ag/rebase-i-rewrite-todo
>  (this branch uses ag/rebase-i-append-todo-help.)
> 
>  Stepwise rewriting of the machinery of "rebase -i" into C continues.

Alban abided by your wish and smooshed those patch series into one honking big one, to grow even further with his ongoing work. I have to admit that I am not enthusiastic about this, as it will make it even harder to squeeze in the time for me to review those patches.

The latest iteration of this is here: https://public-inbox.org/git/20180702105717.26386-5-alban.gruin@gmail.com/T/#r8eea71077745d6f2c839acb6200bb8b2bea579d3

I would *strongly* encourage you to allow Alban to go back to the small, incremental patch series he sent before, because it will make it *substantially* easier to not only review, but also develop, and for you to merge.

In this case, for example, the two patches in https://public-inbox.org/git/20180702105717.26386-4-alban.gruin@gmail.com/ and https://public-inbox.org/git/20180702105717.26386-5-alban.gruin@gmail.com/ are ready to go into `next`, like, right now.

Yet because they are now embedded in a larger patch series, they will have to be re-rolled over and over again until *all* of `git-rebase--interactive.sh` is rewritten in C? That's really wasteful of reviewers' time.

Ciao, Dscho

Junio C Hamano· Jul 6, 2018, 17:50 UTC · re: Johannes Schindelin · lore

Re: ag/rebase-i-rewrite-todo, was Re: What's cooking in git.git (Jun 2018, #07; Thu, 28)

Johannes Schindelin <Johannes.Schindelin@gmx.de> writes:
> The latest iteration of this is here:
> https://public-inbox.org/git/20180702105717.26386-5-alban.gruin@gmail.com/T/#r8eea71077745d6f2c839acb6200bb8b2bea579d3
Good.  I think we have it in tree now.
> I would *strongly* encourage you to allow Alban to go back to the small,
> incremental patch series he sent before, because it will make it
> *substantially* easier to not only review, but also develop, and for you
> to merge.

An organization in which you can make sure that the order of dependency and which ones have been updated since previous rounds are clear, even to those who are looking from the sidelines ("these 4 patches are to replace patch 3, 7 and 8 from the previous round" is already hostile to late reviewers and doing so without a pointer to the archive is even worse---a full reroll with the unchanged ones marked below the three-dash lines would be perfect), would be good. A random collection of seemingly separate but actually interdependent topics is very hard to work with with limited mental bandwidth.

Once the core of _a_ topic hits 'next', we can go incremental (because by definition things get quiet and require small updates by then), but not before.

I think the 7 patches in ag/rebase-i-in-c are more or less in good shape, modulo the issues pointed out on the list yet to be addressed, which I do not think require redesign. Which is good.

Johannes Schindelin· Jul 6, 2018, 19:21 UTC · re: Junio C Hamano · lore

Re: ag/rebase-i-rewrite-todo, was Re: What's cooking in git.git (Jun 2018, #07; Thu, 28)

Hi Junio,
On Fri, 6 Jul 2018, Junio C Hamano wrote:
Show 24 quoted lines
> Johannes Schindelin <Johannes.Schindelin@gmx.de> writes:
> 
> > I would *strongly* encourage you to allow Alban to go back to the small,
> > incremental patch series he sent before, because it will make it
> > *substantially* easier to not only review, but also develop, and for
> > you to merge.
> 
> An organization in which you can make sure that the order of dependency
> and which ones have been updated since previous rounds are clear, even
> to those who are looking from the sidelines ("these 4 patches are to
> replace patch 3, 7 and 8 from the previous round" is already hostile to
> late reviewers and doing so without a pointer to the archive is even
> worse---a full reroll with the unchanged ones marked below the
> three-dash lines would be perfect), would be good.  A random collection
> of seemingly separate but actually interdependent topics is very hard to
> work with with limited mental bandwidth.
> 
> Once the core of _a_ topic hits 'next', we can go incremental (because
> by definition things get quiet and require small updates by then), but
> not before.
> 
> I think the 7 patches in ag/rebase-i-in-c are more or less in good
> shape, modulo the issues pointed out on the list yet to be addressed,
> which I do not think require redesign.  Which is good.

You do understand that with your proposed "let's just roll them up into one big patch series, and just add freely whatever you need on top", these 7 patches (3 of which I reviewed I think four times on the list now, and more times on GitHub, which is quite taxing on my time) will be soon joined by 6 more patches: https://github.com/git/git/pull/518

Of course, at that point I will have to look through those 7 patches again, if only to verify that yes, they are still the same.

And Alban is not sitting on his hands, either.

So after reviewing those 13 patches, which undoubtedly will not be integrated into `next` under the premise that they are still in flux, they will most likely be joined by another dozen patches until the interactive rebase is rewritten completely in C. After which time, I will have reviewed the first 3 patches over 15 times.

I wish there was a better way.

Ciao, Dscho

Junio C Hamano· Jul 6, 2018, 19:40 UTC · re: Johannes Schindelin · lore

Re: ag/rebase-i-rewrite-todo, was Re: What's cooking in git.git (Jun 2018, #07; Thu, 28)

Johannes Schindelin <Johannes.Schindelin@gmx.de> writes:
> Of course, at that point I will have to look through those 7 patches
> again, if only to verify that yes, they are still the same.
That is something the patch author must help the reviewer with, no?

Have uncontroversial stuff early in the series, concentrate on stabilizing them before moving on to shiny new toys, then mark them "unchanged since the last round" after three dashes when sending a reroll to update later parts of the series that are in flux. After a few rounds, it may become apparent to reviewers that these early parts can stand on their own merit as a separate series, on top of which the remaining patches can build as a separate (sub)topic, at which time we may have two or more topics, among which the early one that has become stable may already be 'next'.

> And Alban is not sitting on his hands, either.

If you are saying that all of these 13 are constantly in flux, whether these 13 patches are spread across 3 series or a single one (assuming that all 13 are about the same topic that are interdependant), the need to look at their updated incarnation does not change, so I do not know what you are complaining about.

Show 7 quoted lines
> So after reviewing those 13 patches, which undoubtedly will not be
> integrated into `next` under the premise that they are still in flux, they
> will most likely be joined by another dozen patches until the interactive
> rebase is rewritten completely in C. After which time, I will have
> reviewed the first 3 patches over 15 times.
>
> I wish there was a better way.
Johannes Schindelin· Jul 6, 2018, 22:38 UTC · re: Junio C Hamano · lore

Re: ag/rebase-i-rewrite-todo, was Re: What's cooking in git.git (Jun 2018, #07; Thu, 28)

Hi Junio,
On Fri, 6 Jul 2018, Junio C Hamano wrote:
Show 16 quoted lines
> Johannes Schindelin <Johannes.Schindelin@gmx.de> writes:
> 
> > Of course, at that point I will have to look through those 7 patches
> > again, if only to verify that yes, they are still the same.
> 
> That is something the patch author must help the reviewer with, no?
> 
> Have uncontroversial stuff early in the series, concentrate on
> stabilizing them before moving on to shiny new toys, then mark them
> "unchanged since the last round" after three dashes when sending a
> reroll to update later parts of the series that are in flux.  After
> a few rounds, it may become apparent to reviewers that these early
> parts can stand on their own merit as a separate series, on top of
> which the remaining patches can build as a separate (sub)topic, at
> which time we may have two or more topics, among which the early one
> that has become stable may already be 'next'.

Okay, I give up. I tried to show you that I thought that Alban's partitioning made sense on its own, that they could (and should) stabilize independently, even if they technically build on one another. In fact, it was I who suggested to keep them in bite-sized patch series. And you simply disagree from your maintaining point of view, and I violently disagree with your decision from the reviewer's point of view.

In the end, you win. Dscho

Johannes Schindelin· Jul 3, 2018, 13:05 UTC · re: Junio C Hamano · lore

js/branch-diff, was Re: What's cooking in git.git (Jun 2018, #07; Thu, 28)

Hi Junio,
On Thu, 28 Jun 2018, Junio C Hamano wrote:
Show 27 quoted lines
> * js/branch-diff (2018-05-16) 19 commits
>  - fixup! Add a function to solve least-cost assignment problems
>  - completion: support branch-diff
>  - branch-diff: add a man page
>  - branch-diff --dual-color: work around bogus white-space warning
>  - branch-diff: offer to dual-color the diffs
>  - diff: add an internal option to dual-color diffs of diffs
>  - color: provide inverted colors, too
>  - branch-diff: use color for the commit pairs
>  - branch-diff: add tests
>  - branch-diff: do not show "function names" in hunk headers
>  - branch-diff: adjust the output of the commit pairs
>  - branch-diff: suppress the diff headers
>  - branch-diff: indent the diffs just like tbdiff
>  - branch-diff: right-trim commit messages
>  - branch-diff: also show the diff between patches
>  - branch-diff: improve the order of the shown commits
>  - branch-diff: first rudimentary implementation
>  - Add a new builtin: branch-diff
>  - Add a function to solve least-cost assignment problems
> 
>  "git tbdiff" that lets us compare individual patches in two
>  iterations of a topic has been rewritten and made into a built-in
>  command.
> 
>  Expecting a reroll.
>  cf. <nycvar.QRO.7.76.6.1805052351560.77@tvgsbejvaqbjf.bet>
I rolled it around so hard that it got dizzy.
(Yes, I just made fun of this Git speak "to re-roll".)
Seriously again, I sent a new iteration:
	https://public-inbox.org/git/pull.1.v3.git.gitgitgadget@gmail.com/

Ciao, Dscho

Junio C Hamano· Jul 3, 2018, 22:12 UTC · re: Johannes Schindelin · lore

Re: js/branch-diff, was Re: What's cooking in git.git (Jun 2018, #07; Thu, 28)

Johannes Schindelin <Johannes.Schindelin@gmx.de> writes:
> Seriously again, I sent a new iteration:
>
> 	https://public-inbox.org/git/pull.1.v3.git.gitgitgadget@gmail.com/

Thanks, will take a look but it is likely that I'll run out of time today, so please be a bit patient.

Johannes Schindelin· Jul 4, 2018, 09:27 UTC · re: Junio C Hamano · lore

Re: js/branch-diff, was Re: What's cooking in git.git (Jun 2018, #07; Thu, 28)

Hi Junio,
On Tue, 3 Jul 2018, Junio C Hamano wrote:
Show 8 quoted lines
> Johannes Schindelin <Johannes.Schindelin@gmx.de> writes:
> 
> > Seriously again, I sent a new iteration:
> >
> > 	https://public-inbox.org/git/pull.1.v3.git.gitgitgadget@gmail.com/
> 
> Thanks, will take a look but it is likely that I'll run out of time
> today, so please be a bit patient.

Sorry, I should have clarified that I do not at all expect reviews to be performed *right now*. It took *me* over a month to get this iteration out, after all, and as the changes list suggests: there had been a *ton* of changes.

While I need the functionality of `range-diff` myself, I'm quite fine over here with my custom build (which I use via an alias from my default git.exe).

Ciao, Dscho

Elijah Newren· Jul 3, 2018, 18:30 UTC · re: Junio C Hamano · lore

Re: What's cooking in git.git (Jun 2018, #07; Thu, 28)

Hi Derrick,
On Thu, Jun 28, 2018 at 2:40 PM, Junio C Hamano <gitster@pobox.com> wrote:
Show 30 quoted lines
> * ds/multi-pack-index (2018-06-25) 24 commits
>  - midx: clear midx on repack
>  - packfile: skip loading index if in multi-pack-index
>  - midx: prevent duplicate packfile loads
>  - midx: use midx in approximate_object_count
>  - midx: use existing midx when writing new one
>  - midx: use midx in abbreviation calculations
>  - midx: read objects from multi-pack-index
>  - midx: prepare midxed_git struct
>  - config: create core.multiPackIndex setting
>  - midx: write object offsets
>  - midx: write object id fanout chunk
>  - midx: write object ids in a chunk
>  - midx: sort and deduplicate objects from packfiles
>  - midx: read pack names into array
>  - multi-pack-index: write pack names in chunk
>  - multi-pack-index: read packfile list
>  - packfile: generalize pack directory list
>  - multi-pack-index: expand test data
>  - multi-pack-index: load into memory
>  - midx: write header information to lockfile
>  - multi-pack-index: add 'write' verb
>  - multi-pack-index: add builtin
>  - multi-pack-index: add format details
>  - multi-pack-index: add design document
>
>  When there are too many packfiles in a repository (which is not
>  recommended), looking up an object in these would require
>  consulting many pack .idx files; a new mechanism to have a single
>  file that consolidates all of these .idx files is introduced.

pu fails to build for me (with the standard 'make DEVELOPER=1 ...'), and it appears to come from this series:

midx.c:567:15: error: ‘write_midx_pack_lookup’ defined but not used
[-Werror=unused-function]
 static size_t write_midx_pack_lookup(struct hashfile *f,

It looks like the use of this function was removed in v2 of your series, but the function itself was left behind? Could you take a look?

Thanks, Elijah

Eric Sunshine· Jul 3, 2018, 18:33 UTC · re: Elijah Newren · lore

Re: What's cooking in git.git (Jun 2018, #07; Thu, 28)

On Tue, Jul 3, 2018 at 2:31 PM Elijah Newren <newren@gmail.com> wrote:
Show 13 quoted lines
> On Thu, Jun 28, 2018 at 2:40 PM, Junio C Hamano <gitster@pobox.com> wrote:
> > * ds/multi-pack-index (2018-06-25) 24 commits
>
> pu fails to build for me (with the standard 'make DEVELOPER=1 ...'),
> and it appears to come from this series:
>
> midx.c:567:15: error: ‘write_midx_pack_lookup’ defined but not used
> [-Werror=unused-function]
>  static size_t write_midx_pack_lookup(struct hashfile *f,
>
> It looks like the use of this function was removed in v2 of your
> series, but the function itself was left behind?  Could you take a
> look?
Also reported here [1].
[1]: https://public-inbox.org/git/CAPig+cRWwTAFRJOwQOi4USk5UZke=2sz_JVDh3+XRKCcBGD5ow@mail.gmail.com/
Jonathan Tan· Jul 9, 2018, 17:31 UTC · re: Junio C Hamano · lore

Re: What's cooking in git.git (Jun 2018, #07; Thu, 28)

Show 14 quoted lines
> * jt/fetch-pack-negotiator (2018-06-15) 7 commits
>  - fetch-pack: introduce negotiator API
>  - fetch-pack: move common check and marking together
>  - fetch-pack: make negotiation-related vars local
>  - fetch-pack: use ref adv. to prune "have" sent
>  - fetch-pack: directly end negotiation if ACK ready
>  - fetch-pack: clear marks before re-marking
>  - fetch-pack: split up everything_local()
>  (this branch is used by jt/fetch-nego-tip.)
> 
>  Code restructuring and a small fix to transport protocol v2 during
>  fetching.
> 
>  Is this ready for 'next'?

Sorry for the late reply - I think that this is ready for "next". I have addressed all the comments that you, Jonathan Nieder and Brandon Williams have given, and Brandon has given his OK [1].

[1] https://public-inbox.org/git/20180625182434.GE19910@google.com/
Junio C Hamano· Jul 9, 2018, 18:13 UTC · re: Jonathan Tan · lore

Re: What's cooking in git.git (Jun 2018, #07; Thu, 28)

Jonathan Tan <jonathantanmy@google.com> writes:
Show 20 quoted lines
>> * jt/fetch-pack-negotiator (2018-06-15) 7 commits
>>  - fetch-pack: introduce negotiator API
>>  - fetch-pack: move common check and marking together
>>  - fetch-pack: make negotiation-related vars local
>>  - fetch-pack: use ref adv. to prune "have" sent
>>  - fetch-pack: directly end negotiation if ACK ready
>>  - fetch-pack: clear marks before re-marking
>>  - fetch-pack: split up everything_local()
>>  (this branch is used by jt/fetch-nego-tip.)
>> 
>>  Code restructuring and a small fix to transport protocol v2 during
>>  fetching.
>> 
>>  Is this ready for 'next'?
>
> Sorry for the late reply - I think that this is ready for "next". I have
> addressed all the comments that you, Jonathan Nieder and Brandon
> Williams have given, and Brandon has given his OK [1].
>
> [1] https://public-inbox.org/git/20180625182434.GE19910@google.com/

Thanks, I've re-read it over the weekend and agree that we are in a good shape here.

← back to recent threads