{"thread":{"id":"36326","subject":"What's cooking in git.git (Mar 2014, #08; Mon, 31)","startedAt":"2014-04-01T00:29:03Z","lastAt":"2014-04-03T17:15:04Z","messageCount":10,"participants":["Junio C Hamano","Duy Nguyen","Stephen Leake","Heiko Voigt","David Kastrup"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"238185","messageId":"xmqqppl128q8.fsf@gitster.dls.corp.google.com","threadId":"36326","inReplyTo":null,"subject":"What's cooking in git.git (Mar 2014, #08; Mon, 31)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2014-04-01T00:29:03Z","receivedAt":"2014-04-01T00:29:03Z","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\nMore topics merged to 'master', some of which are fallouts from GSoC\nmicroprojects, some are updates to docs.  C/C++ funcname pattern\nupdate is now in.\n\nI haven't reverted the merge of that \"submodule update\" topic yet; I\nshould do that soonish.\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[Graduated to \"master\"]\n\n* an/branch-config-message (2014-03-24) 1 commit\n  (merged to 'next' on 2014-03-26 at 26f9741)\n + branch.c: install_branch_config: simplify if chain\n\n\n* ca/doc-config-third-party (2014-03-21) 1 commit\n  (merged to 'next' on 2014-03-25 at 731e011)\n + config.txt: third-party tools may and do use their own variables\n\n\n* dp/makefile-charset-lib-doc (2014-03-23) 1 commit\n  (merged to 'next' on 2014-03-25 at b32e3ad)\n + Makefile: describe CHARSET_LIB better\n\n\n* dt/tests-with-env-not-subshell (2014-03-19) 1 commit\n  (merged to 'next' on 2014-03-25 at 19fe25f)\n + tests: use \"env\" to run commands with temporary env-var settings\n (this branch is used by jk/tests-cleanup.)\n\n\n* dw/doc-status-no-longer-shows-pound-prefix (2014-03-21) 1 commit\n  (merged to 'next' on 2014-03-25 at 2683eb6)\n + doc: status, remove leftover statement about '#' prefix\n\n\n* hs/simplify-bit-setting-in-fsck-tree (2014-03-20) 1 commit\n  (merged to 'next' on 2014-03-25 at 08efd68)\n + fsck: use bitwise-or assignment operator to set flag\n\n\n* ib/rev-parse-parseopt-argh (2014-03-23) 2 commits\n  (merged to 'next' on 2014-03-25 at d9083ed)\n + t1502: protect runs of SPs used in the indentation\n + rev-parse --parseopt: option argument name hints\n (this branch is used by jc/rev-parse-argh-dashed-multi-words.)\n\n Teaches the \"rev-parse --parseopt\" mechanism used by scripted\n Porcelains to parse command line options and give help text how to\n supply argv-help (the placeholder string for an option parameter,\n e.g. \"key-id\" in \"--gpg-sign=<key-id>\").\n\n\n* jk/tests-cleanup (2014-03-21) 12 commits\n  (merged to 'next' on 2014-03-26 at 4a72b49)\n + t0001: drop subshells just for \"cd\"\n + t0001: drop useless subshells\n + t0001: use test_must_fail\n + t0001: use test_config_global\n + t0001: use test_path_is_*\n + t0001: make symlink reinit test more careful\n + t: prefer \"git config --file\" to GIT_CONFIG\n + t: prefer \"git config --file\" to GIT_CONFIG with test_must_fail\n + t: stop using GIT_CONFIG to cross repo boundaries\n + t: drop useless sane_unset GIT_* calls\n + t/test-lib: drop redundant unset of GIT_CONFIG\n + t/Makefile: stop setting GIT_CONFIG\n (this branch uses dt/tests-with-env-not-subshell.)\n\n\n* js/userdiff-cc (2014-03-21) 10 commits\n  (merged to 'next' on 2014-03-25 at 8c0e585)\n + userdiff: have 'cpp' hunk header pattern catch more C++ anchor points\n + t4018: test cases showing that the cpp pattern misses many anchor points\n + t4018: test cases for the built-in cpp pattern\n + t4018: reduce test files for pattern compilation tests\n + t4018: convert custom pattern test to the new infrastructure\n + t4018: convert java pattern test to the new infrastructure\n + t4018: convert perl pattern tests to the new infrastructure\n + t4018: an infrastructure to test hunk headers\n + userdiff: support unsigned and long long suffixes of integer constants\n + userdiff: support C++ ->* and .* operators in the word regexp\n\n Improves the pattern to match the hunk-header for C/C++.\n\n\n* mm/status-porcelain-format-i18n-fix (2014-03-26) 1 commit\n  (merged to 'next' on 2014-03-26 at 41680fc)\n + status: disable translation when --porcelain is used\n\n\n* mr/msvc-link-with-lcurl (2014-03-27) 1 commit\n  (merged to 'next' on 2014-03-28 at 3281dab)\n + MSVC: allow linking with the cURL library\n\n\n* wt/doc-submodule-name-path-confusion-1 (2014-03-27) 1 commit\n  (merged to 'next' on 2014-03-28 at 225f241)\n + doc: submodule.* config are keyed by submodule names\n\n\n* wt/doc-submodule-name-path-confusion-2 (2014-03-27) 1 commit\n  (merged to 'next' on 2014-03-28 at ec5bcf3)\n + doc: submodule.*.branch config is keyed by name\n\n--------------------------------------------------\n[New Topics]\n\n* cb/aix (2014-03-31) 2 commits\n - tests: don't rely on strerror text when testing rmdir failure\n - dir.c: make git_fnmatch() not inline\n\n Will merge to 'next'.\n\n\n* jc/fix-diff-no-index-diff-opt-parse (2014-03-31) 1 commit\n - diff-no-index: correctly diagnose error return from diff_opt_parse()\n\n \"diff --no-index -Mq a b\" fell into an infinite loop.\n\n Will merge to 'next'.\n\n\n* mr/opt-set-ptr (2014-03-31) 3 commits\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\n We may just want directly to go to the endgame of removing\n OPT_SET_PTR macro, which nobody seems to use.\n\n\n* mt/patch-id-stable (2014-03-31) 3 commits\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\n* jl/nor-or-nand-and (2014-03-31) 4 commits\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/ref-transaction (2014-03-31) 27 commits\n - ref_transaction_commit(): work with transaction->updates in place\n - struct ref_update: add type field\n - struct ref_update: add a lock member\n - ref_transaction_commit(): simplify expressions\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 breakage 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(): tighten 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 test name and expected result of one test\n\n Update \"update-ref --stdin [-z]\" and then introduce a transactional\n support for (multi-)reference updates.\n\n I think the earlier part should be merged to 'next' soonish in\n order to keep the topic that is still in flux reasonably short.  I\n haven't read the \"transaction\" part carefully yet.\n\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 RFC.  This latest round clashes with the kb/fast-hashmap topic in\n 'master'.\n\n\n* sz/mingw-index-pack-threaded (2014-03-19) 1 commit\n - Enable index-pack threading in msysgit.\n\n Still under discussion among Windows folks.  A failure report\n exists ($gmane/245170).\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* kb/fast-hashmap-pack-struct (2014-02-24) 1 commit\n - hashmap.h: make sure map entries are tightly packed\n\n Will discard.\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 Will hold.\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* hv/submodule-ignore-fix (2013-12-06) 4 commits\n - disable complete ignorance of submodules for index <-> HEAD diff\n - always show committed submodules in summary after commit\n - teach add -f option for ignored submodules\n - fix 'git add' to skip submodules configured as ignored\n\n Teach \"git add\" to be consistent with \"git status\" when changes to\n submodules are set to be ignored, to avoid surprises after checking\n with \"git status\" to see there isn't any change to be further added\n and then see that \"git add .\" adds changes to them.\n\n I think a reroll is coming, so this may need to be replaced, but I\n needed some practice with heavy conflict resolution.  It conflicts\n with two changes to \"git add\" that have been scheduled for Git 2.0\n quite badly, and I think I got the resolution right this time.\n\n Waiting for a reroll.\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 Temporarily ejected from 'pu', to try out jk/pack-bitmap, which\n this topic conflicts with.\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* jc/apply-ignore-whitespace (2014-03-26) 1 commit\n - apply --ignore-space-change: lines with and without leading whitespaces do not match\n\n An RFC.  \"--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 hold.\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 Will merge to 'master'.\n\n\n* jk/commit-dates-parsing-fix (2014-03-26) 1 commit\n - t4212: loosen far-in-future test for AIX\n\n I think we agreed that a simpler test would be a better way\n forward.\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 Will merge to 'master'.\n\n\n* ep/shell-command-substitution (2014-03-25) 2 commits\n  (merged to 'next' on 2014-03-28 at 99a512a)\n + git-am.sh: use the $(...) construct for command substitution\n + check-builtins.sh: use the $(...) construct for command substitution\n\n Will merge to 'master'.\n\n\n* rs/pickaxe-i (2014-03-24) 10 commits\n  (merged to 'next' on 2014-03-25 at 3b6f21f)\n + pickaxe: simplify kwset loop in contains()\n + pickaxe: call strlen only when necessary in diffcore_pickaxe_count()\n + pickaxe: move pickaxe() after pickaxe_match()\n + pickaxe: merge diffcore_pickaxe_grep() and diffcore_pickaxe_count() into diffcore_pickaxe()\n + pickaxe: honor -i when used with -S and --pickaxe-regex\n + t4209: use helper functions to test --author\n + t4209: use helper functions to test --grep\n + t4209: factor out helper function test_log_icase()\n + t4209: factor out helper function test_log()\n + t4209: set up expectations up front\n\n Allow the options -i/--regexp-ignore-case, --pickaxe-regex, and -S\n to be used together and work as expected to perform a pickaxe\n search using case-insensitive regular expression matching.\n\n Will merge to 'master'.\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 hold.\n\n\n* nd/gc-aggressive (2014-03-31) 2 commits\n - environment.c: fix constness for odb_pack_keep()\n - gc --aggressive: make --depth configurable\n\n Allow tweaking the maximum length of the delta-chain produced by\n \"gc --aggressive\".\n\n Will merge to 'next'.\n\n\n* ap/remote-hg-skip-null-bookmarks (2014-03-25) 1 commit\n  (merged to 'next' on 2014-03-25 at a8cd922)\n + remote-hg: do not fail on invalid bookmarks\n\n Will merge to 'master'.\n\n\n* cn/fetch-prune-overlapping-destination (2014-03-26) 2 commits\n  (merged to 'next' on 2014-03-28 at 954513a)\n + fetch: handle overlaping refspecs on --prune\n + fetch: add a failing test for prunning with overlapping refspecs\n\n Protect refs in a hierarchy that can come from more than one remote\n hierarcies from incorrect removal by \"git fetch --prune\".\n\n Since I didn't get any responses to my earlier \"Comments?\", I ended\n up reading it myself again and found a small leak, whose fix has\n been squashed in to the tip commit.\n\n Will merge to 'master'.\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 What is the doneness of this thing?  I remember reading it through\n once and sent review comments on earlier parts, but have there been\n a lot of discussions on this topic?\n\n\n* ks/tree-diff-nway (2014-03-27) 19 commits\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 The preparatory steps are all in 'next'.  The real fun part should\n follow soonish (hopefully).\n\n\n* nd/log-show-linear-break (2014-03-25) 2 commits\n  (merged to 'next' on 2014-03-28 at ea4a8db)\n + log: add --show-linear-break to help see non-linear history\n + object.h: centralize object flag allocation\n\n Attempts to show where a single-strand-of-pearls break in \"git log\"\n output.\n\n Will merge to 'master'.\n\n\n* cc/interpret-trailers (2014-03-26) 12 commits\n - trailers: 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 - Move lower case functions into wrapper.c\n - trailer: process trailers from stdin and arguments\n - trailers: add data structures and basic functions\n\n A new filter to programatically edit the tail end of the commit log\n messages.\n"},{"id":"238186","messageId":"CACsJy8A9b2eccm_BJMjW5RwWLux90xs7HfUQEn0_jrzhHWeajQ@mail.gmail.com","threadId":"36326","inReplyTo":"xmqqppl128q8.fsf@gitster.dls.corp.google.com","subject":"Re: What's cooking in git.git (Mar 2014, #08; Mon, 31)","fromName":"Duy Nguyen","fromEmail":"pclouds@gmail.com","sentAt":"2014-04-01T00:53:37Z","receivedAt":"2014-04-01T00:53:37Z","isPatch":false,"sender":{"key":"pclouds@gmail.com","avatar":"https://avatars.githubusercontent.com/u/720?v=4"},"body":"On Tue, Apr 1, 2014 at 7:29 AM, Junio C Hamano <gitster@pobox.com> wrote:\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>  What is the doneness of this thing?  I remember reading it through\n>  once and sent review comments on earlier parts, but have there been\n>  a lot of discussions on this topic?\n\nThe basic support is there. Some bells and whistles (e.g. listing\ncheckouts) are not, but we can add them when we see the needs. Eric\nand Torsten helped review but no, there hasn't much discussion about\nit, which may be because it's already perfect, or people are not\ninterested. Unfortunately, this multiple checkout thing conflicts with\nhow I use emacs (--daemon) so I'm not one of its heavy users either. I\nonly occastionally make new, short-lived checkouts to test things.\n-- \nDuy\n"},{"id":"238237","messageId":"xmqq38hw272q.fsf@gitster.dls.corp.google.com","threadId":"36326","inReplyTo":"CACsJy8A9b2eccm_BJMjW5RwWLux90xs7HfUQEn0_jrzhHWeajQ@mail.gmail.com","subject":"Re: What's cooking in git.git (Mar 2014, #08; Mon, 31)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2014-04-01T19:17:01Z","receivedAt":"2014-04-01T19:17:01Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Duy Nguyen <pclouds@gmail.com> writes:\n\n> The basic support is there. Some bells and whistles (e.g. listing\n> checkouts) are not, but we can add them when we see the needs. Eric\n> and Torsten helped review but no, there hasn't much discussion about\n> it, which may be because it's already perfect, or people are not\n> interested.\n\n;-)\n\n> Unfortunately, this multiple checkout thing conflicts with how I\n> use emacs (--daemon) so I'm not one of its heavy users either. I\n> only occastionally make new, short-lived checkouts to test things.\n\nFWIW, I do keep a permanent multi-checkout and I sometimes make\nedits, but it is mostly for building other branches while I work on\nsomething else.  When I edit working-tree files there, I do let\nemacsclient talk to the emacs I have files open in the primary\nworking tree, though, so I am not sure what your issues are with\nemacs (--daemon).\n"},{"id":"238277","messageId":"xmqqppl0yaf9.fsf@gitster.dls.corp.google.com","threadId":"36326","inReplyTo":"xmqqppl128q8.fsf@gitster.dls.corp.google.com","subject":"Re: What's cooking in git.git (Mar 2014, #08; Mon, 31)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2014-04-01T22:03:38Z","receivedAt":"2014-04-01T22:03:38Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Junio C Hamano <gitster@pobox.com> writes:\n\n> I haven't reverted the merge of that \"submodule update\" topic yet; I\n> should do that soonish.\n> ...\n\nSigh...  This is giving me a lot of headache.\n\nAs 23d25e48 (submodule: explicit local branch creation in\nmodule_clone, 2014-01-26) has been in 'master' since fairly early\nduring this cycle, a lot of topics that are not planned to be on the\n'maint' branch has forked from the tip of 'master' and are now\ncontaminated by that commit.\n\nI think I have a preparatory patch to correctly revert 00d4ff1a\n(Merge branch 'wt/doc-submodule-name-path-confusion-2', 2014-03-31)\nand 06c27689 (Merge branch 'wk/submodule-on-branch', 2014-02-27),\nand also a part of 384364b (Start preparing for Git 2.0,\n2014-03-07), but I am not sure what to do with them ;-<))\n\nAnyhow...\n"},{"id":"238288","messageId":"CACsJy8BVXk248VDBi-pAMDY4QpDSbFuAf5qxRxCCq71yGNzpqA@mail.gmail.com","threadId":"36326","inReplyTo":"xmqq38hw272q.fsf@gitster.dls.corp.google.com","subject":"Re: What's cooking in git.git (Mar 2014, #08; Mon, 31)","fromName":"Duy Nguyen","fromEmail":"pclouds@gmail.com","sentAt":"2014-04-01T23:40:59Z","receivedAt":"2014-04-01T23:40:59Z","isPatch":false,"sender":{"key":"pclouds@gmail.com","avatar":"https://avatars.githubusercontent.com/u/720?v=4"},"body":"On Wed, Apr 2, 2014 at 2:17 AM, Junio C Hamano <gitster@pobox.com> wrote:\n>> Unfortunately, this multiple checkout thing conflicts with how I\n>> use emacs (--daemon) so I'm not one of its heavy users either. I\n>> only occastionally make new, short-lived checkouts to test things.\n>\n> FWIW, I do keep a permanent multi-checkout and I sometimes make\n> edits, but it is mostly for building other branches while I work on\n> something else.  When I edit working-tree files there, I do let\n> emacsclient talk to the emacs I have files open in the primary\n> working tree, though, so I am not sure what your issues are with\n> emacs (--daemon).\n\nThe --daemon part is probably not worth mentioning because I always\nhave one emacs window open. The problem is switch-buffer based on file\nname can be confusing (\"git.c\" and \"git.c<2>\", which belongs to which\ncheckout?). I ended up modifying files in the wrong checkout all the\ntime.\n-- \nDuy\n"},{"id":"238321","messageId":"87ha6cian0.fsf@fencepost.gnu.org","threadId":"36326","inReplyTo":"xmqqppl0yaf9.fsf@gitster.dls.corp.google.com","subject":"Re: What's cooking in git.git (Mar 2014, #08; Mon, 31)","fromName":"David Kastrup","fromEmail":"dak@gnu.org","sentAt":"2014-04-02T05:05:39Z","receivedAt":"2014-04-02T05:05:39Z","isPatch":false,"sender":{"key":"dak@gnu.org","avatar":"https://avatars.githubusercontent.com/u/52141349?v=4"},"body":"Junio C Hamano <gitster@pobox.com> writes:\n\n> Junio C Hamano <gitster@pobox.com> writes:\n>\n>> I haven't reverted the merge of that \"submodule update\" topic yet; I\n>> should do that soonish.\n>> ...\n>\n> Sigh...  This is giving me a lot of headache.\n>\n> As 23d25e48 (submodule: explicit local branch creation in\n> module_clone, 2014-01-26) has been in 'master' since fairly early\n> during this cycle, a lot of topics that are not planned to be on the\n> 'maint' branch has forked from the tip of 'master' and are now\n> contaminated by that commit.\n>\n> I think I have a preparatory patch to correctly revert 00d4ff1a\n> (Merge branch 'wt/doc-submodule-name-path-confusion-2', 2014-03-31)\n> and 06c27689 (Merge branch 'wk/submodule-on-branch', 2014-02-27),\n> and also a part of 384364b (Start preparing for Git 2.0,\n> 2014-03-07), but I am not sure what to do with them ;-<))\n\nWhy not just revert on master?  When merging with the topic branches,\nthe revert should then override the contamination.\n\nIt makes some sense then to rewrite the \"submodule update\" topic branch\nso that when it gets reintroduced, its commits are not negated by the\nbranch revert on master.\n\nSounds like a stock case out of the \"using Git with topic branches\"\nbook, so what am I missing?\n\n-- \nDavid Kastrup\n"},{"id":"238312","messageId":"xmqqlhvntt6t.fsf@gitster.dls.corp.google.com","threadId":"36326","inReplyTo":"87ha6cian0.fsf@fencepost.gnu.org","subject":"Re: What's cooking in git.git (Mar 2014, #08; Mon, 31)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2014-04-02T19:41:46Z","receivedAt":"2014-04-02T19:41:46Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"David Kastrup <dak@gnu.org> writes:\n\n> Junio C Hamano <gitster@pobox.com> writes:\n>\n>> Junio C Hamano <gitster@pobox.com> writes:\n>>\n>>> I haven't reverted the merge of that \"submodule update\" topic yet; I\n>>> should do that soonish.\n>>> ...\n>>\n>> Sigh...  This is giving me a lot of headache.\n>>\n>> As 23d25e48 (submodule: explicit local branch creation in\n>> module_clone, 2014-01-26) has been in 'master' since fairly early\n>> during this cycle, a lot of topics that are not planned to be on the\n>> 'maint' branch has forked from the tip of 'master' and are now\n>> contaminated by that commit.\n>>\n>> I think I have a preparatory patch to correctly revert 00d4ff1a\n>> (Merge branch 'wt/doc-submodule-name-path-confusion-2', 2014-03-31)\n>> and 06c27689 (Merge branch 'wk/submodule-on-branch', 2014-02-27),\n>> and also a part of 384364b (Start preparing for Git 2.0,\n>> 2014-03-07), but I am not sure what to do with them ;-<))\n>\n> Why not just revert on master?  When merging with the topic branches,\n> the revert should then override the contamination.\n\nThat was actually not the cumbersome part.  I wanted to be very sure\nthat the revert was correctly done, and one way I know to get an\nindependent verification is to rebuild the master branch starting\nall the way back from the point before the problematic topic was\nmerged to it.  Some of the topics merged to 'master' after that\npoint, however, were forked after that original problematic merge\nwas made, so they needed to be rebuilt before I could do so.\n\nIt is worth noting that this verification can also be done in a\ndifferent way.  You can start at 06c27689^1, and \"cherry-pick -m1\"\n(or \"cherry-pick\" for non-merge commits that update the release\nnotes) the commits in \"git rev-list --reverse --first-parent\n06c27689..master\" on top of it.  That should result in the same tree\nobject as a correct revert on top of 'master' would have.\n\nBecause \"cherry-pick -m1\" loses the other parents, the resulting\nhistory does not reflect the reality, but I am not doing this in\norder to replace the history of the 'master' with the result but\nonly to make sure that the final tree matches what would have\nhappened if the topic were not merged to 'master', so it is\nsufficient for the purpose of this exercise.\n\nHope it clarifies.\n"},{"id":"238316","messageId":"20140402200409.GA9984@sandbox-ub","threadId":"36326","inReplyTo":"xmqqppl128q8.fsf@gitster.dls.corp.google.com","subject":"Re: What's cooking in git.git (Mar 2014, #08; Mon, 31)","fromName":"Heiko Voigt","fromEmail":"hvoigt@hvoigt.net","sentAt":"2014-04-02T20:04:09Z","receivedAt":"2014-04-02T20:04:09Z","isPatch":false,"sender":{"key":"hvoigt@hvoigt.net","avatar":"https://avatars.githubusercontent.com/u/184958?v=4"},"body":"On Mon, Mar 31, 2014 at 05:29:03PM -0700, Junio C Hamano wrote:\n> * hv/submodule-ignore-fix (2013-12-06) 4 commits\n>  - disable complete ignorance of submodules for index <-> HEAD diff\n>  - always show committed submodules in summary after commit\n>  - teach add -f option for ignored submodules\n>  - fix 'git add' to skip submodules configured as ignored\n> \n>  Teach \"git add\" to be consistent with \"git status\" when changes to\n>  submodules are set to be ignored, to avoid surprises after checking\n>  with \"git status\" to see there isn't any change to be further added\n>  and then see that \"git add .\" adds changes to them.\n> \n>  I think a reroll is coming, so this may need to be replaced, but I\n>  needed some practice with heavy conflict resolution.  It conflicts\n>  with two changes to \"git add\" that have been scheduled for Git 2.0\n>  quite badly, and I think I got the resolution right this time.\n> \n>  Waiting for a reroll.\n\nSince Ronald and Jens picked up this topic[1], I think you can discard\nmy series.\n\nCheers Heiko\n\n[1] http://thread.gmane.org/gmane.comp.version-control.git/245341\n"},{"id":"238290","messageId":"8538hvwc3e.fsf_-_@stephe-leake.org","threadId":"36326","inReplyTo":"CACsJy8BVXk248VDBi-pAMDY4QpDSbFuAf5qxRxCCq71yGNzpqA@mail.gmail.com","subject":"Re: emacs buffer names","fromName":"Stephen Leake","fromEmail":"stephen_leake@stephe-leake.org","sentAt":"2014-04-02T23:22:45Z","receivedAt":"2014-04-02T23:22:45Z","isPatch":false,"sender":{"key":"stephen_leake@stephe-leake.org","avatar":null},"body":"Duy Nguyen <pclouds@gmail.com> writes:\n\n> The --daemon part is probably not worth mentioning because I always\n> have one emacs window open. The problem is switch-buffer based on file\n> name can be confusing (\"git.c\" and \"git.c<2>\", which belongs to which\n> checkout?). I ended up modifying files in the wrong checkout all the\n> time.\n\n(setq uniquify-buffer-name-style (quote post-forward-angle-brackets) nil (uniquify))\n\nThis puts enough of the directory name in the buffer name to make the\nbuffer names unique; very helpful!\n\n-- \n-- Stephe\n"},{"id":"238355","messageId":"xmqqmwg2qqqv.fsf@gitster.dls.corp.google.com","threadId":"36326","inReplyTo":"20140402200409.GA9984@sandbox-ub","subject":"Re: What's cooking in git.git (Mar 2014, #08; Mon, 31)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2014-04-03T17:15:04Z","receivedAt":"2014-04-03T17:15:04Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Heiko Voigt <hvoigt@hvoigt.net> writes:\n\n> On Mon, Mar 31, 2014 at 05:29:03PM -0700, Junio C Hamano wrote:\n>> * hv/submodule-ignore-fix (2013-12-06) 4 commits\n>>  - disable complete ignorance of submodules for index <-> HEAD diff\n>>  - always show committed submodules in summary after commit\n>>  - teach add -f option for ignored submodules\n>>  - fix 'git add' to skip submodules configured as ignored\n>> \n>>  Teach \"git add\" to be consistent with \"git status\" when changes to\n>>  submodules are set to be ignored, to avoid surprises after checking\n>>  with \"git status\" to see there isn't any change to be further added\n>>  and then see that \"git add .\" adds changes to them.\n>> \n>>  I think a reroll is coming, so this may need to be replaced, but I\n>>  needed some practice with heavy conflict resolution.  It conflicts\n>>  with two changes to \"git add\" that have been scheduled for Git 2.0\n>>  quite badly, and I think I got the resolution right this time.\n>> \n>>  Waiting for a reroll.\n>\n> Since Ronald and Jens picked up this topic[1], I think you can discard\n> my series.\n\nThanks, will do.\n"}]}