{"thread":{"id":"34784","subject":"What's cooking in git.git (Aug 2013, #06; Tue, 27)","startedAt":"2013-08-27T19:22:30Z","lastAt":"2013-08-28T06:39:16Z","messageCount":10,"participants":["Junio C Hamano","Jeff King","Antoine Pelisse","Kacper Kornet","Johannes Sixt"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"226047","messageId":"xmqqsixvaqh5.fsf@gitster.dls.corp.google.com","threadId":"34784","inReplyTo":null,"subject":"What's cooking in git.git (Aug 2013, #06; Tue, 27)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2013-08-27T19:22:30Z","receivedAt":"2013-08-27T19:22:30Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"What's cooking in git.git (Aug 2013, #06; Tue, 27)\n--------------------------------------------------\n\nHere are the topics that have been cooking.  Commits prefixed with\n'-' are only in 'pu' (proposed updates) while commits prefixed with\n'+' are in 'next'.\n\nGit 1.8.4 was tagged and released recently, and we will shortly go\ninto a new development cycle for the next one, likely to be 1.8.5.\n\nIn this issue of \"What's cooking\" report, I haven't started sifting\nthe topics, most of which are marked as \"will cook in next\", into\nseparate bins to indicate what order they would graduate yet.  After\ndoing so, the tip of 'next' will be rewound, hopefully tomorrow but\nit may slip by a day or so.\n\nI expect this cycle to conclude at around the end of October, and we\nwill have another release by the end of the year.  The first release\nin the coming year may be named \"Git 2.0\" with the promised\ncompatibility breakages.\n\nYou can find the changes described here in the integration branches\nof the repositories listed at\n\n    http://git-blame.blogspot.com/p/git-public-repositories.html\n\n--------------------------------------------------\n[New Topics]\n\n* js/xread-in-full (2013-08-20) 1 commit\n  (merged to 'next' on 2013-08-20 at 95baa13)\n + stream_to_pack: xread does not guarantee to read all requested bytes\n\n Originally merged to 'next' on 2013-08-20\n\n A call to xread() was used without a loop around to cope with short\n read in the codepath to stream new contents to a pack.\n\n Will cook in 'next'.\n\n\n* sb/mailmap-freeing-NULL-is-ok (2013-08-20) 1 commit\n  (merged to 'next' on 2013-08-20 at 303b16c)\n + mailmap: remove redundant check for freeing memory\n\n Originally merged to 'next' on 2013-08-20\n\n Will cook in 'next'.\n\n\n* tg/index-struct-sizes (2013-08-20) 1 commit\n  (merged to 'next' on 2013-08-22 at df6b8e2)\n + read-cache: use fixed width integer types\n\n Originally merged to 'next' on 2013-08-22\n\n The code that reads from a region that mmaps an on-disk index\n assumed that \"int\"/\"short\" are always 32/16 bits.\n\n Will cook in 'next'.\n\n\n* bc/completion-for-bash-3.0 (2013-08-22) 3 commits\n  (merged to 'next' on 2013-08-22 at 46c5bb2)\n + contrib/git-prompt.sh: handle missing 'printf -v' more gracefully\n + t9902-completion.sh: old Bash still does not support array+=('') notation\n + git-completion.bash: use correct Bash/Zsh array length syntax\n\n Originally merged to 'next' on 2013-08-22\n\n Some people still use rather old versions of bash, which cannot\n grok some constructs like 'printf -v varname' the prompt and\n completion code started to use recently.\n\n Will cook in 'next'.\n\n\n* bc/submodule-status-ignored (2013-08-20) 2 commits\n  (merged to 'next' on 2013-08-22 at 3dfd2a3)\n + submodule: don't print status output with ignore=all\n + submodule: fix confusing variable name\n\n Originally merged to 'next' on 2013-08-22\n\n Will cook in 'next'.\n\n\n* jk/config-int-range-check (2013-08-21) 2 commits\n  (merged to 'next' on 2013-08-22 at 465efb3)\n + teach git-config to output large integers\n + config: properly range-check integer values\n\n Originally merged to 'next' on 2013-08-22\n\n \"git config --int section.var 3g\" should somehow diagnose that the\n number does not fit in \"int\" (on 32-bit platforms anyway) but it\n did not.\n\n Will cook in 'next'.\n\n\n* jk/duplicate-objects-in-packs (2013-08-24) 6 commits\n - default pack.indexDuplicates to false\n - index-pack: optionally reject packs with duplicate objects\n - test index-pack on packs with recoverable delta cycles\n - add tests for indexing packs with delta cycles\n - sha1-lookup: handle duplicate keys with GIT_USE_LOOKUP\n - test-sha1: add a binary output mode\n\n A packfile that stores the same object more than once is broken and\n will be rejected.\n\n Will merge to 'next'.\n\n\n* mm/mediawiki-dumb-push-fix (2013-08-21) 2 commits\n - git-remote-mediawiki: add test and check Makefile targets\n - git-remote-mediawiki: reset private ref after non-dumb push\n\n Waiting for a reroll.\n\n\n* rt/rebase-p-no-merge-summary (2013-08-21) 1 commit\n  (merged to 'next' on 2013-08-22 at 5310599)\n + rebase --preserve-merges: ignore \"merge.log\" config\n\n Originally merged to 'next' on 2013-08-22\n\n \"git rebase -p\" internally used the merge machinery, but when\n rebasing, there should not be a need for merge summary.\n\n Will cook in 'next'.\n\n\n* rv/send-email-cache-generated-mid (2013-08-21) 2 commits\n - git-send-email: Cache generated message-ids, use them when prompting\n - git-send-email: add optional 'choices' parameter to the ask sub\n\n\n* sp/clip-read-write-to-8mb (2013-08-20) 2 commits\n  (merged to 'next' on 2013-08-22 at 254e75d)\n + Revert \"compat/clipped-write.c: large write(2) fails on Mac OS X/XNU\"\n + xread, xwrite: limit size of IO to 8MB\n\n Originally merged to 'next' on 2013-08-22\n\n Send a large request to read(2)/write(2) as a smaller but still\n reasonably large chunks, which would improve the latency when the\n operation needs to be killed and incidentally works around broken\n 64-bit systems that cannot take a 2GB write or read in one go.\n\n Will cook in 'next'.\n\n\n* sp/doc-smart-http (2013-08-21) 1 commit\n  (merged to 'next' on 2013-08-22 at 4a16937)\n + Document the HTTP transport protocols\n\n Originally merged to 'next' on 2013-08-22\n\n Will cook in 'next'.\n\n\n* dw/diff-no-index-doc (2013-08-22) 1 commit\n - diff --no-index: clarify operation when not inside a repository\n\n When the user types \"git diff\" outside a working tree, thinking he\n is inside one, the current error message that is a single-liner\n \"usage: git diff --no-index <path> <path>\" may not be sufficient to\n make him realize the mistake. Add \"Not a git repository\" to the\n error message when we fell into the \"--no-index\" mode without an\n explicit command line option to instruct us to do so.\n\n\n* sb/repack-in-c (2013-08-22) 3 commits\n - repack: rewrite the shell script in C (squashing proposal)\n - repack: retain the return value of pack-objects\n - repack: rewrite the shell script in C\n\n\n* ap/commit-author-mailmap (2013-08-24) 1 commit\n - commit: search author pattern against mailmap\n\n \"git commit --author=$name\", when $name is not in the canonical\n \"A. U. Thor <au.thor@example.xz>\" format, looks for a matching name\n from existing history, but did not consult mailmap to grab the\n preferred author name.\n\n Will merge to 'next'.\n\n\n* hv/config-from-blob (2013-08-26) 1 commit\n  (merged to 'next' on 2013-08-27 at 7bc9019)\n + config: do not use C function names as struct members\n\n Originally merged to 'next' on 2013-08-27\n\n Portability fix.\n\n Will merge to 'master', aiming to later apply to 1.8.4.x maintenance track.\n\n\n* jk/mailmap-incomplete-line (2013-08-25) 2 commits\n - mailmap: avoid allocation when reading from blob\n - mailmap: handle mailmap blobs without trailing newlines\n\n Will merge to 'next'.\n\n\n* jk/write-broken-index-with-nul-sha1 (2013-08-26) 1 commit\n - write_index: optionally allow broken null sha1s\n\n Am I waiting for another reroll?\n\n\n* kk/tests-with-no-perl (2013-08-24) 4 commits\n - reset test: modernize style\n - t/t7106-reset-unborn-branch.sh: Add PERL prerequisite\n - add -i test: use skip_all instead of repeated PERL prerequisite\n - Make test \"using invalid commit with -C\" more strict\n\n Am I waiting for another reroll?\n\n\n* mm/fast-import-feature-doc (2013-08-25) 1 commit\n - Documentation/fast-import: clarify summary for `feature` command\n\n Will merge to 'next'.\n\n\n* mm/remote-helpers-doc (2013-08-26) 1 commit\n - Documentation/remote-helpers: document common use-case for private ref\n\n Will merge to 'next'.\n\n\n* mn/doc-pack-heu-remove-dead-pastebin (2013-08-23) 1 commit\n - remove dead pastebin link from pack-heuristics document\n\n Will merge to 'next'.\n\n\n* nd/fetch-pack-shallow-fix (2013-08-25) 1 commit\n  (merged to 'next' on 2013-08-27 at 7c2a162)\n + fetch-pack: do not remove .git/shallow file when --depth is not specified\n\n Originally merged to 'next' on 2013-08-27\n\n Recent \"short-cut clone connectivity check\" topic broke a shallow\n repository when a fetch operation tries to auto-follow tags.\n\n Will merge to 'master', aiming to later apply to 1.8.4.x maintenance track.\n\n\n* ta/user-manual (2013-08-24) 6 commits\n - Remove irrelevant reference from \"Tying it all together\"\n - Remove unnecessary historical note from \"Object storage format\"\n - Simplify \"How to make a commit\"\n - Use \"git merge\" instead of \"git pull .\"\n - Use current output for \"git repack\"\n - Call it \"Git User Manual\" and remove reference to very old Git version\n\n Update the user's manual to more recent versions of Git.\n\n I've picked up only the least questionable bits from a slightly\n larger series.\n\n\n* tb/precompose-autodetect-fix (2013-08-27) 1 commit\n - Set core.precomposeunicode to true on e.g. HFS+\n\n Will merge to 'next'.\n\n--------------------------------------------------\n[Stalled]\n\n* rj/read-default-config-in-show-ref-pack-refs (2013-06-17) 3 commits\n - ### DONTMERGE: needs better explanation on what config they need\n - pack-refs.c: Add missing call to git_config()\n - show-ref.c: Add missing call to git_config()\n\n The changes themselves are probably good, but it is unclear what\n basic setting needs to be read for which exact operation.\n\n Waiting for clarification.\n $gmane/228294\n\n\n* jh/shorten-refname (2013-05-07) 4 commits\n - t1514: refname shortening is done after dereferencing symbolic refs\n - shorten_unambiguous_ref(): Fix shortening refs/remotes/origin/HEAD to origin\n - t1514: Demonstrate failure to correctly shorten \"refs/remotes/origin/HEAD\"\n - t1514: Add tests of shortening refnames in strict/loose mode\n\n When remotes/origin/HEAD is not a symbolic ref, \"rev-parse\n --abbrev-ref remotes/origin/HEAD\" ought to show \"origin\", not\n \"origin/HEAD\", which is fixed with this series (if it is a symbolic\n ref that points at remotes/origin/something, then it should show\n \"origin/something\" and it already does).\n\n Expecting a reroll, as an early part of a larger series.\n $gmane/225137\n\n\n* jk/list-objects-sans-blobs (2013-06-06) 4 commits\n . archive: ignore blob objects when checking reachability\n . list-objects: optimize \"revs->blob_objects = 0\" case\n . upload-archive: restrict remote objects with reachability check\n . clear parsed flag when we free tree buffers\n\n Attempt to allow \"archive --remote=$there $arbitrary_sha1\" while\n keeping the reachability safety.\n\n Seems to break some tests in a trivial and obvious way.\n\n\n* mg/more-textconv (2013-05-10) 7 commits\n - grep: honor --textconv for the case rev:path\n - grep: allow to use textconv filters\n - t7008: demonstrate behavior of grep with textconv\n - cat-file: do not die on --textconv without textconv filters\n - show: honor --textconv for blobs\n - diff_opt: track whether flags have been set explicitly\n - t4030: demonstrate behavior of show with textconv\n\n Make \"git grep\" and \"git show\" pay attention to --textconv when\n dealing with blob objects.\n\n I thought this was pretty well designed and executed, but it seems\n there are some doubts on the list; kicked back to 'pu'.\n\n\n* jc/format-patch (2013-04-22) 2 commits\n - format-patch: --inline-single\n - format-patch: rename \"no_inline\" field\n\n A new option to send a single patch to the standard output to be\n appended at the bottom of a message.  I personally have no need for\n this, but it was easy enough to cobble together.  Tests, docs and\n stripping out more MIMEy stuff are left as exercises to interested\n parties.\n\n Not ready for inclusion.\n\n Will discard unless we hear from anybody who is interested in\n tying its loose ends.\n\n\n* jk/gitweb-utf8 (2013-04-08) 4 commits\n - gitweb: Fix broken blob action parameters on blob/commitdiff pages\n - gitweb: Don't append ';js=(0|1)' to external links\n - gitweb: Make feed title valid utf8\n - gitweb: Fix utf8 encoding for blob_plain, blobdiff_plain, commitdiff_plain, and patch\n\n Various fixes to gitweb.\n\n Drew Northup volunteered to take a look into this.\n $gmane/226216\n\n\n* jc/show-branch (2013-06-07) 5 commits\n - show-branch: use commit slab to represent bitflags of arbitrary width\n - show-branch.c: remove \"all_mask\"\n - show-branch.c: abstract out \"flags\" operation\n - show-branch.c: lift all_mask/all_revs to a global static\n - show-branch.c: update comment style\n\n Waiting for the final step to lift the hard-limit before sending it out.\n\n--------------------------------------------------\n[Cooking]\n\n* tf/gitweb-ss-tweak (2013-08-20) 4 commits\n  (merged to 'next' on 2013-08-22 at 3978368)\n + gitweb: make search help link less ugly\n + gitweb: omit the repository owner when it is unset\n + gitweb: vertically centre contents of page footer\n + gitweb: ensure OPML text fits inside its box\n\n Originally merged to 'next' on 2013-08-22\n\n Tweak Gitweb CSS to layout some elements better.\n\n Will cook in 'next'.\n\n\n* es/rebase-i-respect-core-commentchar (2013-08-18) 1 commit\n  (merged to 'next' on 2013-08-20 at 9f3d0c8)\n + rebase -i: fix cases ignoring core.commentchar\n\n Originally merged to 'next' on 2013-08-20\n\n Will cook in 'next'.\n\n\n* jx/branch-vv-always-compare-with-upstream (2013-08-26) 2 commits\n - status: always show tracking branch even no change\n - branch: report invalid tracking branch as gone\n\n \"git branch -v -v\" (and \"git status\") did not distinguish among a\n branch that does not build on any other branch, a branch that is in\n sync with the branch it builds on, and a branch that is configured\n to build on some other branch that no longer exists.\n\n Am I waiting for another reroll?\n\n\n* nd/fetch-into-shallow (2013-08-18) 6 commits\n - list-objects: mark more commits as edges in mark_edges_uninteresting\n - list-objects: reduce one argument in mark_edges_uninteresting\n - upload-pack: delegate rev walking in shallow fetch to pack-objects\n - shallow: add setup_temporary_shallow()\n - shallow: only add shallow graft points to new shallow file\n - move setup_alternate_shallow and write_shallow_commits to shallow.c\n\n When there is no sufficient overlap between old and new history\n during a fetch into a shallow repository, we unnecessarily sent\n objects the sending side knows the receiving end has.\n\n Will merge to 'next'.\n\n\n* sb/diff-delta-remove-needless-comparison (2013-08-18) 1 commit\n  (merged to 'next' on 2013-08-20 at c35730b)\n + create_delta_index: simplify condition always evaluating to true\n\n Originally merged to 'next' on 2013-08-20\n\n Will cook in 'next'.\n\n\n* sg/bash-prompt-lf-in-cwd-test (2013-08-18) 1 commit\n  (merged to 'next' on 2013-08-20 at 5544f62)\n + bash prompt: test the prompt with newline in repository path\n\n Originally merged to 'next' on 2013-08-20\n\n Will cook in 'next'.\n\n\n* jl/some-submodule-config-are-not-boolean (2013-08-19) 1 commit\n  (merged to 'next' on 2013-08-20 at 140c2e4)\n + avoid segfault on submodule.*.path set to an empty \"true\"\n\n Originally merged to 'next' on 2013-08-20\n\n Will cook in 'next'.\n\n\n* jc/ls-files-killed-optim (2013-08-23) 4 commits\n  (merged to 'next' on 2013-08-27 at f7687d1)\n + dir.c::test_one_path(): work around directory_exists_in_index_icase() breakage\n  (merged to 'next' on 2013-08-20 at d4bf1ad)\n + t3010: update to demonstrate \"ls-files -k\" optimization pitfalls\n + ls-files -k: a directory only can be killed if the index has a non-directory\n + dir.c: use the cache_* macro to access the current index\n\n Originally merged to 'next' on 2013-08-27\n\n \"git ls-files -k\" needs to crawl only the part of the working tree\n that may overlap the paths in the index to find killed files, but\n shared code with the logic to find all the untracked files, which\n made it unnecessarily inefficient.\n\n Will cook in 'next'.\n\n\n* es/blame-L-twice (2013-08-06) 16 commits\n  (merged to 'next' on 2013-08-08 at 9d6f821)\n + line-range: reject -L line numbers less than 1\n + t8001/t8002: blame: add tests of -L line numbers less than 1\n + line-range: teach -L^:RE to search from start of file\n + line-range: teach -L:RE to search from end of previous -L range\n + line-range: teach -L^/RE/ to search from start of file\n + line-range-format.txt: document -L/RE/ relative search\n + log: teach -L/RE/ to search from end of previous -L range\n + blame: teach -L/RE/ to search from end of previous -L range\n + line-range: teach -L/RE/ to search relative to anchor point\n + blame: document multiple -L support\n + t8001/t8002: blame: add tests of multiple -L options\n + blame: accept multiple -L ranges\n + blame: inline one-line function into its lone caller\n + range-set: publish API for re-use by git-blame -L\n + line-range-format.txt: clarify -L:regex usage form\n + git-log.txt: place each -L option variation on its own line\n (this branch is used by es/contacts-blame-L-multi; uses es/blame-L-more.)\n\n Originally merged to 'next' on 2013-08-08\n\n Teaches \"git blame\" to take more than one -L ranges.\n\n Will cook in 'next'.\n\n\n* mm/no-shell-escape-in-die-message (2013-08-07) 1 commit\n  (merged to 'next' on 2013-08-08 at bddff86)\n + die_with_status: use \"printf '%s\\n'\", not \"echo\"\n\n Originally merged to 'next' on 2013-08-08\n\n Fixes a minor bug in \"git rebase -i\" (there could be others, as the\n root cause is pretty generic) where the code feeds a random, data\n dependeant string to 'echo' and expects it to come out literally.\n\n Will cook in 'next'.\n\n\n* tr/fd-gotcha-fixes (2013-08-06) 1 commit\n  (merged to 'next' on 2013-08-08 at f452c5b)\n + t0070: test that git_mkstemps correctly checks return value of open()\n\n Originally merged to 'next' on 2013-08-08\n\n Finishing touches to an earlier fix already in 'master'.\n\n Will cook in 'next'.\n\n\n* ap/remote-hg-tilde-is-home-directory (2013-08-09) 1 commit\n  (merged to 'next' on 2013-08-14 at cd963e3)\n + remote-hg: fix path when cloning with tilde expansion\n\n Originally merged to 'next' on 2013-08-14\n\n Will cook in 'next'.\n\n\n* es/rebase-i-no-abbrev (2013-08-25) 3 commits\n  (merged to 'next' on 2013-08-26 at 1c3d45d)\n + rebase -i: fix short SHA-1 collision\n + t3404: rebase -i: demonstrate short SHA-1 collision\n + t3404: make tests more self-contained\n\n Originally merged to 'next' on 2013-08-26\n\n Will cook in 'next'.\n\n\n* fc/remote-hg-shared-setup (2013-08-11) 2 commits\n  (merged to 'next' on 2013-08-14 at aae6858)\n + remote-hg: add shared repo upgrade\n + remote-hg: ensure shared repo is initialized\n\n Originally merged to 'next' on 2013-08-14\n\n Will cook in 'next'.\n\n\n* jc/transport-do-not-use-connect-twice-in-fetch (2013-08-07) 5 commits\n  (merged to 'next' on 2013-08-20 at 5414722)\n + fetch: work around \"transport-take-over\" hack\n + fetch: refactor code that fetches leftover tags\n + fetch: refactor code that prepares a transport\n + fetch: rename file-scope global \"transport\" to \"gtransport\"\n + t5802: add test for connect helper\n\n Originally merged to 'next' on 2013-08-20\n\n The auto-tag-following code in \"git fetch\" tries to reuse the same\n transport twice when the serving end does not cooperate and does\n not give tags that point to commits that are asked for as part of\n the primary transfer.  Unfortunately, Git-aware transport helper\n interface is not designed to be used more than once, hence this\n does not work over smart-http transfer.\n\n Will cook in 'next'.\n\n\n* ks/p4-view-spec (2013-08-11) 3 commits\n - WAITING FOR ACK\n - git p4: implement view spec wildcards with \"p4 where\"\n - git p4 test: sanitize P4CHARSET\n\n Waiting for an ack.\n\n\n* mm/war-on-whatchanged (2013-08-13) 2 commits\n  (merged to 'next' on 2013-08-14 at fe77c11)\n + whatchanged: document its historical nature\n + core-tutorial: trim the section on Inspecting Changes\n\n Originally merged to 'next' on 2013-08-14\n\n Will cook in 'next'.\n\n\n* nd/gc-lock-against-each-other (2013-08-09) 1 commit\n  (merged to 'next' on 2013-08-14 at 6999651)\n + gc: reject if another gc is running, unless --force is given\n\n Originally merged to 'next' on 2013-08-14\n\n Will cook in 'next'.\n\n\n* rt/doc-merge-file-diff3 (2013-08-09) 1 commit\n  (merged to 'next' on 2013-08-14 at 1e5847b)\n + Documentation/git-merge-file: document option \"--diff3\"\n\n Originally merged to 'next' on 2013-08-14\n\n Will cook in 'next'.\n\n\n* sb/misc-cleanup (2013-08-09) 3 commits\n  (merged to 'next' on 2013-08-14 at 9e7ff9a)\n + rm: remove unneeded null pointer check\n + diff: fix a possible null pointer dereference\n + diff: remove ternary operator evaluating always to true\n\n Originally merged to 'next' on 2013-08-14\n\n Will cook in 'next'.\n\n\n* aj/p4-symlink-lose-nl (2013-08-12) 1 commit\n  (merged to 'next' on 2013-08-14 at a4959b7)\n + git-p4: Fix occasional truncation of symlink contents.\n\n Originally merged to 'next' on 2013-08-14\n\n Will cook in 'next'.\n\n\n* es/contacts-blame-L-multi (2013-08-13) 3 commits\n  (merged to 'next' on 2013-08-14 at 551ed3d)\n + contacts: reduce git-blame invocations\n + contacts: gather all blame sources prior to invoking git-blame\n + contacts: validate hunk length earlier\n (this branch uses es/blame-L-more and es/blame-L-twice.)\n\n Originally merged to 'next' on 2013-08-14\n\n Will cook in 'next'.\n\n\n* fc/unpack-trees-leakfix (2013-08-13) 1 commit\n  (merged to 'next' on 2013-08-14 at 989f329)\n + unpack-trees: plug a memory leak\n\n Originally merged to 'next' on 2013-08-14\n\n Will cook in 'next'.\n\n\n* nd/push-no-thin (2013-08-13) 1 commit\n  (merged to 'next' on 2013-08-14 at b2d0fa9)\n + push: respect --no-thin\n\n Originally merged to 'next' on 2013-08-14\n\n Will cook in 'next'.\n\n\n* sh/pull-rebase-preserve (2013-08-13) 1 commit\n  (merged to 'next' on 2013-08-14 at 99a85dd)\n + pull: Allow pull to preserve merges when rebasing.\n\n Originally merged to 'next' on 2013-08-14\n\n Will cook in 'next'.\n\n\n* es/blame-L-more (2013-08-05) 11 commits\n  (merged to 'next' on 2013-08-06 at 2679f3e)\n + blame: reject empty ranges -L,+0 and -L,-0\n + t8001/t8002: blame: demonstrate acceptance of bogus -L,+0 and -L,-0\n + blame: reject empty ranges -LX,+0 and -LX,-0\n + t8001/t8002: blame: demonstrate acceptance of bogus -LX,+0 and -LX,-0\n + log: fix -L bounds checking bug\n + t4211: retire soon-to-be unimplementable tests\n + t4211: log: demonstrate -L bounds checking bug\n + blame: fix -L bounds checking bug\n + t8001/t8002: blame: add empty file & partial-line tests\n + t8001/t8002: blame: demonstrate -L bounds checking bug\n + t8001/t8002: blame: decompose overly-large test\n (this branch is used by es/blame-L-twice and es/contacts-blame-L-multi.)\n\n Originally merged to 'next' on 2013-08-06\n\n More fixes to the code to parse the \"-L\" option in \"log\" and \"blame\".\n\n Will cook in 'next'.\n\n\n* jk/cat-file-batch-optim (2013-08-05) 1 commit\n  (merged to 'next' on 2013-08-06 at cd48621)\n + cat-file: only split on whitespace when %(rest) is used\n\n Originally merged to 'next' on 2013-08-06\n\n Rework the reverted change to `cat-file --batch-check`.\n\n Will cook in 'next'.\n\n\n* jn/post-receive-utf8 (2013-08-05) 3 commits\n  (merged to 'next' on 2013-08-20 at 1623e8f)\n + hooks/post-receive-email: set declared encoding to utf-8\n + hooks/post-receive-email: force log messages in UTF-8\n + hooks/post-receive-email: use plumbing instead of git log/show\n\n Originally merged to 'next' on 2013-08-20\n\n Update post-receive-email script to make sure the message contents\n and pathnames are encoded consistently in UTF-8.\n\n I have a feeling that it is a lost cause to solve the issue the\n topic tries to address in general, because the patch text can have\n payload in any encodings that are different from either the\n pathnames or the log message.  Patches that touch paths that use an\n encoding that conflicts with the encoding of the payload and/or the\n log message could be transferred with core.quotepath set and patch\n generated as all binary, but that would be pretty much useless.\n\n Will cook in 'next'.\n\n\n* sb/parseopt-boolean-removal (2013-08-07) 9 commits\n  (merged to 'next' on 2013-08-08 at b138a2d)\n + revert: use the OPT_CMDMODE for parsing, reducing code\n + checkout-index: fix negations of even numbers of -n\n + config parsing options: allow one flag multiple times\n + hash-object: replace stdin parsing OPT_BOOLEAN by OPT_COUNTUP\n + branch, commit, name-rev: ease up boolean conditions\n + checkout: remove superfluous local variable\n + log, format-patch: parsing uses OPT__QUIET\n + Replace deprecated OPT_BOOLEAN by OPT_BOOL\n + Remove deprecated OPTION_BOOLEAN for parsing arguments\n (this branch uses jc/parseopt-command-modes.)\n\n Originally merged to 'next' on 2013-08-08\n\n Convert most uses of OPT_BOOLEAN/OPTION_BOOLEAN that can use\n OPT_BOOL/OPTION_BOOLEAN which have much saner semantics, and turn\n remaining ones into OPT_SET_INT, OPT_COUNTUP, etc. as necessary.\n\n Will cook in 'next'.\n\n\n* mb/docs-favor-en-us (2013-08-01) 1 commit\n  (merged to 'next' on 2013-08-06 at 763d868)\n + Provide some linguistic guidance for the documentation.\n\n Originally merged to 'next' on 2013-08-06\n\n Declare that the official grammar & spelling of the source of this\n project is en_US, but strongly discourage patches only to \"fix\"\n existing en_UK strings to avoid unnecessary churns.\n\n Will cook in 'next'.\n\n\n* jc/parseopt-command-modes (2013-07-30) 2 commits\n  (merged to 'next' on 2013-08-08 at 43abba9)\n + tag: use OPT_CMDMODE\n + parse-options: add OPT_CMDMODE()\n (this branch is used by sb/parseopt-boolean-removal.)\n\n Originally merged to 'next' on 2013-08-08\n\n Many commands use --dashed-option as a operation mode selector\n (e.g. \"git tag --delete\") that the user can use at most one\n (e.g. \"git tag --delete --verify\" is a nonsense) and you cannot\n negate (e.g. \"git tag --no-delete\" is a nonsense).  Make it easier\n for users of parse_options() to enforce these restrictions.\n\n Will cook in 'next'.\n\n\n* tr/log-full-diff-keep-true-parents (2013-08-05) 2 commits\n  (merged to 'next' on 2013-08-08 at 2fb633f)\n + log: use true parents for diff when walking reflogs\n + log: use true parents for diff even when rewriting\n\n Originally merged to 'next' on 2013-08-08\n\n Output from \"git log --full-diff -- <pathspec>\" looked strange,\n because comparison was done with the previous ancestor that touched\n the specified <pathspec>, causing the patches for paths outside the\n pathspec to show more than the single commit has changed.\n\n Tweak \"git reflog -p\" for the same reason using the same mechanism.\n\n Will cook in 'next'.\n\n\n* bc/unuse-packfile (2013-08-02) 2 commits\n  (merged to 'next' on 2013-08-06 at 01aa4fd)\n + Don't close pack fd when free'ing pack windows\n + sha1_file: introduce close_one_pack() to close packs on fd pressure\n\n Originally merged to 'next' on 2013-08-06\n\n Handle memory pressure and file descriptor pressure separately when\n deciding to release pack windows to honor resource limits.\n\n Will cook in 'next'.\n\n\n* da/darwin (2013-08-05) 3 commits\n  (merged to 'next' on 2013-08-05 at 5c54cfa)\n + OS X: Fix redeclaration of die warning\n + Makefile: Fix APPLE_COMMON_CRYPTO with BLK_SHA1\n  (merged to 'next' on 2013-08-01 at d151a94)\n + imap-send: use Apple's Security framework for base64 encoding\n\n Originally merged to 'next' on 2013-08-05\n\n Will cook in 'next'.\n\n\n* nd/sq-quote-buf (2013-07-30) 3 commits\n  (merged to 'next' on 2013-08-01 at dc7934a)\n + quote: remove sq_quote_print()\n + tar-tree: remove dependency on sq_quote_print()\n + for-each-ref, quote: convert *_quote_print -> *_quote_buf\n\n Originally merged to 'next' on 2013-08-01\n\n Code simplification as a preparatory step to something larger.\n\n Will cook in 'next'.\n\n\n* jc/url-match (2013-08-09) 7 commits\n  (merged to 'next' on 2013-08-12 at 381d358)\n + builtin/config.c: compilation fix\n  (merged to 'next' on 2013-08-08 at c1ee470)\n + config: \"git config --get-urlmatch\" parses section.<url>.key\n + builtin/config: refactor collect_config()\n + config: parse http.<url>.<variable> using urlmatch\n + config: add generic callback wrapper to parse section.<url>.key\n + config: add helper to normalize and match URLs\n + http.c: fix parsing of http.sslCertPasswordProtected variable\n\n Originally merged to 'next' on 2013-08-12\n\n Allow section.<urlpattern>.var configuration variables to be\n treated as a \"virtual\" section.var given a URL, and use the\n mechanism to enhance http.* configuration variables.\n\n This is a reroll of Kyle J. McKay's work.\n\n Will cook in 'next'.\n\n\n* jl/submodule-mv (2013-08-06) 5 commits\n  (merged to 'next' on 2013-08-08 at 1a57bdf)\n + rm: delete .gitmodules entry of submodules removed from the work tree\n + mv: update the path entry in .gitmodules for moved submodules\n + submodule.c: add .gitmodules staging helper functions\n + mv: move submodules using a gitfile\n + mv: move submodules together with their work trees\n (this branch uses nd/magic-pathspec.)\n\n Originally merged to 'next' on 2013-08-08\n\n \"git mv A B\" when moving a submodule A does \"the right thing\",\n inclusing relocating its working tree and adjusting the paths in\n the .gitmodules file.\n\n Will cook in 'next'.\n\n\n* nd/clone-connectivity-shortcut (2013-07-23) 1 commit\n  (merged to 'next' on 2013-07-30 at 8d16721)\n + smart http: use the same connectivity check on cloning\n\n Originally merged to 'next' on 2013-07-30\n\n Will cook in 'next'.\n\n\n* db/http-savecookies (2013-08-05) 2 commits\n  (merged to 'next' on 2013-08-05 at be6f3a3)\n + t5551: Remove header from curl cookie file\n  (merged to 'next' on 2013-07-30 at d2ad6e5)\n + http: add http.savecookies option to write out HTTP cookies\n\n Originally merged to 'next' on 2013-08-05\n\n Will cook in 'next'.\n\n\n* sb/fsck-opt-bool (2013-07-29) 1 commit\n  (merged to 'next' on 2013-07-30 at 8a9964c)\n + fsck: Replace deprecated OPT_BOOLEAN by OPT_BOOL\n\n Originally merged to 'next' on 2013-07-30\n\n Will drop.\n\n sb/parseopt-boolean-removal contains the moral equivalent of this\n change.\n\n\n* rr/feed-real-path-to-editor (2013-07-29) 1 commit\n  (merged to 'next' on 2013-07-30 at 72c9bcb)\n + editor: use canonicalized absolute path\n\n Originally merged to 'next' on 2013-07-30\n\n Will cook in 'next'.\n\n\n* rj/doc-rev-parse (2013-07-22) 2 commits\n  (merged to 'next' on 2013-07-22 at 8188667)\n + rev-parse(1): logically group options\n + rev-parse: remove restrictions on some options\n\n Originally merged to 'next' on 2013-07-22\n\n Will cook in 'next'.\n\n\n* jk/fast-import-empty-ls (2013-06-23) 4 commits\n  (merged to 'next' on 2013-07-22 at d26dab6)\n + fast-import: allow moving the root tree\n + fast-import: allow ls or filecopy of the root tree\n + fast-import: set valid mode on root tree in \"ls\"\n + t9300: document fast-import empty path issues\n\n Originally merged to 'next' on 2013-07-22\n\n Will cook in 'next'.\n\n\n* jc/diff-filter-negation (2013-07-19) 6 commits\n  (merged to 'next' on 2013-07-22 at 80d167d)\n + diff: deprecate -q option to diff-files\n + diff: allow lowercase letter to specify what change class to exclude\n + diff: reject unknown change class given to --diff-filter\n + diff: preparse --diff-filter string argument\n + diff: factor out match_filter()\n + diff: pass the whole diff_options to diffcore_apply_filter()\n (this branch is used by jc/hold-diff-remove-q-synonym-for-no-deletion.)\n\n Originally merged to 'next' on 2013-07-22\n\n Teach \"git diff --diff-filter\" to express \"I do not want to see\n these classes of changes\" more directly by listing only the\n unwanted ones in lowercase (e.g. \"--diff-filter=d\" will show\n everything but deletion) and deprecate \"diff-files -q\" which did\n the same thing as \"--diff-filter=d\".\n\n Will cook in 'next'.\n\n\n* nd/magic-pathspec (2013-07-23) 48 commits\n  (merged to 'next' on 2013-07-23 at 784c3a1)\n + rm: do not set a variable twice without intermediate reading.\n  (merged to 'next' on 2013-07-22 at 39f7194)\n + t6131 - skip tests if on case-insensitive file system\n + parse_pathspec: accept :(icase)path syntax\n + pathspec: support :(glob) syntax\n + pathspec: make --literal-pathspecs disable pathspec magic\n + pathspec: support :(literal) syntax for noglob pathspec\n + kill limit_pathspec_to_literal() as it's only used by parse_pathspec()\n + parse_pathspec: preserve prefix length via PATHSPEC_PREFIX_ORIGIN\n + parse_pathspec: make sure the prefix part is wildcard-free\n + rename field \"raw\" to \"_raw\" in struct pathspec\n + tree-diff: remove the use of pathspec's raw[] in follow-rename codepath\n + remove match_pathspec() in favor of match_pathspec_depth()\n + remove init_pathspec() in favor of parse_pathspec()\n + remove diff_tree_{setup,release}_paths\n + convert common_prefix() to use struct pathspec\n + convert add_files_to_cache to take struct pathspec\n + convert {read,fill}_directory to take struct pathspec\n + convert refresh_index to take struct pathspec\n + convert report_path_error to take struct pathspec\n + checkout: convert read_tree_some to take struct pathspec\n + convert unmerge_cache to take struct pathspec\n + convert run_add_interactive to use struct pathspec\n + convert read_cache_preload() to take struct pathspec\n + line-log: convert to use parse_pathspec\n + reset: convert to use parse_pathspec\n + add: convert to use parse_pathspec\n + check-ignore: convert to use parse_pathspec\n + archive: convert to use parse_pathspec\n + ls-files: convert to use parse_pathspec\n + rm: convert to use parse_pathspec\n + checkout: convert to use parse_pathspec\n + rerere: convert to use parse_pathspec\n + status: convert to use parse_pathspec\n + commit: convert to use parse_pathspec\n + clean: convert to use parse_pathspec\n + guard against new pathspec magic in pathspec matching code\n + parse_pathspec: support prefixing original patterns\n + parse_pathspec: support stripping/checking submodule paths\n + parse_pathspec: support stripping submodule trailing slashes\n + parse_pathspec: add special flag for max_depth feature\n + convert some get_pathspec() calls to parse_pathspec()\n + parse_pathspec: add PATHSPEC_PREFER_{CWD,FULL} flags\n + parse_pathspec: save original pathspec for reporting\n + add parse_pathspec() that converts cmdline args to struct pathspec\n + pathspec: add copy_pathspec\n + pathspec: i18n-ize error strings in pathspec parsing code\n + move struct pathspec and related functions to pathspec.[ch]\n + clean: remove unused variable \"seen\"\n (this branch is used by jl/submodule-mv.)\n\n Originally merged to 'next' on 2013-07-23\n\n Use \"struct pathspec\" interface in more places, instead of array of\n characters, the latter of which cannot express magic pathspecs\n (e.g. \":(icase)makefile\" that matches both Makefile and makefile).\n\n Will cook in 'next'.\n\n\n* jc/check-x-z (2013-07-11) 4 commits\n  (merged to 'next' on 2013-07-15 at 62751bc)\n + check-attr -z: a single -z should apply to both input and output\n + check-ignore -z: a single -z should apply to both input and output\n + check-attr: the name of the character is NUL, not NULL\n + check-ignore: the name of the character is NUL, not NULL\n\n Originally merged to 'next' on 2013-07-15\n\n \"git check-ignore -z\" applied the NUL termination to both its input\n (with --stdin) and its output, but \"git check-attr -z\" ignored the\n option on the output side.\n\n This is potentially a backward incompatible fix.  I am tempted to\n merge this to and keep it in 'next' for a while to see if anybody\n screams before deciding if we want to do anything to help existing\n users (there may be none).\n\n Will cook in 'next'.\n\n\n* ms/fetch-prune-configuration (2013-07-18) 1 commit\n  (merged to 'next' on 2013-07-21 at 7ffb690)\n + fetch: make --prune configurable\n\n Originally merged to 'next' on 2013-07-21\n\n Allow fetch.prune and remote.*.prune configuration variables to be set,\n and \"git fetch\" to behave as if \"--prune\" is given.\n\n \"git fetch\" that honors remote.*.prune is fine, but I wonder if we\n should somehow make \"git push\" aware of it as well.  Perhaps\n remote.*.prune should not be just a boolean, but a 4-way \"none\",\n \"push\", \"fetch\", \"both\"?\n\n Will cook in 'next'.\n\n\n* km/svn-1.8-serf-only (2013-07-18) 3 commits\n  (merged to 'next' on 2013-07-19 at ab8d034)\n + Git.pm: revert _temp_cache use of temp_is_locked\n  (merged to 'next' on 2013-07-12 at 70e0b95)\n + git-svn: allow git-svn fetching to work using serf\n + Git.pm: add new temp_is_locked function\n\n Originally merged to 'next' on 2013-07-19\n\n Subversion 1.8.0 that was recently released breaks older subversion\n clients coming over http/https in various ways.\n\n Will cook in 'next'.\n\n\n* tr/merge-recursive-index-only (2013-07-07) 3 commits\n - merge-recursive: -Xindex-only to leave worktree unchanged\n - merge-recursive: untangle double meaning of o->call_depth\n - merge-recursive: remove dead conditional in update_stages()\n\n Will hold, until we get any user.\n\n\n* jc/push-cas (2013-08-02) 9 commits\n  (merged to 'next' on 2013-08-05 at 34f4cf3)\n + push: teach --force-with-lease to smart-http transport\n + send-pack: fix parsing of --force-with-lease option\n  (merged to 'next' on 2013-08-01 at a7604ee)\n + t5540/5541: smart-http does not support \"--force-with-lease\"\n  (merged to 'next' on 2013-07-23 at 72371b0)\n + t5533: test \"push --force-with-lease\"\n + push --force-with-lease: tie it all together\n + push --force-with-lease: implement logic to populate old_sha1_expect[]\n + remote.c: add command line option parser for \"--force-with-lease\"\n + builtin/push.c: use OPT_BOOL, not OPT_BOOLEAN\n + cache.h: move remote/connect API out of it\n\n Originally merged to 'next' on 2013-08-05\n\n Allow a safer \"rewind of the remote tip\" push than blind \"--force\",\n by requiring that the overwritten remote ref to be unchanged since\n the new history to replace it was prepared.\n\n The machinery is more or less ready.  The \"--force\" option is again\n the big red button to override any safety, thanks to J6t's sanity\n (the original round allowed --lockref to defeat --force).\n\n The logic to choose the default implemented here is fragile\n (e.g. \"git fetch\" after seeing a failure will update the\n remote-tracking branch and will make the next \"push\" pass,\n defeating the safety pretty easily).  It is suitable only for the\n simplest workflows, and it may hurt users more than it helps them.\n\n Will cook in 'next'.\n\n\n* jc/pull-training-wheel (2013-07-19) 1 commit\n  (merged to 'next' on 2013-07-22 at 748180e)\n + pull: require choice between rebase/merge on non-fast-forward pull\n\n Originally merged to 'next' on 2013-07-22\n\n Make \"git pull\" (without arguments that say what branch to\n integrate from where) refuse with \"it does not fast forward; choose\n between 'pull --merge' and 'pull --rebase'\".\n\n See\n\n http://thread.gmane.org/gmane.comp.version-control.git/225146/focus=225326\n http://thread.gmane.org/gmane.comp.version-control.git/225146/focus=229221\n\n for full discussion.\n\n Will cook in 'next'.\n\n\n* jc/reflog-doc (2013-06-19) 1 commit\n - setup_reflog_action: document the rules for using GIT_REFLOG_ACTION\n\n Document rules to use GIT_REFLOG_ACTION variable in the scripted\n Porcelain.  git-rebase--interactive locally violates this, but it\n is a leaf user that does not call out to or dot-sources other\n scripts, so fixing it is not all that urgent.\n\n\n* jn/add-2.0-u-A-sans-pathspec (2013-04-26) 1 commit\n - git add: -u/-A now affects the entire working tree\n\n Will cook in 'next' until Git 2.0.\n\n\n* jc/core-checkstat-2.0 (2013-05-06) 1 commit\n - core.statinfo: remove as promised in Git 2.0\n\n Will cook in 'next' until Git 2.0.\n\n\n* jc/push-2.0-default-to-simple (2013-06-18) 1 commit\n - push: switch default from \"matching\" to \"simple\"\n\n Will cook in 'next' until Git 2.0.\n\n\n* jc/add-2.0-ignore-removal (2013-04-22) 1 commit\n - git add <pathspec>... defaults to \"-A\"\n\n Updated endgame for \"git add <pathspec>\" that defaults to \"--all\"\n aka \"--no-ignore-removal\".\n\n Will cook in 'next' until Git 2.0.\n\n\n* jc/hold-diff-remove-q-synonym-for-no-deletion (2013-07-19) 1 commit\n - diff: remove \"diff-files -q\" in a version of Git in a distant future\n (this branch uses jc/diff-filter-negation.)\n\n Will cook in 'next' until a distant future.\n\n--------------------------------------------------\n[Discarded]\n\n* jh/remote-hg-fetch-fix (2013-07-25) 2 commits\n  (merged to 'next' on 2013-07-25 at 33161ad)\n + Revert \"remotes-hg: bugfix for fetching non local remotes\"\n  (merged to 'next' on 2013-07-24 at 9c96641)\n + remotes-hg: bugfix for fetching non local remotes\n\n Originally merged to 'next' on 2013-07-25\n\n Reverted.\n\n Waiting for the final patch to replace, after discussion settles.\n\n\n* km/http-curl-config-per-url (2013-07-31) 7 commits\n  (merged to 'next' on 2013-07-31 at e88aad5)\n + Revert km/http-curl-config-per-url in its entirety\n  (merged to 'next' on 2013-07-25 at 531e8cb)\n + docs: update http.<url>.* options documentation\n  (merged to 'next' on 2013-07-23 at f84689b)\n + config: allow http.<url>.* any user matching\n + tests: add new test for the url_normalize function\n + config: improve support for http.<url>.* settings\n + config: add support for http.<url>.* settings\n + http.c: fix parsing of http.sslCertPasswordProtected variable\n\n Originally merged to 'next' on 2013-07-31\n\n Reverted to be replaced with a reroll (jc/url-match topic).\n"},{"id":"226063","messageId":"20130827205125.GA23783@sigill.intra.peff.net","threadId":"34784","inReplyTo":"xmqqsixvaqh5.fsf@gitster.dls.corp.google.com","subject":"Re: What's cooking in git.git (Aug 2013, #06; Tue, 27)","fromName":"Jeff King","fromEmail":"peff@peff.net","sentAt":"2013-08-27T20:51:25Z","receivedAt":"2013-08-27T20:51:25Z","isPatch":false,"sender":{"key":"peff@peff.net","avatar":"https://avatars.githubusercontent.com/u/45925?v=4"},"body":"On Tue, Aug 27, 2013 at 12:22:30PM -0700, Junio C Hamano wrote:\n\n> * jk/config-int-range-check (2013-08-21) 2 commits\n>   (merged to 'next' on 2013-08-22 at 465efb3)\n>  + teach git-config to output large integers\n>  + config: properly range-check integer values\n> \n>  Originally merged to 'next' on 2013-08-22\n> \n>  \"git config --int section.var 3g\" should somehow diagnose that the\n>  number does not fit in \"int\" (on 32-bit platforms anyway) but it\n>  did not.\n> \n>  Will cook in 'next'.\n\nI think Jonathan had some concerns about the test in the first one, and\nthere was an open question in the second of whether we wanted to add\nsomething like --ulong, call it something more agnostic like\n--file-size, or simply teach --int to use 64-bit integers everywhere for\nsimplicity.\n\nThoughts?\n\n> * jk/mailmap-incomplete-line (2013-08-25) 2 commits\n>  - mailmap: avoid allocation when reading from blob\n>  - mailmap: handle mailmap blobs without trailing newlines\n> \n>  Will merge to 'next'.\n\nDid you want me to squash these? The second one more or less eradicates\nthe changes made to the first one. I mainly did them separately in case\nwe were going to only do the first half on maint.\n\n> * jk/write-broken-index-with-nul-sha1 (2013-08-26) 1 commit\n>  - write_index: optionally allow broken null sha1s\n> \n>  Am I waiting for another reroll?\n\nYep, just sent v3.\n\n> [Stalled]\n> [...]\n> * jk/list-objects-sans-blobs (2013-06-06) 4 commits\n>  . archive: ignore blob objects when checking reachability\n>  . list-objects: optimize \"revs->blob_objects = 0\" case\n>  . upload-archive: restrict remote objects with reachability check\n>  . clear parsed flag when we free tree buffers\n> \n>  Attempt to allow \"archive --remote=$there $arbitrary_sha1\" while\n>  keeping the reachability safety.\n> \n>  Seems to break some tests in a trivial and obvious way.\n\nYou can probably discard this one (though you may want to take the\nbottom as a separate cleanup). I think we decided that the right\nstrategy is to do the \":\" split as we do now, but then do the normal\ncommit-level reachability check on the left-hand side. I just haven't\ngotten around to writing the code yet.\n\n-Peff\n"},{"id":"226064","messageId":"xmqqbo4ic0ap.fsf@gitster.dls.corp.google.com","threadId":"34784","inReplyTo":"20130827205125.GA23783@sigill.intra.peff.net","subject":"Re: What's cooking in git.git (Aug 2013, #06; Tue, 27)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2013-08-27T21:05:01Z","receivedAt":"2013-08-27T21:05:01Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Jeff King <peff@peff.net> writes:\n\n> On Tue, Aug 27, 2013 at 12:22:30PM -0700, Junio C Hamano wrote:\n>\n>> * jk/config-int-range-check (2013-08-21) 2 commits\n>>   (merged to 'next' on 2013-08-22 at 465efb3)\n>>  + teach git-config to output large integers\n>>  + config: properly range-check integer values\n>> \n>>  Originally merged to 'next' on 2013-08-22\n>> \n>>  \"git config --int section.var 3g\" should somehow diagnose that the\n>>  number does not fit in \"int\" (on 32-bit platforms anyway) but it\n>>  did not.\n>> \n>>  Will cook in 'next'.\n>\n> I think Jonathan had some concerns about the test in the first one, and\n> there was an open question in the second of whether we wanted to add\n> something like --ulong, call it something more agnostic like\n> --file-size, or simply teach --int to use 64-bit integers everywhere for\n> simplicity.\n>\n> Thoughts?\n\nAre the scripts that use \"git config --<type>\" expected to know the\nrepresentation type used by C binaries on the platform?  If so,\nletting them say \"git config --ulong 3g\" when setting a new value,\nand \"git config --ulong\" when asking the current value with range\nchecking does make sense.  When the underlying code uses \"int\" (as\nopposed to \"int32_t\") to read the value for a variable on any\nplatform, then \"git config --int 3g\" that does not warn only because\nit is running on 64-bit platform may not help very much.  The users\ncan protect themselves by learning to use \"config --int32 3g\", but I\nam not sure that is a sensible approach---rather, \"config --int\"\nthat makes sure that the current value or the value being set is\nwithin range on any sensible platform may be a lot more user-friendly.\n\n>> * jk/mailmap-incomplete-line (2013-08-25) 2 commits\n>>  - mailmap: avoid allocation when reading from blob\n>>  - mailmap: handle mailmap blobs without trailing newlines\n>> \n>>  Will merge to 'next'.\n>\n> Did you want me to squash these? The second one more or less eradicates\n> the changes made to the first one. I mainly did them separately in case\n> we were going to only do the first half on maint.\n\nHmm, perhaps.  Is reading mailmap from a blob commonly done and\ndeserves a maint update down for 1.8.3/1.8.2 series?\n\nI'll be rewinding the 'next' soonish (either tomorrow or Thursday),\nso I'll try to remember not to merge this (yet).\n\n>> * jk/write-broken-index-with-nul-sha1 (2013-08-26) 1 commit\n>>  - write_index: optionally allow broken null sha1s\n>> \n>>  Am I waiting for another reroll?\n>\n> Yep, just sent v3.\n\nThanks.\n"},{"id":"226066","messageId":"CALWbr2wrHsa7LwZcEPYKe=kCrSHKsdrU6Bwevtb=2MwLrXtMnw@mail.gmail.com","threadId":"34784","inReplyTo":"xmqqsixvaqh5.fsf@gitster.dls.corp.google.com","subject":"Re: What's cooking in git.git (Aug 2013, #06; Tue, 27)","fromName":"Antoine Pelisse","fromEmail":"apelisse@gmail.com","sentAt":"2013-08-27T21:25:15Z","receivedAt":"2013-08-27T21:25:15Z","isPatch":false,"sender":{"key":"apelisse@gmail.com","avatar":"https://avatars.githubusercontent.com/u/1929644?v=4"},"body":"On Tue, Aug 27, 2013 at 9:22 PM, Junio C Hamano <gitster@pobox.com> wrote:\n> * jh/remote-hg-fetch-fix (2013-07-25) 2 commits\n>   (merged to 'next' on 2013-07-25 at 33161ad)\n>  + Revert \"remotes-hg: bugfix for fetching non local remotes\"\n>   (merged to 'next' on 2013-07-24 at 9c96641)\n>  + remotes-hg: bugfix for fetching non local remotes\n>\n>  Originally merged to 'next' on 2013-07-25\n>\n>  Reverted.\n>\n>  Waiting for the final patch to replace, after discussion settles.\n\nI think it has already been replaced by:\n\n> * fc/remote-hg-shared-setup (2013-08-11) 2 commits\n>   (merged to 'next' on 2013-08-14 at aae6858)\n>  + remote-hg: add shared repo upgrade\n>  + remote-hg: ensure shared repo is initialized\n>\n>  Originally merged to 'next' on 2013-08-14\n>\n>  Will cook in 'next'.\n"},{"id":"226068","messageId":"20130827214808.GA26350@sigill.intra.peff.net","threadId":"34784","inReplyTo":"xmqqbo4ic0ap.fsf@gitster.dls.corp.google.com","subject":"Re: What's cooking in git.git (Aug 2013, #06; Tue, 27)","fromName":"Jeff King","fromEmail":"peff@peff.net","sentAt":"2013-08-27T21:48:08Z","receivedAt":"2013-08-27T21:48:08Z","isPatch":false,"sender":{"key":"peff@peff.net","avatar":"https://avatars.githubusercontent.com/u/45925?v=4"},"body":"[+cc Jonathan]\n\nOn Tue, Aug 27, 2013 at 02:05:01PM -0700, Junio C Hamano wrote:\n\n> >> * jk/config-int-range-check (2013-08-21) 2 commits\n> [...]\n> >\n> > I think Jonathan had some concerns about the test in the first one, and\n> > there was an open question in the second of whether we wanted to add\n> > something like --ulong, call it something more agnostic like\n> > --file-size, or simply teach --int to use 64-bit integers everywhere for\n> > simplicity.\n> \n> Are the scripts that use \"git config --<type>\" expected to know the\n> representation type used by C binaries on the platform?\n\nI think that's an open question. My argument was that you would want to\nbe able to get the same range errors that git would see internally. So I\nknow that if \"git config --ulong pack.packSizeLimit 10g\" does not fail,\nthen pack-objects itself will be fine when reading the value. So if you\nbuy the argument that such a thing is useful, then:\n\n  1. We would want to keep the range checks we have for --int.\n\n  2. We would need new types to represent items beyond --int, and they\n     should match what git will do internally.  You can call it --ulong,\n     or --uint64, or --file-size, or whatever you like, but --int does\n     not cut it.\n\nThe counterarguments I can see are:\n\n  1. Who cares? If you want to know whether pack-objects will choke on\n     your huge config value, then run pack-objects.\n\n  2. Such a check would involve knowing which type we use internally to\n     look at packSizeLimit, and that is utterly undocumented (and\n     subject to change; e.g., it seems kind of senseless that we have a\n     4G pack-size limit on 32-bit platforms, and we may want to fix\n     that).\n\nSo if you do not buy the argument that communicating git's internal\nrange checks is useful, then we can simply say \"--int is magically long\non every platform, and you can use it for everything numeric\". And\nimplement it with int64_t. You may be able to read or write some values\nfor certain keys that git will barf on internally, but that is git's\nproblem.\n\nThe one thing it doesn't get you is that you can currently set unsigned\nvalues to \"-1\" in the config to have them treated as ULONG_MAX. This is\nundocumented and as far as I know not used by anyone. But it would be\none place where the interpretation of \"git config key\" is not the same\nas what git does internally, and you do not get a warning or death, but\nrather you just get a completely different value.\n\nI don't feel too strongly either way. I mostly kept the range checks for\n--int because that is how the code already worked, and I assumed that\nwas what was desired. But given what I know of the history of the config\ncode, it is probably a completely random side effect of how it is\nimplemented. :)\n\nI can try to prepare a series going in that direction (we still need to\nfix the internal truncation that currently happens, though).\n\n> >> * jk/mailmap-incomplete-line (2013-08-25) 2 commits\n> >>  - mailmap: avoid allocation when reading from blob\n> >>  - mailmap: handle mailmap blobs without trailing newlines\n> >> \n> >>  Will merge to 'next'.\n> >\n> > Did you want me to squash these? The second one more or less eradicates\n> > the changes made to the first one. I mainly did them separately in case\n> > we were going to only do the first half on maint.\n> \n> Hmm, perhaps.  Is reading mailmap from a blob commonly done and\n> deserves a maint update down for 1.8.3/1.8.2 series?\n\nYes. The tip of jk/mailmap-from-blob turned on blob reading by default\nin bare repositories. So if you have a .mailmap without a terminating\nnewline, \"git shortlog\" will segfault by default in a bare version of\nyour repository.\n\nI do not know if it is so serious a fix that you need to go back to\nv1.8.2 series, but I think it is definitely maint-worthy. I was worried\ninitially that the second part of the patch would involve too much\nrefactoring for maint, but it actually turned out pretty simple.\n\nI'll prepare a squashed version that I think should be suitable for\nmaint.\n\n-Peff\n"},{"id":"226069","messageId":"xmqqtxiaajlh.fsf@gitster.dls.corp.google.com","threadId":"34784","inReplyTo":"xmqqsixvaqh5.fsf@gitster.dls.corp.google.com","subject":"Re: What's cooking in git.git (Aug 2013, #06; Tue, 27)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2013-08-27T21:51:06Z","receivedAt":"2013-08-27T21:51:06Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Junio C Hamano <gitster@pobox.com> writes:\n\n> What's cooking in git.git (Aug 2013, #06; Tue, 27)\n> --------------------------------------------------\n>\n> Here are the topics that have been cooking.  Commits prefixed with\n> '-' are only in 'pu' (proposed updates) while commits prefixed with\n> '+' are in 'next'.\n>\n> Git 1.8.4 was tagged and released recently, and we will shortly go\n> into a new development cycle for the next one, likely to be 1.8.5.\n>\n> In this issue of \"What's cooking\" report, I haven't started sifting\n> the topics, most of which are marked as \"will cook in next\", into\n> separate bins to indicate what order they would graduate yet.  After\n> doing so, the tip of 'next' will be rewound, hopefully tomorrow but\n> it may slip by a day or so.\n\nI am nominating the following topics to graduate to 'master' by the\nend of this week (or in the middle of the next week at the latest).\n\nThose graduating earlier than others are:\n\n (1) trivially correct and safe;\n\n (2) of minor impact and even if they are broken, no real harm will\n     be done; or\n\n (3) touch parts of the system that are so important that we would\n     want to learn unforseen breakages sooner rather than later in\n     the cycle, and we have done sufficient reviews and testing on\n     'next' already.\n\nThe last category is of particular importance, as we seemed to have\nseen a few regression reports _after_ topics that have been cooking\nfor a long time in 'next' graduated to 'master' during the latest\ncycle.  We need to recruit more people from minority platforms and\nwith various different workflows to test 'next' more, but that will\nnot happen overnight, so the next best thing we can do is to feed\ntopics that are reasonably well cooked in 'next' early to 'master'.\n\n* nd/fetch-pack-shallow-fix (2013-08-25) 1 commit\n  (merged to 'next' on 2013-08-27 at 7c2a162)\n + fetch-pack: do not remove .git/shallow file when --depth is not specified\n\n Recent \"short-cut clone connectivity check\" topic broke a shallow\n repository when a fetch operation tries to auto-follow tags.\n\n Will merge to 'master', aiming to later apply to 1.8.4.x maintenance track.\n\n\n* hv/config-from-blob (2013-08-26) 1 commit\n  (merged to 'next' on 2013-08-27 at 7bc9019)\n + config: do not use C function names as struct members\n\n Portability fix.\n\n Will merge to 'master', aiming to later apply to 1.8.4.x maintenance track.\n\n\n* rj/doc-rev-parse (2013-07-22) 2 commits\n  (merged to 'next' on 2013-07-22 at 8188667)\n + rev-parse(1): logically group options\n + rev-parse: remove restrictions on some options\n\n Will merge to 'master'.\n\n\n* mb/docs-favor-en-us (2013-08-01) 1 commit\n  (merged to 'next' on 2013-08-06 at 763d868)\n + Provide some linguistic guidance for the documentation.\n\n Declare that the official grammar & spelling of the source of this\n project is en_US, but strongly discourage patches only to \"fix\"\n existing en_UK strings to avoid unnecessary churns.\n\n Will merge to 'master'.\n\n\n* nd/sq-quote-buf (2013-07-30) 3 commits\n  (merged to 'next' on 2013-08-01 at dc7934a)\n + quote: remove sq_quote_print()\n + tar-tree: remove dependency on sq_quote_print()\n + for-each-ref, quote: convert *_quote_print -> *_quote_buf\n\n Code simplification as a preparatory step to something larger.\n\n Will merge to 'master'.\n\n\n* mm/no-shell-escape-in-die-message (2013-08-07) 1 commit\n  (merged to 'next' on 2013-08-08 at bddff86)\n + die_with_status: use \"printf '%s\\n'\", not \"echo\"\n\n Fixes a minor bug in \"git rebase -i\" (there could be others, as the\n root cause is pretty generic) where the code feeds a random, data\n dependeant string to 'echo' and expects it to come out literally.\n\n Will merge to 'master'.\n\n\n* sb/parseopt-boolean-removal (2013-08-07) 9 commits\n  (merged to 'next' on 2013-08-08 at b138a2d)\n + revert: use the OPT_CMDMODE for parsing, reducing code\n + checkout-index: fix negations of even numbers of -n\n + config parsing options: allow one flag multiple times\n + hash-object: replace stdin parsing OPT_BOOLEAN by OPT_COUNTUP\n + branch, commit, name-rev: ease up boolean conditions\n + checkout: remove superfluous local variable\n + log, format-patch: parsing uses OPT__QUIET\n + Replace deprecated OPT_BOOLEAN by OPT_BOOL\n + Remove deprecated OPTION_BOOLEAN for parsing arguments\n (this branch uses jc/parseopt-command-modes.)\n\n Convert most uses of OPT_BOOLEAN/OPTION_BOOLEAN that can use\n OPT_BOOL/OPTION_BOOLEAN which have much saner semantics, and turn\n remaining ones into OPT_SET_INT, OPT_COUNTUP, etc. as necessary.\n\n Will merge to 'master'.\n\n\n* ap/remote-hg-tilde-is-home-directory (2013-08-09) 1 commit\n  (merged to 'next' on 2013-08-14 at cd963e3)\n + remote-hg: fix path when cloning with tilde expansion\n\n Will merge to 'master'.\n\n\n* rt/doc-merge-file-diff3 (2013-08-09) 1 commit\n  (merged to 'next' on 2013-08-14 at 1e5847b)\n + Documentation/git-merge-file: document option \"--diff3\"\n\n Will merge to 'master'.\n\n\n* sb/misc-cleanup (2013-08-09) 3 commits\n  (merged to 'next' on 2013-08-14 at 9e7ff9a)\n + rm: remove unneeded null pointer check\n + diff: fix a possible null pointer dereference\n + diff: remove ternary operator evaluating always to true\n\n Will merge to 'master'.\n\n\n* mm/war-on-whatchanged (2013-08-13) 2 commits\n  (merged to 'next' on 2013-08-14 at fe77c11)\n + whatchanged: document its historical nature\n + core-tutorial: trim the section on Inspecting Changes\n\n Will merge to 'master'.\n\n\n* sb/diff-delta-remove-needless-comparison (2013-08-18) 1 commit\n  (merged to 'next' on 2013-08-20 at c35730b)\n + create_delta_index: simplify condition always evaluating to true\n\n Will merge to 'master'.\n\n\n* sg/bash-prompt-lf-in-cwd-test (2013-08-18) 1 commit\n  (merged to 'next' on 2013-08-20 at 5544f62)\n + bash prompt: test the prompt with newline in repository path\n\n Will merge to 'master'.\n\n\n* jl/some-submodule-config-are-not-boolean (2013-08-19) 1 commit\n  (merged to 'next' on 2013-08-20 at 140c2e4)\n + avoid segfault on submodule.*.path set to an empty \"true\"\n\n Will merge to 'master'.\n\n\n* sp/doc-smart-http (2013-08-21) 1 commit\n  (merged to 'next' on 2013-08-22 at 4a16937)\n + Document the HTTP transport protocols\n\n Will merge to 'master'.\n\n\n* bc/completion-for-bash-3.0 (2013-08-22) 3 commits\n  (merged to 'next' on 2013-08-22 at 46c5bb2)\n + contrib/git-prompt.sh: handle missing 'printf -v' more gracefully\n + t9902-completion.sh: old Bash still does not support array+=('') notation\n + git-completion.bash: use correct Bash/Zsh array length syntax\n\n Some people still use rather old versions of bash, which cannot\n grok some constructs like 'printf -v varname' the prompt and\n completion code started to use recently.\n\n Will merge to 'master'.\n\n\n* km/svn-1.8-serf-only (2013-07-18) 3 commits\n  (merged to 'next' on 2013-07-19 at ab8d034)\n + Git.pm: revert _temp_cache use of temp_is_locked\n  (merged to 'next' on 2013-07-12 at 70e0b95)\n + git-svn: allow git-svn fetching to work using serf\n + Git.pm: add new temp_is_locked function\n\n Subversion 1.8.0 that was recently released breaks older subversion\n clients coming over http/https in various ways.\n\n Will merge to 'master'.\n\n\n* jc/check-x-z (2013-07-11) 4 commits\n  (merged to 'next' on 2013-07-15 at 62751bc)\n + check-attr -z: a single -z should apply to both input and output\n + check-ignore -z: a single -z should apply to both input and output\n + check-attr: the name of the character is NUL, not NULL\n + check-ignore: the name of the character is NUL, not NULL\n\n \"git check-ignore -z\" applied the NUL termination to both its input\n (with --stdin) and its output, but \"git check-attr -z\" ignored the\n option on the output side.\n\n This is potentially a backward incompatible fix.  Let's see if\n anybody screams before deciding if we want to do anything to help\n existing users (there may be none).\n\n Will merge to 'master'.\n\n\n* rr/feed-real-path-to-editor (2013-07-29) 1 commit\n  (merged to 'next' on 2013-07-30 at 72c9bcb)\n + editor: use canonicalized absolute path\n\n Will merge to 'master'.\n\n\n* jk/fast-import-empty-ls (2013-06-23) 4 commits\n  (merged to 'next' on 2013-07-22 at d26dab6)\n + fast-import: allow moving the root tree\n + fast-import: allow ls or filecopy of the root tree\n + fast-import: set valid mode on root tree in \"ls\"\n + t9300: document fast-import empty path issues\n\n Will merge to 'master'.\n\n\n* da/darwin (2013-08-05) 3 commits\n  (merged to 'next' on 2013-08-05 at 5c54cfa)\n + OS X: Fix redeclaration of die warning\n + Makefile: Fix APPLE_COMMON_CRYPTO with BLK_SHA1\n  (merged to 'next' on 2013-08-01 at d151a94)\n + imap-send: use Apple's Security framework for base64 encoding\n\n Will merge to 'master'.\n\n\n* bc/unuse-packfile (2013-08-02) 2 commits\n  (merged to 'next' on 2013-08-06 at 01aa4fd)\n + Don't close pack fd when free'ing pack windows\n + sha1_file: introduce close_one_pack() to close packs on fd pressure\n\n Handle memory pressure and file descriptor pressure separately when\n deciding to release pack windows to honor resource limits.\n\n Will merge to 'master'.\n\n\n* tr/fd-gotcha-fixes (2013-08-06) 1 commit\n  (merged to 'next' on 2013-08-08 at f452c5b)\n + t0070: test that git_mkstemps correctly checks return value of open()\n\n Finishing touches to an earlier fix already in 'master'.\n\n Will merge to 'master'.\n\n\n* nd/gc-lock-against-each-other (2013-08-09) 1 commit\n  (merged to 'next' on 2013-08-14 at 6999651)\n + gc: reject if another gc is running, unless --force is given\n\n Will merge to 'master'.\n\n\n* fc/remote-hg-shared-setup (2013-08-11) 2 commits\n  (merged to 'next' on 2013-08-14 at aae6858)\n + remote-hg: add shared repo upgrade\n + remote-hg: ensure shared repo is initialized\n\n Will merge to 'master'.\n\n\n* aj/p4-symlink-lose-nl (2013-08-12) 1 commit\n  (merged to 'next' on 2013-08-14 at a4959b7)\n + git-p4: Fix occasional truncation of symlink contents.\n\n Will merge to 'master'.\n\n\n* fc/unpack-trees-leakfix (2013-08-13) 1 commit\n  (merged to 'next' on 2013-08-14 at 989f329)\n + unpack-trees: plug a memory leak\n\n Will merge to 'master'.\n"},{"id":"226070","messageId":"xmqqppsyahor.fsf@gitster.dls.corp.google.com","threadId":"34784","inReplyTo":"20130827214808.GA26350@sigill.intra.peff.net","subject":"Re: What's cooking in git.git (Aug 2013, #06; Tue, 27)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2013-08-27T22:32:20Z","receivedAt":"2013-08-27T22:32:20Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Jeff King <peff@peff.net> writes:\n\n> I don't feel too strongly either way. I mostly kept the range checks for\n> --int because that is how the code already worked, and I assumed that\n> was what was desired. But given what I know of the history of the config\n> code, it is probably a completely random side effect of how it is\n> implemented. :)\n\n;-)\n\n> I can try to prepare a series going in that direction (we still need to\n> fix the internal truncation that currently happens, though).\n\nYeah, allowing range checks to allow those who do set using \"git\nconfig\" from the command line to protect themselves is in theory\na good idea, but in practice that means they need to know the\ninternal type (and they need to know to pass --int in the first\nplace), so it may be a losing proposition.\n\n> I do not know if it is so serious a fix that you need to go back to\n> v1.8.2 series, but I think it is definitely maint-worthy. I was worried\n> initially that the second part of the patch would involve too much\n> refactoring for maint, but it actually turned out pretty simple.\n>\n> I'll prepare a squashed version that I think should be suitable for\n> maint.\n\nThanks.\n"},{"id":"226071","messageId":"xmqqli3mahnc.fsf@gitster.dls.corp.google.com","threadId":"34784","inReplyTo":"CALWbr2wrHsa7LwZcEPYKe=kCrSHKsdrU6Bwevtb=2MwLrXtMnw@mail.gmail.com","subject":"Re: What's cooking in git.git (Aug 2013, #06; Tue, 27)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2013-08-27T22:33:11Z","receivedAt":"2013-08-27T22:33:11Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Antoine Pelisse <apelisse@gmail.com> writes:\n\n> On Tue, Aug 27, 2013 at 9:22 PM, Junio C Hamano <gitster@pobox.com> wrote:\n>> * jh/remote-hg-fetch-fix (2013-07-25) 2 commits\n>>   (merged to 'next' on 2013-07-25 at 33161ad)\n>>  + Revert \"remotes-hg: bugfix for fetching non local remotes\"\n>>   (merged to 'next' on 2013-07-24 at 9c96641)\n>>  + remotes-hg: bugfix for fetching non local remotes\n>>\n>>  Originally merged to 'next' on 2013-07-25\n>>\n>>  Reverted.\n>>\n>>  Waiting for the final patch to replace, after discussion settles.\n>\n> I think it has already been replaced by:\n\nSurely, of course.  I think I just ignored the text altogether when\nI moved it to the \"discarded\" section.\n\nThanks.\n"},{"id":"226072","messageId":"20130828000548.GA5238@camk.edu.pl","threadId":"34784","inReplyTo":"xmqqsixvaqh5.fsf@gitster.dls.corp.google.com","subject":"Re: What's cooking in git.git (Aug 2013, #06; Tue, 27)","fromName":"Kacper Kornet","fromEmail":"draenog@pld-linux.org","sentAt":"2013-08-28T00:05:48Z","receivedAt":"2013-08-28T00:05:48Z","isPatch":false,"sender":{"key":"draenog@pld-linux.org","avatar":"https://avatars.githubusercontent.com/u/608762?v=4"},"body":"On Tue, Aug 27, 2013 at 12:22:30PM -0700, Junio C Hamano wrote:\n> * kk/tests-with-no-perl (2013-08-24) 4 commits\n>  - reset test: modernize style\n>  - t/t7106-reset-unborn-branch.sh: Add PERL prerequisite\n>  - add -i test: use skip_all instead of repeated PERL prerequisite\n>  - Make test \"using invalid commit with -C\" more strict\n\n>  Am I waiting for another reroll?\n\n>From these four commits only the second and last one are from me and I'm\nhappy with their form as in pu (I see that you have already introduced\nyour improvement of  \"commit --allow-empty\" to the last one). Unless\nthere are some remarks to them.\n\nBut I don't know what about Jonathan's commits.\n\nAs a matter of fact I have no idea how I even should reroll the topic\nthat includes commits with mixed authorship.\n\n-- \n  Kacper\n"},{"id":"226079","messageId":"521D9B14.2070408@viscovery.net","threadId":"34784","inReplyTo":"20130827214808.GA26350@sigill.intra.peff.net","subject":"Re: What's cooking in git.git (Aug 2013, #06; Tue, 27)","fromName":"Johannes Sixt","fromEmail":"j.sixt@viscovery.net","sentAt":"2013-08-28T06:39:16Z","receivedAt":"2013-08-28T06:39:16Z","isPatch":false,"sender":{"key":"j6t@kdbg.org","avatar":"https://avatars.githubusercontent.com/u/14810926?v=4"},"body":"Am 8/27/2013 23:48, schrieb Jeff King:\n> The counterarguments I can see are:\n> \n>   1. Who cares? If you want to know whether pack-objects will choke on\n>      your huge config value, then run pack-objects.\n> \n>   2. Such a check would involve knowing which type we use internally to\n>      look at packSizeLimit, and that is utterly undocumented (and\n>      subject to change; e.g., it seems kind of senseless that we have a\n>      4G pack-size limit on 32-bit platforms, and we may want to fix\n>      that).\n> \n> So if you do not buy the argument that communicating git's internal\n> range checks is useful, then we can simply say \"--int is magically long\n> on every platform, and you can use it for everything numeric\". And\n> implement it with int64_t. You may be able to read or write some values\n> for certain keys that git will barf on internally, but that is git's\n> problem.\n\nI'm in the camp of these (counter) arguments.\n\nWhen my shell script asks for 'git config --int 3g', I expect to be\nreturned a positive 10-digit. What would I care which type Git or any\nother tool is using internally? I only care whether my shell can work with\nnumbers that large. Or the next tool that I feed the number to. But that's\nmy business, not Git's.\n\n> The one thing it doesn't get you is that you can currently set unsigned\n> values to \"-1\" in the config to have them treated as ULONG_MAX. This is\n> undocumented and as far as I know not used by anyone.\n\nAnd it better stays that way. Magic numbers should be encoded with magic\nstrings in the config file.\n\n-- Hannes\n"}]}