{"thread":{"id":"30201","subject":"What's cooking in git.git (Apr 2012, #04; Tue, 10)","startedAt":"2012-04-10T22:27:57Z","lastAt":"2012-04-12T06:35:39Z","messageCount":4,"participants":["Junio C Hamano","Jakub Narebski","Lucian Poston"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"188912","messageId":"7vty0rnrwy.fsf@alter.siamese.dyndns.org","threadId":"30201","inReplyTo":null,"subject":"What's cooking in git.git (Apr 2012, #04; Tue, 10)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2012-04-10T22:27:57Z","receivedAt":"2012-04-10T22:27:57Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"What's cooking in git.git (Apr 2012, #04; Tue, 10)\n--------------------------------------------------\n\nThis issue describes the state of my tree in preparation for opening the\npost 1.7.10 cycle.  The 'next' branch has not been rewound yet, but it\nsoon will be.  The Stalled topics will be discarded by default.\n\n\"git subtree\" is in contrib/ now.\n\nYou can find the changes described here in the integration branches of the\nrepositories listed at\n\n    http://git-blame.blogspot.com/p/git-public-repositories.html\n\n--------------------------------------------------\n[New Topics]\n\n* bw/submodule-sed-solaris (2012-04-09) 1 commit\n - Avoid bug in Solaris xpg4/sed as used in submodule\n\nSensible.\nWill merge to 'next'.\n\n* cb/http-multi-curl-auth (2012-04-10) 2 commits\n - fix http auth with multiple curl handles\n - http auth fails with multiple curl handles\n\nFixes http authentication breakage when we keep multiple HTTP requests in\nflight using curl-multi.\n\nPeff may want to sign this off, but otherwise looked sensible.\n\n* jk/rebase-i-submodule-conflict-only (2012-04-09) 1 commit\n - rebase -i continue: don't skip commits that only change submodules\n\nGiving \"--continue\" to a conflicted \"rebase -i\" session skipped a commit\nthat only results in changes to submodules.\n\nSensible.\nWill merge to 'next'.\n\n* ms/remote-usage-string (2012-04-09) 1 commit\n - remote: update builtin usage\n\nAs an independent follow-up, we may want to rethink how the overall usage\nstring and subcommand usage strings are maintained.\n\nWill merge to 'next'.\n\n* pw/fast-import-dataref-parsing (2012-04-10) 1 commit\n - fast-import: tighten parsing of datarefs\n\n* pw/git-p4 (2012-04-09) 3 commits\n - git p4: use \"git p4\" directly in tests\n - git p4: update name in script\n - git-p4: move to toplevel\n\nWill merge to 'next'.\n\n* rl/show-empty-prefix (2012-04-10) 1 commit\n - rev-parse --show-prefix: add in trailing newline\n\nWill merge to 'next'.\n\n* tr/xdiff-fast-hash (2012-04-09) 1 commit\n - xdiff: load full words in the inner loop of xdl_hash_record\n\n--------------------------------------------------\n[Stalled]\n\n* lp/maint-diff-three-dash-with-graph (2012-03-20) 3 commits\n - t4202: add test for \"log --graph --stat -p\" separator lines\n - log --graph: fix break in graph lines\n - log --graph --stat: three-dash separator should come after graph lines\n\nWith a review from Zbigniew, I have been expecting that this would be\nrerolled, but that didn't happen.\n\nWill discard shortly unless nothing happens.\n\n* cn/apply-fix-ws-can-lengthen-lines (2012-03-11) 1 commit\n . apply: reallocate the postimage buffer when needed\n\nAttempts to address an ancient bug that dates back to the addition\nof an oddball \"tab-in-indent\" whitespace breakage class that wants\nto have longer lines than the original when fixing things up; broken\nand results in double-frees.\n\nWill discard shortly unless nothing happens.\n\n* nd/columns (2012-03-13) 12 commits\n - column: support grouping entries\n - column: support \"denser\" mode\n - ls-files: support --column\n - tag: add --column\n - column: support piping stdout to external git-column process\n - status: add --column\n - branch: add --column\n - help: reuse print_columns() for help -a\n - column: add dense layout support\n - column: add columnar layout\n - Stop starting pager recursively\n - Add column layout skeleton and git-column\n\nModulo minor nits, looked nicer than the previous round and getting closer\nto completion but nothing has happened recently.\n\nWill discard shortly unless nothing happens.\n\n* nd/threaded-index-pack (2012-03-11) 2 commits\n - index-pack: support multithreaded delta resolving\n - index-pack: split second pass obj handling into own function\n\nAnother reroll after a bugreport on pthread usage discovered by Ramsey,\nbut it seems the topic is cooking between Ramsay and Duy out of tree.\n\nWill discard shortly unless nothing happens.\n\n* jh/apply-free-patch (2012-03-28) 7 commits\n - apply.c: WIP ownership audit\n - apply: free unused fragments for submodule patch\n - apply: free patch->result\n - apply: release memory for fn_table\n - apply: free patch->{def,old,new}_name fields\n - apply: rename free_patch() to free_patch_list()\n - apply: do not leak patches and fragments\n\nValgrind reports quite a lot of discarded memory inside apply.  I started\nauditing the memory ownership rules in the command, and am almost done.\n\n* ss/git-svn-prompt-sans-terminal (2012-01-04) 3 commits\n - fixup! 15eaaf4\n - git-svn, perl/Git.pm: extend Git::prompt helper for querying users\n - perl/Git.pm: \"prompt\" helper to honor GIT_ASKPASS and SSH_ASKPASS\n\nThe bottom one has been replaced with a rewrite based on comments\nfrom Ævar. The second one needs more work, both in perl/Git.pm and\nprompt.c, to give precedence to tty over SSH_ASKPASS when terminal\nis available.\n\nWill discard shortly unless nothing happens.\n\n* jc/split-blob (2012-04-03) 7 commits\n - chunked-object: streaming checkout\n - chunked-object: fallback checkout codepaths\n - bulk-checkin: support chunked-object encoding\n - bulk-checkin: allow the same data to be multiply hashed\n - new representation types in the packstream\n - packfile: use varint functions\n - varint: make it available outside the context of pack\n (this branch is tangled with jc/index-v4.)\n\nNot ready.\n\nI finished the streaming checkout codepath, but as explained in\n127b177 (bulk-checkin: support chunked-object encoding, 2011-11-30),\nthese are still early steps of a long and painful journey. At least\npack-objects and fsck need to learn the new encoding for the series\nto be usable locally, and then index-pack/unpack-objects needs to\nlearn it to be used remotely.\n\nGiven that I heard a lot of noise that people want large files, and\nthat I was asked by somebody at GitTogether'11 privately for an\nadvice on how to pay developers (not me) to help adding necessary\nsupport, I am somewhat dissapointed that the original patch series\nthat was sent almost two months ago still remains here without much\ncomments and updates from the developer community. I even made the\ninterface to the logic that decides where to split chunks easily\nreplaceable, and I deliberately made the logic in the original patch\nextremely stupid to entice others, especially the \"bup\" fanboys, to\ncome up with a better logic, thinking that giving people an easy\ntarget to shoot for, they may be encouraged to help out. The plan is\nnot working :-(.\n\n--------------------------------------------------\n[Cooking]\n\n* jc/index-v4 (2012-04-04) 10 commits\n - update-index: upgrade/downgrade on-disk index version\n - read-cache.c: write prefix-compressed names in the index\n - read-cache.c: read prefix-compressed names in index on-disk version v4\n - read-cache.c: move code to copy incore to ondisk cache to a helper function\n - read-cache.c: move code to copy ondisk to incore cache to a helper function\n - read-cache.c: report the header version we do not understand\n - read-cache.c: make create_from_disk() report number of bytes it consumed\n - read-cache.c: allow unaligned mapping of the index file\n - cache.h: hide on-disk index details\n - varint: make it available outside the context of pack\n (this branch is tangled with jc/split-blob.)\n\nTrivially shrinks the on-disk size of the index file to save both I/O and\nchecksum overhead.  The \"v4\" format represented here may not be what we\neventually want to have, but the preparatory refactoring steps and the way\nhow backward compatibility is retained should give plenty to learn to GSoC\nstudent-hopefuls.\n\n* jk/add-p-skip-conflicts (2012-04-05) 1 commit\n - add--interactive: ignore unmerged entries in patch mode\n\nExcludes conflicted paths from \"add -p\" processing, as it is not prepared\nto handle them.\n\nWill merge to 'next'.\n\n* bw/spawn-via-shell-path (2012-04-03) 1 commit\n - Use SHELL_PATH from build system in run_command.c:prepare_shell_cmd\n\n\"sh\" on the user's PATH may be utterly broken on some systems;\nconsistently use SHELL_PATH even from inside run-command API.\n\nWill merge to 'next'.\n\n* jc/commit-unedited-template (2012-04-03) 5 commits\n - Documentation/git-commit: rephrase the \"initial-ness\" of templates\n - git-commit.txt: clarify -t requires editing message\n - commit: rephrase the error when user did not touch templated log message\n - commit: do not trigger bogus \"has templated message edited\" check\n - t7501: test the right kind of breakage\n\nWhen \"git commit --template F\" errors out because the user did not touch\nthe message, it claimed that it aborts due to \"empty message\", which was\nutterly wrong.\n\nWill merge to 'next'.\n\n* jc/push-upstream-sanity (2012-04-05) 1 commit\n - push: error out when the \"upstream\" semantics does not make sense\n\nFix broken 'push to upstream' implementation.  \"git push $there\" without\nrefspec, when the current branch is set to push to a remote different from\n$there, used to push to $there using the upstream information to a remote\nunreleated to $there.\n\nWill merge to 'next'.\n\n* jk/http-backend-keep-committer-ident-env (2012-03-30) 1 commit\n - http-backend: respect existing GIT_COMMITTER_* variables\n\nThe smart-http backend used to always override GIT_COMMITTER_* variables\nwith REMOTE_USER and REMOTE_ADDR.\n\nWill merge to 'next'.\n\n* mk/gitweb-diff-hl (2012-04-05) 8 commits\n - gitweb: Refinement highlightning in combined diffs\n - gitweb: Highlight interesting parts of diff\n - gitweb: Push formatting diff lines to print_diff_chunk()\n - gitweb: Use print_diff_chunk() for both side-by-side and inline diffs\n - gitweb: Extract print_sidebyside_diff_lines()\n - gitweb: Pass esc_html_hl_regions() options to esc_html()\n - gitweb: esc_html_hl_regions(): Don't create empty <span> elements\n - gitweb: Use descriptive names in esc_html_hl_regions()\n\nRerolled.\n\n* it/fetch-pack-many-refs (2012-04-10) 4 commits\n - remote-curl: main test case for the OS command line overflow\n - fetch-pack: test cases for the new --stdin option\n - remote-curl: send the refs to fetch-pack on stdin\n - fetch-pack: new --stdin option to read refs from stdin\n\nWill merge to 'next'.\n\n* jn/debian-customizes-default-editor (2012-04-10) 2 commits\n - var doc: advertise current DEFAULT_PAGER and DEFAULT_EDITOR settings\n - var doc: default editor and pager are configurable at build time\n\nWill merge to 'next'.\n\n* rs/commit-list-sort-in-batch (2012-04-09) 3 commits\n - revision: insert unsorted, then sort in prepare_revision_walk()\n - commit: use mergesort() in commit_list_sort_by_date()\n - add mergesort() for linked lists\n\n* hv/submodule-recurse-push (2012-03-30) 3 commits\n - push: teach --recurse-submodules the on-demand option\n - Refactor submodule push check to use string list instead of integer\n - Teach revision walking machinery to walk multiple times sequencially\n\n* jk/branch-quiet (2012-03-26) 2 commits\n - teach \"git branch\" a --quiet option\n - checkout: suppress tracking message with \"-q\"\n\nEven with \"-q\"uiet option, \"checkout\" used to report setting up tracking.\nAlso \"branch\" learns \"-q\"uiet option to squelch informational message.\n\nWill merge to 'next'.\n\n* jk/run-command-eacces (2012-04-05) 2 commits\n - run-command: treat inaccessible directories as ENOENT\n - compat/mingw.[ch]: Change return type of exec functions to int\n\nWhen PATH contains an unreadable directory, alias expansion code did not\nkick in, and failed with an error that said \"git-subcmd\" was not found.\n\nWill merge to 'next'.\n\n* jb/am-include (2012-03-28) 1 commit\n - am: support --include option\n\nWill merge to 'next'.\n\n* jc/am-report-3way (2012-03-28) 1 commit\n - am -3: list the paths that needed 3-way fallback\n\nWill merge to 'next'.\n\n* rs/combine-diff-zero-context-at-the-beginning (2012-03-25) 1 commit\n - combine-diff: fix loop index underflow\n\nFixes an age old corner case bug in combine diff (only triggered with -U0\nand the hunk at the beginning of the file needs to be shown).\n\nWill merge to 'next'.\n\n* sl/autoconf (2012-03-26) 3 commits\n - configure: be more idiomatic\n - configure: avoid some code repetitions thanks to m4_{push,pop}def\n - configure: move definitions of private m4 macros before AC_INIT invocation\n\nUpdates our configure.ac to follow a better \"autoconf\" style.\n\nWill merge to 'next'.\n\n* wk/gitweb-snapshot-use-if-modified-since (2012-03-30) 3 commits\n - gitweb: add If-Modified-Since handling to git_snapshot().\n - gitweb: refactor If-Modified-Since handling\n - gitweb: add `status` headers to git_feed() responses.\n\nMakes 'snapshot' request to \"gitweb\" honor If-Modified-Since: header,\nbased on the commit date.\n\nWill merge to 'next'.\n\n* jk/diff-no-rename-empty (2012-03-23) 4 commits\n - merge-recursive: don't detect renames of empty files\n - teach diffcore-rename to optionally ignore empty content\n - make is_empty_blob_sha1 available everywhere\n - drop casts from users EMPTY_TREE_SHA1_BIN\n\nForbids rename detection logic from matching two empty files as renames\nduring merge-recursive to prevent mismerges.\n\nWill merge to 'next'.\n\n* th/difftool-diffall (2012-04-04) 8 commits\n - difftool: print list of valid tools with '--tool-help'\n - difftool: teach difftool to handle directory diffs\n - difftool: eliminate setup_environment function\n - difftool: stop appending '.exe' to git\n - difftool: remove explicit change of PATH\n - difftool: exit(0) when usage is printed\n - difftool: add '--no-gui' option\n - difftool: parse options using Getopt::Long\n\nRolls the two-directory-diff logic from diffall script (in contrib/) into\n\"git difftool\" framework. \n\n* jc/maint-clean-nested-worktree-in-subdir (2012-03-15) 2 commits\n  (merged to 'next' on 2012-03-20 at fb5485e)\n + clean: preserve nested git worktree in subdirectories\n + remove_dir_recursively(): Add flag for skipping removal of toplevel dir\n (this branch is tangled with jh/notes-merge-in-git-dir-worktree.)\n\n\"git clean -d -f\" (not \"-d -f -f\") is supposed to protect nested working\ntrees of independent git repositories that exist in the current project\nworking tree from getting removed, but the protection applied only to such\nworking trees that are at the top-level of the current project by mistake.\n\nWill merge to 'next'.\n\n* ct/advise-push-default (2012-03-26) 2 commits\n  (merged to 'next' on 2012-03-28 at 62764ae)\n + clean up struct ref's nonfastforward field\n + push: Provide situational hints for non-fast-forward errors\n\nBreaks down the cases in which \"git push\" fails due to non-ff into three\ncategories, and gives separate advise messages.  This should be a good\nchange regardless of mm/push-default-switch-warning topic.\n\nWill merge to 'next'.\n\n* nl/rebase-i-cheat-sheet (2012-03-20) 1 commit\n  (merged to 'next' on 2012-03-20 at 3092a2b)\n + rebase -i: remind that the lines are top-to-bottom\n\nWill merge to 'next'.\n\n* da/difftool-test (2012-03-19) 1 commit\n  (merged to 'next' on 2012-03-20 at 0ada7d4)\n + t7800: Test difftool passing arguments to diff\n\nMakes sure \"difftool\" options can be given in any order.\n\nWill merge to 'master'.\n\n* jh/notes-merge-in-git-dir-worktree (2012-03-15) 4 commits\n  (merged to 'next' on 2012-03-20 at 0c1b1de)\n + notes-merge: Don't remove .git/NOTES_MERGE_WORKTREE; it may be the user's cwd\n + notes-merge: use opendir/readdir instead of using read_directory()\n + t3310: illustrate failure to \"notes merge --commit\" inside $GIT_DIR/\n + remove_dir_recursively(): Add flag for skipping removal of toplevel dir\n (this branch is tangled with jc/maint-clean-nested-worktree-in-subdir.)\n\nRunning \"notes merge --commit\" failed to perform correctly when run\nfrom any directory inside $GIT_DIR/.  When \"notes merge\" stops with\nconflicts, $GIT_DIR/NOTES_MERGE_WORKTREE is the place a user edits\nto resolve it.\n\nWill merge to 'master'.\n\n* jn/diffstat-tests (2012-03-13) 7 commits\n  (merged to 'next' on 2012-03-20 at 8791b2f)\n + diffstat summary line varies by locale: miscellany\n + test: use numstat instead of diffstat in binary-diff test\n + test: use --numstat instead of --stat in \"git stash show\" tests\n + test: test cherry-pick functionality and output separately\n + test: modernize funny-names test style\n + test: use numstat instead of diffstat in funny-names test\n + test: use test_i18ncmp when checking --stat output\n\nSome tests checked the \"diff --stat\" output when they do not have to,\nwhich unnecessarily made things harder to verify under GETTEXT_POISON.\n\nWill merge to 'master'.\n\n* tr/maint-word-diff-regex-sticky (2012-03-14) 3 commits\n  (merged to 'next' on 2012-03-20 at b3f67cd)\n + diff: tweak a _copy_ of diff_options with word-diff\n + diff: refactor the word-diff setup from builtin_diff_cmd\n + t4034: diff.*.wordregex should not be \"sticky\" in --word-diff\n\nThe regexp configured with wordregex was incorrectly reused across\nfiles.\n\nWill merge to 'master'.\n\n* zj/test-cred-helper-nicer-prove (2012-03-15) 2 commits\n  (merged to 'next' on 2012-03-20 at b675ec1)\n + t0303: resurrect commit message as test documentation\n + t0303: immediately bail out w/o GIT_TEST_CREDENTIAL_HELPER\n\nMinor improvement to t0303.\nWill merge to 'master'.\n\n* jc/commit-hook-authorship (2012-03-11) 3 commits\n  (merged to 'next' on 2012-03-12 at 05ca7f8)\n + commit: pass author/committer info to hooks\n + t7503: does pre-commit-hook learn authorship?\n + ident.c: add split_ident_line() to parse formatted ident line\n (this branch is tangled with jc/run-hook-env-1.)\n\n\"git commit --author=$name\" did not tell the name that was being\nrecorded in the resulting commit to hooks, even though it does do so\nwhen the end user overrode the authorship via the \"GIT_AUTHOR_NAME\"\nenvironment variable.\n\nWill merge to 'master'.\n\n* jc/run-hook-env-1 (2012-03-11) 3 commits\n - run_hook(): enhance the interface to pass arbitrary environment\n + t7503: does pre-commit-hook learn authorship?\n + ident.c: add split_ident_line() to parse formatted ident line\n (this branch is tangled with jc/commit-hook-authorship.)\n\nNot urgent.\n\nUpdates run_hook() API to be much less specific to \"commit\".  It would\nonly be useful if people start doing more interesting things with hooks.\n\n* jc/diff-algo-cleanup (2012-02-19) 2 commits\n  (merged to 'next' on 2012-03-15 at cca0032)\n + xdiff: PATIENCE/HISTOGRAM are not independent option bits\n + xdiff: remove XDL_PATCH_* macros\n\nResurrects the preparatory clean-up patches from another topic that was\ndiscarded, as this would give a saner foundation to build on diff.algo\nconfiguration option series.\n\nWill merge to 'master'.\n\n* rs/unpack-trees-leakfix (2012-03-06) 1 commit\n  (merged to 'next' on 2012-03-07 at 69a69cd)\n + unpack-trees: plug minor memory leak\n\nWill sneak a fix-up under this one when I rebuild 'next' tomorrow.\n\n* mm/push-default-switch-warning (2012-03-09) 1 commit\n  (merged to 'next' on 2012-03-28 at 074b16b)\n + push: start warning upcoming default change for push.default\n\nThis resurrects an ancient patch I wrote during a discussion we had in the\n1.6.3-1.6.4 era.  This should probably come after ct/advise-push-default\ntopic and at that point the advise messages need to be rephrased, taking\nthe future default change into account.\n\nWill kick back to 'pu'.\n\n* jc/fmt-merge-msg-people (2012-03-13) 1 commit\n - fmt-merge-msg: show those involved in a merged series\n\nThe \"fmt-merge-msg\" command learns to list the primary contributors\ninvolved in the side topic you are merging.\n\nWill merge to 'next'.\n\n* nl/http-proxy-more (2012-04-10) 6 commits\n - fixup? 6e17b8c\n  (merged to 'next' on 2012-03-20 at c004001)\n + http: rename HTTP_REAUTH to HTTP_AUTH_RETRY\n + http: Avoid limit of retrying request only twice\n + http: handle proxy authentication failure (error 407)\n + http: handle proxy proactive authentication\n + http: try http_proxy env var when http.proxy config option is not set\n\nThe code to talk to http proxies learn to use the same credential\nAPI used to talk to the final http destinations.\n\nWill kick back to 'pu'.\n\n* nd/stream-more (2012-03-07) 7 commits\n  (merged to 'next' on 2012-03-07 at 7325922)\n + update-server-info: respect core.bigfilethreshold\n + fsck: use streaming API for writing lost-found blobs\n + show: use streaming API for showing blobs\n + parse_object: avoid putting whole blob in core\n + cat-file: use streaming API to print blobs\n + Add more large blob test cases\n + streaming: make streaming-write-entry to be more reusable\n\nUse API to read blob data in smaller chunks in more places to\nreduce the memory footprint.  In general, looked fairly good.\n\nWill merge to 'master'.\n"},{"id":"188973","messageId":"m3ty0q9sco.fsf@localhost.localdomain","threadId":"30201","inReplyTo":"7vty0rnrwy.fsf@alter.siamese.dyndns.org","subject":"Re: What's cooking in git.git (Apr 2012, #04; Tue, 10)","fromName":"Jakub Narebski","fromEmail":"jnareb@gmail.com","sentAt":"2012-04-11T15:54:32Z","receivedAt":"2012-04-11T15:54:32Z","isPatch":false,"sender":{"key":"jnareb@gmail.com","avatar":"https://avatars.githubusercontent.com/u/2706?v=4"},"body":"Junio C Hamano <gitster@pobox.com> writes:\n\n> Will discard shortly unless nothing happens.\n\nDouble negation?  \"Unless nothing\"?\n \n> * mk/gitweb-diff-hl (2012-04-05) 8 commits\n> * wk/gitweb-snapshot-use-if-modified-since (2012-03-30) 3 commits\n\nWhat about \"[PATCH (bugfix)] gitweb: Fix unintended \"--no-merges\" for\nregular Atom feed\" from Sebastian Pipping?\n\n  http://thread.gmane.org/gmane.comp.version-control.git/194535/focus=194695\n\n-- \nJakub Narebski\n"},{"id":"188991","messageId":"7vhawqmd1m.fsf@alter.siamese.dyndns.org","threadId":"30201","inReplyTo":"m3ty0q9sco.fsf@localhost.localdomain","subject":"Re: What's cooking in git.git (Apr 2012, #04; Tue, 10)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2012-04-11T16:46:45Z","receivedAt":"2012-04-11T16:46:45Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Jakub Narebski <jnareb@gmail.com> writes:\n\n> Junio C Hamano <gitster@pobox.com> writes:\n>\n>> Will discard shortly unless nothing happens.\n>\n> Double negation?  \"Unless nothing\"?\n>  \n>> * mk/gitweb-diff-hl (2012-04-05) 8 commits\n>> * wk/gitweb-snapshot-use-if-modified-since (2012-03-30) 3 commits\n>\n> What about \"[PATCH (bugfix)] gitweb: Fix unintended \"--no-merges\" for\n> regular Atom feed\" from Sebastian Pipping?\n>\n>   http://thread.gmane.org/gmane.comp.version-control.git/194535/focus=194695\n\nPlease stop saying \"What about\" on Gitweb topics.  I am not keeping track\nand mostly trusting your judgement; resending with Ack is sufficient.\n\nThanks.\n"},{"id":"189078","messageId":"CACz_eycyOFmPqsJiB+1OvM4GwpOi-xD0RoxGvn3SJiwqVKS4mw@mail.gmail.com","threadId":"30201","inReplyTo":"7vty0rnrwy.fsf@alter.siamese.dyndns.org","subject":"Re: What's cooking in git.git (Apr 2012, #04; Tue, 10)","fromName":"Lucian Poston","fromEmail":"lucian.poston@gmail.com","sentAt":"2012-04-12T06:35:39Z","receivedAt":"2012-04-12T06:35:39Z","isPatch":false,"sender":{"key":"lucian.poston@gmail.com","avatar":"https://avatars.githubusercontent.com/u/646121?v=4"},"body":"On Tue, Apr 10, 2012 at 15:27, Junio C Hamano <gitster@pobox.com> wrote:\n> --------------------------------------------------\n> [Stalled]\n>\n> * lp/maint-diff-three-dash-with-graph (2012-03-20) 3 commits\n>  - t4202: add test for \"log --graph --stat -p\" separator lines\n>  - log --graph: fix break in graph lines\n>  - log --graph --stat: three-dash separator should come after graph lines\n>\n> With a review from Zbigniew, I have been expecting that this would be\n> rerolled, but that didn't happen.\n>\n> Will discard shortly unless nothing happens.\n\nThis weekend I'll post an updated version of the patch Zbigniew reviewed.\n"}]}