{"thread":{"id":"32631","subject":"What's cooking in git.git (Jan 2013, #06; Mon, 14)","startedAt":"2013-01-14T22:23:56Z","lastAt":"2013-01-16T22:20:12Z","messageCount":9,"participants":["Junio C Hamano","Chris Rorvick","Adam Spiers"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"206846","messageId":"7vehhn8kub.fsf@alter.siamese.dyndns.org","threadId":"32631","inReplyTo":null,"subject":"What's cooking in git.git (Jan 2013, #06; Mon, 14)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2013-01-14T22:23:56Z","receivedAt":"2013-01-14T22:23:56Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Here are the topics that have been cooking.  Commits prefixed with\n'-' are only in 'pu' (proposed updates) while commits prefixed with\n'+' are in 'next'.\n\nAs usual, this cycle is expected to last for 8 to 10 weeks, with a\npreview -rc0 sometime in the middle of next month.\n\nYou can find the changes described here in the integration branches of the\nrepositories listed at\n\n    http://git-blame.blogspot.com/p/git-public-repositories.html\n\n--------------------------------------------------\n[New Topics]\n\n* jc/cvsimport-upgrade (2013-01-14) 8 commits\n - t9600: adjust for new cvsimport\n - t9600: further prepare for sharing\n - cvsimport-3: add a sample test\n - cvsimport: make tests reusable for cvsimport-3\n - cvsimport: start adding cvsps 3.x support\n - cvsimport: introduce a version-switch wrapper\n - cvsimport: allow setting a custom cvsps (2.x) program name\n - Makefile: add description on PERL/PYTHON_PATH\n\n The most important part of this series is the addition of the new\n cvsimport by Eric Raymond that works with cvsps 3.x.  Given some\n distros have inertia to be conservative, Git with cvsimport that\n does not work with both 3.x will block adoption of cvsps 3.x by\n them, and shipping Git with cvsimport that does not work with cvsps\n 2.x will block such a version of Git, so we'll do the proven \"both\n old and new are available, but we aim to deprecate and remove the\n old one in due time\" strategy that we used successfully in the\n past.\n\n\n* as/pre-push-hook (2013-01-14) 3 commits\n - Add sample pre-push hook script\n - push: Add support for pre-push hooks\n - hooks: Add function to check if a hook exists\n\n Add an extra hook so that \"git push\" that is run without making\n sure what is being pushed is sane can be checked and rejected (as\n opposed to the user deciding not pushing).\n\n\n* dl/am-hg-locale (2013-01-14) 1 commit\n - am: invoke perl's strftime in C locale\n\n Datestamp recorded in \"Hg\" format patch was reformatted incorrectly\n to an e-mail looking date using locale dependant strftime, causing\n patch application to fail.\n\n\n* jk/config-parsing-cleanup (2013-01-14) 7 commits\n - [DONTMERGE] reroll coming\n - submodule: simplify memory handling in config parsing\n - submodule: use match_config_key when parsing config\n - userdiff: drop parse_driver function\n - convert some config callbacks to match_config_key\n - archive-tar: use match_config_key when parsing config\n - config: add helper function for parsing key names\n\n Expecting a reroll.\n\n* mp/diff-algo-config (2013-01-14) 3 commits\n - diff: Introduce --diff-algorithm command line option\n - config: Introduce diff.algorithm variable\n - git-completion.bash: Autocomplete --minimal and --histogram for git-diff\n\n Add diff.algorithm configuration so that the user does not type\n \"diff --histogram\".\n\n Expecting a reroll.\n\n* ph/rebase-preserve-all-merges (2013-01-14) 1 commit\n - rebase --preserve-merges: keep all merge commits including empty ones\n\n An earlier change to add --keep-empty option broke \"git rebase\n --preserve-merges\" and lost merge commits that end up being the\n same as its parent.\n\n Will merge to 'next'.\n\n* rs/archive-tar-config-parsing-fix (2013-01-14) 1 commit\n - archive-tar: fix sanity check in config parsing\n\n Configuration parsing for tar.* configuration variables were\n broken; Peff's config parsing clean-up topic will address the same\n breakage, so this may be superseded by that other topic.\n\n\n* rs/pretty-use-prefixcmp (2013-01-14) 1 commit\n - pretty: use prefixcmp instead of memcmp on NUL-terminated strings\n\n Will merge to 'next'.\n\n--------------------------------------------------\n[Graduated to \"master\"]\n\n* ap/status-ignored-in-ignored-directory (2013-01-07) 3 commits\n  (merged to 'next' on 2013-01-10 at 20f7476)\n + status: always report ignored tracked directories\n  (merged to 'next' on 2013-01-07 at 2a20b19)\n + git-status: Test --ignored behavior\n + dir.c: Make git-status --ignored more consistent\n\n Output from \"git status --ignored\" showed an unexpected interaction\n with \"--untracked\".\n\n\n* fc/remote-testgit-feature-done (2012-10-29) 1 commit\n  (merged to 'next' on 2013-01-10 at 3132a60)\n + remote-testgit: properly check for errors\n\n In the longer term, tightening rules is a good thing to do, and\n because nobody who has worked in the remote helper area seems to be\n interested in reviewing this, I would assume they do not think\n such a retroactive tightening will affect their remote helpers.  So\n let's advance this topic to see what happens.\n\n\n* jc/blame-no-follow (2012-09-21) 2 commits\n  (merged to 'next' on 2013-01-10 at 201c7f4)\n + blame: pay attention to --no-follow\n + diff: accept --no-follow option\n\n Teaches \"--no-follow\" option to \"git blame\" to disable its\n whole-file rename detection.\n\n\n* jc/format-patch-reroll (2013-01-03) 9 commits\n  (merged to 'next' on 2013-01-07 at 0e007e6)\n + format-patch: give --reroll-count a short synonym -v\n + format-patch: document and test --reroll-count\n + format-patch: add --reroll-count=$N option\n + get_patch_filename(): split into two functions\n + get_patch_filename(): drop \"just-numbers\" hack\n + get_patch_filename(): simplify function signature\n + builtin/log.c: stop using global patch_suffix\n + builtin/log.c: drop redundant \"numbered_files\" parameter from make_cover_letter()\n + builtin/log.c: drop unused \"numbered\" parameter from make_cover_letter()\n\n Originally merged to 'next' on 2013-01-04\n\n Teach \"format-patch\" to prefix v4- to its output files for the\n fourth iteration of a patch series, to make it easier for the\n submitter to keep separate copies for iterations.\n\n\n* jc/merge-blobs (2012-12-26) 5 commits\n  (merged to 'next' on 2013-01-08 at 582ca38)\n + merge-tree: fix d/f conflicts\n + merge-tree: add comments to clarify what these functions are doing\n + merge-tree: lose unused \"resolve_directories\"\n + merge-tree: lose unused \"flags\" from merge_list\n + Which merge_file() function do you mean?\n\n Update the disused merge-tree proof-of-concept code.\n\n\n* jk/maint-fast-import-doc-reorder (2013-01-09) 2 commits\n  (merged to 'next' on 2013-01-10 at 9f3950d)\n + git-fast-import(1): reorganise options\n + git-fast-import(1): combine documentation of --[no-]relative-marks\n\n\n* jk/shortlog-no-wrap-doc (2013-01-09) 1 commit\n  (merged to 'next' on 2013-01-10 at c79898a)\n + git-shortlog(1): document behaviour of zero-width wrap\n\n\n* jk/unify-exit-code-by-receiving-signal (2013-01-06) 1 commit\n  (merged to 'next' on 2013-01-08 at 5ebf940)\n + run-command: encode signal death as a positive integer\n\n The internal logic had to deal with two representations of a death\n of a child process by a signal.\n\n\n* jn/xml-depends-on-asciidoc-conf (2013-01-06) 1 commit\n  (merged to 'next' on 2013-01-08 at 4faf8d4)\n + docs: manpage XML depends on asciidoc.conf\n\n\n* nd/upload-pack-shallow-must-be-commit (2013-01-08) 1 commit\n  (merged to 'next' on 2013-01-10 at a8b3ba5)\n + upload-pack: only accept commits from \"shallow\" line\n\n A minor consistency check patch that does not have much relevance\n to the real world.\n\n\n* nz/send-email-headers-are-case-insensitive (2013-01-06) 1 commit\n  (merged to 'next' on 2013-01-10 at cf4c9c9)\n + git-send-email: treat field names as case-insensitively\n\n When user spells \"cc:\" in lowercase in the fake \"header\" in the\n trailer part, send-email failed to pick up the addresses from\n there. As e-mail headers field names are case insensitive, this\n script should follow suit and treat \"cc:\" and \"Cc:\" the same way.\n\n\n* rs/zip-tests (2013-01-07) 4 commits\n  (merged to 'next' on 2013-01-08 at 8e37423)\n + t5003: check if unzip supports symlinks\n + t5000, t5003: move ZIP tests into their own script\n + t0024, t5000: use test_lazy_prereq for UNZIP\n + t0024, t5000: clear variable UNZIP, use GIT_UNZIP instead\n\n Updates zip tests to skip some that cannot be handled on platform\n unzip.\n\n\n* rs/zip-with-uncompressed-size-in-the-header (2013-01-06) 1 commit\n  (merged to 'next' on 2013-01-08 at d9ec30e)\n + archive-zip: write uncompressed size into header even with streaming\n\n Improve compatibility of our zip output to fill uncompressed size\n in the header, which we can do without seeking back (even though it\n should not be necessary).\n\n--------------------------------------------------\n[Stalled]\n\n* mp/complete-paths (2013-01-11) 1 commit\n - git-completion.bash: add support for path completion\n\n The completion script used to let the default completer to suggest\n pathnames, which gave too many irrelevant choices (e.g. \"git add\"\n would not want to add an unmodified path).  Teach it to use a more\n git-aware logic to enumerate only relevant ones.\n\n Waiting for area-experts' help and review.\n\n\n* jl/submodule-deinit (2012-12-04) 1 commit\n - submodule: add 'deinit' command\n\n There was no Porcelain way to say \"I no longer am interested in\n this submodule\", once you express your interest in a submodule with\n \"submodule init\".  \"submodule deinit\" is the way to do so.\n\n Expecting a reroll.\n $gmane/212884\n\n\n* jk/lua-hackery (2012-10-07) 6 commits\n - pretty: fix up one-off format_commit_message calls\n - Minimum compilation fixup\n - Makefile: make \"lua\" a bit more configurable\n - add a \"lua\" pretty format\n - add basic lua infrastructure\n - pretty: make some commit-parsing helpers more public\n\n Interesting exercise. When we do this for real, we probably would want\n to wrap a commit to make it more like an \"object\" with methods like\n \"parents\", etc.\n\n\n* rc/maint-complete-git-p4 (2012-09-24) 1 commit\n - Teach git-completion about git p4\n\n Comment from Pete will need to be addressed ($gmane/206172).\n\n\n* jc/maint-name-rev (2012-09-17) 7 commits\n - describe --contains: use \"name-rev --algorithm=weight\"\n - name-rev --algorithm=weight: tests and documentation\n - name-rev --algorithm=weight: cache the computed weight in notes\n - name-rev --algorithm=weight: trivial optimization\n - name-rev: --algorithm option\n - name_rev: clarify the logic to assign a new tip-name to a commit\n - name-rev: lose unnecessary typedef\n\n \"git name-rev\" names the given revision based on a ref that can be\n reached in the smallest number of steps from the rev, but that is\n not useful when the caller wants to know which tag is the oldest one\n that contains the rev.  This teaches a new mode to the command that\n uses the oldest ref among those which contain the rev.\n\n I am not sure if this is worth it; for one thing, even with the help\n from notes-cache, it seems to make the \"describe --contains\" even\n slower. Also the command will be unusably slow for a user who does\n not have a write access (hence unable to create or update the\n notes-cache).\n\n Stalled mostly due to lack of responses.\n\n\n* jc/xprm-generation (2012-09-14) 1 commit\n - test-generation: compute generation numbers and clock skews\n\n A toy to analyze how bad the clock skews are in histories of real\n world projects.\n\n Stalled mostly due to lack of responses.\n\n\n* jc/add-delete-default (2012-08-13) 1 commit\n - git add: notice removal of tracked paths by default\n\n \"git add dir/\" updated modified files and added new files, but does\n not notice removed files, which may be \"Huh?\" to some users.  They\n can of course use \"git add -A dir/\", but why should they?\n\n Resurrected from graveyard, as I thought it was a worthwhile thing\n to do in the longer term.\n\n Stalled mostly due to lack of responses.\n\n\n* mb/remote-default-nn-origin (2012-07-11) 6 commits\n - Teach get_default_remote to respect remote.default.\n - Test that plain \"git fetch\" uses remote.default when on a detached HEAD.\n - Teach clone to set remote.default.\n - Teach \"git remote\" about remote.default.\n - Teach remote.c about the remote.default configuration setting.\n - Rename remote.c's default_remote_name static variables.\n\n When the user does not specify what remote to interact with, we\n often attempt to use 'origin'.  This can now be customized via a\n configuration variable.\n\n Expecting a reroll.\n $gmane/210151\n\n \"The first remote becomes the default\" bit is better done as a\n separate step.\n\n--------------------------------------------------\n[Cooking]\n\n* rt/commit-cleanup-config (2013-01-10) 1 commit\n - commit: make default of \"cleanup\" option configurable\n\n Add a configuration variable to set default clean-up mode other\n than \"strip\".\n\n Will merge to 'next'.\n\n\n* jc/custom-comment-char (2013-01-10) 1 commit\n - Allow custom \"comment char\"\n\n An illustration to show codepaths that need to be touched to change\n the hint lines in the edited text to begin with something other\n than '#'.\n\n\n* jn/maint-trim-vim-contrib (2013-01-10) 1 commit\n - contrib/vim: simplify instructions for old vim support\n\n Will merge to 'next'.\n\n\n* mz/reset-misc (2013-01-10) 22 commits\n - reset [--mixed]: use diff-based reset whether or not pathspec was given\n - [SQUASH???] script portability fixes\n - reset: allow reset on unborn branch\n - reset $sha1 $pathspec: require $sha1 only to be treeish\n - reset [--mixed] --quiet: don't refresh index\n - reset.c: finish entire cmd_reset() whether or not pathspec is given\n - reset [--mixed]: don't write index file twice\n - reset.c: move lock, write and commit out of update_index_refresh()\n - reset.c: move update_index_refresh() call out of read_from_tree()\n - reset: avoid redundant error message\n - reset --keep: only write index file once\n - reset.c: replace switch by if-else\n - reset.c: share call to die_if_unmerged_cache()\n - [SQUASH???] style fixes\n - reset.c: extract function for updating {ORIG,}HEAD\n - reset.c: remove unnecessary variable 'i'\n - [SQUASH???] style fix\n - reset.c: extract function for parsing arguments\n - reset: don't allow \"git reset -- $pathspec\" in bare repo\n - reset.c: pass pathspec around instead of (prefix, argv) pair\n - reset $pathspec: exit with code 0 if successful\n - reset $pathspec: no need to discard index\n\n Various 'reset' optimizations and clean-ups, followed by a change\n to allow \"git reset\" to work even on an unborn branch.\n\n\n* pe/doc-email-env-is-trumped-by-config (2013-01-10) 1 commit\n  (merged to 'next' on 2013-01-14 at 6b4d555)\n + git-commit-tree(1): correct description of defaults\n\n In the precedence order, the environment variable $EMAIL comes\n between the built-in default (i.e. taking value by asking the\n system's gethostname() etc.) and the user.email configuration\n variable; the documentation implied that it is stronger than the\n configuration like $GIT_COMMITTER_EMAIL is, which is wrong.\n\n\n* ds/completion-silence-in-tree-path-probe (2013-01-11) 1 commit\n - git-completion.bash: silence \"not a valid object\" errors\n\n An internal ls-tree call made by completion code only to probe if\n a path exists in the tree recorded in a commit object leaked error\n messages when the path is not there.  It is not an error at all and\n should not be shown to the end user.\n\n Will merge to 'next'.\n\n\n* nd/fetch-depth-is-broken (2013-01-11) 3 commits\n - fetch: elaborate --depth action\n - upload-pack: fix off-by-one depth calculation in shallow clone\n - fetch: add --unshallow for turning shallow repo into complete one\n\n \"git fetch --depth\" was broken in at least three ways.  The\n resulting history was deeper than specified by one commit, it was\n unclear how to wipe the shallowness of the repository with the\n command, and documentation was misleading.\n\n Will merge to 'next'.\n\n\n* jc/no-git-config-in-clone (2013-01-11) 1 commit\n - clone: do not export and unexport GIT_CONFIG\n\n We stopped paying attention to $GIT_CONFIG environment that points\n at a single configuration file from any command other than \"git config\"\n quite a while ago, but \"git clone\" internally set, exported, and\n then unexported the variable during its operation unnecessarily.\n\n\n* mk/complete-tcsh (2013-01-07) 1 commit\n  (merged to 'next' on 2013-01-11 at b8b30b1)\n + Prevent space after directories in tcsh completion\n\n Update tcsh command line completion so that an unwanted space is\n not added to a single directory name.\n\n\n* dg/subtree-fixes (2013-01-08) 7 commits\n - contrib/subtree: mkdir the manual directory if needed\n - contrib/subtree: honor $(DESTDIR)\n - contrib/subtree: fix synopsis and command help\n - contrib/subtree: better error handling for \"add\"\n - contrib/subtree: add --unannotate option\n - contrib/subtree: use %B for split Subject/Body\n - t7900: remove test number comments\n\n contrib/subtree updates.\n\n Rerolled?\n\n\n* ap/log-mailmap (2013-01-10) 11 commits\n  (merged to 'next' on 2013-01-10 at 8544084)\n + log --use-mailmap: optimize for cases without --author/--committer search\n + log: add log.mailmap configuration option\n + log: grep author/committer using mailmap\n + test: add test for --use-mailmap option\n + log: add --use-mailmap option\n + pretty: use mailmap to display username and email\n + mailmap: add mailmap structure to rev_info and pp\n + mailmap: simplify map_user() interface\n + mailmap: remove email copy and length limitation\n + Use split_ident_line to parse author and committer\n + string-list: allow case-insensitive string list\n\n Teach commands in the \"log\" family to optionally pay attention to\n the mailmap.\n\n\n* jc/push-2.0-default-to-simple (2013-01-08) 11 commits\n  (merged to 'next' on 2013-01-09 at 74c3498)\n + doc: push.default is no longer \"matching\"\n + push: switch default from \"matching\" to \"simple\"\n + t9401: do not assume the \"matching\" push is the default\n + t9400: do not assume the \"matching\" push is the default\n + t7406: do not assume the \"matching\" push is the default\n + t5531: do not assume the \"matching\" push is the default\n + t5519: do not assume the \"matching\" push is the default\n + t5517: do not assume the \"matching\" push is the default\n + t5516: do not assume the \"matching\" push is the default\n + t5505: do not assume the \"matching\" push is the default\n + t5404: do not assume the \"matching\" push is the default\n\n Will cook in 'next' until Git 2.0 ;-).\n\n\n* nd/clone-no-separate-git-dir-with-bare (2013-01-10) 1 commit\n - clone: forbid --bare --separate-git-dir <dir>\n\n Will merge to 'next'.\n\n\n* nd/parse-pathspec (2013-01-11) 20 commits\n . Convert more init_pathspec() to parse_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 read_cache_preload() to take struct pathspec\n . add: 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 . Export parse_pathspec() and convert some get_pathspec() calls\n . Add parse_pathspec() that converts cmdline args to struct pathspec\n . pathspec: save the non-wildcard length part\n\n Uses the parsed pathspec structure in more places where we used to\n use the raw \"array of strings\" pathspec.\n\n Ejected from 'pu' for now; will take a look at the rerolled one\n later ($gmane/213340).\n\n\n* jc/doc-maintainer (2013-01-03) 2 commits\n  (merged to 'next' on 2013-01-11 at f35d582)\n + howto/maintain: mark titles for asciidoc\n + Documentation: update \"howto maintain git\"\n\n Describe tools for automation that were invented since this\n document was originally written.\n\n\n* mo/cvs-server-updates (2012-12-09) 18 commits\n  (merged to 'next' on 2013-01-08 at 75e2d11)\n + t9402: Use TABs for indentation\n + t9402: Rename check.cvsCount and check.list\n + t9402: Simplify git ls-tree\n + t9402: Add missing &&; Code style\n + t9402: No space after IO-redirection\n + t9402: Dont use test_must_fail cvs\n + t9402: improve check_end_tree() and check_end_full_tree()\n + t9402: sed -i is not portable\n + cvsserver Documentation: new cvs ... -r support\n + cvsserver: add t9402 to test branch and tag refs\n + cvsserver: support -r and sticky tags for most operations\n + cvsserver: Add version awareness to argsfromdir\n + cvsserver: generalize getmeta() to recognize commit refs\n + cvsserver: implement req_Sticky and related utilities\n + cvsserver: add misc commit lookup, file meta data, and file listing functions\n + cvsserver: define a tag name character escape mechanism\n + cvsserver: cleanup extra slashes in filename arguments\n + cvsserver: factor out git-log parsing logic\n\n Various git-cvsserver updates.\n\n Will cook in 'next' for a while to see if anybody screams.\n\n\n* as/check-ignore (2013-01-10) 12 commits\n  (merged to 'next' on 2013-01-14 at 9df2afc)\n + t0008: avoid brace expansion\n + add git-check-ignore sub-command\n + setup.c: document get_pathspec()\n + add.c: extract new die_if_path_beyond_symlink() for reuse\n + add.c: extract check_path_for_gitlink() from treat_gitlinks() for reuse\n + pathspec.c: rename newly public functions for clarity\n + add.c: move pathspec matchers into new pathspec.c for reuse\n + add.c: remove unused argument from validate_pathspec()\n + dir.c: improve docs for match_pathspec() and match_pathspec_depth()\n + dir.c: provide clear_directory() for reclaiming dir_struct memory\n + dir.c: keep track of where patterns came from\n + dir.c: use a single struct exclude_list per source of excludes\n\n Add a new command \"git check-ignore\" for debugging .gitignore\n files.\n\n\n* nd/retire-fnmatch (2013-01-01) 7 commits\n  (merged to 'next' on 2013-01-07 at ab31f9b)\n + Makefile: add USE_WILDMATCH to use wildmatch as fnmatch\n + wildmatch: advance faster in <asterisk> + <literal> patterns\n + wildmatch: make a special case for \"*/\" with FNM_PATHNAME\n + test-wildmatch: add \"perf\" command to compare wildmatch and fnmatch\n + wildmatch: support \"no FNM_PATHNAME\" mode\n + wildmatch: make dowild() take arbitrary flags\n + wildmatch: rename constants and update prototype\n\n Originally merged to 'next' on 2013-01-04\n\n Replace our use of fnmatch(3) with a more feature-rich wildmatch.\n A handful patches at the bottom have been moved to nd/wildmatch to\n graduate as part of that branch, before this series solidifies.\n\n Will cook in 'next' a bit longer than other topics.\n\n\n* mb/gitweb-highlight-link-target (2012-12-20) 1 commit\n - Highlight the link target line in Gitweb using CSS\n\n Expecting a reroll.\n $gmane/211935\n\n\n* zk/clean-report-failure (2013-01-14) 1 commit\n - git-clean: Display more accurate delete messages\n\n \"git clean\" states what it is going to remove and then goes on to\n remove it, but sometimes it only discovers things that cannot be\n removed after recursing into a directory, which makes the output\n confusing and even wrong.\n\n Will merge to 'next'.\n\n\n* bc/append-signed-off-by (2013-01-01) 12 commits\n - t4014: do not use echo -n\n - Unify appending signoff in format-patch, commit and sequencer\n - format-patch: update append_signoff prototype\n - format-patch: stricter S-o-b detection\n - t4014: more tests about appending s-o-b lines\n - sequencer.c: teach append_signoff to avoid adding a duplicate newline\n - sequencer.c: teach append_signoff how to detect duplicate s-o-b\n - sequencer.c: always separate \"(cherry picked from\" from commit body\n - sequencer.c: recognize \"(cherry picked from ...\" as part of s-o-b footer\n - t/t3511: add some tests of 'cherry-pick -s' functionality\n - t/test-lib-functions.sh: allow to specify the tag name to test_commit\n - sequencer.c: remove broken support for rfc2822 continuation in footer\n\n Expecting a reroll.\n $gmane/212507\n\n* er/replace-cvsimport (2013-01-12) 7 commits\n . t/lib-cvs: cvsimport no longer works without Python >= 2.7\n . t9605: test for cvsps commit ordering bug\n . t9604: fixup for new cvsimport\n . t9600: fixup for new cvsimport\n . t/lib-cvs.sh: allow cvsps version 3.x.\n . t/t960[123]: remove leftover scripts\n . cvsimport: rewrite to use cvsps 3.x to fix major bugs\n\n Rerolled as jc/cvsimport-upgrade and ejected from 'pu'.\n"},{"id":"206880","messageId":"7v1udn6tdg.fsf@alter.siamese.dyndns.org","threadId":"32631","inReplyTo":"7vehhn8kub.fsf@alter.siamese.dyndns.org","subject":"Re: What's cooking in git.git (Jan 2013, #06; Mon, 14)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2013-01-15T03:02:35Z","receivedAt":"2013-01-15T03:02:35Z","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> [New Topics]\n>\n> * jc/cvsimport-upgrade (2013-01-14) 8 commits\n>  - t9600: adjust for new cvsimport\n>  - t9600: further prepare for sharing\n>  - cvsimport-3: add a sample test\n>  - cvsimport: make tests reusable for cvsimport-3\n>  - cvsimport: start adding cvsps 3.x support\n>  - cvsimport: introduce a version-switch wrapper\n>  - cvsimport: allow setting a custom cvsps (2.x) program name\n>  - Makefile: add description on PERL/PYTHON_PATH\n>\n>  The most important part of this series is the addition of the new\n>  cvsimport by Eric Raymond that works with cvsps 3.x.  Given some\n>  distros have inertia to be conservative, Git with cvsimport that\n>  does not work with both 3.x will block adoption of cvsps 3.x by\n>  them, and shipping Git with cvsimport that does not work with cvsps\n>  2.x will block such a version of Git, so we'll do the proven \"both\n>  old and new are available, but we aim to deprecate and remove the\n>  old one in due time\" strategy that we used successfully in the\n>  past.\n\nMy reading of the review discussion of this series, and the\ndiscussion in the $gmane/213170 thread, is that the approach\noutlined in this series is something Git-side is comfortable working\nwith.\n\nI personally think it will be slightly less work on your side to\nkeep the cvsps 3.x + new cvsimport combo improving, because you no\nlonger need to worry about punting to the old cvsimport.  In\naddition, I think the new layout would make it easier for the new\ncombo to gain trust of existing Git userbase over time by adding\nmore t965x series of tests that correspond to the tests in the t960x\nseries, working on the same (simple) CVS histories, demonstrating\nthat the result would be what users expect, and guarding the code\nfrom future breakage.  By giving options to pick and choose both old\nand new cvsps, I think it will make it easier for distros to include\ncvsps 3.x sooner, promoting its adoption, which will in turn benefit\nus.\n\nI converted one of Chris's follow-up test tweaks to this to\nillustrate how it can be done without breaking tests for the\noriginal cvsimport, but didn't do all of them.  Chris, is this a\nfoundation we can work together on top?\n\nEven though I assigned Author: to the \"start adding cvsps 3\" patch,\nI forgot to forge Eric's sign-off to it.  If Eric is OK with the\ndirection this series is going, I'll do so and advance the rerolled\nseries to 'next'.\n\nThanks.\n"},{"id":"206903","messageId":"CAEUsAPZrp=racc4mY7thugPvGPhH9B_wNPwNx6HkvqUMnXzJ_Q@mail.gmail.com","threadId":"32631","inReplyTo":"7v1udn6tdg.fsf@alter.siamese.dyndns.org","subject":"Re: What's cooking in git.git (Jan 2013, #06; Mon, 14)","fromName":"Chris Rorvick","fromEmail":"chris@rorvick.com","sentAt":"2013-01-15T06:10:34Z","receivedAt":"2013-01-15T06:10:34Z","isPatch":false,"sender":{"key":"chris@rorvick.com","avatar":"https://avatars.githubusercontent.com/u/824726?v=4"},"body":"On Mon, Jan 14, 2013 at 9:02 PM, Junio C Hamano <gitster@pobox.com> wrote:\n> I converted one of Chris's follow-up test tweaks to this to\n> illustrate how it can be done without breaking tests for the\n> original cvsimport, but didn't do all of them.  Chris, is this a\n> foundation we can work together on top?\n\nSure, looks straightforward and makes things easier.\n\nThanks,\n\nChris\n"},{"id":"207001","messageId":"CAOkDyE_a4R7=A318VL2TxDn6X8Tu2+m2KNWWYqwBbygRrALQzg@mail.gmail.com","threadId":"32631","inReplyTo":"7vehhn8kub.fsf@alter.siamese.dyndns.org","subject":"Re: What's cooking in git.git (Jan 2013, #06; Mon, 14)","fromName":"Adam Spiers","fromEmail":"git@adamspiers.org","sentAt":"2013-01-16T01:34:36Z","receivedAt":"2013-01-16T01:34:36Z","isPatch":false,"sender":{"key":"git@adamspiers.org","avatar":"https://avatars.githubusercontent.com/u/100738?v=4"},"body":"On Mon, Jan 14, 2013 at 10:23 PM, Junio C Hamano <gitster@pobox.com> wrote:\n> * as/check-ignore (2013-01-10) 12 commits\n>   (merged to 'next' on 2013-01-14 at 9df2afc)\n>  + t0008: avoid brace expansion\n>  + add git-check-ignore sub-command\n>  + setup.c: document get_pathspec()\n>  + add.c: extract new die_if_path_beyond_symlink() for reuse\n>  + add.c: extract check_path_for_gitlink() from treat_gitlinks() for reuse\n>  + pathspec.c: rename newly public functions for clarity\n>  + add.c: move pathspec matchers into new pathspec.c for reuse\n>  + add.c: remove unused argument from validate_pathspec()\n>  + dir.c: improve docs for match_pathspec() and match_pathspec_depth()\n>  + dir.c: provide clear_directory() for reclaiming dir_struct memory\n>  + dir.c: keep track of where patterns came from\n>  + dir.c: use a single struct exclude_list per source of excludes\n>\n>  Add a new command \"git check-ignore\" for debugging .gitignore\n>  files.\n\nThe above is v4 plus the \"t0008: avoid brace expansion\" fix.  v4 is\nslightly outdated and not quite the right version to merge to 'next'.\nI'll post a v5 re-roll as per:\n\nhttp://thread.gmane.org/gmane.comp.version-control.git/212184/focus=212856\n\nin the next 24 hours or so.\n\nI think the \"t0008: avoid brace expansion\" fix at the tip should\nprobably be squashed into its parent.  I've amended the commit message\naccordingly in my github fork.\n\nThanks,\nAdam\n"},{"id":"207017","messageId":"7v7gnd26pr.fsf@alter.siamese.dyndns.org","threadId":"32631","inReplyTo":"CAOkDyE_a4R7=A318VL2TxDn6X8Tu2+m2KNWWYqwBbygRrALQzg@mail.gmail.com","subject":"Re: What's cooking in git.git (Jan 2013, #06; Mon, 14)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2013-01-16T02:37:52Z","receivedAt":"2013-01-16T02:37:52Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Adam Spiers <git@adamspiers.org> writes:\n\n> On Mon, Jan 14, 2013 at 10:23 PM, Junio C Hamano <gitster@pobox.com> wrote:\n>> * as/check-ignore (2013-01-10) 12 commits\n>>   (merged to 'next' on 2013-01-14 at 9df2afc)\n>>  + t0008: avoid brace expansion\n>>  + add git-check-ignore sub-command\n>>  + setup.c: document get_pathspec()\n>>  + add.c: extract new die_if_path_beyond_symlink() for reuse\n>>  + add.c: extract check_path_for_gitlink() from treat_gitlinks() for reuse\n>>  + pathspec.c: rename newly public functions for clarity\n>>  + add.c: move pathspec matchers into new pathspec.c for reuse\n>>  + add.c: remove unused argument from validate_pathspec()\n>>  + dir.c: improve docs for match_pathspec() and match_pathspec_depth()\n>>  + dir.c: provide clear_directory() for reclaiming dir_struct memory\n>>  + dir.c: keep track of where patterns came from\n>>  + dir.c: use a single struct exclude_list per source of excludes\n>>\n>>  Add a new command \"git check-ignore\" for debugging .gitignore\n>>  files.\n>\n> The above is v4 plus the \"t0008: avoid brace expansion\" fix.  v4 is\n> slightly outdated and not quite the right version to merge to 'next'.\n\nSigh.\n\nThe \"What's cooking\" is a report of what _has_ already happened.  I\nwould have appreciated if you said the above _before_ this happened.\n\n> I'll post a v5 re-roll as per:\n\nNow the series is in 'next', it is too late to _replace_ it X-<.\nCould you instead make an incremental updates on top?  That way, we\ndo not have to re-review the whole thing; we only need to review the\nchanges relative to the old one, making sure that the fixes in the\nupdates are better than the v4 version.\n\nThanks.\n"},{"id":"207046","messageId":"CAOkDyE-p9WLrsFZjPb9sY+YEypkF2wDxMybBkCT-76jBbKOmCA@mail.gmail.com","threadId":"32631","inReplyTo":"7v7gnd26pr.fsf@alter.siamese.dyndns.org","subject":"Re: What's cooking in git.git (Jan 2013, #06; Mon, 14)","fromName":"Adam Spiers","fromEmail":"git@adamspiers.org","sentAt":"2013-01-16T12:06:56Z","receivedAt":"2013-01-16T12:06:56Z","isPatch":false,"sender":{"key":"git@adamspiers.org","avatar":"https://avatars.githubusercontent.com/u/100738?v=4"},"body":"On Wed, Jan 16, 2013 at 2:37 AM, Junio C Hamano <gitster@pobox.com> wrote:\n> Adam Spiers <git@adamspiers.org> writes:\n>\n>> On Mon, Jan 14, 2013 at 10:23 PM, Junio C Hamano <gitster@pobox.com> wrote:\n>>> * as/check-ignore (2013-01-10) 12 commits\n>>>   (merged to 'next' on 2013-01-14 at 9df2afc)\n>>>  + t0008: avoid brace expansion\n>>>  + add git-check-ignore sub-command\n>>>  + setup.c: document get_pathspec()\n>>>  + add.c: extract new die_if_path_beyond_symlink() for reuse\n>>>  + add.c: extract check_path_for_gitlink() from treat_gitlinks() for reuse\n>>>  + pathspec.c: rename newly public functions for clarity\n>>>  + add.c: move pathspec matchers into new pathspec.c for reuse\n>>>  + add.c: remove unused argument from validate_pathspec()\n>>>  + dir.c: improve docs for match_pathspec() and match_pathspec_depth()\n>>>  + dir.c: provide clear_directory() for reclaiming dir_struct memory\n>>>  + dir.c: keep track of where patterns came from\n>>>  + dir.c: use a single struct exclude_list per source of excludes\n>>>\n>>>  Add a new command \"git check-ignore\" for debugging .gitignore\n>>>  files.\n>>\n>> The above is v4 plus the \"t0008: avoid brace expansion\" fix.  v4 is\n>> slightly outdated and not quite the right version to merge to 'next'.\n>\n> Sigh.\n>\n> The \"What's cooking\" is a report of what _has_ already happened.  I\n> would have appreciated if you said the above _before_ this happened.\n\nI did, 8 days ago in the link which you just trimmed from your reply:\n\n  http://thread.gmane.org/gmane.comp.version-control.git/212184/focus=212856\n\nThe additional issues with t0008 were discovered after I posted v4, as\nreflected in last Wednesday's \"What's cooking\" (#04):\n\n    \"The test it adds seems to break under dash.\n     Expecting a reroll or fixup.\"\n\nI assumed that for brevity you had deliberately omitted mentioning the\noutstanding dir.exclude_list_group[EXC_CMDL].el[0] encapsulation\nissue linked above, so I thought we were aligned at this point.\n\nHowever I see now that you changed the status to an intention to merge\nthis to 'next' in last Friday's \"What's cooking\" (#05).  That gave me\na window of under 72 hours in which to reiterate the need for a final\nre-roll.  Unfortunately with other commitments and illness over the\nweekend, I didn't catch this in time.\n\nHowever, the damage is very small:\n\n>> I'll post a v5 re-roll as per:\n>\n> Now the series is in 'next', it is too late to _replace_ it X-<.\n> Could you instead make an incremental updates on top?  That way, we\n> do not have to re-review the whole thing; we only need to review the\n> changes relative to the old one, making sure that the fixes in the\n> updates are better than the v4 version.\n\nSure, that's easy to do.  It'll be a single small patch very similar\nto this one:\n\n  http://article.gmane.org/gmane.comp.version-control.git/212852\n\nminus the superfluous printf() debug statements.  I'll do that now.\n\nThanks,\nAdam\n"},{"id":"207051","messageId":"1358342758-30503-1-git-send-email-git@adamspiers.org","threadId":"32631","inReplyTo":"CAOkDyE-p9WLrsFZjPb9sY+YEypkF2wDxMybBkCT-76jBbKOmCA@mail.gmail.com","subject":"[PATCH] clean.c, ls-files.c: respect encapsulation of exclude_list_groups","fromName":"Adam Spiers","fromEmail":"git@adamspiers.org","sentAt":"2013-01-16T13:25:58Z","receivedAt":"2013-01-16T13:25:58Z","isPatch":true,"sender":{"key":"git@adamspiers.org","avatar":"https://avatars.githubusercontent.com/u/100738?v=4"},"body":"Consumers of the dir.c traversal API should avoid assuming knowledge\nof the internal implementation of exclude_list_groups.  Therefore\nwhen adding items to an exclude list, it should be accessed via the\npointer returned from add_exclude_list(), rather than by referencing\na location within dir.exclude_list_groups[EXC_CMDL].\n\nSigned-off-by: Adam Spiers <git@adamspiers.org>\n---\n builtin/clean.c    |  6 +++---\n builtin/ls-files.c | 15 ++++++++++-----\n 2 files changed, 13 insertions(+), 8 deletions(-)\n\ndiff --git a/builtin/clean.c b/builtin/clean.c\nindex b098288..b9cb7ad 100644\n--- a/builtin/clean.c\n+++ b/builtin/clean.c\n@@ -45,6 +45,7 @@ int cmd_clean(int argc, const char **argv, const char *prefix)\n \tstatic const char **pathspec;\n \tstruct strbuf buf = STRBUF_INIT;\n \tstruct string_list exclude_list = STRING_LIST_INIT_NODUP;\n+\tstruct exclude_list *el;\n \tconst char *qname;\n \tchar *seen = NULL;\n \tstruct option options[] = {\n@@ -97,10 +98,9 @@ int cmd_clean(int argc, const char **argv, const char *prefix)\n \tif (!ignored)\n \t\tsetup_standard_excludes(&dir);\n \n-\tadd_exclude_list(&dir, EXC_CMDL, \"--exclude option\");\n+\tel = add_exclude_list(&dir, EXC_CMDL, \"--exclude option\");\n \tfor (i = 0; i < exclude_list.nr; i++)\n-\t\tadd_exclude(exclude_list.items[i].string, \"\", 0,\n-\t\t\t    &dir.exclude_list_group[EXC_CMDL].el[0], -(i+1));\n+\t\tadd_exclude(exclude_list.items[i].string, \"\", 0, el, -(i+1));\n \n \tpathspec = get_pathspec(prefix, argv);\n \ndiff --git a/builtin/ls-files.c b/builtin/ls-files.c\nindex fa9ccb8..b4d8b01 100644\n--- a/builtin/ls-files.c\n+++ b/builtin/ls-files.c\n@@ -421,10 +421,10 @@ static int option_parse_z(const struct option *opt,\n static int option_parse_exclude(const struct option *opt,\n \t\t\t\tconst char *arg, int unset)\n {\n-\tstruct exclude_list_group *group = opt->value;\n+\tstruct string_list *exclude_list = opt->value;\n \n \texc_given = 1;\n-\tadd_exclude(arg, \"\", 0, &group->el[0], --exclude_args);\n+\tstring_list_append(exclude_list, arg);\n \n \treturn 0;\n }\n@@ -453,9 +453,11 @@ static int option_parse_exclude_standard(const struct option *opt,\n \n int cmd_ls_files(int argc, const char **argv, const char *cmd_prefix)\n {\n-\tint require_work_tree = 0, show_tag = 0;\n+\tint require_work_tree = 0, show_tag = 0, i;\n \tconst char *max_prefix;\n \tstruct dir_struct dir;\n+\tstruct exclude_list *el;\n+\tstruct string_list exclude_list = STRING_LIST_INIT_NODUP;\n \tstruct option builtin_ls_files_options[] = {\n \t\t{ OPTION_CALLBACK, 'z', NULL, NULL, NULL,\n \t\t\t\"paths are separated with NUL character\",\n@@ -490,7 +492,7 @@ int cmd_ls_files(int argc, const char **argv, const char *cmd_prefix)\n \t\tOPT_BOOLEAN(0, \"resolve-undo\", &show_resolve_undo,\n \t\t\t    \"show resolve-undo information\"),\n \t\t{ OPTION_CALLBACK, 'x', \"exclude\",\n-\t\t\t&dir.exclude_list_group[EXC_CMDL], \"pattern\",\n+\t\t\t&exclude_list, \"pattern\",\n \t\t\t\"skip files matching pattern\",\n \t\t\t0, option_parse_exclude },\n \t\t{ OPTION_CALLBACK, 'X', \"exclude-from\", &dir, \"file\",\n@@ -525,9 +527,12 @@ int cmd_ls_files(int argc, const char **argv, const char *cmd_prefix)\n \tif (read_cache() < 0)\n \t\tdie(\"index file corrupt\");\n \n-\tadd_exclude_list(&dir, EXC_CMDL, \"--exclude option\");\n \targc = parse_options(argc, argv, prefix, builtin_ls_files_options,\n \t\t\tls_files_usage, 0);\n+\tel = add_exclude_list(&dir, EXC_CMDL, \"--exclude option\");\n+\tfor (i = 0; i < exclude_list.nr; i++) {\n+\t\tadd_exclude(exclude_list.items[i].string, \"\", 0, el, --exclude_args);\n+\t}\n \tif (show_tag || show_valid_bit) {\n \t\ttag_cached = \"H \";\n \t\ttag_unmerged = \"M \";\n-- \n1.8.1.291.g0730ed6\n"},{"id":"207076","messageId":"7v7gndxbu5.fsf@alter.siamese.dyndns.org","threadId":"32631","inReplyTo":"1358342758-30503-1-git-send-email-git@adamspiers.org","subject":"Re: [PATCH] clean.c, ls-files.c: respect encapsulation of exclude_list_groups","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2013-01-16T17:43:46Z","receivedAt":"2013-01-16T17:43:46Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Adam Spiers <git@adamspiers.org> writes:\n\n> Consumers of the dir.c traversal API should avoid assuming knowledge\n> of the internal implementation of exclude_list_groups.  Therefore\n> when adding items to an exclude list, it should be accessed via the\n> pointer returned from add_exclude_list(), rather than by referencing\n> a location within dir.exclude_list_groups[EXC_CMDL].\n\nSounds sensible.\n"},{"id":"207110","messageId":"7v1udkwz1f.fsf@alter.siamese.dyndns.org","threadId":"32631","inReplyTo":"1358342758-30503-1-git-send-email-git@adamspiers.org","subject":"Re: [PATCH] clean.c, ls-files.c: respect encapsulation of exclude_list_groups","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2013-01-16T22:20:12Z","receivedAt":"2013-01-16T22:20:12Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Adam Spiers <git@adamspiers.org> writes:\n\n> Consumers of the dir.c traversal API should avoid assuming knowledge\n> of the internal implementation of exclude_list_groups.  Therefore\n> when adding items to an exclude list, it should be accessed via the\n> pointer returned from add_exclude_list(), rather than by referencing\n> a location within dir.exclude_list_groups[EXC_CMDL].\n>\n> Signed-off-by: Adam Spiers <git@adamspiers.org>\n> ---\n>  builtin/clean.c    |  6 +++---\n>  builtin/ls-files.c | 15 ++++++++++-----\n>  2 files changed, 13 insertions(+), 8 deletions(-)\n>\n> diff --git a/builtin/clean.c b/builtin/clean.c\n> index b098288..b9cb7ad 100644\n> --- a/builtin/clean.c\n> +++ b/builtin/clean.c\n> @@ -45,6 +45,7 @@ int cmd_clean(int argc, const char **argv, const char *prefix)\n>  \tstatic const char **pathspec;\n>  \tstruct strbuf buf = STRBUF_INIT;\n>  \tstruct string_list exclude_list = STRING_LIST_INIT_NODUP;\n> +\tstruct exclude_list *el;\n\nWhen a type \"exclude_list\" exists and used in the same function,\nhaving a local variable of the same name but of a different type\nbecomes a bit awkward.\n\nbuiltin/ls-files.c shares the same structure.  Does the file-scope\n\"exclude_args\" variable need to be a file-scope static over there?\nIt seems that it is closely tied to the elements of the string list,\nso it may make sense to:\n\n    * remove the file-scope static \"exclude_args\";\n\n    * rename \"exclude_list\" string list variable to \"exclude_args\";\n      and\n\n    * replace \"--exclude_args\" in the loop that iterates over\n      exclude_list (now exclude_args) with \"-(i+1)\" or something,\n      just like you do in \"builtin/clean.c\" below.\n\n> -\tadd_exclude_list(&dir, EXC_CMDL, \"--exclude option\");\n> +\tel = add_exclude_list(&dir, EXC_CMDL, \"--exclude option\");\n>  \tfor (i = 0; i < exclude_list.nr; i++)\n> -\t\tadd_exclude(exclude_list.items[i].string, \"\", 0,\n> -\t\t\t    &dir.exclude_list_group[EXC_CMDL].el[0], -(i+1));\n> +\t\tadd_exclude(exclude_list.items[i].string, \"\", 0, el, -(i+1));\n\nWe may want to use for_each_string_list_item() here and in the\ncorresponding loop in builtin/ls-files.c, but because we do need to\ngive the -(i + 1) label to each element, I think the code is OK\nas-is.\n"}]}