{"thread":{"id":"30090","subject":"What's cooking in git.git (Mar 2012, #10; Wed, 28)","startedAt":"2012-03-28T22:23:25Z","lastAt":"2012-03-30T15:56:51Z","messageCount":5,"participants":["Junio C Hamano","Jakub Narebski"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"188019","messageId":"7vsjgss6ua.fsf@alter.siamese.dyndns.org","threadId":"30090","inReplyTo":null,"subject":"What's cooking in git.git (Mar 2012, #10; Wed, 28)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2012-03-28T22:23:25Z","receivedAt":"2012-03-28T22:23:25Z","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 '-' are\nonly in 'pu' (proposed updates) while commits prefixed with '+' are in 'next'.\n\nHopefully the release will be done early next week.  The final rc was\ntagged and pushed out today so that people can play with it and report\npossible issues.\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* dg/subtree (2012-03-25) 112 commits\n - Add 'contrib/subtree/' from commit '2e63f75b8f49abe220ef55ec4e978e7a3b8dc351'\n - Add Subtree Test Makefile\n - Build Subtree\n - Use Project Config Files\n - Remove Unneeded Files\n - ...\n\nA test merge of the 'subtree'.\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\n* jk/run-command-eacces (2012-03-27) 3 commits\n - [DONTMERGE] semi-squashable test addition\n - git: continue alias lookup on EACCES errors\n - run-command: propagate EACCES errors to parent\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\n* jb/am-include (2012-03-28) 1 commit\n - am: support --include option\n\n* jc/am-report-3way (2012-03-28) 1 commit\n - am -3: list the paths that needed 3-way fallback\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\nThe combination of two options \"log --graph --stat\" was an obscure corner\ncase nobody cared about, and did not correctly show the ancestry graph\nlines.\n\nI've split the original patch into three pieces, one for fixes to two\ndifferent issues and a test.  Also the test is adjusted so that the series\ncan be back-merged to older codebase that did not have 7f81463 (Use\ncorrect grammar in diffstat summary line, 2012-02-01) that first appeared\nin v1.7.9.2\n\nWith a review from Zbigniew, I would expect that this would be rerolled\nagain.\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.\n\nNeeds more work; results in double-frees.\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\nRerolled again.  Modulo minor nits, looked nicer than the previous round.\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.\nWaiting for resolution.\n\n* jh/apply-free-patch (2012-03-27) 6 commits\n - apply.c: WIP ownership audit\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\nWill defer til 1.7.10.\n\n* hv/submodule-recurse-push (2012-02-13) 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\nThe bottom one was not clearly explained and needs a reroll.\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\n* jc/split-blob (2012-03-16) 6 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 - varint-in-pack: refactor varint encoding/decoding\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* 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\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\n* wk/gitweb-snapshot-use-if-modified-since (2012-03-26) 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\n* jc/maint-merge-autoedit (2012-03-20) 1 commit\n - merge: backport GIT_MERGE_AUTOEDIT support\n\nIn 1.7.10, we added GIT_MERGE_AUTOEDIT=no environment variable to help\nolder scripts to let them refuse giving users a chance to explain the\nmerge, but forgot that 1.7.9 automatically opens an editor when merging an\nannotated tag, and there is no equivalent escape hatch.  A merge of this\ntopic to 1.7.10 track becomes a no-op, but we may want to apply this to\nthe 1.7.9.x series.\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\n* th/difftool-diffall (2012-03-26) 9 commits\n - difftool: print list of valid tools with '--tool-help'\n - difftool: teach difftool to handle directory diffs\n - difftool: replace system call with Git::command_noisy\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\nNot urgent.\n\n* ct/advise-push-default (2012-03-26) 2 commits\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\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\nNot urgent.\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\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\nNot urgent.\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\nNot urgent.\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.\nNot urgent.\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.\nNot urgent.\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.  This is a simpler of the two approaches.\n\nWill defer til 1.7.10.\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\nNot urgent.\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 defer til 1.7.10.\n\n* mm/push-default-switch-warning (2012-03-09) 1 commit\n - push: start warning upcoming default change for push.default\n\nNot urgent.\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\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 defer til 1.7.10.\n\n* nl/http-proxy-more (2012-03-15) 5 commits\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 defer til 1.7.10.\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 defer til 1.7.10.\n\n--------------------------------------------------\n[Discarded]\n\n* tb/maint-remove-irrelevant-i18n-test (2012-03-06) 1 commit\n  (merged to 'next' on 2012-03-07 at 23f2dd1)\n + t0204: remove a test that checks undefined behaviour\n\nI tentatively parked this in 'next' but later reverted the merge.\n\n* dg/test-from-elsewhere (2012-03-04) 2 commits\n . Support out-of-tree Valgrind tests\n . Allow overriding GIT_BUILD_DIR\n\nNo immediate need; dropped.\n"},{"id":"188108","messageId":"m3fwcrarwo.fsf@localhost.localdomain","threadId":"30090","inReplyTo":"7vsjgss6ua.fsf@alter.siamese.dyndns.org","subject":"Re: What's cooking in git.git (Mar 2012, #10; Wed, 28)","fromName":"Jakub Narebski","fromEmail":"jnareb@gmail.com","sentAt":"2012-03-29T23:51:01Z","receivedAt":"2012-03-29T23:51:01Z","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> --------------------------------------------------\n> [Cooking]\n\n> * wk/gitweb-snapshot-use-if-modified-since (2012-03-26) 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> \n> Makes 'snapshot' request to \"gitweb\" honor If-Modified-Since: header,\n> based on the commit date.\n\nWhat about \"[PATCH v2 0/8] gitweb: Highlight interesting parts of diff\"\nseries by Michał Kiedrowicz:\n\n  Message-ID: <1332543417-19664-1-git-send-email-michal.kiedrowicz@gmail.com>\n  http://thread.gmane.org/gmane.comp.version-control.git/193804\n\nI have copy of them in 'gitweb/diff-hl' branch:\n\n-- >8 --\nThe following changes since commit 455cf268dbaf227bdbd5e9fbf96525452bcfe44f:\n\n  Git 1.7.10-rc3 (2012-03-28 11:18:42 -0700)\n\nare available in the git repository at:\n\n  git://repo.or.cz/git/jnareb-git.git gitweb/diff-hl\n\nfor you to fetch changes up to 60b06478f0349729dc9a4ddeb1abf5e28986fb7e:\n\n  gitweb: Refinement highlightning in combined diffs (2012-03-29 21:26:36 +0200)\n\n----------------------------------------------------------------\nMichał Kiedrowicz (8):\n      gitweb: esc_html_hl_regions(): Don't create empty <span> elements\n      gitweb: Pass esc_html_hl_regions() options to esc_html()\n      gitweb: Extract print_sidebyside_diff_lines()\n      gitweb: Use print_diff_chunk() for both side-by-side and inline diffs\n      gitweb: Move HTML-formatting diff line back to process_diff_line()\n      gitweb: Push formatting diff lines to print_diff_chunk()\n      gitweb: Highlight interesting parts of diff\n      gitweb: Refinement highlightning in combined diffs\n\n gitweb/gitweb.perl       |  299 +++++++++++++++++++++++++++++++++-------------\n gitweb/static/gitweb.css |    8 ++\n 2 files changed, 225 insertions(+), 82 deletions(-)\n\n-- \nJakub Narebski\n"},{"id":"188111","messageId":"7vobreq2zp.fsf@alter.siamese.dyndns.org","threadId":"30090","inReplyTo":"m3fwcrarwo.fsf@localhost.localdomain","subject":"Re: What's cooking in git.git (Mar 2012, #10; Wed, 28)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2012-03-30T01:41:46Z","receivedAt":"2012-03-30T01:41:46Z","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>> --------------------------------------------------\n>> [Cooking]\n>\n>> * wk/gitweb-snapshot-use-if-modified-since (2012-03-26) 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>> \n>> Makes 'snapshot' request to \"gitweb\" honor If-Modified-Since: header,\n>> based on the commit date.\n>\n> What about \"[PATCH v2 0/8] gitweb: Highlight interesting parts of diff\"\n> series by Michał Kiedrowicz:\n>\n>   Message-ID: <1332543417-19664-1-git-send-email-michal.kiedrowicz@gmail.com>\n>   http://thread.gmane.org/gmane.comp.version-control.git/193804\n\nI was waiting for the review cycle to come to a resolution.\n\n> I have copy of them in 'gitweb/diff-hl' branch:\n\nThanks for collecting them. I wasn't paying much attention to the list\ndiscussion.\n\nDo you mean \"I collected them for you to review, I do not yet vouch for\nthem\"?  I see one of them has your Ack, but otherwise there is no Sign-off\nby you on any of the commit, so it is unclear if your \"What about\" was\nyour way of saying \"I think this round is ready, go ahead and pull.\" with\na rhetorical question, or a pure question \"What is your opinion on this\nseries?\"\n\n\n> -- >8 --\n> The following changes since commit 455cf268dbaf227bdbd5e9fbf96525452bcfe44f:\n>\n>   Git 1.7.10-rc3 (2012-03-28 11:18:42 -0700)\n>\n> are available in the git repository at:\n>\n>   git://repo.or.cz/git/jnareb-git.git gitweb/diff-hl\n>\n> for you to fetch changes up to 60b06478f0349729dc9a4ddeb1abf5e28986fb7e:\n>\n>   gitweb: Refinement highlightning in combined diffs (2012-03-29 21:26:36 +0200)\n>\n> ----------------------------------------------------------------\n> Michał Kiedrowicz (8):\n>       gitweb: esc_html_hl_regions(): Don't create empty <span> elements\n>       gitweb: Pass esc_html_hl_regions() options to esc_html()\n>       gitweb: Extract print_sidebyside_diff_lines()\n>       gitweb: Use print_diff_chunk() for both side-by-side and inline diffs\n>       gitweb: Move HTML-formatting diff line back to process_diff_line()\n>       gitweb: Push formatting diff lines to print_diff_chunk()\n>       gitweb: Highlight interesting parts of diff\n>       gitweb: Refinement highlightning in combined diffs\n>\n>  gitweb/gitweb.perl       |  299 +++++++++++++++++++++++++++++++++-------------\n>  gitweb/static/gitweb.css |    8 ++\n>  2 files changed, 225 insertions(+), 82 deletions(-)\n"},{"id":"188146","messageId":"201203301313.01432.jnareb@gmail.com","threadId":"30090","inReplyTo":"7vobreq2zp.fsf@alter.siamese.dyndns.org","subject":"Re: What's cooking in git.git (Mar 2012, #10; Wed, 28)","fromName":"Jakub Narebski","fromEmail":"jnareb@gmail.com","sentAt":"2012-03-30T11:13:00Z","receivedAt":"2012-03-30T11:13:00Z","isPatch":false,"sender":{"key":"jnareb@gmail.com","avatar":"https://avatars.githubusercontent.com/u/2706?v=4"},"body":"On Fri, 30 Mar 2012, Junio C Hamano wrote:\n> Jakub Narebski <jnareb@gmail.com> writes:\n>> Junio C Hamano <gitster@pobox.com> writes:\n>>\n>>> --------------------------------------------------\n>>> [Cooking]\n>>\n>>> * wk/gitweb-snapshot-use-if-modified-since (2012-03-26) 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>>> \n>>> Makes 'snapshot' request to \"gitweb\" honor If-Modified-Since: header,\n>>> based on the commit date.\n>>\n>> What about \"[PATCH v2 0/8] gitweb: Highlight interesting parts of diff\"\n>> series by Michał Kiedrowicz:\n>>\n>>   Message-ID: <1332543417-19664-1-git-send-email-michal.kiedrowicz@gmail.com>\n>>   http://thread.gmane.org/gmane.comp.version-control.git/193804\n> \n> I was waiting for the review cycle to come to a resolution.\n\nAh, so you were waiting for review before even considering this\nseries even for 'pu'...\n\n...which I have only just finished (the review I mean).\n\n>> I have copy of them in 'gitweb/diff-hl' branch:\n> \n> Thanks for collecting them. I wasn't paying much attention to the list\n> discussion.\n> \n> Do you mean \"I collected them for you to review, I do not yet vouch for\n> them\"?  I see one of them has your Ack, but otherwise there is no Sign-off\n> by you on any of the commit, so it is unclear if your \"What about\" was\n> your way of saying \"I think this round is ready, go ahead and pull.\" with\n> a rhetorical question, or a pure question \"What is your opinion on this\n> series?\"\n\nI meant \"I collected them for you to have an option to pull rather than\napply set of 8 patches\".  Anyway I applied them to examine how well they\nwork.\n\nI think this series is in quite good shape, and I expect in next revision\nor two might be even ready for 'next'.  One more serious issue that I'd\nlike to see solved is requiring untabify() run before format_diff*() vs\nhaving format_diff*() do it itself which is more future-proof I think.\n\n>> -->8 --\n>> The following changes since commit 455cf268dbaf227bdbd5e9fbf96525452bcfe44f:\n>>\n>>   Git 1.7.10-rc3 (2012-03-28 11:18:42 -0700)\n>>\n>> are available in the git repository at:\n>>\n>>   git://repo.or.cz/git/jnareb-git.git gitweb/diff-hl\n>>\n>> for you to fetch changes up to 60b06478f0349729dc9a4ddeb1abf5e28986fb7e:\n>>\n>>   gitweb: Refinement highlightning in combined diffs (2012-03-29 21:26:36 +0200)\n\n-- \nJakub Narebski\nPoland\n"},{"id":"188168","messageId":"7v4nt6nku4.fsf@alter.siamese.dyndns.org","threadId":"30090","inReplyTo":"201203301313.01432.jnareb@gmail.com","subject":"Re: What's cooking in git.git (Mar 2012, #10; Wed, 28)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2012-03-30T15:56:51Z","receivedAt":"2012-03-30T15:56:51Z","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>> I was waiting for the review cycle to come to a resolution.\n>\n> Ah, so you were waiting for review before even considering this\n> series even for 'pu'...\n\nIt was more like \"Putting a version from a random day I happened to pick\nthe series up and queue them in 'pu' and leaving it there while if it is\nstill daily being churned would not make any sense, because, I wouldn't be\nable to keep up with it anyway; it is not my area.\"\n\n> I think this series is in quite good shape, and I expect in next revision\n> or two might be even ready for 'next'.  One more serious issue that I'd\n> like to see solved is requiring untabify() run before format_diff*() vs\n> having format_diff*() do it itself which is more future-proof I think.\n\nThanks for reviewing and helping the topic move forward.\n"}]}