{"thread":{"id":"36397","subject":"What's cooking in git.git (Apr 2014, #03; Fri, 11)","startedAt":"2014-04-11T22:22:58Z","lastAt":"2014-04-17T17:52:45Z","messageCount":10,"participants":["Junio C Hamano","Felipe Contreras","Ramsay Jones","Michael Haggerty","Duy Nguyen","W. Trevor King"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"238754","messageId":"xmqq1tx3qzel.fsf@gitster.dls.corp.google.com","threadId":"36397","inReplyTo":null,"subject":"What's cooking in git.git (Apr 2014, #03; Fri, 11)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2014-04-11T22:22:58Z","receivedAt":"2014-04-11T22:22:58Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Here are the topics that have been cooking.  Commits prefixed with\n'-' are only in 'pu' (proposed updates) while commits prefixed with\n'+' are in 'next'.\n\nThe number of topics cooking in 'next' has been shrinking, and the\ncycle is getting long. Hopefully we will have -rc0 late next week to\nclose the 'master' branch and a few rounds of -rc iterations until\n2.0 final. Other topics under discussion may continue to flow to\n'next', as usual, during that time, to be pushed out early in the\nnext cycle.\n\nYou can find the changes described here in the integration branches\nof the repositories listed at\n\n    http://git-blame.blogspot.com/p/git-public-repositories.html\n\n--------------------------------------------------\n[New Topics]\n\n* nd/index-pack-one-fd-per-thread (2014-04-09) 1 commit\n - index-pack: work around thread-unsafe pread()\n\n Enable threaded index-pack on platforms without thread-unsafe\n pread() emulation.\n\n Will merge to 'next' and keep it there for the remainder of the cycle.\n\n\n* tb/unicode-6.3-zero-width (2014-04-09) 1 commit\n - utf8.c: partially update to version 6.3\n\n Teach our display-column-counting logic about decomposed umlauts\n and friends.\n\n Will merge to 'next'.\n\n\n* fc/complete-aliased-push (2014-04-09) 1 commit\n - completion: fix completing args of aliased \"push\", \"fetch\", etc.\n\n Will merge to 'next'.\n\n\n* fc/remote-helper-fixes (2014-04-09) 4 commits\n - remote-bzr: include authors field in pushed commits\n - remote-bzr: add support for older versions\n - remote-hg: always normalize paths\n - remote-helpers: allow all tests running from any dir\n\n Will merge to 'next'.\n\n\n* fc/publish-vs-upstream (2014-04-11) 8 commits\n - sha1_name: add support for @{publish} marks\n - sha1_name: simplify track finding\n - sha1_name: cleanup interpret_branch_name()\n - branch: display publish branch\n - push: add --set-publish option\n - branch: add --set-publish-to option\n - Add concept of 'publish' branch\n - t5516 (fetch-push): fix test restoration\n\n Add branch@{publish}; this round v3 hasn't yet seen much reviews\n yet.\n\n Seems to have some interactions to break tests when merged to 'pu'.\n\n\n* ym/fix-opportunistic-index-update-race (2014-04-10) 2 commits\n - read-cache.c: verify index file before we opportunistically update it\n - wrapper.c: add xpread() similar to xread()\n\n Duy raised a good point that we may need to do the same for the\n normal writeout codepath, not just the \"opportunistic\" update\n codepath.  While that is true, nobody sane would be running two\n simultaneous operations that are clearly write-oriented competing\n with each other against the same index file.  So in that sense that\n can be done as a less urgent follow-up for this topic.\n\n\n* km/avoid-bs-in-shell-glob (2014-04-11) 1 commit\n - test: fix t5560 on FreeBSD\n\n Portability fix.\n Will merge to 'next' and to 'master'.\n\n\n* km/avoid-cp-a (2014-04-11) 1 commit\n - test: fix t7001 cp to use POSIX options\n\n Portability fix.\n Will merge to 'next' and hopefully to 'master'.\n\n\n* rs/ref-closer-to-atomic (2014-04-11) 3 commits\n - refs.c: change ref_transaction_commit to run the commit loops once all work is finished\n - refs.c: split delete_ref_loose() into a separate flag-for-deletion and commit phase\n - refs.c: split writing and commiting a ref into two separate functions\n (this branch uses mh/ref-transaction.)\n\n Builds on top of Michael's ref-transaction series to shrink the\n race window during multiple ref updates.\n\n Appears to break some tests even standalone, and somewhat conflicts\n with Michael's mh/lockfile topic when merging to 'pu'.\n\n--------------------------------------------------\n[Graduated to \"master\"]\n\n* ib/rev-parse-parseopt-argh (2014-04-01) 1 commit\n  (merged to 'next' on 2014-04-01 at 025578d)\n + rev-parse: fix typo in example on manpage\n\n Finishing touch to a new topic scheduled for 2.0\n\n\n* jc/rev-parse-argh-dashed-multi-words (2014-03-24) 3 commits\n  (merged to 'next' on 2014-03-31 at 1c48649)\n + parse-options: make sure argh string does not have SP or _\n + update-index: teach --cacheinfo a new syntax \"mode,sha1,path\"\n + parse-options: multi-word argh should use dash to separate words\n\n Make sure that the help text given to describe the \"<param>\" part\n of the \"git cmd --option=<param>\" does not contain SP or _,\n e.g. \"--gpg-sign=<key-id>\" option for \"git commit\" is not spelled\n as \"--gpg-sign=<key id>\".\n\n\n* jk/commit-dates-parsing-fix (2014-04-01) 2 commits\n  (merged to 'next' on 2014-04-04 at c16eeb0)\n + t4212: loosen far-in-future test for AIX\n + date: recognize bogus FreeBSD gmtime output\n\n Finishing touches for portability.\n\n\n* jk/pack-bitmap (2014-04-04) 2 commits\n  (merged to 'next' on 2014-04-04 at 0306834)\n + pack-objects: do not reuse packfiles without --delta-base-offset\n + add `ignore_missing_links` mode to revwalk\n\n Fixes the pack-bitmap already in 'master'.\n\n\n* jl/nor-or-nand-and (2014-03-31) 4 commits\n  (merged to 'next' on 2014-04-04 at b5d1ac5)\n + code and test: fix misuses of \"nor\"\n + comments: fix misuses of \"nor\"\n + contrib: fix misuses of \"nor\"\n + Documentation: fix misuses of \"nor\"\n\n Eradicate mistaken use of \"nor\" (that is, essentially \"nor\" used\n not in \"neither A nor B\" ;-)) from in-code comments, command output\n strings, and documentations.\n\n\n* mh/update-ref-batch-create-fix (2014-04-02) 1 commit\n  (merged to 'next' on 2014-04-04 at 97e3f12)\n + update-ref: fail create operation over stdin if ref already exists\n\n Requesting \"update-ref --stdin\" to create a ref that already exists\n should have errored out, but didn't.\n\n\n* mr/msvc-link-with-invalidcontinue (2014-03-28) 1 commit\n  (merged to 'next' on 2014-03-31 at 051a29e)\n + MSVC: link in invalidcontinue.obj for better POSIX compatibility\n\n\n* mr/opt-set-ptr (2014-03-31) 3 commits\n  (merged to 'next' on 2014-04-03 at a26385b)\n + parse-options: remove unused OPT_SET_PTR\n + parse-options: add cast to correct pointer type to OPT_SET_PTR\n + MSVC: fix t0040-parse-options crash\n\n OPT_SET_PTR() implementation was broken on IL32P64 platforms;\n it turns out that the macro is not used by any real user.\n\n--------------------------------------------------\n[Stalled]\n\n* tr/merge-recursive-index-only (2014-02-05) 3 commits\n - merge-recursive: -Xindex-only to leave worktree unchanged\n - merge-recursive: internal flag to avoid touching the worktree\n - merge-recursive: remove dead conditional in update_stages()\n (this branch is used by tr/remerge-diff.)\n\n Will hold.\n\n\n* tr/remerge-diff (2014-02-26) 5 commits\n . log --remerge-diff: show what the conflict resolution changed\n . name-hash: allow dir hashing even when !ignore_case\n . merge-recursive: allow storing conflict hunks in index\n . revision: fold all merge diff variants into an enum merge_diff_mode\n . combine-diff: do not pass revs->dense_combined_merges redundantly\n (this branch uses tr/merge-recursive-index-only.)\n\n \"log -p\" output learns a new way to let users inspect a merge\n commit by showing the differences between the automerged result\n with conflicts the person who recorded the merge would have seen\n and the final conflict resolution that was recorded in the merge.\n\n Needs to be rebased, now kb/fast-hashmap topic is in.\n\n\n* bc/blame-crlf-test (2014-02-18) 1 commit\n - blame: add a failing test for a CRLF issue.\n\n I have a feeling that a fix for this should be fairly isolated and\n trivial (it should be just the matter of paying attention to the\n crlf settings when synthesizing the fake commit)---perhaps somebody\n can squash in a fix to this?\n\n\n* jk/makefile (2014-02-05) 16 commits\n - FIXUP\n - move LESS/LV pager environment to Makefile\n - Makefile: teach scripts to include make variables\n - FIXUP\n - Makefile: auto-build C strings from make variables\n - Makefile: drop *_SQ variables\n - FIXUP\n - Makefile: add c-quote helper function\n - Makefile: introduce sq function for shell-quoting\n - Makefile: always create files via make-var\n - Makefile: store GIT-* sentinel files in MAKE/\n - Makefile: prefer printf to echo for GIT-*\n - Makefile: use tempfile/mv strategy for GIT-*\n - Makefile: introduce make-var helper function\n - Makefile: fix git-instaweb dependency on gitweb\n - Makefile: drop USE_GETTEXT_SCHEME from GIT-CFLAGS\n\n Simplify the Makefile rules and macros that exist primarily for\n quoting purposes, and make it easier to robustly express the\n dependency rules.\n\n Expecting a reroll.\n\n\n* po/everyday-doc (2014-01-27) 1 commit\n - Make 'git help everyday' work\n\n This may make the said command to emit something, but the source is\n not meant to be formatted into a manual pages to begin with, and\n also its contents are a bit stale.  It may be a good first step in\n the right direction, but needs more work to at least get the\n mark-up right before public consumption.\n\n Will hold.\n\n\n* jk/branch-at-publish-rebased (2014-01-17) 5 commits\n . t1507 (rev-parse-upstream): fix typo in test title\n . implement @{publish} shorthand\n . branch_get: provide per-branch pushremote pointers\n . branch_get: return early on error\n . sha1_name: refactor upstream_mark\n\n Give an easier access to the tracking branches from \"other\" side in\n a triangular workflow by introducing B@{publish} that works in a\n similar way to how B@{upstream} does.\n\n Meant to be used as a basis for whatever Ram wants to build on.\n\n Ejected from 'pu' to make room for fc/publish-vs-upstream topic.\n\n\n* rb/merge-prepare-commit-msg-hook (2014-01-10) 4 commits\n - merge: drop unused arg from abort_commit method signature\n - merge: make prepare_to_commit responsible for write_merge_state\n - t7505: ensure cleanup after hook blocks merge\n - t7505: add missing &&\n\n Expose more merge states (e.g. $GIT_DIR/MERGE_MODE) to hooks that\n run during \"git merge\".  The log message stresses too much on one\n hook, prepare-commit-msg, but it would equally apply to other hooks\n like post-merge, I think.\n\n Waiting for a reroll.\n\n\n* jl/submodule-recursive-checkout (2013-12-26) 5 commits\n - Teach checkout to recursively checkout submodules\n - submodule: teach unpack_trees() to update submodules\n - submodule: teach unpack_trees() to repopulate submodules\n - submodule: teach unpack_trees() to remove submodule contents\n - submodule: prepare for recursive checkout of submodules\n\n Expecting a reroll.\n\n\n* jc/graph-post-root-gap (2013-12-30) 3 commits\n - WIP: document what we want at the end\n - graph: remove unused code a bit\n - graph: stuff the current commit into graph->columns[]\n\n This was primarily a RFH ($gmane/239580).\n\n\n* np/pack-v4 (2013-09-18) 90 commits\n . packv4-parse.c: add tree offset caching\n . t1050: replace one instance of show-index with verify-pack\n . index-pack, pack-objects: allow creating .idx v2 with .pack v4\n . unpack-objects: decode v4 trees\n . unpack-objects: allow to save processed bytes to a buffer\n - ...\n\n Nico and Duy advancing the eternal vaporware pack-v4.  This is here\n primarily for wider distribution of the preview edition.\n\n Needs to be rebased, now the pack-bitmap series is in.\n\n\n* tg/perf-lib-test-perf-cleanup (2013-09-19) 2 commits\n - perf-lib: add test_perf_cleanup target\n - perf-lib: split starting the test from the execution\n\n Add test_perf_cleanup shell function to the perf suite, that allows\n the script writers to define a test with a clean-up action.\n\n Will hold.\n\n\n* jc/format-patch (2013-04-22) 2 commits\n - format-patch: --inline-single\n - format-patch: rename \"no_inline\" field\n\n A new option to send a single patch to the standard output to be\n appended at the bottom of a message.  I personally have no need for\n this, but it was easy enough to cobble together.  Tests, docs and\n stripping out more MIMEy stuff are left as exercises to interested\n parties.\n\n\n* jc/show-branch (2014-03-24) 5 commits\n - show-branch: use commit slab to represent bitflags of arbitrary width\n - show-branch.c: remove \"all_mask\"\n - show-branch.c: abstract out \"flags\" operation\n - show-branch.c: lift all_mask/all_revs to a global static\n - show-branch.c: update comment style\n\n Waiting for the final step to lift the hard-limit before sending it out.\n\n--------------------------------------------------\n[Cooking]\n\n* jl/status-added-submodule-is-never-ignored (2014-04-07) 2 commits\n - commit -m: commit staged submodules regardless of ignore config\n - status/commit: show staged submodules regardless of ignore config\n\n\n* mh/multimail (2014-04-07) 1 commit\n - git-multimail: update to version 1.0.0\n\n\n* mh/lockfile (2014-04-07) 25 commits\n - trim_last_path_elm(): replace last_path_elm()\n - resolve_symlink(): take a strbuf parameter\n - resolve_symlink(): use a strbuf for internal scratch space\n - Change lock_file::filename into a strbuf\n - commit_lock_file(): use a strbuf to manage temporary space\n - try_merge_strategy(): use a statically-allocated lock_file object\n - try_merge_strategy(): remove redundant lock_file allocation\n - lockfile: avoid transitory invalid states\n - commit_lock_file(): make committing an unlocked lockfile a NOP\n - commit_lock_file(): inline temporary variable\n - remove_lock_file(): call rollback_lock_file()\n - lock_file(): exit early if lockfile cannot be opened\n - write_packed_entry_fn(): convert cb_data into a (const int *)\n - prepare_index(): declare return value to be (const char *)\n - delete_ref_loose(): don't muck around in the lock_file's filename\n - lockfile: define a constant LOCK_SUFFIX_LEN\n - lockfile.c: document the various states of lock_file objects\n - struct lock_file: replace on_list field with flags field\n - lock_file(): always add lock_file object to lock_file_list\n - hold_lock_file_for_append(): release lock on errors\n - lockfile: unlock file if lockfile permissions cannot be adjusted\n - rollback_lock_file(): set fd to -1\n - rollback_lock_file(): do not clear filename redundantly\n - unable_to_lock_die(): rename function from unable_to_lock_index_die()\n - api-lockfile: expand the documentation\n\n Refactor and fix corner-case bugs in the lockfile API.\n\n\n* mt/patch-id-stable (2014-03-31) 3 commits\n  (merged to 'next' on 2014-04-08 at 0188d44)\n + patch-id-test: test --stable and --unstable flags\n + patch-id: document new behaviour\n + patch-id: make it stable against hunk reordering\n\n Introduce a new way to compute patch-id for a patch that is not\n affected by the order of the paths that appear in the input.\n\n This changes the generated patch-id unless the users add an extra\n option to their command lines, but I deliberately queued the series\n to 'next' without reverting that compatibility breakage to see if\n people complain.  It could be that we do not have to worry about\n the default flipping at all.  We'll see.\n\n Will keep in 'next' for the remainder of this cycle.\n\n\n* mh/ref-transaction (2014-04-07) 27 commits\n - ref_transaction_commit(): work with transaction->updates in place\n - struct ref_update: add a type field\n - struct ref_update: add a lock field\n - ref_transaction_commit(): simplify code using temporary variables\n - struct ref_update: store refname as a FLEX_ARRAY\n - struct ref_update: rename field \"ref_name\" to \"refname\"\n - refs: remove API function update_refs()\n - update-ref --stdin: reimplement using reference transactions\n - refs: add a concept of a reference transaction\n - update-ref --stdin: harmonize error messages\n - update-ref --stdin: improve the error message for unexpected EOF\n - t1400: test one mistake at a time\n - update-ref --stdin -z: deprecate interpreting the empty string as zeros\n - update-ref.c: extract a new function, parse_next_sha1()\n - t1400: test that stdin -z update treats empty <newvalue> as zeros\n - update-ref --stdin: simplify error messages for missing oldvalues\n - update-ref --stdin: make error messages more consistent\n - update-ref --stdin: improve error messages for invalid values\n - update-ref.c: extract a new function, parse_refname()\n - parse_cmd_verify(): copy old_sha1 instead of evaluating <oldvalue> twice\n - update-ref --stdin: read the whole input at once\n - update_refs(): fix constness\n - refs.h: rename the action_on_err constants\n - t1400: add some more tests involving quoted arguments\n - parse_arg(): really test that argument is properly terminated\n - t1400: provide more usual input to the command\n - t1400: fix name and expected result of one test\n (this branch is used by rs/ref-closer-to-atomic.)\n\n Update \"update-ref --stdin [-z]\" and then introduce a transactional\n support for (multi-)reference updates.\n\n Is this ready to be merged to 'next' for wider exposure?\n\n\n* jc/apply-ignore-whitespace (2014-03-26) 1 commit\n  (merged to 'next' on 2014-04-04 at 53779a7)\n + apply --ignore-space-change: lines with and without leading whitespaces do not match\n\n \"--ignore-space-change\" option of \"git apply\" ignored the\n spaces at the beginning of line too aggressively, which is\n inconsistent with the option of the same name \"diff\" and \"git diff\"\n have.\n\n Will keep in 'next' for the remainder of this cycle.\n\n\n* as/grep-fullname-config (2014-03-20) 1 commit\n  (merged to 'next' on 2014-03-28 at 810a076)\n + grep: add grep.fullName config variable\n\n Add a configuration variable to force --full-name to be default for\n \"git grep\".\n\n This may cause regressions on scripted users that do not expect\n this new behaviour.\n\n Will keep in 'next' for the remainder of this cycle.\n\n\n* nd/multiple-work-trees (2014-03-25) 28 commits\n - count-objects: report unused files in $GIT_DIR/repos/...\n - gc: support prune --repos\n - gc: style change -- no SP before closing bracket\n - prune: strategies for linked checkouts\n - checkout: detach if the branch is already checked out elsewhere\n - checkout: clean up half-prepared directories in --to mode\n - checkout: support checking out into a new working directory\n - use new wrapper write_file() for simple file writing\n - wrapper.c: wrapper to open a file, fprintf then close\n - setup.c: support multi-checkout repo setup\n - setup.c: detect $GIT_COMMON_DIR check_repository_format_gently()\n - setup.c: convert check_repository_format_gently to use strbuf\n - setup.c: detect $GIT_COMMON_DIR in is_git_directory()\n - setup.c: convert is_git_directory() to use strbuf\n - git-stash: avoid hardcoding $GIT_DIR/logs/....\n - *.sh: avoid hardcoding $GIT_DIR/hooks/...\n - git-sh-setup.sh: use rev-parse --git-path to get $GIT_DIR/objects\n - $GIT_COMMON_DIR: a new environment variable\n - commit: use SEQ_DIR instead of hardcoding \"sequencer\"\n - fast-import: use git_path() for accessing .git dir instead of get_git_dir()\n - reflog: avoid constructing .lock path with git_path\n - *.sh: respect $GIT_INDEX_FILE\n - git_path(): be aware of file relocation in $GIT_DIR\n - path.c: group git_path(), git_pathdup() and strbuf_git_path() together\n - path.c: rename vsnpath() to do_git_path()\n - git_snpath(): retire and replace with strbuf_git_path()\n - path.c: make get_pathname() call sites return const char *\n - path.c: make get_pathname() return strbuf instead of static buffer\n\n A replacement for contrib/workdir/git-new-workdir that does not\n rely on symbolic links and make sharing of objects and refs safer\n by making the borrowee and borrowers aware of each other.\n\n Will hold.\n\n\n* ks/tree-diff-nway (2014-04-09) 20 commits\n  (merged to 'next' on 2014-04-09 at c17228e)\n + mingw: activate alloca\n  (merged to 'next' on 2014-04-08 at 6b74773)\n + combine-diff: speed it up, by using multiparent diff tree-walker directly\n + tree-diff: rework diff_tree() to generate diffs for multiparent cases as well\n + Portable alloca for Git\n  (merged to 'next' on 2014-03-31 at 16a7bd4)\n + tree-diff: reuse base str(buf) memory on sub-tree recursion\n + tree-diff: no need to call \"full\" diff_tree_sha1 from show_path()\n + tree-diff: rework diff_tree interface to be sha1 based\n + tree-diff: diff_tree() should now be static\n + tree-diff: remove special-case diff-emitting code for empty-tree cases\n  (merged to 'next' on 2014-03-25 at cfcbdac)\n + tree-diff: simplify tree_entry_pathcmp\n + tree-diff: show_path prototype is not needed anymore\n + tree-diff: rename compare_tree_entry -> tree_entry_pathcmp\n + tree-diff: move all action-taking code out of compare_tree_entry()\n + tree-diff: don't assume compare_tree_entry() returns -1,0,1\n  (merged to 'next' on 2014-03-21 at d872679)\n + tree-diff: consolidate code for emitting diffs and recursion in one place\n + tree-diff: show_tree() is not needed\n + tree-diff: no need to pass match to skip_uninteresting()\n + tree-diff: no need to manually verify that there is no mode change for a path\n + combine-diff: move changed-paths scanning logic into its own function\n + combine-diff: move show_log_first logic/action out of paths scanning\n\n Instead of running N pair-wise diff-trees when inspecting a\n N-parent merge, find the set of paths that were touched by walking\n N+1 trees in parallel.  These set of paths can then be turned into\n N pair-wise diff-tree results to be processed through rename\n detections and such.  And N=2 case nicely degenerates to the usual\n 2-way diff-tree, which is very nice.\n\n Will keep in 'next' for the remainder of this cycle.\n\n\n* cc/interpret-trailers (2014-04-07) 12 commits\n - trailer: add blank line before the trailers if needed\n - Documentation: add documentation for 'git interpret-trailers'\n - trailer: add tests for commands in config file\n - trailer: execute command from 'trailer.<name>.command'\n - trailer: add tests for \"git interpret-trailers\"\n - trailer: add interpret-trailers command\n - trailer: put all the processing together and print\n - trailer: parse trailers from stdin\n - trailer: process command line trailer arguments\n - trailer: read and process config information\n - trailer: process trailers from stdin and arguments\n - trailer: add data structures and basic functions\n\n A new filter to programatically edit the tail end of the commit log\n messages.\n\n I was planning to merge it to 'next' and keep it there for the\n remainder of this cycle, but it appears that there still will be\n another round of reroll, at least for the documentation?\n\n--------------------------------------------------\n[Discarded]\n\n* sz/mingw-index-pack-threaded (2014-03-19) 1 commit\n . Enable index-pack threading in msysgit.\n\n Queued a different attempt by Duy on nd/index-pack-one-fd-per-thread\n"},{"id":"238759","messageId":"53487a78f38bf_3b596b93106e@nysa.notmuch","threadId":"36397","inReplyTo":"xmqq1tx3qzel.fsf@gitster.dls.corp.google.com","subject":"RE: What's cooking in git.git (Apr 2014, #03; Fri, 11)","fromName":"Felipe Contreras","fromEmail":"felipe.contreras@gmail.com","sentAt":"2014-04-11T23:27:53Z","receivedAt":"2014-04-11T23:27:53Z","isPatch":false,"sender":{"key":"felipe.contreras@gmail.com","avatar":"https://avatars.githubusercontent.com/u/8358?v=4"},"body":"Junio C Hamano wrote:\n> * fc/complete-aliased-push (2014-04-09) 1 commit\n>  - completion: fix completing args of aliased \"push\", \"fetch\", etc.\n> \n>  Will merge to 'next'.\n> \n> \n> * fc/remote-helper-fixes (2014-04-09) 4 commits\n>  - remote-bzr: include authors field in pushed commits\n\nBefore you do, you might want to pick the unrelated fix I just sent[1],\notherwise this test might fail on some machines.\n\n>  - remote-bzr: add support for older versions\n>  - remote-hg: always normalize paths\n>  - remote-helpers: allow all tests running from any dir\n> \n>  Will merge to 'next'.\n\n> * fc/publish-vs-upstream (2014-04-11) 8 commits\n>  - sha1_name: add support for @{publish} marks\n>  - sha1_name: simplify track finding\n>  - sha1_name: cleanup interpret_branch_name()\n>  - branch: display publish branch\n>  - push: add --set-publish option\n>  - branch: add --set-publish-to option\n>  - Add concept of 'publish' branch\n>  - t5516 (fetch-push): fix test restoration\n> \n>  Add branch@{publish}; this round v3 hasn't yet seen much reviews\n>  yet.\n> \n>  Seems to have some interactions to break tests when merged to 'pu'.\n\nThat was v2, wasn't it?\n\n[1] http://article.gmane.org/gmane.comp.version-control.git/246157\n\n-- \nFelipe Contreras\n"},{"id":"238764","messageId":"7v61mfm48o.fsf@alter.siamese.dyndns.org","threadId":"36397","inReplyTo":"53487a78f38bf_3b596b93106e@nysa.notmuch","subject":"Re: What's cooking in git.git (Apr 2014, #03; Fri, 11)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2014-04-12T06:49:43Z","receivedAt":"2014-04-12T06:49:43Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Felipe Contreras <felipe.contreras@gmail.com> writes:\n\n> Junio C Hamano wrote:\n>> * fc/complete-aliased-push (2014-04-09) 1 commit\n>>  - completion: fix completing args of aliased \"push\", \"fetch\", etc.\n>> \n>>  Will merge to 'next'.\n>> \n>> \n>> * fc/remote-helper-fixes (2014-04-09) 4 commits\n>>  - remote-bzr: include authors field in pushed commits\n>\n> Before you do, you might want to pick the unrelated fix I just sent[1],\n> otherwise this test might fail on some machines.\n\nWill pick up $gmane/246157 (or at least will try to remember ;-).\nThanks.\n\n>>  - remote-bzr: add support for older versions\n>>  - remote-hg: always normalize paths\n>>  - remote-helpers: allow all tests running from any dir\n>> \n>>  Will merge to 'next'.\n>\n>> * fc/publish-vs-upstream (2014-04-11) 8 commits\n>>  - sha1_name: add support for @{publish} marks\n>>  - sha1_name: simplify track finding\n>>  - sha1_name: cleanup interpret_branch_name()\n>>  - branch: display publish branch\n>>  - push: add --set-publish option\n>>  - branch: add --set-publish-to option\n>>  - Add concept of 'publish' branch\n>>  - t5516 (fetch-push): fix test restoration\n>> \n>>  Add branch@{publish}; this round v3 hasn't yet seen much reviews\n>>  yet.\n>> \n>>  Seems to have some interactions to break tests when merged to 'pu'.\n>\n> That was v2, wasn't it?\n\nThe tip d95807a0e1b7 (sha1_name: add support for @{publish} marks,\n2014-04-11) is with timestamp \"Fri Apr 11 12:59:11 2014 -0500\",\nwhich matches $gmane/246122, so I am reasonably sure that I applied\nthe v3 on the same base as the base I queued the v2 on, compared\nthem and then updated the branch.  Also I can tell by seeing some\npatches lost the retitle I did for the previous round.\n"},{"id":"238777","messageId":"534985F3.6060501@ramsay1.demon.co.uk","threadId":"36397","inReplyTo":"xmqq1tx3qzel.fsf@gitster.dls.corp.google.com","subject":"Re: What's cooking in git.git (Apr 2014, #03; Fri, 11)","fromName":"Ramsay Jones","fromEmail":"ramsay@ramsay1.demon.co.uk","sentAt":"2014-04-12T18:29:07Z","receivedAt":"2014-04-12T18:29:07Z","isPatch":false,"sender":{"key":"ramsay@ramsayjones.plus.com","avatar":"https://avatars.githubusercontent.com/u/33702710?v=4"},"body":"On 11/04/14 23:22, Junio C Hamano wrote:\n[...]\n> [New Topics]\n> \n> * nd/index-pack-one-fd-per-thread (2014-04-09) 1 commit\n>  - index-pack: work around thread-unsafe pread()\n> \n>  Enable threaded index-pack on platforms without thread-unsafe\n>  pread() emulation.\n> \n>  Will merge to 'next' and keep it there for the remainder of the cycle.\n\nThe commit message for commit 512ebe5d (\"index-pack: work around\nthread-unsafe pread()\", 25-03-2014) is a little misleading.\n\nThe pread() implementation is only thread-unsafe on older versions of\ncygwin (anything prior to v1.7 - aka v1.5). Indeed, index-pack has been\nthreaded on cygwin (v1.7) since commit 103d530f (\"Cygwin 1.7 has thread-\nsafe pread\", 19-07-2013). (So, it was released in v1.8.4, I guess).\n\nSince I upgraded my cygwin installation long ago (well about 9 months\nago), I can no longer test patches on cygwin v1.5. Also, since commit\na50dec22 (\"Makefile: update defaults for modern Cygwin\", 01-04-2010)\ntells us that cygwin v1.5 is no longer being actively supported, it\nmay be difficult to find anyone who can do that testing. (v1.5 was\nthe last version supported on windows 95, 98 and ME, so maybe ... :)\n\nYesterday, I briefly compared the performance of the master and pu\nbranches (respectively with and without this patch) on cygwin.\n(There seems to be little difference in the performance, as expected).\n\n  $ uname -a\n  CYGWIN_NT-5.1 toshiba 1.7.29(0.272/5/3) 2014-04-07 13:44 i686 Cygwin\n\nFirst, on the master branch:\n\n  $ ./git version\n  git version 1.9.2.459.g68773ac\n  $ git log -1 --decorate --oneline\n  68773ac (HEAD, origin/master, origin/HEAD, master) Sync with 1.9.2\n\n  $ cd t\n  $ time ./t5302-pack-index.sh\n  ok 1 - setup\n  ...\n  ok 31 - running index-pack in the object store\n  # passed all 31 test(s)\n  1..31\n  \n  real    1m11.984s\n  user    1m24.986s\n  sys     0m41.521s\n  $\n\n  $ cd perf\n  $ ./p5302-pack-index.sh\n  warning: $GIT_PERF_LARGE_REPO is $GIT_BUILD_DIR.\n  warning: This will work, but may not be a sufficiently large repo\n  warning: for representative measurements.\n  ok 1 - repack\n  perf 2 - index-pack 0 threads: 1 2 3 ok\n  perf 3 - index-pack 1 thread : 1 2 3 ok\n  perf 4 - index-pack 2 threads: 1 2 3 ok\n  perf 5 - index-pack 4 threads: 1 2 3 ok\n  perf 6 - index-pack 8 threads: 1 2 3 ok\n  perf 7 - index-pack default number of threads: 1 2 3 ok\n  # passed all 7 test(s)\n  1..7\n  Can't locate Git.pm in @INC (@INC contains: /usr/lib/perl5/site_perl/5.14/i686-cygwin-threads-64int /usr/lib/perl5/site_perl/5.14 /usr/lib/perl5/vendor_perl/5.14/i686-cygwin-threads-64int /usr/lib/perl5/vendor_perl/5.14 /usr/lib/perl5/5.14/i686-cygwin-threads-64int /usr/lib/perl5/5.14 /usr/lib/perl5/site_perl/5.10 /usr/lib/perl5/vendor_perl/5.10 /usr/lib/perl5/site_perl/5.8 .) at ./aggregate.perl line 5.\n  BEGIN failed--compilation aborted at ./aggregate.perl line 5.\n\n  $ PERL5LIB=/home/ramsay/lib/perl5/site_perl/5.14 ./aggregate.perl ./p5302-pack-index.sh\n  Test                                           this tree\n  ----------------------------------------------------------------\n  5302.2: index-pack 0 threads                   38.60(34.38+0.93)\n  5302.3: index-pack 1 thread                    36.48(34.74+0.90)\n  5302.4: index-pack 2 threads                   26.60(36.57+1.87)\n  5302.5: index-pack 4 threads                   27.32(37.46+2.38)\n  5302.6: index-pack 8 threads                   27.18(38.38+3.01)\n  5302.7: index-pack default number of threads   25.40(36.07+2.09)\n  $\n  \nNOTE: that t/perf/aggregate.perl can't locate Git.pm; I haven't investigated\nthat.\n\nNow, on the pu branch:\n  \n  $ ./git version\n  git version 1.9.2.667.ge5b74e1\n  $ git log -1 --decorate --oneline\n  e5b74e1 (HEAD, origin/pu, pu) Merge branch 'jc/graph-post-root-gap' into pu\n\n  $ cd t\n  $ time ./t5302-pack-index.sh\n  ok 1 - setup\n  ...\n  ok 31 - running index-pack in the object store\n  # passed all 31 test(s)\n  1..31\n  \n  real    1m8.063s\n  user    1m26.151s\n  sys     0m40.477s\n  $\n\n  $ cd perf\n  $ PERL5LIB=/home/ramsay/lib/perl5/site_perl/5.14 ./p5302-pack-index.sh\n  warning: $GIT_PERF_LARGE_REPO is $GIT_BUILD_DIR.\n  warning: This will work, but may not be a sufficiently large repo\n  warning: for representative measurements.\n  ok 1 - repack\n  perf 2 - index-pack 0 threads: 1 2 3 ok\n  perf 3 - index-pack 1 thread : 1 2 3 ok\n  perf 4 - index-pack 2 threads: 1 2 3 ok\n  perf 5 - index-pack 4 threads: 1 2 3 ok\n  perf 6 - index-pack 8 threads: 1 2 3 ok\n  perf 7 - index-pack default number of threads: 1 2 3 ok\n  # passed all 7 test(s)\n  1..7\n  Test                                           this tree\n  ----------------------------------------------------------------\n  5302.2: index-pack 0 threads                   37.90(34.40+0.82)\n  5302.3: index-pack 1 thread                    37.93(34.79+0.96)\n  5302.4: index-pack 2 threads                   27.75(36.46+1.98)\n  5302.5: index-pack 4 threads                   27.93(37.24+2.82)\n  5302.6: index-pack 8 threads                   29.34(38.23+3.04)\n  5302.7: index-pack default number of threads   26.25(36.55+1.84)\n  $\n\n\nHTH\n\nATB,\nRamsay Jones\n"},{"id":"238813","messageId":"534AF118.5040501@alum.mit.edu","threadId":"36397","inReplyTo":"xmqq1tx3qzel.fsf@gitster.dls.corp.google.com","subject":"Re: What's cooking in git.git (Apr 2014, #03; Fri, 11)","fromName":"Michael Haggerty","fromEmail":"mhagger@alum.mit.edu","sentAt":"2014-04-13T20:18:32Z","receivedAt":"2014-04-13T20:18:32Z","isPatch":false,"sender":{"key":"mhagger@alum.mit.edu","avatar":"https://avatars.githubusercontent.com/u/119718?v=4"},"body":"On 04/12/2014 12:22 AM, Junio C Hamano wrote:\n> [...]\n> * mh/ref-transaction (2014-04-07) 27 commits\n>  - ref_transaction_commit(): work with transaction->updates in place\n>  - struct ref_update: add a type field\n>  - struct ref_update: add a lock field\n>  - ref_transaction_commit(): simplify code using temporary variables\n>  - struct ref_update: store refname as a FLEX_ARRAY\n>  - struct ref_update: rename field \"ref_name\" to \"refname\"\n>  - refs: remove API function update_refs()\n>  - update-ref --stdin: reimplement using reference transactions\n>  - refs: add a concept of a reference transaction\n>  - update-ref --stdin: harmonize error messages\n>  - update-ref --stdin: improve the error message for unexpected EOF\n>  - t1400: test one mistake at a time\n>  - update-ref --stdin -z: deprecate interpreting the empty string as zeros\n>  - update-ref.c: extract a new function, parse_next_sha1()\n>  - t1400: test that stdin -z update treats empty <newvalue> as zeros\n>  - update-ref --stdin: simplify error messages for missing oldvalues\n>  - update-ref --stdin: make error messages more consistent\n>  - update-ref --stdin: improve error messages for invalid values\n>  - update-ref.c: extract a new function, parse_refname()\n>  - parse_cmd_verify(): copy old_sha1 instead of evaluating <oldvalue> twice\n>  - update-ref --stdin: read the whole input at once\n>  - update_refs(): fix constness\n>  - refs.h: rename the action_on_err constants\n>  - t1400: add some more tests involving quoted arguments\n>  - parse_arg(): really test that argument is properly terminated\n>  - t1400: provide more usual input to the command\n>  - t1400: fix name and expected result of one test\n>  (this branch is used by rs/ref-closer-to-atomic.)\n> \n>  Update \"update-ref --stdin [-z]\" and then introduce a transactional\n>  support for (multi-)reference updates.\n> \n>  Is this ready to be merged to 'next' for wider exposure?\n\nYes, as far as I know there are no outstanding objections to this\nversion (v3) of this series and it is ready for 'next'.\n\nMichael\n\n-- \nMichael Haggerty\nmhagger@alum.mit.edu\nhttp://softwareswirl.blogspot.com/\n"},{"id":"238891","messageId":"xmqqeh0ylext.fsf@gitster.dls.corp.google.com","threadId":"36397","inReplyTo":"534985F3.6060501@ramsay1.demon.co.uk","subject":"Re: What's cooking in git.git (Apr 2014, #03; Fri, 11)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2014-04-15T16:45:18Z","receivedAt":"2014-04-15T16:45:18Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Ramsay Jones <ramsay@ramsay1.demon.co.uk> writes:\n\n> On 11/04/14 23:22, Junio C Hamano wrote:\n> [...]\n>> [New Topics]\n>> \n>> * nd/index-pack-one-fd-per-thread (2014-04-09) 1 commit\n>>  - index-pack: work around thread-unsafe pread()\n>> \n>>  Enable threaded index-pack on platforms without thread-unsafe\n>>  pread() emulation.\n>> \n>>  Will merge to 'next' and keep it there for the remainder of the cycle.\n>\n> The commit message for commit 512ebe5d (\"index-pack: work around\n> thread-unsafe pread()\", 25-03-2014) is a little misleading.\n\nOK.  Can we have a concrete alternative?\n\n    Multi-threaing of index-pack was disabled with c0f8654\n    (index-pack: Disable threading on cygwin - 2012-06-26), because\n    pread() implementations for Cygwin and MSYS were not thread\n    safe.  Recent Cygwin does offer usable pread() and we enabled\n    multi-threading with 103d530f (Cygwin 1.7 has thread-safe pread,\n    2013-07-19).\n\n    Work around this problem on platforms with a thread-unsafe\n    pread() emulation by opening one file handle per thread; it\n    would prevent parallel pread() on different file handles from\n    stepping on each other.\n\n    Also remove NO_THREAD_SAFE_PREAD that was introduced in c0f8654\n    because it's no longer used anywhere.\n\n    This workaround is unconditional, even for platforms with\n    thread-safe pread() because the overhead is small (a couple file\n    handles more) and not worth fragmenting the code.\n"},{"id":"238909","messageId":"CACsJy8C_WU_-73+VF0KV1X1BLDSWH_C35jPjMcrjR2OW3s9z-Q@mail.gmail.com","threadId":"36397","inReplyTo":"xmqqeh0ylext.fsf@gitster.dls.corp.google.com","subject":"Re: What's cooking in git.git (Apr 2014, #03; Fri, 11)","fromName":"Duy Nguyen","fromEmail":"pclouds@gmail.com","sentAt":"2014-04-15T23:18:33Z","receivedAt":"2014-04-15T23:18:33Z","isPatch":false,"sender":{"key":"pclouds@gmail.com","avatar":"https://avatars.githubusercontent.com/u/720?v=4"},"body":"On Tue, Apr 15, 2014 at 11:45 PM, Junio C Hamano <gitster@pobox.com> wrote:\n> Ramsay Jones <ramsay@ramsay1.demon.co.uk> writes:\n>\n>> On 11/04/14 23:22, Junio C Hamano wrote:\n>> [...]\n>>> [New Topics]\n>>>\n>>> * nd/index-pack-one-fd-per-thread (2014-04-09) 1 commit\n>>>  - index-pack: work around thread-unsafe pread()\n>>>\n>>>  Enable threaded index-pack on platforms without thread-unsafe\n>>>  pread() emulation.\n>>>\n>>>  Will merge to 'next' and keep it there for the remainder of the cycle.\n>>\n>> The commit message for commit 512ebe5d (\"index-pack: work around\n>> thread-unsafe pread()\", 25-03-2014) is a little misleading.\n>\n> OK.  Can we have a concrete alternative?\n>\n>     Multi-threaing of index-pack was disabled with c0f8654\n>     (index-pack: Disable threading on cygwin - 2012-06-26), because\n>     pread() implementations for Cygwin and MSYS were not thread\n>     safe.  Recent Cygwin does offer usable pread() and we enabled\n>     multi-threading with 103d530f (Cygwin 1.7 has thread-safe pread,\n>     2013-07-19).\n>\n>     Work around this problem on platforms with a thread-unsafe\n>     pread() emulation by opening one file handle per thread; it\n>     would prevent parallel pread() on different file handles from\n>     stepping on each other.\n>\n>     Also remove NO_THREAD_SAFE_PREAD that was introduced in c0f8654\n>     because it's no longer used anywhere.\n>\n>     This workaround is unconditional, even for platforms with\n>     thread-safe pread() because the overhead is small (a couple file\n>     handles more) and not worth fragmenting the code.\n>\n\nOK to me.\n-- \nDuy\n"},{"id":"238930","messageId":"534E6A2B.5060903@ramsay1.demon.co.uk","threadId":"36397","inReplyTo":"CACsJy8C_WU_-73+VF0KV1X1BLDSWH_C35jPjMcrjR2OW3s9z-Q@mail.gmail.com","subject":"Re: What's cooking in git.git (Apr 2014, #03; Fri, 11)","fromName":"Ramsay Jones","fromEmail":"ramsay@ramsay1.demon.co.uk","sentAt":"2014-04-16T11:31:55Z","receivedAt":"2014-04-16T11:31:55Z","isPatch":false,"sender":{"key":"ramsay@ramsayjones.plus.com","avatar":"https://avatars.githubusercontent.com/u/33702710?v=4"},"body":"On 16/04/14 00:18, Duy Nguyen wrote:\n> On Tue, Apr 15, 2014 at 11:45 PM, Junio C Hamano <gitster@pobox.com> wrote:\n>> Ramsay Jones <ramsay@ramsay1.demon.co.uk> writes:\n>>\n>>> On 11/04/14 23:22, Junio C Hamano wrote:\n>>> [...]\n>>>> [New Topics]\n>>>>\n>>>> * nd/index-pack-one-fd-per-thread (2014-04-09) 1 commit\n>>>>  - index-pack: work around thread-unsafe pread()\n>>>>\n>>>>  Enable threaded index-pack on platforms without thread-unsafe\n>>>>  pread() emulation.\n>>>>\n>>>>  Will merge to 'next' and keep it there for the remainder of the cycle.\n>>>\n>>> The commit message for commit 512ebe5d (\"index-pack: work around\n>>> thread-unsafe pread()\", 25-03-2014) is a little misleading.\n>>\n>> OK.  Can we have a concrete alternative?\n>>\n>>     Multi-threaing of index-pack was disabled with c0f8654\n>>     (index-pack: Disable threading on cygwin - 2012-06-26), because\n>>     pread() implementations for Cygwin and MSYS were not thread\n>>     safe.  Recent Cygwin does offer usable pread() and we enabled\n>>     multi-threading with 103d530f (Cygwin 1.7 has thread-safe pread,\n>>     2013-07-19).\n>>\n>>     Work around this problem on platforms with a thread-unsafe\n>>     pread() emulation by opening one file handle per thread; it\n>>     would prevent parallel pread() on different file handles from\n>>     stepping on each other.\n>>\n>>     Also remove NO_THREAD_SAFE_PREAD that was introduced in c0f8654\n>>     because it's no longer used anywhere.\n>>\n>>     This workaround is unconditional, even for platforms with\n>>     thread-safe pread() because the overhead is small (a couple file\n>>     handles more) and not worth fragmenting the code.\n>>\n> \n> OK to me.\n\n\nYep, this looks good to me too.\n\nThanks!\n\nATB,\nRamsay Jones\n"},{"id":"239020","messageId":"20140417165247.GQ21805@odin.tremily.us","threadId":"36397","inReplyTo":"xmqq1tx3qzel.fsf@gitster.dls.corp.google.com","subject":"Re: What's cooking in git.git (Apr 2014, #03; Fri, 11)","fromName":"W. Trevor King","fromEmail":"wking@tremily.us","sentAt":"2014-04-17T16:52:47Z","receivedAt":"2014-04-17T16:52:47Z","isPatch":false,"sender":{"key":"wking@tremily.us","avatar":"https://avatars.githubusercontent.com/u/209920?v=4"},"body":"On Fri, Apr 11, 2014 at 03:22:58PM -0700, Junio C Hamano wrote:\n> * jl/submodule-recursive-checkout (2013-12-26) 5 commits\n>  - Teach checkout to recursively checkout submodules\n>  - submodule: teach unpack_trees() to update submodules\n>  - submodule: teach unpack_trees() to repopulate submodules\n>  - submodule: teach unpack_trees() to remove submodule contents\n>  - submodule: prepare for recursive checkout of submodules\n> \n>  Expecting a reroll.\n\nI think this was rerolled with Jens' v2 [1]:\n\n  * jl/submodule-recursive-checkout (2014-02-03) 9 commits\n  - submodule: prepare for recursive checkout of submodules\n  - Teach reset the --[no-]recurse-submodules option\n  - Teach checkout the --[no-]recurse-submodules option\n  - Teach merge the --[no-]recurse-submodules option\n  - Teach bisect--helper the --[no-]recurse-submodules option\n  - Teach bisect the --[no-]recurse-submodules option\n  - submodule: teach unpack_trees() to remove submodule contents\n  - submodule: teach unpack_trees() to repopulate submodules\n  - submodule: teach unpack_trees() to update submodules\n\nCheers,\nTrevor\n\n[1]: http://thread.gmane.org/gmane.comp.version-control.git/241455\n\n-- \nThis email may be signed or encrypted with GnuPG (http://www.gnupg.org).\nFor more information, see http://en.wikipedia.org/wiki/Pretty_Good_Privacy\n"},{"id":"239027","messageId":"xmqqsipbetci.fsf@gitster.dls.corp.google.com","threadId":"36397","inReplyTo":"20140417165247.GQ21805@odin.tremily.us","subject":"Re: What's cooking in git.git (Apr 2014, #03; Fri, 11)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2014-04-17T17:52:45Z","receivedAt":"2014-04-17T17:52:45Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"\"W. Trevor King\" <wking@tremily.us> writes:\n\n> On Fri, Apr 11, 2014 at 03:22:58PM -0700, Junio C Hamano wrote:\n>> * jl/submodule-recursive-checkout (2013-12-26) 5 commits\n>>  - Teach checkout to recursively checkout submodules\n>>  - submodule: teach unpack_trees() to update submodules\n>>  - submodule: teach unpack_trees() to repopulate submodules\n>>  - submodule: teach unpack_trees() to remove submodule contents\n>>  - submodule: prepare for recursive checkout of submodules\n>> \n>>  Expecting a reroll.\n>\n> I think this was rerolled with Jens' v2 [1]:\n>\n>   * jl/submodule-recursive-checkout (2014-02-03) 9 commits\n>   - submodule: prepare for recursive checkout of submodules\n>   - Teach reset the --[no-]recurse-submodules option\n>   - Teach checkout the --[no-]recurse-submodules option\n>   - Teach merge the --[no-]recurse-submodules option\n>   - Teach bisect--helper the --[no-]recurse-submodules option\n>   - Teach bisect the --[no-]recurse-submodules option\n>   - submodule: teach unpack_trees() to remove submodule contents\n>   - submodule: teach unpack_trees() to repopulate submodules\n>   - submodule: teach unpack_trees() to update submodules\n>\n> Cheers,\n> Trevor\n>\n> [1]: http://thread.gmane.org/gmane.comp.version-control.git/241455\n\nThanks, I'll add a reference to the RFC v2, but it also had\nsignificant discussion threads on it, so it would still be in\n\"Expecting a reroll\" state anyway.\n"}]}