{"thread":{"id":"21320","subject":"What's cooking in git.git (Oct 2009, #04; Wed, 21)","startedAt":"2009-10-22T06:52:30Z","lastAt":"2009-10-28T14:47:26Z","messageCount":33,"participants":["Junio C Hamano","Stephen Boyd","Jakub Narebski","Nguyen Thai Ngoc Duy","skillzero@gmail.com","Sverre Rabbelier","A Large Angry SCM","Johannes Schindelin","Jens Lehmann","Clemens Buchacher","Shawn O. Pearce"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"125679","messageId":"7veiovly35.fsf@alter.siamese.dyndns.org","threadId":"21320","inReplyTo":null,"subject":"What's cooking in git.git (Oct 2009, #04; Wed, 21)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2009-10-22T06:52:30Z","receivedAt":"2009-10-22T06:52:30Z","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' while commits prefixed with '+' are in 'next'.  The ones\nmarked with '.' do not appear in any of the integration branches, but I am\nstill holding onto them.\n\nIn 1.7.0, we plan to correct handful of warts in the interfaces everybody\nagrees that they were mistakes.  The resulting system may not be strictly\nbackward compatible.  Currently planeed changes are:\n\n * refuse push to update the checked out branch in a non-bare repo by\n   default\n\n   Make \"git push\" into a repository to update the branch that is checked\n   out fail by default.  You can countermand this default by setting a\n   configuration variable in the receiving repository.\n\n   http://thread.gmane.org/gmane.comp.version-control.git/107758/focus=108007\n\n * refuse push to delete the current branch by default\n\n   Make \"git push $there :$killed\" to delete the branch that is pointed at\n   by its HEAD fail by default.  You can countermand this default by\n   setting a configuration variable in the receiving repository.\n\n   http://thread.gmane.org/gmane.comp.version-control.git/108862/focus=108936\n\n * git-send-email won't make deep threads by default\n\n   Many people said that by default when sending more than 2 patches the\n   threading git-send-email makes by default is hard to read, and they\n   prefer the default be one cover letter and each patch as a direct\n   follow-up to the cover letter.  You can countermand this by setting a\n   configuration variable.\n\n   http://article.gmane.org/gmane.comp.version-control.git/109790\n\n * git-status won't be \"git-commit --dry-run\" anymore\n\n   http://thread.gmane.org/gmane.comp.version-control.git/125989/focus=125993\n\n * \"git-diff -w --exit-code\" will exit success if only differences it\n   found are whitespace changes that are stripped away from the output.\n\n   http://thread.gmane.org/gmane.comp.version-control.git/119731/focus=119751\n\n--------------------------------------------------\n[Graduated to \"master\"]\n\n* gb/maint-gitweb-esc-param (2009-10-13) 1 commit.\n  (merged to 'next' on 2009-10-14 at 105f997)\n + gitweb: fix esc_param\n (this branch is used by sb/gitweb-link-author.)\n\n--------------------------------------------------\n[New Topics]\n\n* vl/git-gui (2009-10-16) 1 commit.\n - git-gui: adjust the minimum height of diff pane for shorter screen height\n\nShawn?\n\n* cb/doc-fetch-pull-merge (2009-10-21) 1 commit.\n  (merged to 'next' on 2009-10-21 at 1d9190d)\n + modernize fetch/merge/pull examples\n\n* ja/fetch-doc (2009-10-21) 1 commit.\n  (merged to 'next' on 2009-10-21 at bf09f62)\n + Documentation/fetch-options.txt: order options alphabetically\n\nHelps 'git-fetch.{1,html}' without helping 'git-pull.{1,html}'.\n\n* jc/1.7.0-no-commit-no-ff-2 (2009-10-21) 1 commit.\n - git-merge: forbid fast-forward and up-to-date when --no-commit is given\n\nThis makes \"git merge --no-commit\" fail when it results in fast-forward or\nup-to-date.  I haven't described this at the beginning of this message\nyet, as it is not clear if this is even necessary, but since I already\nwrote it and many people seem to be interested in UI and behaviour\nwarts,...\n\nSome tests expect the traditional behaviour of silently ignoring --no-commit\nupon fast-forward, and tonight's 'pu' does not pass them.\n\n* jc/1.7.0-no-commit-no-ff (2009-10-21) 1 commit.\n . git-merge: imply --no-ff when --no-commit is given\n\nThis is an alternative patch to the same issue.\n\n* jc/fsck-default-full (2009-10-20) 1 commit\n  (merged to 'next' on 2009-10-21 at 1375192)\n + fsck: default to \"git fsck --full\"\n\nShould be safe enough to be in 'master' soon.\n\n* jc/maint-fix-unpack-zlib-check (2009-10-21) 1 commit.\n - Fix incorrect error check while reading deflated pack data\n\nThis is the final round from 2009-10-21, not my earlier botched attempts.\n\n* jc/receive-pack-auto (2009-10-20) 2 commits.\n  (merged to 'next' on 2009-10-21 at fef13ef)\n + receive-pack: run \"gc --auto --quiet\" and optionally \"update-server-info\"\n + gc --auto --quiet: make the notice a bit less verboase\n\n* jp/dirty-describe (2009-10-21) 1 commit.\n - Teach \"git describe\" --dirty option\n\nAck?\n\n* tr/filter-branch (2009-10-21) 2 commits.\n - filter-branch: nearest-ancestor rewriting outside subdir filter\n - filter-branch: stop special-casing $filter_subdir argument\n\nJ6t already has some comments on this.\n\n* tr/maint-roff-quote (2009-10-21) 2 commits.\n - Document GNU_ROFF in Makefile\n - Quote ' as \\(aq in manpages\n\nThe doc may need to be clarified a bit more.\n\n* bg/clone-doc (2009-10-20) 1 commit.\n  (merged to 'next' on 2009-10-21 at 3016736)\n + git-clone.txt: Fix grammar and formatting\n\nShould be correct enough to be in 'master' soon.\n\n* iv/tar-lzma-xz (2009-10-20) 1 commit.\n  (merged to 'next' on 2009-10-21 at cb0df8a)\n + import-tars: Add support for tarballs compressed with lzma, xz\n\nShould be safe enough to be in 'master' soon.\n\n* rs/pretty-wrap (2009-10-17) 1 commit\n - Implement wrap format %w() as if it is a mode switch\n (this branch uses js/log-rewrap; is related to jc/strbuf-nested-expand.)\n\nWhen it comes to design issues to keep unnecessary complexity out, I tend\nto trust R辿ne (and Nico) a lot more than I trust myself.  Tonight's 'pu'\nqueues this series instead of my \"nested\" one.\n\n* sr/blame-incomplete (2009-10-19) 1 commit.\n - blame: make sure that the last line ends in an LF\n\nI think this is _good enough_ as-is; although it would be better if we\nadded some hint to the output for Porcelain implementations, that can be\ndone as a follow-up fix.\n\n--------------------------------------------------\n[Stalled]\n\n* mr/gitweb-snapshot (2009-09-26) 2 commits.\n - gitweb: append short hash ids to snapshot files\n  (merged to 'next' on 2009-10-11 at 22ba047)\n + gitweb: check given hash before trying to create snapshot\n\nI lost track of the discussion around the tip commit.  The bottom one may\nbetter go to 'master' regardless.\n\n* db/vcs-helper-rest (2009-09-03) 6 commits.\n - Allow helpers to report in \"list\" command that the ref is unchanged\n - Add support for \"import\" helper command\n - Add a config option for remotes to specify a foreign vcs\n - Allow programs to not depend on remotes having urls\n - Allow fetch to modify refs\n - Use a function to determine whether a remote is valid\n (this branch is used by jh/cvs-helper.)\n\nThis holds the remainder of the db/vcs-helper topic that has already\nmerged in 1.6.5.  If people want to replace this with improvements it\nwould be a good time to do so.\n\n* jl/submodule-add-noname (2009-09-22) 1 commit.\n - git submodule add: make the <path> parameter optional\n\nDscho started an interesting discussion regarding the larger workflow in\nwhich the \"submodule add\" is used.  I think the patch itself makes sense\nbut at the same time it probably makes sense to also take the <path> and\ninfer the <repository> as Dscho suggested, probably in \"git submodule\nadd\", not in \"git add\" proper, at least initially.\n\n* sr/gfi-options (2009-09-06) 6 commits.\n - fast-import: test the new option command\n - fast-import: add option command\n - fast-import: test the new feature command\n - fast-import: add feature command\n - fast-import: put marks reading in it's own function\n - fast-import: put option parsing code in separate functions\n\n* je/send-email-no-subject (2009-08-05) 1 commit.\n  (merged to 'next' on 2009-10-11 at 1b99c56)\n + send-email: confirm on empty mail subjects\n\nThe existing tests cover the positive case (i.e. as long as the user says\n\"yes\" to the \"do you really want to send this message that lacks subject\",\nthe message is sent) of this feature, but the feature itself needs its own\ntest to verify the negative case (i.e. does it correctly stop if the user\nsays \"no\"?)\n\n* jh/cvs-helper (2009-08-18) 8 commits.\n . More fixes to the git-remote-cvs installation procedure\n . Fix the Makefile-generated path to the git_remote_cvs package in git-remote-cvs\n . Add simple selftests of git-remote-cvs functionality\n . git-remote-cvs: Remote helper program for CVS repositories\n . 2/2: Add Python support library for CVS remote helper\n . 1/2: Add Python support library for CVS remote helper\n . Basic build infrastructure for Python scripts\n . Allow helpers to request marks for fast-import\n (this branch uses db/vcs-helper-rest.)\n\n* jc/strbuf-nested-expand (2009-10-18) 3 commits\n . Teach --wrap to only indent without wrapping\n . Add %[wrap(width,in1,in2)<<any-string>>%] implementation\n . strbuf_nested_expand(): allow expansion to interrupt in the middle\n (this branch uses js/log-rewrap; is related to rs/pretty-wrap.)\n\nEjected from 'pu' to let rs/pretty-wrap in as described above.\n\n--------------------------------------------------\n[Cooking]\n\n* ne/rev-cache (2009-10-19) 7 commits.\n - support for commit grafts, slight change to general mechanism\n - support for path name caching in rev-cache\n - full integration of rev-cache into git, completed test suite\n - administrative functions for rev-cache, start of integration into git\n - support for non-commit object caching in rev-cache\n - basic revision cache system, no integration or features\n - man page and technical discussion for rev-cache\n\nStill unstable?  Has an extra test squashed in; tonight's 'pu' does not\npass tests.\n\n* ak/bisect-reset-to-switch (2009-10-13) 1 commit.\n - bisect reset: Allow resetting to any commit, not just a branch\n\nSoon in 'next'.\n\n* fc/doc-fast-forward (2009-10-11) 1 commit.\n - user-manual: use 'fast-forward'\n\n* jc/maint-1.6.3-graft-trailing-space (2009-10-14) 1 commit.\n - info/grafts: allow trailing whitespaces at the end of line\n\nSoon in 'next'.\n\n* jk/maint-cvsimport-pathname (2009-10-19) 1 commit.\n  (merged to 'next' on 2009-10-19 at 77824f2)\n + cvsimport: fix relative argument filenames\n\nShould be safe enough to be in 'master' soon.\n\n* jn/show-normalized-refs (2009-10-12) 3 commits.\n - check-ref-format: simplify --print implementation\n - git check-ref-format --print\n - Add tests for git check-ref-format\n\nThis was for helping Porcelains like git-gui to sanely cope with user\ninput that has redundant // in refnames.  Are potential users happy with\nthe series?  I think this is ready for 'next'.\n\n* sb/gitweb-link-author (2009-10-15) 1 commit\n - gitweb: linkify author/committer names with search\n\nSoon in 'next'.\n\n* jc/checkout-auto-track (2009-10-18) 3 commits\n - git checkout --no-guess\n - DWIM \"git checkout frotz\" to \"git checkout -b frotz origin/frotz\"\n - check_filename(): make verify_filename() callable without dying\n\nThe final shape of this series ended up to be more or less exactly what\nDscho hinted he wanted to have in one of the discussion. Is everybody\nhappy with this kind of new user-friendliness?  I think it is safe enough\nto be queued to 'next'.\n\n* tr/stash-format (2009-10-19) 5 commits\n - stash list: drop the default limit of 10 stashes\n - stash list: use new %g formats instead of sed\n - Introduce new pretty formats %g[sdD] for reflog information\n - reflog-walk: refactor the branch@{num} formatting\n - Refactor pretty_print_commit arguments into a struct\n\nSoon in 'next'.\n\n* ks/precompute-completion (2009-10-05) 1 commit.\n  (merged to 'next' on 2009-10-14 at adf722a)\n + Speedup bash completion loading\n\nAre people happy with this?\n\n* sp/smart-http (2009-10-14) 17 commits\n - Smart HTTP fetch: gzip requests\n - Smart fetch over HTTP: client side\n - Smart push over HTTP: client side\n - Discover refs via smart HTTP server when available\n - Smart fetch and push over HTTP: server side\n - Add stateless RPC options to upload-pack, receive-pack\n - Git-aware CGI to provide dumb HTTP transport\n - Move WebDAV HTTP push under remote-curl\n - remote-helpers: Support custom transport options\n - remote-helpers: Fetch more than one ref in a batch\n - fetch: Allow transport -v -v -v to set verbosity to 3\n - remote-curl: Refactor walker initialization\n - Add multi_ack_detailed capability to fetch-pack/upload-pack\n - Move \"get_ack()\" back to fetch-pack\n - fetch-pack: Use a strbuf to compose the want list\n - pkt-line: Make packet_read_line easier to debug\n - pkt-line: Add strbuf based functions\n\nWhat's the doneness of this series?\n\n* ef/msys-imap (2009-10-21) 8 commits.\n - MSVC: Enable OpenSSL, and translate -lcrypto\n - mingw: enable OpenSSL\n - mingw: wrap SSL_set_(w|r)fd to call _get_osfhandle\n - imap-send: build imap-send on Windows\n - imap-send: fix compilation-error on Windows\n - imap-send: use run-command API for tunneling\n - imap-send: use separate read and write fds\n - imap-send: remove useless uid code\n\nAnother re-roll.  Waiting for an Ack from MSVC folks but otherwise it is\nready for 'next', I think.\n\n* jc/pretty-lf (2009-10-04) 1 commit.\n - Pretty-format: %[+-]x to tweak inter-item newlines\n\n* js/log-rewrap (2008-11-10) 2 commits\n - Add strbuf_add_wrapped_text() to utf8.[ch]\n - print_wrapped_text(): allow hard newlines\n (this branch is used by jc/strbuf-nested-expand and rs/pretty-wrap.)\n\nSoon in 'next'; regardless of how wrapping is exposed to --pretty=format,\nthis code will be used, and it seems to be leak-free and reasonably done.\n\nWe _might_ want to cherry-pick the tip of jc/strbuf-nested-expand to this\nseries, though.\n\n* js/diff-verbose-submodule (2009-10-14) 2 commits.\n - add tests for git diff --submodule-summary\n - Add the --submodule option to the diff option family\n\nI should retitle and fix some comments in the tip commit (the tests have\nalready been adjusted to use the real option name), but otherwise I think\nthis is ready for 'next'.\n\n* jc/fix-tree-walk (2009-09-14) 10 commits.\n - read-tree --debug-unpack\n  (merged to 'next' on 2009-10-11 at 0b058e2)\n + unpack-trees.c: look ahead in the index\n + unpack-trees.c: prepare for looking ahead in the index\n + Aggressive three-way merge: fix D/F case\n + traverse_trees(): handle D/F conflict case sanely\n + more D/F conflict tests\n + tests: move convenience regexp to match object names to test-lib.sh\n + unpack_callback(): use unpack_failed() consistently\n + unpack-trees: typofix\n + diff-lib.c: fix misleading comments on oneway_diff()\n\nThis is my replacement for Linus's lt/maint-traverse-trees-fix patch.  It\nis not so much as a counter-proposal; I originally thought it might make\nsense to walk the index and drive the walker to return the entries from\ntrees to match entries from the index, but I ended up doing pretty much\nwhat Linus outlined --- walk the trees, and have the index walker follow\nit.  It turned out that the index side also needed some hairy look-ahead,\n\nThis includes the fix to aggressive mode of three-way merge used by the\nresolve strategy.\n\n* jh/notes (2009-10-09) 22 commits.\n - fast-import: Proper notes tree manipulation using the notes API\n - Refactor notes concatenation into a flexible interface for combining notes\n - Notes API: Allow multiple concurrent notes trees with new struct notes_tree\n - Notes API: for_each_note(): Traverse the entire notes tree with a callback\n - Notes API: get_note(): Return the note annotating the given object\n - Notes API: add_note(): Add note objects to the internal notes tree structure\n - Notes API: init_notes(): Initialize the notes tree from the given notes ref\n - Notes API: get_commit_notes() -> format_note() + remove the commit restriction\n - Add selftests verifying concatenation of multiple notes for the same commit\n - Refactor notes code to concatenate multiple notes annotating the same object\n - Add selftests verifying that we can parse notes trees with various fanouts\n - Teach the notes lookup code to parse notes trees with various fanout schemes\n - Teach notes code to free its internal data structures on request\n - Add '%N'-format for pretty-printing commit notes\n - Add flags to get_commit_notes() to control the format of the note string\n - t3302-notes-index-expensive: Speed up create_repo()\n - fast-import: Add support for importing commit notes\n - Teach \"-m <msg>\" and \"-F <file>\" to \"git notes edit\"\n - Add an expensive test for git-notes\n - Speed up git notes lookup\n - Add a script to edit/inspect notes\n - Introduce commit notes\n\nRebased so that it does not pull in anything else.  Presumably it is ready\nfor next?\n\n* jn/gitweb-blame (2009-09-01) 5 commits.\n - gitweb: Minify gitweb.js if JSMIN is defined\n - gitweb: Create links leading to 'blame_incremental' using JavaScript\n  (merged to 'next' on 2009-10-11 at 73c4a83)\n + gitweb: Colorize 'blame_incremental' view during processing\n + gitweb: Incremental blame (using JavaScript)\n + gitweb: Add optional \"time to generate page\" info in footer\n\nAjax-y blame.  Probably the first three should go to 'master' by now?\n\n* nd/sparse (2009-08-20) 19 commits.\n - sparse checkout: inhibit empty worktree\n - Add tests for sparse checkout\n - read-tree: add --no-sparse-checkout to disable sparse checkout support\n - unpack-trees(): ignore worktree check outside checkout area\n - unpack_trees(): apply $GIT_DIR/info/sparse-checkout to the final index\n - unpack-trees(): \"enable\" sparse checkout and load $GIT_DIR/info/sparse-checkout\n - unpack-trees.c: generalize verify_* functions\n - unpack-trees(): add CE_WT_REMOVE to remove on worktree alone\n - Introduce \"sparse checkout\"\n - dir.c: export excluded_1() and add_excludes_from_file_1()\n - excluded_1(): support exclude files in index\n - unpack-trees(): carry skip-worktree bit over in merged_entry()\n - Read .gitignore from index if it is skip-worktree\n - Avoid writing to buffer in add_excludes_from_file_1()\n - Teach Git to respect skip-worktree bit (writing part)\n - Teach Git to respect skip-worktree bit (reading part)\n - Introduce \"skip-worktree\" bit in index, teach Git to get/set this bit\n - Add test-index-version\n - update-index: refactor mark_valid() in preparation for new options\n\n--------------------------------------------------\n[For 1.7.0]\n\n* jk/1.7.0-status (2009-09-05) 5 commits.\n - docs: note that status configuration affects only long format\n  (merged to 'next' on 2009-10-11 at 65c8513)\n + commit: support alternate status formats\n + status: add --porcelain output format\n + status: refactor format option parsing\n + status: refactor short-mode printing to its own function\n (this branch uses jc/1.7.0-status.)\n\nGives the --short output format to post 1.7.0 \"git commit --dry-run\" that\nis similar to that of post 1.7.0 \"git status\".\n\n* jc/1.7.0-status (2009-09-05) 4 commits.\n  (merged to 'next' on 2009-10-11 at 9558627)\n + status: typo fix in usage\n + git status: not \"commit --dry-run\" anymore\n + git stat -s: short status output\n + git stat: the beginning of \"status that is not a dry-run of commit\"\n (this branch is used by jk/1.7.0-status.)\n\nWith this, \"git status\" is no longer \"git commit --dry-run\".\n\n* jc/1.7.0-send-email-no-thread-default (2009-08-22) 1 commit.\n  (merged to 'next' on 2009-10-11 at 043acdf)\n + send-email: make --no-chain-reply-to the default\n\n* jc/1.7.0-diff-whitespace-only-status (2009-08-30) 4 commits.\n  (merged to 'next' on 2009-10-11 at 546c74d)\n + diff.c: fix typoes in comments\n + Make test case number unique\n + diff: Rename QUIET internal option to QUICK\n + diff: change semantics of \"ignore whitespace\" options\n\nThis changes exit code from \"git diff --ignore-whitespace\" and friends\nwhen there is no actual output.  It is a backward incompatible change, but\nwe could argue that it is a bugfix.\n\n* jc/1.7.0-push-safety (2009-02-09) 2 commits.\n  (merged to 'next' on 2009-10-11 at 81b8128)\n + Refuse deleting the current branch via push\n + Refuse updating the current branch in a non-bare repository via push\n\n--------------------------------------------------\n[I have been too busy to purge these]\n\n* jc/log-tz (2009-03-03) 1 commit.\n - Allow --date=local --date=other-format to work as expected\n\nMaybe some people care about this.  I dunno.\n\n* jc/mailinfo-remove-brackets (2009-07-15) 1 commit.\n - mailinfo: -b option keeps [bracketed] strings that is not a [PATCH] marker\n\nMaybe some people care about this.  I dunno.\n\n* jg/log-format-body-indent (2009-09-19) 1 commit.\n . git-log --format: Add %B tag with %B(x) option\n"},{"id":"125690","messageId":"4AE0190E.8020803@gmail.com","threadId":"21320","inReplyTo":"7veiovly35.fsf@alter.siamese.dyndns.org","subject":"ks/precompute-completion (was Re: What's cooking in git.git (Oct 2009, #04; Wed, 21))","fromName":"Stephen Boyd","fromEmail":"bebarino@gmail.com","sentAt":"2009-10-22T08:34:22Z","receivedAt":"2009-10-22T08:34:22Z","isPatch":false,"sender":{"key":"bebarino@gmail.com","avatar":"https://avatars.githubusercontent.com/u/38832?v=4"},"body":"Junio C Hamano wrote:\n> * ks/precompute-completion (2009-10-05) 1 commit.\n>   (merged to 'next' on 2009-10-14 at adf722a)\n>  + Speedup bash completion loading\n>\n> Are people happy with this?\n\nNo. I now have rebase.sh, am.sh, etc. in my completion because of how\ngit help -a fully lists git commands in libexec and elsewhere in my\n$PATH (which gets pointed to my build directory during make).\n\nIt's late and I'm tired, but I think we can just ignore files ending in\n*.sh, *.perl, etc.\n\ndiff --git a/contrib/completion/git-completion.bash.generate b/contrib/completion/git-completion.bash.generate\nindex 33b1d1d..6487fd5 100755\n--- a/contrib/completion/git-completion.bash.generate\n+++ b/contrib/completion/git-completion.bash.generate\n@@ -24,6 +24,7 @@ __git_all_commands ()\n        do\n                case $i in\n                *--*)             : helper pattern;;\n+               *.sh|*.perl)      : build scripts;;\n                *) echo $i;;\n                esac\n        done\n"},{"id":"125691","messageId":"m3ljj3es02.fsf@localhost.localdomain","threadId":"21320","inReplyTo":"7veiovly35.fsf@alter.siamese.dyndns.org","subject":"Re: What's cooking in git.git (Oct 2009, #04; Wed, 21)","fromName":"Jakub Narebski","fromEmail":"jnareb@gmail.com","sentAt":"2009-10-22T08:46:50Z","receivedAt":"2009-10-22T08:46:50Z","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> * gb/maint-gitweb-esc-param (2009-10-13) 1 commit.\n>   (merged to 'next' on 2009-10-14 at 105f997)\n>  + gitweb: fix esc_param\n>  (this branch is used by sb/gitweb-link-author.)\n\nGood.  Beside fixing excaping of multibyte Unicode characters this\nalso finally makes gitweb use '+' and not '%20' to encode space ' '\nin CGI parameters.\n\nThis reminds me that gitweb really should do conversion / marking data\nas UTF-8 _on input_, to avoid situations where output is mangled\nbecause of problems with Unicode.  That goes to my gitweb's TODO.\n\n> * mr/gitweb-snapshot (2009-09-26) 2 commits.\n>  - gitweb: append short hash ids to snapshot files\n>   (merged to 'next' on 2009-10-11 at 22ba047)\n>  + gitweb: check given hash before trying to create snapshot\n> \n> I lost track of the discussion around the tip commit.  The bottom one may\n> better go to 'master' regardless.\n\nThe tip commit should be fixed before accepting.  There are some\nproblems with it as it is now:\n\n * $hash parameter is abused to hold version suffix of snapshot\n   filename (and archive prefix), e.g. 'next-ae4ab03'; it really\n   should be done using separate variable, and perhaps even separate\n   subroutine which would generate snapshot name.\n\n * I don't think it works with fully qualified refnames that gitweb\n   itself generate, e.g. 'refs/heads/next' or 'refs/tags/v1.6.0',\n   nor with hierarchical branch names such as 'mr/gitweb-snapshot';\n   snapshot name can't include '/', and prefix shouldn't include '/'\n \n * when new test is running with --debug option, it dumps whole output\n   of gitweb for 'snapshot' action, which includes *binary data*, and\n   not only HTTP headers like it should (at least in first version).\n\n> * sb/gitweb-link-author (2009-10-15) 1 commit\n>  - gitweb: linkify author/committer names with search\n> \n> Soon in 'next'.\n\nIs this version that uses title attribute to show that this link is\ndifferent in that it leads to search results, not an action view?\n\n> * jn/gitweb-blame (2009-09-01) 5 commits.\n>  - gitweb: Minify gitweb.js if JSMIN is defined\n>  - gitweb: Create links leading to 'blame_incremental' using JavaScript\n>   (merged to 'next' on 2009-10-11 at 73c4a83)\n>  + gitweb: Colorize 'blame_incremental' view during processing\n>  + gitweb: Incremental blame (using JavaScript)\n>  + gitweb: Add optional \"time to generate page\" info in footer\n> \n> Ajax-y blame.  Probably the first three should go to 'master' by now?\n\nThe first three makes fairly 'invisible' change, but introduce\npossibility of using JavaScript in gitweb.  I'd like more testing with\ndifferent browsers than mine, but corrections if any can be done\nin-tree.\n\nThe \"Create links\" patch is not ready yet.\n\n\n> * rs/pretty-wrap (2009-10-17) 1 commit\n>  - Implement wrap format %w() as if it is a mode switch\n>  (this branch uses js/log-rewrap; is related to jc/strbuf-nested-expand.)\n> \n> When it comes to design issues to keep unnecessary complexity out, I tend\n> to trust Rene (and Nico) a lot more than I trust myself.  Tonight's 'pu'\n> queues this series instead of my \"nested\" one.\n\n> * jc/strbuf-nested-expand (2009-10-18) 3 commits\n>  . Teach --wrap to only indent without wrapping\n>  . Add %[wrap(width,in1,in2)<<any-string>>%] implementation\n>  . strbuf_nested_expand(): allow expansion to interrupt in the middle\n>  (this branch uses js/log-rewrap; is related to rs/pretty-wrap.)\n> \n> Ejected from 'pu' to let rs/pretty-wrap in as described above.\n\nI think nested expand is easier to use than a mode switch: using\nscoping (well, kind of) like in high-level programming languages is\nIMVHO easier than programming a state machine like in assembler (or\ne.g. OpenGL).\n\nOn the other hand this makes pretty format into a mini-language; also\nwe already have and use mode switches in the form of color codes.\nPerhaps if color also used wrapping / nested expand, so one doesn't\nhave to track where to turn off and on which toggle...\n\n> * jg/log-format-body-indent (2009-09-19) 1 commit.\n>  . git-log --format: Add %B tag with %B(x) option\n\n...and this was yet another alternate solution (less generic, though)\n\n\n> * jc/pretty-lf (2009-10-04) 1 commit.\n>  - Pretty-format: %[+-]x to tweak inter-item newlines\n\nI understand that %a%+b expands to %a%n%b if %b has non-empty\nexpansion, and to %a if %b is empty, but what %-b is used for?\n\n> * js/log-rewrap (2008-11-10) 2 commits\n>  - Add strbuf_add_wrapped_text() to utf8.[ch]\n>  - print_wrapped_text(): allow hard newlines\n>  (this branch is used by jc/strbuf-nested-expand and rs/pretty-wrap.)\n> \n> Soon in 'next'; regardless of how wrapping is exposed to --pretty=format,\n> this code will be used, and it seems to be leak-free and reasonably done.\n> \n> We _might_ want to cherry-pick the tip of jc/strbuf-nested-expand to this\n> series, though.\n \n\n\n \n> --------------------------------------------------\n> [Cooking]\n> \n> * ne/rev-cache (2009-10-19) 7 commits.\n>  - support for commit grafts, slight change to general mechanism\n>  - support for path name caching in rev-cache\n>  - full integration of rev-cache into git, completed test suite\n>  - administrative functions for rev-cache, start of integration into git\n>  - support for non-commit object caching in rev-cache\n>  - basic revision cache system, no integration or features\n>  - man page and technical discussion for rev-cache\n> \n> Still unstable?  Has an extra test squashed in; tonight's 'pu' does not\n> pass tests.\n\nBTW. I would really prefer if this series was send with cover letter\nexplaining series and perhaps differences from previous version of\nseries as a whole (reordering, splitting and joining patches, new\npatches etc.), and individual patches in series replies to this cover\nletter, without 'Re: ' prefix in subject, and with explanation of the\ndifference from previous version (if any) in the commentary area.\n\n\n> * nd/sparse (2009-08-20) 19 commits.\n>  - sparse checkout: inhibit empty worktree\n>  - Add tests for sparse checkout\n>  - read-tree: add --no-sparse-checkout to disable sparse checkout support\n>  - unpack-trees(): ignore worktree check outside checkout area\n>  - unpack_trees(): apply $GIT_DIR/info/sparse-checkout to the final index\n>  - unpack-trees(): \"enable\" sparse checkout and load $GIT_DIR/info/sparse-checkout\n>  - unpack-trees.c: generalize verify_* functions\n>  - unpack-trees(): add CE_WT_REMOVE to remove on worktree alone\n>  - Introduce \"sparse checkout\"\n>  - dir.c: export excluded_1() and add_excludes_from_file_1()\n>  - excluded_1(): support exclude files in index\n>  - unpack-trees(): carry skip-worktree bit over in merged_entry()\n>  - Read .gitignore from index if it is skip-worktree\n>  - Avoid writing to buffer in add_excludes_from_file_1()\n>  - Teach Git to respect skip-worktree bit (writing part)\n>  - Teach Git to respect skip-worktree bit (reading part)\n>  - Introduce \"skip-worktree\" bit in index, teach Git to get/set this bit\n>  - Add test-index-version\n>  - update-index: refactor mark_valid() in preparation for new options\n\nHmmm... what is happening with that series?\n \n-- \nJakub Narebski\nPoland\nShadeHawk on #git\n"},{"id":"125699","messageId":"fcaeb9bf0910220415v69c22ed9o4ab85b8858fbf187@mail.gmail.com","threadId":"21320","inReplyTo":"m3ljj3es02.fsf@localhost.localdomain","subject":"Re: What's cooking in git.git (Oct 2009, #04; Wed, 21)","fromName":"Nguyen Thai Ngoc Duy","fromEmail":"pclouds@gmail.com","sentAt":"2009-10-22T11:15:16Z","receivedAt":"2009-10-22T11:15:16Z","isPatch":false,"sender":{"key":"pclouds@gmail.com","avatar":"https://avatars.githubusercontent.com/u/720?v=4"},"body":"On Thu, Oct 22, 2009 at 3:46 PM, Jakub Narebski <jnareb@gmail.com> wrote:\n> Junio C Hamano <gitster@pobox.com> writes:\n>> * nd/sparse (2009-08-20) 19 commits.\n>>  - sparse checkout: inhibit empty worktree\n>>  - Add tests for sparse checkout\n>>  - read-tree: add --no-sparse-checkout to disable sparse checkout support\n>>  - unpack-trees(): ignore worktree check outside checkout area\n>>  - unpack_trees(): apply $GIT_DIR/info/sparse-checkout to the final index\n>>  - unpack-trees(): \"enable\" sparse checkout and load $GIT_DIR/info/sparse-checkout\n>>  - unpack-trees.c: generalize verify_* functions\n>>  - unpack-trees(): add CE_WT_REMOVE to remove on worktree alone\n>>  - Introduce \"sparse checkout\"\n>>  - dir.c: export excluded_1() and add_excludes_from_file_1()\n>>  - excluded_1(): support exclude files in index\n>>  - unpack-trees(): carry skip-worktree bit over in merged_entry()\n>>  - Read .gitignore from index if it is skip-worktree\n>>  - Avoid writing to buffer in add_excludes_from_file_1()\n>>  - Teach Git to respect skip-worktree bit (writing part)\n>>  - Teach Git to respect skip-worktree bit (reading part)\n>>  - Introduce \"skip-worktree\" bit in index, teach Git to get/set this bit\n>>  - Add test-index-version\n>>  - update-index: refactor mark_valid() in preparation for new options\n>\n> Hmmm... what is happening with that series?\n\nJunio concerned about CE_MATCH_IGNORE_VALID being used by both\nassume-unchanged and skip-worktree bits, which I did not resolve yet.\nI should really get back to the series when I have time.\n-- \nDuy\n"},{"id":"125707","messageId":"2729632a0910220831x4b67021eg772abc8b751ef7e5@mail.gmail.com","threadId":"21320","inReplyTo":"fcaeb9bf0910220415v69c22ed9o4ab85b8858fbf187@mail.gmail.com","subject":"Re: What's cooking in git.git (Oct 2009, #04; Wed, 21)","fromName":"","fromEmail":"skillzero@gmail.com","sentAt":"2009-10-22T15:31:17Z","receivedAt":"2009-10-22T15:31:17Z","isPatch":false,"sender":{"key":"skillzero@gmail.com","avatar":null},"body":"On Thu, Oct 22, 2009 at 4:15 AM, Nguyen Thai Ngoc Duy <pclouds@gmail.com> wrote:\n> On Thu, Oct 22, 2009 at 3:46 PM, Jakub Narebski <jnareb@gmail.com> wrote:\n>> Junio C Hamano <gitster@pobox.com> writes:\n>>> * nd/sparse (2009-08-20) 19 commits.\n>>>  - sparse checkout: inhibit empty worktree\n>>>  - Add tests for sparse checkout\n>>>  - read-tree: add --no-sparse-checkout to disable sparse checkout support\n>>>  - unpack-trees(): ignore worktree check outside checkout area\n>>>  - unpack_trees(): apply $GIT_DIR/info/sparse-checkout to the final index\n>>>  - unpack-trees(): \"enable\" sparse checkout and load $GIT_DIR/info/sparse-checkout\n>>>  - unpack-trees.c: generalize verify_* functions\n>>>  - unpack-trees(): add CE_WT_REMOVE to remove on worktree alone\n>>>  - Introduce \"sparse checkout\"\n>>>  - dir.c: export excluded_1() and add_excludes_from_file_1()\n>>>  - excluded_1(): support exclude files in index\n>>>  - unpack-trees(): carry skip-worktree bit over in merged_entry()\n>>>  - Read .gitignore from index if it is skip-worktree\n>>>  - Avoid writing to buffer in add_excludes_from_file_1()\n>>>  - Teach Git to respect skip-worktree bit (writing part)\n>>>  - Teach Git to respect skip-worktree bit (reading part)\n>>>  - Introduce \"skip-worktree\" bit in index, teach Git to get/set this bit\n>>>  - Add test-index-version\n>>>  - update-index: refactor mark_valid() in preparation for new options\n>>\n>> Hmmm... what is happening with that series?\n>\n> Junio concerned about CE_MATCH_IGNORE_VALID being used by both\n> assume-unchanged and skip-worktree bits, which I did not resolve yet.\n> I should really get back to the series when I have time.\n\nJust an FYI, but I've been using this series for a while. I'm actually\nrelying on sparse support in our internal build system (via my private\nbuild with this series) so I hope it doesn't go away :) I haven't\nreally noticed any problems (I thought the index state got out of sync\nonce, but I couldn't reproduce the problem later). Here's some\nfeedback though:\n\n1. I found it confusing to have to append '/' to directories in the\nsparse pattern list for directories. I always forget it requires them.\nIt would be nice to support the same rules as .gitignore in terms of\n'/'.\n\n2. It would be nice to have built-in support for a sparse modules file\nand switching between them. Maybe .gitmodules could support \"module\"\nor \"sparsemodule\" sections to list the patterns for that sparse\nmodule. I've written a simple script to do this and it's what I use.\nIt just parses the INI-style file:\n\n[module \"MyProject\"]\n\tApp1\n\tShared1\n\t!FolderIDontWant\n\nThen I have a \"module\" script to read that file for a specified module\nand switch to it:\n\ngit module switch MyProject\n\nThe script just parses `git show HEAD:.gitmodules` (so it works\nwithout a working directory) and switches sparse modules by enabling\nsparse, writing info/sparse-checkout, and doing a checkout:\n\nsub cmd_switch\n{\n\t# Enable sparse.\n\tmy $currentCmd = \"git config core.sparsecheckout true\";\n\tsystem( $currentCmd ) == 0 or die( \"error: $currentCmd\\n\" );\n\t\n\t# Write sparse patterns.\n\tmy $gitDir = `git rev-parse --git-dir`;\n\tchop( $gitDir );\n\tmy $sparsePath = $gitDir . \"/info/sparse-checkout\";\n\tif( $? != 0 ) { die( \"error: read git directory failed $?\\n\" ); }\n\topen( FILE, \">\", $sparsePath ) or die( \"error: can't open '$sparsePath'\\n\" );\n\tforeach( @{$gModules->{$gModuleName}} )\n\t{\n\t\tprint( FILE \"$_\\n\" );\n\t}\n\tclose( FILE );\n\t\n\t# Checkout using new sparse patterns.\n\tsystem( \"git checkout\" ) == 0 or die( \"error: switch module failed\\n\" );\n}\n\nThat said, the current level of sparse support provided by this series\nis good enough for me because I can build my own scripts like this on\ntop of it to automate things.\n"},{"id":"125714","messageId":"fabb9a1e0910221011r957246dx3162cd675ff16800@mail.gmail.com","threadId":"21320","inReplyTo":"4AE0190E.8020803@gmail.com","subject":"Re: ks/precompute-completion (was Re: What's cooking in git.git (Oct 2009, #04; Wed, 21))","fromName":"Sverre Rabbelier","fromEmail":"srabbelier@gmail.com","sentAt":"2009-10-22T17:11:27Z","receivedAt":"2009-10-22T17:11:27Z","isPatch":false,"sender":{"key":"srabbelier@gmail.com","avatar":"https://avatars.githubusercontent.com/u/3098?v=4"},"body":"Heya,\n\nOn Thu, Oct 22, 2009 at 03:34, Stephen Boyd <bebarino@gmail.com> wrote:\n> Junio C Hamano wrote:\n>> * ks/precompute-completion (2009-10-05) 1 commit.\n>>   (merged to 'next' on 2009-10-14 at adf722a)\n>>  + Speedup bash completion loading\n>>\n>> Are people happy with this?\n>\n> No. I now have rebase.sh, am.sh, etc. in my completion\n\nI would really like it if running 'make && make install' in git.git\nwould also build the completion script, I don't want to have to\nremember to run 'cd contrib/completion && make' every time we get new\ncompletion options :P.\n\n-- \nCheers,\n\nSverre Rabbelier\n"},{"id":"125722","messageId":"4AE0AACE.4000205@gmail.com","threadId":"21320","inReplyTo":"fabb9a1e0910221011r957246dx3162cd675ff16800@mail.gmail.com","subject":"Re: ks/precompute-completion (was Re: What's cooking in git.git (Oct 2009, #04; Wed, 21))","fromName":"Stephen Boyd","fromEmail":"bebarino@gmail.com","sentAt":"2009-10-22T18:56:14Z","receivedAt":"2009-10-22T18:56:14Z","isPatch":false,"sender":{"key":"bebarino@gmail.com","avatar":"https://avatars.githubusercontent.com/u/38832?v=4"},"body":"Sverre Rabbelier wrote:\n> Heya,\n>\n> I would really like it if running 'make && make install' in git.git\n> would also build the completion script, I don't want to have to\n> remember to run 'cd contrib/completion && make' every time we get new\n> completion options :P.\n\nPerhaps a top-level completion rule would work? I don't know if building\ncompletion for every user would be appropriate.\n"},{"id":"125723","messageId":"4AE0AC06.1060906@gmail.com","threadId":"21320","inReplyTo":"m3ljj3es02.fsf@localhost.localdomain","subject":"Re: What's cooking in git.git (Oct 2009, #04; Wed, 21)","fromName":"Stephen Boyd","fromEmail":"bebarino@gmail.com","sentAt":"2009-10-22T19:01:26Z","receivedAt":"2009-10-22T19:01:26Z","isPatch":false,"sender":{"key":"bebarino@gmail.com","avatar":"https://avatars.githubusercontent.com/u/38832?v=4"},"body":"Jakub Narebski wrote:\n>> * sb/gitweb-link-author (2009-10-15) 1 commit\n>>  - gitweb: linkify author/committer names with search\n>>\n>> Soon in 'next'.\n>>     \n>\n> Is this version that uses title attribute to show that this link is\n> different in that it leads to search results, not an action view?\n>   \n\nYes.\n"},{"id":"125749","messageId":"7viqe7f6lc.fsf@alter.siamese.dyndns.org","threadId":"21320","inReplyTo":"2729632a0910220831x4b67021eg772abc8b751ef7e5@mail.gmail.com","subject":"Re: What's cooking in git.git (Oct 2009, #04; Wed, 21)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2009-10-22T21:42:55Z","receivedAt":"2009-10-22T21:42:55Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"skillzero@gmail.com writes:\n\n> Just an FYI, but I've been using this series for a while. I'm actually\n> relying on sparse support in our internal build system (via my private\n> build with this series) so I hope it doesn't go away :) I haven't\n> really noticed any problems (I thought the index state got out of sync\n> once, but I couldn't reproduce the problem later). Here's some\n> feedback though:\n> ...\n> That said, the current level of sparse support provided by this series\n> is good enough for me because I can build my own scripts like this on\n> top of it to automate things.\n\nThanks for sharing.  It is the best approach to start by adding minimum\ncore level support and then to let people (like you) with real-world needs\nto experiment with custom wrapper scripts, to figure out what user-level\nconcepts and workflows work and useful (and what don't and aren't), and\nwhat additional core level features is helpful to support them.  Over\ntime, successful custom wrappers will become the best-current-practice and\ncan be folded back into the mainline, and everybody will benefit.\n"},{"id":"125754","messageId":"4AE0DAB3.1030103@gmail.com","threadId":"21320","inReplyTo":"fabb9a1e0910221011r957246dx3162cd675ff16800@mail.gmail.com","subject":"Re: ks/precompute-completion (was Re: What's cooking in git.git (Oct 2009, #04; Wed, 21))","fromName":"A Large Angry SCM","fromEmail":"gitzilla@gmail.com","sentAt":"2009-10-22T22:20:35Z","receivedAt":"2009-10-22T22:20:35Z","isPatch":false,"sender":{"key":"gitzilla@gmail.com","avatar":"https://gravatar.com/avatar/354625c442439908ff3dd99757dee330e29e9df7847472384faf7a00add247fb?d=mp&s=160"},"body":"Sverre Rabbelier wrote:\n> Heya,\n> \n> On Thu, Oct 22, 2009 at 03:34, Stephen Boyd <bebarino@gmail.com> wrote:\n>> Junio C Hamano wrote:\n>>> * ks/precompute-completion (2009-10-05) 1 commit.\n>>>   (merged to 'next' on 2009-10-14 at adf722a)\n>>>  + Speedup bash completion loading\n>>>\n>>> Are people happy with this?\n>> No. I now have rebase.sh, am.sh, etc. in my completion\n> \n> I would really like it if running 'make && make install' in git.git\n> would also build the completion script, I don't want to have to\n> remember to run 'cd contrib/completion && make' every time we get new\n> completion options :P.\n> \n\nPlease do not for completion on those that did not ask for it.\n"},{"id":"125756","messageId":"4AE0E542.8010501@gmail.com","threadId":"21320","inReplyTo":"fabb9a1e0910221556s694a344ag8e5ae07c35351ee4@mail.gmail.com","subject":"Re: ks/precompute-completion (was Re: What's cooking in git.git (Oct 2009, #04; Wed, 21))","fromName":"A Large Angry SCM","fromEmail":"gitzilla@gmail.com","sentAt":"2009-10-22T23:05:38Z","receivedAt":"2009-10-22T23:05:38Z","isPatch":false,"sender":{"key":"gitzilla@gmail.com","avatar":"https://gravatar.com/avatar/354625c442439908ff3dd99757dee330e29e9df7847472384faf7a00add247fb?d=mp&s=160"},"body":"Sverre Rabbelier wrote:\n> How am i forcing completion on those that did not ask for it? Nothing \n> changes  compared to the old situation....\n> \n>> On Oct 22, 2009 5:20 PM, \"A Large Angry SCM\" <gitzilla@gmail.com \n>> <mailto:gitzilla@gmail.com>> wrote:\n>>\n>> Sverre Rabbelier wrote: > > Heya, > > On Thu, Oct 22, 2009 at 03:34, \n>> Stephen Boyd <bebarino@gmail.co...\n>>\n>> Please do not for completion on those that did not ask for it.\n> \n\nYour original email included 'make && make install'; it's the \"make \ninstall\" part I'm concerned about.\n"},{"id":"125755","messageId":"alpine.DEB.1.00.0910230106070.4985@pacific.mpi-cbg.de","threadId":"21320","inReplyTo":"4AE0DAB3.1030103@gmail.com","subject":"Re: ks/precompute-completion (was Re: What's cooking in git.git (Oct 2009, #04; Wed, 21))","fromName":"Johannes Schindelin","fromEmail":"johannes.schindelin@gmx.de","sentAt":"2009-10-22T23:07:14Z","receivedAt":"2009-10-22T23:07:14Z","isPatch":false,"sender":{"key":"johannes.schindelin@gmx.de","avatar":"https://avatars.githubusercontent.com/u/127790?v=4"},"body":"Hi,\n\nOn Thu, 22 Oct 2009, A Large Angry SCM wrote:\n\n> Sverre Rabbelier wrote:\n> > Heya,\n> > \n> > On Thu, Oct 22, 2009 at 03:34, Stephen Boyd <bebarino@gmail.com> wrote:\n> > > Junio C Hamano wrote:\n> > > > * ks/precompute-completion (2009-10-05) 1 commit.\n> > > >   (merged to 'next' on 2009-10-14 at adf722a)\n> > > >  + Speedup bash completion loading\n> > > >\n> > > > Are people happy with this?\n> > > No. I now have rebase.sh, am.sh, etc. in my completion\n> > \n> > I would really like it if running 'make && make install' in git.git\n> > would also build the completion script, I don't want to have to\n> > remember to run 'cd contrib/completion && make' every time we get new\n> > completion options :P.\n> > \n> \n> Please do not for completion on those that did not ask for it.\n\nIt is about installing git-completion.bash, AFAICT, not about forcing your \nshell into loading them by default.  IOW something I wished for already in \nFeb 2008, but was unable to convince anybody of.\n\nCiao,\nDscho\n"},{"id":"125785","messageId":"4AE192AD.70603@web.de","threadId":"21320","inReplyTo":"7veiovly35.fsf@alter.siamese.dyndns.org","subject":"[PATCH] add tests for git diff --submodule","fromName":"Jens Lehmann","fromEmail":"jens.lehmann@web.de","sentAt":"2009-10-23T11:25:33Z","receivedAt":"2009-10-23T11:25:33Z","isPatch":true,"sender":{"key":"jens.lehmann@web.de","avatar":"https://avatars.githubusercontent.com/u/135220?v=4"},"body":"Copied from the submodule summary test and changed to reflect the\ndifferences in the output of git diff --submodule.\n\nSigned-off-by: Jens Lehmann <Jens.Lehmann@web.de>\n---\n\nJunio C Hamano schrieb:\n> * js/diff-verbose-submodule (2009-10-14) 2 commits.\n>  - add tests for git diff --submodule-summary\n>  - Add the --submodule option to the diff option family\n> \n> I should retitle and fix some comments in the tip commit (the tests have\n> already been adjusted to use the real option name), but otherwise I think\n> this is ready for 'next'.\n\nSorry for sending the updated test so late, i haven't had much time for\ngit in the last few days.\n\nApart from your changes necessary to make the test run again my changes are:\n\n- rename from \"t4041-diff-submodule-summary.sh\" to \"t4041-diff-submodule.sh\"\n- corrected all comments still speaking of \"summary\"\n- added tests to test the behaviour of \"--submodule\" and \"--submodule=short\"\n\n\n t/t4041-diff-submodule.sh |  260 +++++++++++++++++++++++++++++++++++++++++++++\n 1 files changed, 260 insertions(+), 0 deletions(-)\n create mode 100755 t/t4041-diff-submodule.sh\n\ndiff --git a/t/t4041-diff-submodule.sh b/t/t4041-diff-submodule.sh\nnew file mode 100755\nindex 0000000..5bb4fed\n--- /dev/null\n+++ b/t/t4041-diff-submodule.sh\n@@ -0,0 +1,260 @@\n+#!/bin/sh\n+#\n+# Copyright (c) 2009 Jens Lehmann, based on t7401 by Ping Yin\n+#\n+\n+test_description='Support for verbose submodule differences in git diff\n+\n+This test tries to verify the sanity of the --submodule option of git diff.\n+'\n+\n+. ./test-lib.sh\n+\n+add_file () {\n+\tsm=$1\n+\tshift\n+\towd=$(pwd)\n+\tcd \"$sm\"\n+\tfor name; do\n+\t\techo \"$name\" > \"$name\" &&\n+\t\tgit add \"$name\" &&\n+\t\ttest_tick &&\n+\t\tgit commit -m \"Add $name\"\n+\tdone >/dev/null\n+\tgit rev-parse --verify HEAD | cut -c1-7\n+\tcd \"$owd\"\n+}\n+commit_file () {\n+\ttest_tick &&\n+\tgit commit \"$@\" -m \"Commit $*\" >/dev/null\n+}\n+\n+test_create_repo sm1 &&\n+add_file . foo >/dev/null\n+\n+head1=$(add_file sm1 foo1 foo2)\n+\n+test_expect_success 'added submodule' \"\n+\tgit add sm1 &&\n+\tgit diff-index -p --submodule=log HEAD >actual &&\n+\tdiff actual - <<-EOF\n+Submodule sm1 0000000...$head1 (new submodule)\n+EOF\n+\"\n+\n+commit_file sm1 &&\n+head2=$(add_file sm1 foo3)\n+\n+test_expect_success 'modified submodule(forward)' \"\n+\tgit diff-index -p --submodule=log HEAD >actual &&\n+\tdiff actual - <<-EOF\n+Submodule sm1 $head1..$head2:\n+  > Add foo3\n+EOF\n+\"\n+\n+test_expect_success 'modified submodule(forward)' \"\n+\tgit diff --submodule=log >actual &&\n+\tdiff actual - <<-EOF\n+Submodule sm1 $head1..$head2:\n+  > Add foo3\n+EOF\n+\"\n+\n+test_expect_success 'modified submodule(forward) --submodule' \"\n+\tgit diff --submodule >actual &&\n+\tdiff actual - <<-EOF\n+Submodule sm1 $head1..$head2:\n+  > Add foo3\n+EOF\n+\"\n+\n+fullhead1=$(cd sm1; git rev-list --max-count=1 $head1)\n+fullhead2=$(cd sm1; git rev-list --max-count=1 $head2)\n+test_expect_success 'modified submodule(forward) --submodule=short' \"\n+\tgit diff --submodule=short >actual &&\n+\tdiff actual - <<-EOF\n+diff --git a/sm1 b/sm1\n+index $head1..$head2 160000\n+--- a/sm1\n++++ b/sm1\n+@@ -1 +1 @@\n+-Subproject commit $fullhead1\n++Subproject commit $fullhead2\n+EOF\n+\"\n+\n+commit_file sm1 &&\n+cd sm1 &&\n+git reset --hard HEAD~2 >/dev/null &&\n+head3=$(git rev-parse --verify HEAD | cut -c1-7) &&\n+cd ..\n+\n+test_expect_success 'modified submodule(backward)' \"\n+\tgit diff-index -p --submodule=log HEAD >actual &&\n+\tdiff actual - <<-EOF\n+Submodule sm1 $head2..$head3 (rewind):\n+  < Add foo3\n+  < Add foo2\n+EOF\n+\"\n+\n+head4=$(add_file sm1 foo4 foo5) &&\n+head4_full=$(GIT_DIR=sm1/.git git rev-parse --verify HEAD)\n+test_expect_success 'modified submodule(backward and forward)' \"\n+\tgit diff-index -p --submodule=log HEAD >actual &&\n+\tdiff actual - <<-EOF\n+Submodule sm1 $head2...$head4:\n+  > Add foo5\n+  > Add foo4\n+  < Add foo3\n+  < Add foo2\n+EOF\n+\"\n+\n+commit_file sm1 &&\n+mv sm1 sm1-bak &&\n+echo sm1 >sm1 &&\n+head5=$(git hash-object sm1 | cut -c1-7) &&\n+git add sm1 &&\n+rm -f sm1 &&\n+mv sm1-bak sm1\n+\n+test_expect_success 'typechanged submodule(submodule->blob), --cached' \"\n+\tgit diff --submodule=log --cached >actual &&\n+\tdiff actual - <<-EOF\n+Submodule sm1 41fbea9...0000000 (submodule deleted)\n+diff --git a/sm1 b/sm1\n+new file mode 100644\n+index 0000000..9da5fb8\n+--- /dev/null\n++++ b/sm1\n+@@ -0,0 +1 @@\n++sm1\n+EOF\n+\"\n+\n+test_expect_success 'typechanged submodule(submodule->blob)' \"\n+\tgit diff --submodule=log >actual &&\n+\tdiff actual - <<-EOF\n+diff --git a/sm1 b/sm1\n+deleted file mode 100644\n+index 9da5fb8..0000000\n+--- a/sm1\n++++ /dev/null\n+@@ -1 +0,0 @@\n+-sm1\n+Submodule sm1 0000000...$head4 (new submodule)\n+EOF\n+\"\n+\n+rm -rf sm1 &&\n+git checkout-index sm1\n+test_expect_success 'typechanged submodule(submodule->blob)' \"\n+\tgit diff-index -p --submodule=log HEAD >actual &&\n+\tdiff actual - <<-EOF\n+Submodule sm1 $head4...0000000 (submodule deleted)\n+diff --git a/sm1 b/sm1\n+new file mode 100644\n+index 0000000..$head5\n+--- /dev/null\n++++ b/sm1\n+@@ -0,0 +1 @@\n++sm1\n+EOF\n+\"\n+\n+rm -f sm1 &&\n+test_create_repo sm1 &&\n+head6=$(add_file sm1 foo6 foo7)\n+fullhead6=$(cd sm1; git rev-list --max-count=1 $head6)\n+test_expect_success 'nonexistent commit' \"\n+\tgit diff-index -p --submodule=log HEAD >actual &&\n+\tdiff actual - <<-EOF\n+Submodule sm1 $head4...$head6 (commits not present)\n+EOF\n+\"\n+\n+commit_file\n+test_expect_success 'typechanged submodule(blob->submodule)' \"\n+\tgit diff-index -p --submodule=log HEAD >actual &&\n+\tdiff actual - <<-EOF\n+diff --git a/sm1 b/sm1\n+deleted file mode 100644\n+index $head5..0000000\n+--- a/sm1\n++++ /dev/null\n+@@ -1 +0,0 @@\n+-sm1\n+Submodule sm1 0000000...$head6 (new submodule)\n+EOF\n+\"\n+\n+commit_file sm1 &&\n+rm -rf sm1\n+test_expect_success 'deleted submodule' \"\n+\tgit diff-index -p --submodule=log HEAD >actual &&\n+\tdiff actual - <<-EOF\n+Submodule sm1 $head6...0000000 (submodule deleted)\n+EOF\n+\"\n+\n+test_create_repo sm2 &&\n+head7=$(add_file sm2 foo8 foo9) &&\n+git add sm2\n+\n+test_expect_success 'multiple submodules' \"\n+\tgit diff-index -p --submodule=log HEAD >actual &&\n+\tdiff actual - <<-EOF\n+Submodule sm1 $head6...0000000 (submodule deleted)\n+Submodule sm2 0000000...$head7 (new submodule)\n+EOF\n+\"\n+\n+test_expect_success 'path filter' \"\n+\tgit diff-index -p --submodule=log HEAD sm2 >actual &&\n+\tdiff actual - <<-EOF\n+Submodule sm2 0000000...$head7 (new submodule)\n+EOF\n+\"\n+\n+commit_file sm2\n+test_expect_success 'given commit' \"\n+\tgit diff-index -p --submodule=log HEAD^ >actual &&\n+\tdiff actual - <<-EOF\n+Submodule sm1 $head6...0000000 (submodule deleted)\n+Submodule sm2 0000000...$head7 (new submodule)\n+EOF\n+\"\n+\n+test_expect_success 'given commit --submodule' \"\n+\tgit diff-index -p --submodule HEAD^ >actual &&\n+\tdiff actual - <<-EOF\n+Submodule sm1 $head6...0000000 (submodule deleted)\n+Submodule sm2 0000000...$head7 (new submodule)\n+EOF\n+\"\n+\n+fullhead7=$(cd sm2; git rev-list --max-count=1 $head7)\n+\n+test_expect_success 'given commit --submodule=short' \"\n+\tgit diff-index -p --submodule=short HEAD^ >actual &&\n+\tdiff actual - <<-EOF\n+diff --git a/sm1 b/sm1\n+deleted file mode 160000\n+index $head6..0000000\n+--- a/sm1\n++++ /dev/null\n+@@ -1 +0,0 @@\n+-Subproject commit $fullhead6\n+diff --git a/sm2 b/sm2\n+new file mode 160000\n+index 0000000..$head7\n+--- /dev/null\n++++ b/sm2\n+@@ -0,0 +1 @@\n++Subproject commit $fullhead7\n+EOF\n+\"\n+\n+test_done\n-- \n1.6.5.rc2.19.gcbaec.dirty\n"},{"id":"125793","messageId":"fabb9a1e0910231127i3ab469qebdc17168a58f22a@mail.gmail.com","threadId":"21320","inReplyTo":"4AE0E542.8010501@gmail.com","subject":"Re: ks/precompute-completion (was Re: What's cooking in git.git (Oct 2009, #04; Wed, 21))","fromName":"Sverre Rabbelier","fromEmail":"srabbelier@gmail.com","sentAt":"2009-10-23T18:27:18Z","receivedAt":"2009-10-23T18:27:18Z","isPatch":false,"sender":{"key":"srabbelier@gmail.com","avatar":"https://avatars.githubusercontent.com/u/3098?v=4"},"body":"Heya,\n\nOn Thu, Oct 22, 2009 at 18:05, A Large Angry SCM <gitzilla@gmail.com> wrote:\n> Your original email included 'make && make install'; it's the \"make install\"\n> part I'm concerned about.\n\nAh, no, I meant that as part of my semi-regular git update (during\nwhich I do 'make && make install') I want to have up-to-date bash\ncompletion, preferably installed somewhere system-wide; currently I am\nforced to have a 'source\n/home/sverre/code/git/contrib/completion/git-completion.bash' in my\n.bashrc,..\n\n-- \nCheers,\n\nSverre Rabbelier\n"},{"id":"125799","messageId":"7vd44eaqc5.fsf@alter.siamese.dyndns.org","threadId":"21320","inReplyTo":"fabb9a1e0910231127i3ab469qebdc17168a58f22a@mail.gmail.com","subject":"Re: ks/precompute-completion","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2009-10-23T18:59:54Z","receivedAt":"2009-10-23T18:59:54Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Sverre Rabbelier <srabbelier@gmail.com> writes:\n\n> Ah, no, I meant that as part of my semi-regular git update (during\n> which I do 'make && make install') I want to have up-to-date bash\n> completion, preferably installed somewhere system-wide; currently I am\n> forced to have a 'source\n> /home/sverre/code/git/contrib/completion/git-completion.bash' in my\n> .bashrc,..\n\nIf you have enough privilege to run 'make && make install' regularly into\na system-wide place, I presume you can have a system-wide rc that sources\n/home/sverre/code/git/contrib/completion/git-completion.bash, no?\n\nI think there are two issues.\n\n 1. The series will break your rc script (either $HOME/.bashrc, or\n    system-side) that sources $git/contrib/completion/git-completion.bash\n    because it has to be built; having \"make\" generate it may alleviate\n    the issue, but \"make clean\" will break it again, so it is not\n    something you can solve in any way other than changing your setting.\n\n 2. Some people have been expecting \"make install\" not to install the bash\n    completion anywhere.\n\nSo perhaps \"make && make install-contrib\"?\n"},{"id":"125800","messageId":"fabb9a1e0910231216j2a024ac5mf5b5ccb5322722f8@mail.gmail.com","threadId":"21320","inReplyTo":"7vd44eaqc5.fsf@alter.siamese.dyndns.org","subject":"Re: ks/precompute-completion","fromName":"Sverre Rabbelier","fromEmail":"srabbelier@gmail.com","sentAt":"2009-10-23T19:16:59Z","receivedAt":"2009-10-23T19:16:59Z","isPatch":false,"sender":{"key":"srabbelier@gmail.com","avatar":"https://avatars.githubusercontent.com/u/3098?v=4"},"body":"Heya,\n\nOn Fri, Oct 23, 2009 at 13:59, Junio C Hamano <gitster@pobox.com> wrote:\n>  1. The series will break your rc script (either $HOME/.bashrc, or\n>    system-side) that sources $git/contrib/completion/git-completion.bash\n>    because it has to be built; having \"make\" generate it may alleviate\n>    the issue, but \"make clean\" will break it again, so it is not\n>    something you can solve in any way other than changing your setting.\n\nThis is my main concern, adding 'bash_completion' as a target to all:\nwould be ok; why would 'make clean' break it? As long as you don't add\n\"make -C contrib/completion clean' to the main clean target there's no\nproblem?\n\n-- \nCheers,\n\nSverre Rabbelier\n"},{"id":"125804","messageId":"7v1vktc1uk.fsf@alter.siamese.dyndns.org","threadId":"21320","inReplyTo":"fabb9a1e0910231216j2a024ac5mf5b5ccb5322722f8@mail.gmail.com","subject":"Re: ks/precompute-completion","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2009-10-23T20:05:55Z","receivedAt":"2009-10-23T20:05:55Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Sverre Rabbelier <srabbelier@gmail.com> writes:\n\n> This is my main concern, adding 'bash_completion' as a target to all:\n> would be ok; why would 'make clean' break it? As long as you don't add\n> \"make -C contrib/completion clean' to the main clean target there's no\n> problem?\n\n\"make clean\" should remove it, because it is a normal build product,\nif you make your \"make all\" build completion scripts.\n\nThe word _should_ is used in the RFC2119 sense: there may exist valid\nreasons in particular circumstances to ignore a particular item, but the\nfull implications must be understood and carefully weighed before choosing\na different course.\n"},{"id":"125805","messageId":"fabb9a1e0910231309r41bc8fb6h61898318ed5568b0@mail.gmail.com","threadId":"21320","inReplyTo":"7v1vktc1uk.fsf@alter.siamese.dyndns.org","subject":"Re: ks/precompute-completion","fromName":"Sverre Rabbelier","fromEmail":"srabbelier@gmail.com","sentAt":"2009-10-23T20:09:26Z","receivedAt":"2009-10-23T20:09:26Z","isPatch":false,"sender":{"key":"srabbelier@gmail.com","avatar":"https://avatars.githubusercontent.com/u/3098?v=4"},"body":"Heya,\n\nOn Fri, Oct 23, 2009 at 15:05, Junio C Hamano <gitster@pobox.com> wrote:\n> \"make clean\" should remove it, because it is a normal build product,\n> if you make your \"make all\" build completion scripts.\n\nHmm, I guess that's fair enough, if you 'make clean' you want all\nbuild products to be removed; this problem would be solved by\ninstalling the completion script in the share dir, that way it doens't\nmatter if 'make clean' removes the one in ~/code/git, as long as the\none in ~/share/git-completion remains.\n\n-- \nCheers,\n\nSverre Rabbelier\n"},{"id":"125808","messageId":"m3d44deu93.fsf@localhost.localdomain","threadId":"21320","inReplyTo":"7v1vktc1uk.fsf@alter.siamese.dyndns.org","subject":"Re: ks/precompute-completion","fromName":"Jakub Narebski","fromEmail":"jnareb@gmail.com","sentAt":"2009-10-23T20:20:19Z","receivedAt":"2009-10-23T20:20:19Z","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> Sverre Rabbelier <srabbelier@gmail.com> writes:\n> \n> > This is my main concern, adding 'bash_completion' as a target to all:\n> > would be ok; why would 'make clean' break it? As long as you don't add\n> > \"make -C contrib/completion clean' to the main clean target there's no\n> > problem?\n> \n> \"make clean\" should remove it, because it is a normal build product,\n> if you make your \"make all\" build completion scripts.\n> \n> The word _should_ is used in the RFC2119 sense: there may exist valid\n> reasons in particular circumstances to ignore a particular item, but the\n> full implications must be understood and carefully weighed before choosing\n> a different course.\n\nIf we take similar approach to the way gitweb can be build to the bash\ncompletion script, which means building it via\n\n  make contrib/completion/git-completion.bash\n\n(and not make this target part of \"make all\"), then there is, I think,\nno reason for \"make clean\" to remove it, isn't it?\n\n-- \nJakub Narebski\nPoland\nShadeHawk on #git\n"},{"id":"125809","messageId":"fabb9a1e0910231322y1355ca8ck209bcd64bd29dec6@mail.gmail.com","threadId":"21320","inReplyTo":"m3d44deu93.fsf@localhost.localdomain","subject":"Re: ks/precompute-completion","fromName":"Sverre Rabbelier","fromEmail":"srabbelier@gmail.com","sentAt":"2009-10-23T20:22:54Z","receivedAt":"2009-10-23T20:22:54Z","isPatch":false,"sender":{"key":"srabbelier@gmail.com","avatar":"https://avatars.githubusercontent.com/u/3098?v=4"},"body":"Heya,\n\nOn Fri, Oct 23, 2009 at 15:20, Jakub Narebski <jnareb@gmail.com> wrote:\n> (and not make this target part of \"make all\")\n\nBut that I can already do through 'make contrib/completion/Makefile',\nwhat I want is to not have to worry about doing that whenever I update\nmy git install (that is, the same way as it was before it became\npre-computed).\n\n-- \nCheers,\n\nSverre Rabbelier\n"},{"id":"125818","messageId":"4AE22E48.9050408@gmail.com","threadId":"21320","inReplyTo":"fabb9a1e0910231322y1355ca8ck209bcd64bd29dec6@mail.gmail.com","subject":"Re: ks/precompute-completion","fromName":"A Large Angry SCM","fromEmail":"gitzilla@gmail.com","sentAt":"2009-10-23T22:29:28Z","receivedAt":"2009-10-23T22:29:28Z","isPatch":false,"sender":{"key":"gitzilla@gmail.com","avatar":"https://gravatar.com/avatar/354625c442439908ff3dd99757dee330e29e9df7847472384faf7a00add247fb?d=mp&s=160"},"body":"Sverre Rabbelier wrote:\n> Heya,\n> \n> On Fri, Oct 23, 2009 at 15:20, Jakub Narebski <jnareb@gmail.com> wrote:\n>> (and not make this target part of \"make all\")\n> \n> But that I can already do through 'make contrib/completion/Makefile',\n> what I want is to not have to worry about doing that whenever I update\n> my git install (that is, the same way as it was before it became\n> pre-computed).\n> \n\nIt seems that you want the completion script promoted out of contrib. \nOtherwise, you're asking for it to be treated special with respect to \neverything else in contrib and have the top level Makefile be aware of \nit and add it to the main targets.\n\nThe promotion I have no problem with as long as the install location is \nnot somewhere where any shell will find it without a config setting in \nthe user's shell. Leaving it in contrib and and adding it to top level \nMakefile, I do have a problem with.\n"},{"id":"125819","messageId":"fabb9a1e0910231539n543ba908k3d57bcd01e5e233c@mail.gmail.com","threadId":"21320","inReplyTo":"4AE22E48.9050408@gmail.com","subject":"Re: ks/precompute-completion","fromName":"Sverre Rabbelier","fromEmail":"srabbelier@gmail.com","sentAt":"2009-10-23T22:39:04Z","receivedAt":"2009-10-23T22:39:04Z","isPatch":false,"sender":{"key":"srabbelier@gmail.com","avatar":"https://avatars.githubusercontent.com/u/3098?v=4"},"body":"Heya,\n\nOn Fri, Oct 23, 2009 at 15:29, A Large Angry SCM <gitzilla@gmail.com> wrote:\n> It seems that you want the completion script promoted out of contrib.\n> Otherwise, you're asking for it to be treated special with respect to\n> everything else in contrib and have the top level Makefile be aware of it\n> and add it to the main targets.\n\nThat would seem like a sensible solution ;).\n\n> The promotion I have no problem with as long as the install location is not\n> somewhere where any shell will find it without a config setting in the\n> user's shell. Leaving it in contrib and and adding it to top level Makefile,\n> I do have a problem\n\nSomething like this then?\n\nhttp://repo.or.cz/w/git/dscho.git?a=commit;h=eb966204d17dcab7abf61621219312a813c87405\n\n-- \nCheers,\n\nSverre Rabbelier\n"},{"id":"125826","messageId":"7vpr8d8f3z.fsf@alter.siamese.dyndns.org","threadId":"21320","inReplyTo":"m3ljj3es02.fsf@localhost.localdomain","subject":"Re: What's cooking in git.git (Oct 2009, #04; Wed, 21)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2009-10-24T06:45:20Z","receivedAt":"2009-10-24T06:45:20Z","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>> * rs/pretty-wrap (2009-10-17) 1 commit\n>>  - Implement wrap format %w() as if it is a mode switch\n>>  (this branch uses js/log-rewrap; is related to jc/strbuf-nested-expand.)\n>> \n>> When it comes to design issues to keep unnecessary complexity out, I tend\n>> to trust Rene (and Nico) a lot more than I trust myself.  Tonight's 'pu'\n>> queues this series instead of my \"nested\" one.\n>\n>> * jc/strbuf-nested-expand (2009-10-18) 3 commits\n>>  . Teach --wrap to only indent without wrapping\n>>  . Add %[wrap(width,in1,in2)<<any-string>>%] implementation\n>>  . strbuf_nested_expand(): allow expansion to interrupt in the middle\n>>  (this branch uses js/log-rewrap; is related to rs/pretty-wrap.)\n>> \n>> Ejected from 'pu' to let rs/pretty-wrap in as described above.\n>\n> I think nested expand is easier to use than a mode switch: using\n> scoping (well, kind of) like in high-level programming languages is\n> IMVHO easier than programming a state machine like in assembler (or\n> e.g. OpenGL).\n>\n> On the other hand this makes pretty format into a mini-language; also\n> we already have and use mode switches in the form of color codes.\n> Perhaps if color also used wrapping / nested expand, so one doesn't\n> have to track where to turn off and on which toggle...\n\nIndeed, the \"mini-language\"-ness was what made Réne worried and reminded\nme that I should be worried, too.  We need to get the design right if we\ndo so---there may come a time that we are better off biting the bullet\nwhen we discover needs (notice, it is not \"wants, because we can\") for\nmany useful string functions, but I do not think we have reached that\npoint yet.\n\n>> * jc/pretty-lf (2009-10-04) 1 commit.\n>>  - Pretty-format: %[+-]x to tweak inter-item newlines\n>\n> I understand that %a%+b expands to %a%n%b if %b has non-empty\n> expansion, and to %a if %b is empty, but what %-b is used for?\n\nI know you can read the commit log message.\n"},{"id":"125854","messageId":"7viqe461ci.fsf@alter.siamese.dyndns.org","threadId":"21320","inReplyTo":"4AE192AD.70603@web.de","subject":"Re: [PATCH] add tests for git diff --submodule","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2009-10-24T19:25:33Z","receivedAt":"2009-10-24T19:25:33Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Jens Lehmann <Jens.Lehmann@web.de> writes:\n\n> Apart from your changes necessary to make the test run again my changes are:\n>\n> - rename from \"t4041-diff-submodule-summary.sh\" to \"t4041-diff-submodule.sh\"\n> - corrected all comments still speaking of \"summary\"\n> - added tests to test the behaviour of \"--submodule\" and \"--submodule=short\"\n\nThanks for a nice summary.\n"},{"id":"125898","messageId":"20091025160213.GA8532@localhost","threadId":"21320","inReplyTo":"7veiovly35.fsf@alter.siamese.dyndns.org","subject":"Re: What's cooking in git.git (Oct 2009, #04; Wed, 21)","fromName":"Clemens Buchacher","fromEmail":"drizzd@aon.at","sentAt":"2009-10-25T16:02:13Z","receivedAt":"2009-10-25T16:02:13Z","isPatch":false,"sender":{"key":"drizzd@gmx.net","avatar":"https://avatars.githubusercontent.com/u/59082?v=4"},"body":"On Wed, Oct 21, 2009 at 11:52:30PM -0700, Junio C Hamano wrote:\n\n> * ks/precompute-completion (2009-10-05) 1 commit.\n>   (merged to 'next' on 2009-10-14 at adf722a)\n>  + Speedup bash completion loading\n> \n> Are people happy with this?\n\nI'm looking forward to this on Windows, where loading the completion script\ncan take about 10 seconds.\n\nClemens\n"},{"id":"125922","messageId":"7vzl7eocd6.fsf@alter.siamese.dyndns.org","threadId":"21320","inReplyTo":"20091025160213.GA8532@localhost","subject":"Re: What's cooking in git.git (Oct 2009, #04; Wed, 21)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2009-10-26T07:14:45Z","receivedAt":"2009-10-26T07:14:45Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Clemens Buchacher <drizzd@aon.at> writes:\n\n> On Wed, Oct 21, 2009 at 11:52:30PM -0700, Junio C Hamano wrote:\n>\n>> * ks/precompute-completion (2009-10-05) 1 commit.\n>>   (merged to 'next' on 2009-10-14 at adf722a)\n>>  + Speedup bash completion loading\n>> \n>> Are people happy with this?\n>\n> I'm looking forward to this on Windows, where loading the completion script\n> can take about 10 seconds.\n\nThanks.\n\nAre you giving this comment after you actually tried it on Windows and\nfound it satisfactory, or is it just based on the general description of\n\"this should make it faster\"?\n\nI need to know, to sift acks/kudos based on facts that I can use to decide\nwhen to release it to 'master', from wishful thinking that I shouldn't,\nespecially after seeing an obvious issue like the one reported by Stephen\nBoyd a few days ago (http://mid.gname.com/4AE0190E.8020803@gmail.com/).\n"},{"id":"125924","messageId":"20091026082931.GA6192@localhost","threadId":"21320","inReplyTo":"7vzl7eocd6.fsf@alter.siamese.dyndns.org","subject":"Re: What's cooking in git.git (Oct 2009, #04; Wed, 21)","fromName":"Clemens Buchacher","fromEmail":"drizzd@aon.at","sentAt":"2009-10-26T08:29:31Z","receivedAt":"2009-10-26T08:29:31Z","isPatch":false,"sender":{"key":"drizzd@gmx.net","avatar":"https://avatars.githubusercontent.com/u/59082?v=4"},"body":"On Mon, Oct 26, 2009 at 12:14:45AM -0700, Junio C Hamano wrote:\n\n> Are you giving this comment after you actually tried it on Windows and\n> found it satisfactory, or is it just based on the general description of\n> \"this should make it faster\"?\n\nI just tried and it went down from several seconds to about half a second.\nOn slower machines I expect the difference to be even more noticable.\n\n> I need to know, to sift acks/kudos based on facts that I can use to decide\n> when to release it to 'master', from wishful thinking that I shouldn't,\n> especially after seeing an obvious issue like the one reported by Stephen\n> Boyd a few days ago (http://mid.gname.com/4AE0190E.8020803@gmail.com/).\n\nI cannot follow that link. If you're referring to the \"completion of\ncommands available only in build environment\" issue, that could also be\nconsidered a feature, because it allows completion of user-defined scripts.\n\nWhy does your PATH include the build directory during make, Stephen?\n\nClemens\n"},{"id":"125925","messageId":"4AE56552.70407@gmail.com","threadId":"21320","inReplyTo":"20091026082931.GA6192@localhost","subject":"Re: What's cooking in git.git (Oct 2009, #04; Wed, 21)","fromName":"Stephen Boyd","fromEmail":"bebarino@gmail.com","sentAt":"2009-10-26T09:01:06Z","receivedAt":"2009-10-26T09:01:06Z","isPatch":false,"sender":{"key":"bebarino@gmail.com","avatar":"https://avatars.githubusercontent.com/u/38832?v=4"},"body":"Clemens Buchacher wrote:\n>> I need to know, to sift acks/kudos based on facts that I can use to decide\n>> when to release it to 'master', from wishful thinking that I shouldn't,\n>> especially after seeing an obvious issue like the one reported by Stephen\n>> Boyd a few days ago (http://mid.gname.com/4AE0190E.8020803@gmail.com/).\n>\n> I cannot follow that link. If you're referring to the \"completion of\n> commands available only in build environment\" issue, that could also be\n> considered a feature, because it allows completion of user-defined scripts.\n>\n> Why does your PATH include the build directory during make, Stephen?\n\nThe Makefile says:\n\ngit-completion.bash: git-completion.bash.in git-completion.bash.generate\n        # Generate completions for binaries we have just built\n        PATH=\"$(shell pwd)/../..:$$PATH\" ./git-completion.bash.generate\n\nHaving user-defined scripts is a good point. Generating the completion\nlike this removes the possibility of such scripts from appearing in the\ncompletion. Unless users are putting their own \"git-*\" scripts in their\nbuild directory (sounds odd to me).\n\nPersonally, I'd rather keep it dynamic but I can see how it's useful to\nget the 10x speedup. It would be really cool if we could have the best\nof both worlds, where I keep my dynamic loading, but others can build\nthe completion and get the speedup.\n"},{"id":"125926","messageId":"20091026100756.GA9874@localhost","threadId":"21320","inReplyTo":"4AE56552.70407@gmail.com","subject":"Re: What's cooking in git.git (Oct 2009, #04; Wed, 21)","fromName":"Clemens Buchacher","fromEmail":"drizzd@aon.at","sentAt":"2009-10-26T10:07:56Z","receivedAt":"2009-10-26T10:07:56Z","isPatch":false,"sender":{"key":"drizzd@gmx.net","avatar":"https://avatars.githubusercontent.com/u/59082?v=4"},"body":"On Mon, Oct 26, 2009 at 02:01:06AM -0700, Stephen Boyd wrote:\n\n> Clemens Buchacher wrote:\n>\n> > Why does your PATH include the build directory during make, Stephen?\n> \n> The Makefile says:\n> \n> git-completion.bash: git-completion.bash.in git-completion.bash.generate\n>         # Generate completions for binaries we have just built\n>         PATH=\"$(shell pwd)/../..:$$PATH\" ./git-completion.bash.generate\n\nRight, of course. I guess ignoring *.sh and *.perl is reasonable then.\n\n> Personally, I'd rather keep it dynamic but I can see how it's useful to\n> get the 10x speedup. It would be really cool if we could have the best\n> of both worlds, where I keep my dynamic loading, but others can build\n> the completion and get the speedup.\n\nShould not be too hard to do using a configuration variable like\ncore.completion = dynamic.\n\nClemens\n"},{"id":"126043","messageId":"20091027182710.GF10505@spearce.org","threadId":"21320","inReplyTo":"7veiovly35.fsf@alter.siamese.dyndns.org","subject":"Re: vl/git-gui topic","fromName":"Shawn O. Pearce","fromEmail":"spearce@spearce.org","sentAt":"2009-10-27T18:27:10Z","receivedAt":"2009-10-27T18:27:10Z","isPatch":false,"sender":{"key":"spearce@spearce.org","avatar":"https://avatars.githubusercontent.com/u/34844?v=4"},"body":"Junio C Hamano <gitster@pobox.com> wrote:\n> * vl/git-gui (2009-10-16) 1 commit.\n>  - git-gui: adjust the minimum height of diff pane for shorter screen height\n> \n> Shawn?\n\nApplied to git-gui tree.\n\n-- \nShawn.\n"},{"id":"126046","messageId":"20091027183747.GG10505@spearce.org","threadId":"21320","inReplyTo":"7veiovly35.fsf@alter.siamese.dyndns.org","subject":"Re: jp/dirty-describe topic","fromName":"Shawn O. Pearce","fromEmail":"spearce@spearce.org","sentAt":"2009-10-27T18:37:47Z","receivedAt":"2009-10-27T18:37:47Z","isPatch":false,"sender":{"key":"spearce@spearce.org","avatar":"https://avatars.githubusercontent.com/u/34844?v=4"},"body":"Junio C Hamano <gitster@pobox.com> wrote:\n> * jp/dirty-describe (2009-10-21) 1 commit.\n>  - Teach \"git describe\" --dirty option\n> \n> Ack?\n\nYup,\n\nAcked-by: Shawn O. Pearce <spearce@spearce.org>\n\n-- \nShawn.\n"},{"id":"126093","messageId":"7v4opkuguz.fsf@alter.siamese.dyndns.org","threadId":"21320","inReplyTo":"20091027182710.GF10505@spearce.org","subject":"Re: vl/git-gui topic","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2009-10-28T07:17:56Z","receivedAt":"2009-10-28T07:17:56Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"\"Shawn O. Pearce\" <spearce@spearce.org> writes:\n\n> Junio C Hamano <gitster@pobox.com> wrote:\n>> * vl/git-gui (2009-10-16) 1 commit.\n>>  - git-gui: adjust the minimum height of diff pane for shorter screen height\n>> \n>> Shawn?\n>\n> Applied to git-gui tree.\n\nThanks, pulled together with bunch of other changes that looked sensible.\n"},{"id":"126147","messageId":"20091028144726.GA17602@spearce.org","threadId":"21320","inReplyTo":"7veiovly35.fsf@alter.siamese.dyndns.org","subject":"Re: sp/smart-http topic","fromName":"Shawn O. Pearce","fromEmail":"spearce@spearce.org","sentAt":"2009-10-28T14:47:26Z","receivedAt":"2009-10-28T14:47:26Z","isPatch":false,"sender":{"key":"spearce@spearce.org","avatar":"https://avatars.githubusercontent.com/u/34844?v=4"},"body":"Junio C Hamano <gitster@pobox.com> wrote:\n> \n> * sp/smart-http (2009-10-14) 17 commits\n>  - Smart fetch over HTTP: client side\n...\n> What's the doneness of this series?\n\nNot done yet.  I want to respin once more before it hits next.\n\nI've picked up a number of test related changes from Clemens\nBuchacher and Tay Ray Chuan, plus some suggestions from the latter\nwere fixed up in the WebDAV code.\n\nJohn \"warthog9\" Hawley and I were spending some time yesterday to\ntry to figure out why smart HTTP serving off kernel.org was giving\nme only 300 KiB/sec during clone, but git-daemon was giving me 12\nMiB/sec for the same server and repository.\n\nPeff noticed the TCP windows for smart HTTP were ~16 KiB in size,\nbut with git-daemon were ~200 KiB on size.  John and I are pretty\nsure this is the throughput problem, but we haven't found why the\nwindow is so much smaller under smart HTTP.\n\nWe also need proper tests for smart HTTP.  I haven't had time to\nwrite tests yet, and the ones that were proposed for t5540-http-push\naren't suitable because you have to run the test suite twice in\norder to test both WebDAV and smart HTTP push for the same build.\n\n-- \nShawn.\n"}]}