{"thread":{"id":"47377","subject":"What's cooking in git.git (Dec 2017, #01; Mon, 4)","startedAt":"2017-12-04T21:46:56Z","lastAt":"2017-12-23T07:27:46Z","messageCount":26,"participants":["Junio C Hamano","Jeff Hostetler","Elijah Newren","Jonathan Tan","Lars Schneider","Torsten Bögershausen","Johannes Schindelin","Todd Zullinger","tboegi@web.de","Eric Sunshine"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"334096","messageId":"xmqqmv2ykvy4.fsf@gitster.mtv.corp.google.com","threadId":"47377","inReplyTo":null,"subject":"What's cooking in git.git (Dec 2017, #01; Mon, 4)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2017-12-04T21:46:43Z","receivedAt":"2017-12-04T21:46: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'. The ones marked with '.' do not appear in any of\nthe integration branches, but I am still holding onto them.\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* ab/simplify-perl-makefile (2017-12-04) 1 commit\n - Makefile: replace perl/Makefile.PL with simple make rules\n\n The build procedure for perl/ part has been greatly simplified by\n weaning ourselves off of MakeMaker.\n\n\n* cc/object-filtering-typofix (2017-12-04) 1 commit\n - list-objects-filter-options: fix 'keword' typo in comment\n (this branch uses jh/object-filtering; is tangled with jh/fsck-promisors and jh/partial-clone.)\n\n Typofix for a topic already in 'next'.\n\n JeffH said that jh/object-filtering needs further polishing a bit\n before graduating to 'master', so it would be appreciated if this\n can also be rolled into such an incremental update.\n\n\n* cc/skip-to-optional-val (2017-12-04) 4 commits\n - diff: use skip_to_optional_val_default()\n - diff: use skip_to_optional_val()\n - index-pack: use skip_to_optional_val()\n - git-compat-util: introduce skip_to_optional_val()\n\n Introduce a helper to simplify code to parse a common pattern that\n expects either \"--key\" or \"--key=\".\n\n This may want a final reroll to make it harder to misuse by\n allowing NULL at the valp part of the argument.\n cf. \n cf. \n\n\n* lb/rebase-i-short-command-names (2017-12-04) 9 commits\n - t3404: add test case for abbreviated commands\n - rebase -i: learn to abbreviate command names\n - rebase -i -x: add exec commands via the rebase--helper\n - rebase -i: update functions to use a flags parameter\n - rebase -i: replace reference to sha1 with oid\n - rebase -i: refactor transform_todo_ids\n - rebase -i: set commit to null in exec commands\n - Documentation: use preferred name for the 'todo list' script\n - Documentation: move rebase.* configs to new file\n\n Allow a single-letter command name in the \"rebase -i\" todo list.\n\n This may want a final reroll to avoid adding new reference to insn\n where todo is the more modern term, among other minor things.\n cf. \n\n\n* ra/prompt-eread-fix (2017-12-04) 2 commits\n - git-prompt: fix reading files with windows line endings\n - git-prompt: make __git_eread intended use explicit\n\n Update the shell prompt script (in contrib/) to strip trailing CR\n from strings read from various \"state\" files.\n\n Proposed log messages of both commits may need small fixes.\n cf. \n cf. \n\n\n* ds/for-each-file-in-obj-micro-optim (2017-12-04) 1 commit\n - sha1_file: use strbuf_add() instead of strbuf_addf()\n\n The code to iterate over loose object files got optimized.\n\n Will merge to 'next'.\n\n--------------------------------------------------\n[Stalled]\n\n* jc/merge-symlink-ours-theirs (2017-09-26) 1 commit\n - merge: teach -Xours/-Xtheirs to symbolic link merge\n\n \"git merge -Xours/-Xtheirs\" learned to use our/their version when\n resolving a conflicting updates to a symbolic link.\n\n Needs review.\n\n\n* mg/merge-base-fork-point (2017-09-17) 3 commits\n - merge-base: find fork-point outside partial reflog\n - merge-base: return fork-point outside reflog\n - t6010: test actual test output\n\n \"merge-base --fork-point $branch $commit\" is used to guess on which\n commit among the commits that were once at the tip of the $branch the\n $commit was built on top of, and it learns these historical tips from\n the reflog of the $branch. When the true fork-point is lost due to\n pruning of old reflog entries, the command does not give any output,\n because it has no way to guess correctly and does not want to mislead\n the user with a wrong guess.\n\n The command has been updated to give the best but not known to be\n correct guess, based on a hope that a merge-base between $commit and a\n virtual merge across all the reflog entries that still are available\n for $branch may still be a closer to the true fork-point than the\n merge-base between $commit and the current tip of the $branch.\n\n This may have to be offered by an additional option, to allow the\n users that are prepared to see a potentially incorrect guess to opt\n into the feature, without affecting the current callers that may not\n be prepared to accept a guess that is not known to be correct.\n\n What's the doneness of this one?\n\n\n* jk/drop-ancient-curl (2017-08-09) 5 commits\n - http: #error on too-old curl\n - curl: remove ifdef'd code never used with curl >=7.19.4\n - http: drop support for curl < 7.19.4\n - http: drop support for curl < 7.16.0\n - http: drop support for curl < 7.11.1\n\n Some code in http.c that has bitrot is being removed.\n\n Expecting a reroll.\n\n\n* mk/use-size-t-in-zlib (2017-08-10) 1 commit\n . zlib.c: use size_t for size\n\n The wrapper to call into zlib followed our long tradition to use\n \"unsigned long\" for sizes of regions in memory, which have been\n updated to use \"size_t\".\n\n Needs resurrecting by making sure the fix is good and still applies\n (or adjusted to today's codebase).\n\n\n* mg/status-in-progress-info (2017-05-10) 2 commits\n - status --short --inprogress: spell it as --in-progress\n - status: show in-progress info for short status\n\n \"git status\" learns an option to report various operations\n (e.g. \"merging\") that the user is in the middle of.\n\n cf. \n\n\n* nd/worktree-move (2017-04-20) 6 commits\n - worktree remove: new command\n - worktree move: refuse to move worktrees with submodules\n - worktree move: accept destination as directory\n - worktree move: new command\n - worktree.c: add update_worktree_location()\n - worktree.c: add validate_worktree()\n\n \"git worktree\" learned move and remove subcommands.\n\n Expecting a reroll.\n cf. <20170420101024.7593-1-pclouds@gmail.com>\n cf. <20170421145916.mknekgqzhxffu7di@sigill.intra.peff.net>\n cf. \n\n--------------------------------------------------\n[Cooking]\n\n* en/merge-recursive-icase-removal (2017-11-27) 1 commit\n - merge-recursive: ignore_case shouldn't reject intentional removals\n\n The code internal to the recursive merge strategy was not fully\n prepared to see a path that is renamed to try overwriting another\n path that is only different in case on case insensitive systems.\n This does not matter in the current code, but will start to matter\n once the rename detection logic starts taking hints from nearby\n paths moving to some directory and moves a new path along with them.\n\n\n* gk/tracing-optimization (2017-11-27) 3 commits\n - SQUASH??? compilation fix-up\n - trace: improve performance while category is disabled\n - trace: remove trace key normalization\n\n The tracing infrastructure has been optimized for cases where no\n tracing is requested.\n\n\n* pc/submodule-helper (2017-11-26) 1 commit\n - submodule--helper.c: i18n: add a missing space in message\n\n A message fix.\n\n Will merge to 'next'.\n\n\n* tb/check-crlf-for-safe-crlf (2017-11-27) 1 commit\n - convert: tighten the safe autocrlf handling\n\n The \"safe crlf\" check incorrectly triggered for contents that does\n not use CRLF as line endings, which has been corrected.\n\n Will merge to 'next'.\n\n\n* tg/t-readme-updates (2017-11-27) 2 commits\n - t/README: document test_cmp_rev\n - t/README: remove mention of adding copyright notices\n\n Developer doc updates.\n\n Will merge to 'next'.\n\n\n* cc/require-tcl-tk-for-build (2017-11-29) 2 commits\n - travis-ci: avoid new tcl/tk build requirement\n - Makefile: check that tcl/tk is installed\n\n A first-time builder of Git may have installed neither tclsh nor\n msgfmt, in which case git-gui and gitk part will fail and break the\n build. As a workaround, refuse to run a build when tclsh is not\n installed and NO_TCLTK is not set.\n\n Undecided.\n I still feel that requring tclsh to be installed, with or without\n \"escape hatch\" for experts, may be too heavy-handed.\n\n\n* mk/http-backend-content-length (2017-11-27) 4 commits\n - SQUASH???\n - t5560-http-backend-noserver.sh: add CONTENT_LENGTH cases\n - SQUASH???\n - http-backend: respect CONTENT_LENGTH as specified by rfc3875\n\n The http-backend (used for smart-http transport) used to slurp the\n whole input until EOF, without paying attention to CONTENT_LENGTH\n that is supplied in the environment and instead expecting the Web\n server to close the input stream. This has been fixed.\n\n Expecting a reroll.\n Suggested fixes to be used when rerolling is queued, but I'd\n prefer _not_ squashing them myself.\n\n\n* fk/sendmail-from-path (2017-11-28) 1 commit\n - git-send-email: honor $PATH for sendmail binary\n\n \"git send-email\" tries to see if the sendmail program is available\n in /usr/lib and /usr/sbin; extend the list of locations to be\n checked to also include directories on $PATH.\n\n Will merge to 'next'.\n\n\n* jk/no-optional-locks (2017-11-27) 1 commit\n - git-status.txt: mention --no-optional-locks\n\n Doc update for a feature available in Git v2.14 and upwards.\n\n\n* jt/diff-anchored-patience (2017-11-28) 1 commit\n - diff: support anchoring line(s)\n\n \"git diff\" learned a variant of the \"--patience\" algorithm, to\n which the user can specify which 'unique' line to be used as\n anchoring points.\n\n\n* ks/doc-checkout-previous (2017-11-28) 1 commit\n - Doc/checkout: checking out using @{-N} can lead to detached state\n\n @{-N} in \"git checkout @{-N}\" may refer to a detached HEAD state,\n but the documentation was not clear about it, which has been fixed.\n\n Will merge to 'next'.\n\n\n* en/rename-directory-detection (2017-11-21) 33 commits\n - merge-recursive: ensure we write updates for directory-renamed file\n - merge-recursive: avoid spurious rename/rename conflict from dir renames\n - directory rename detection: new testcases showcasing a pair of bugs\n - merge-recursive: fix remaining directory rename + dirty overwrite cases\n - merge-recursive: fix overwriting dirty files involved in renames\n - merge-recursive: avoid clobbering untracked files with directory renames\n - merge-recursive: apply necessary modifications for directory renames\n - merge-recursive: when comparing files, don't include trees\n - merge-recursive: check for file level conflicts then get new name\n - merge-recursive: add computation of collisions due to dir rename & merging\n - merge-recursive: add a new hashmap for storing file collisions\n - merge-recursive: check for directory level conflicts\n - merge-recursive: add get_directory_renames()\n - merge-recursive: add a new hashmap for storing directory renames\n - merge-recursive: split out code for determining diff_filepairs\n - merge-recursive: make !o->detect_rename codepath more obvious\n - merge-recursive: fix leaks of allocated renames and diff_filepairs\n - merge-recursive: introduce new functions to handle rename logic\n - merge-recursive: move the get_renames() function\n - directory rename detection: tests for handling overwriting dirty files\n - directory rename detection: tests for handling overwriting untracked files\n - directory rename detection: miscellaneous testcases to complete coverage\n - directory rename detection: testcases exploring possibly suboptimal merges\n - directory rename detection: more involved edge/corner testcases\n - directory rename detection: testcases checking which side did the rename\n - directory rename detection: files/directories in the way of some renames\n - directory rename detection: partially renamed directory testcase/discussion\n - directory rename detection: testcases to avoid taking detection too far\n - directory rename detection: directory splitting testcases\n - directory rename detection: basic testcases\n - merge-recursive: add explanation for src_entry and dst_entry\n - merge-recursive: fix logic ordering issue\n - Tighten and correct a few testcases for merging and cherry-picking\n\n Rename detection logic in \"diff\" family that is used in \"merge\" has\n learned to guess when all of x/a, x/b and x/c have moved to z/a,\n z/b and z/c, it is likely that x/d added in the meantime would also\n want to move to z/d by taking the hint that the entire directory\n 'x' moved to 'z'.\n\n Needs review.\n\n\n* ac/complete-pull-autostash (2017-11-22) 1 commit\n (merged to 'next' on 2017-11-27 at 802d204eda)\n + completion: add --autostash and --no-autostash to pull\n\n The shell completion (in contrib/) learned that \"git pull\" can take\n the \"--autostash\" option.\n\n Will merge to 'master'.\n\n\n* cc/git-packet-pm (2017-11-22) 2 commits\n (merged to 'next' on 2017-11-27 at 1527ab3519)\n + Git/Packet.pm: use 'if' instead of 'unless'\n + Git/Packet: clarify that packet_required_key_val_read allows EOF\n\n Code clean-up.\n\n Will merge to 'master'.\n\n\n* jn/reproducible-build (2017-11-22) 3 commits\n (merged to 'next' on 2017-11-27 at 6ae6946f8c)\n + Merge branch 'jn/reproducible-build' of ../git-gui into jn/reproducible-build\n + git-gui: sort entries in optimized tclIndex\n + generate-cmdlist: avoid non-deterministic output\n\n The build procedure has been taught to avoid some unnecessary\n instability in the build products.\n\n Will merge to 'master'.\n\n\n* jt/submodule-tests-cleanup (2017-11-22) 1 commit\n (merged to 'next' on 2017-11-27 at 5f0e4bcde1)\n + Tests: clean up submodule recursive helpers\n\n Further test clean-up.\n\n Will merge to 'master'.\n\n\n* rd/man-prune-progress (2017-11-22) 1 commit\n (merged to 'next' on 2017-11-27 at 1680e6a2f5)\n + prune: add \"--progress\" to man page and usage msg\n\n Doc update.\n\n Will merge to 'master'.\n\n\n* rd/man-reflog-add-n (2017-11-22) 1 commit\n (merged to 'next' on 2017-11-27 at eacb065aa6)\n + doc: add missing \"-n\" (dry-run) option to reflog man page\n\n Doc update.\n\n Will merge to 'master'.\n\n\n* ph/stash-save-m-option-fix (2017-11-24) 1 commit\n (merged to 'next' on 2017-11-27 at 96eadf9372)\n + stash: learn to parse -m/--message like commit does\n\n In addition to \"git stash -m message\", the command learned to\n accept \"git stash -mmessage\" form.\n\n Will merge to 'master'.\n\n\n* ra/decorate-limit-refs (2017-11-22) 1 commit\n - log: add option to choose which refs to decorate\n\n The tagnames \"git log --decorate\" uses to annotate the commits can\n now be limited to subset of available refs with the two additional\n options, --decorate-refs[-exclude]=.\n\n Will merge to 'next'.\n\n\n* rd/doc-notes-prune-fix (2017-11-22) 1 commit\n (merged to 'next' on 2017-11-27 at 4a657474ee)\n + notes: correct 'git notes prune' options to '[-n] [-v]'\n\n Doc update.\n\n Will merge to 'master'.\n\n\n* tg/deprecate-stash-save (2017-11-22) 1 commit\n (merged to 'next' on 2017-11-27 at 6bcb61b267)\n + doc: prefer 'stash push' over 'stash save'\n\n Doc update.\n\n Will merge to 'master'.\n\n\n* ab/pcre2-grep (2017-11-24) 2 commits\n - grep: fix segfault under -P + PCRE2 <=10.30 + (*NO_JIT)\n - test-lib: add LIBPCRE1 & LIBPCRE2 prerequisites\n\n \"git grep\" compiled with libpcre2 sometimes triggered a segfault,\n which is being fixed.\n\n Will merge to 'next'.\n\n\n* jc/receive-pack-hook-doc (2017-11-24) 1 commit\n - hooks doc: clarify when receive-pack invokes its hooks\n\n Doc update.\n\n Will merge to 'next'.\n\n\n* sb/diff-blobfind (2017-11-24) 1 commit\n - diff: add a new diffcore transformation to find a specific blob\n\n \"git diff\" family of commands learned --blobfind= that\n allows you to limit the output only to a change that involves the\n named blob object (either changing the contents from or to it).\n\n Needs review.\n\n\n* sp/doc-info-attributes (2017-11-24) 1 commit\n (merged to 'next' on 2017-11-27 at 4e509c2d25)\n + doc: Mention info/attributes in gitrepository-layout\n\n Doc update.\n\n Will merge to 'master'.\n\n\n* tg/worktree-create-tracking (2017-11-27) 6 commits\n - add worktree.guessRemote config option\n - worktree: add --guess-remote flag to add subcommand\n - worktree: make add dwim\n - worktree: add --[no-]track option to the add subcommand\n - worktree: add can be created from any commit-ish\n - checkout: factor out functions to new lib file\n\n The way \"git worktree add\" determines what branch to create from\n where and checkout in the new worktree has been updated a bit.\n\n\n* cc/perf-run-config (2017-09-24) 9 commits\n (merged to 'next' on 2017-11-27 at d75a2469eb)\n + perf: store subsection results in \"test-results/$GIT_PERF_SUBSECTION/\"\n + perf/run: show name of rev being built\n + perf/run: add run_subsection()\n + perf/run: update get_var_from_env_or_config() for subsections\n + perf/run: add get_subsections()\n + perf/run: add calls to get_var_from_env_or_config()\n + perf/run: add GIT_PERF_DIRS_OR_REVS\n + perf/run: add get_var_from_env_or_config()\n + perf/run: add '--config' option to the 'run' script\n\n Will merge to 'master'.\n\n\n* pw/sequencer-in-process-commit (2017-11-24) 9 commits\n - t3512/t3513: remove KNOWN_FAILURE_CHERRY_PICK_SEES_EMPTY_COMMIT=1\n - sequencer: try to commit without forking 'git commit'\n - sequencer: load commit related config\n - sequencer: simplify adding Signed-off-by: trailer\n - commit: move print_commit_summary() to libgit\n - commit: move post-rewrite code to libgit\n - Add a function to update HEAD after creating a commit\n - commit: move empty message checks to libgit\n - t3404: check intermediate squash messages\n\n The sequencer infrastructure is shared across \"git cherry-pick\",\n \"git rebase -i\", etc., and has always spawned \"git commit\" when it\n needs to create a commit. It has been taught to do so internally,\n when able, by reusing the codepath \"git commit\" itself uses, which\n gives performance boost for a few tens of percents in some sample\n scenarios.\n\n Will merge to and cook in 'next'.\n\n\n* jk/fewer-pack-rescan (2017-11-22) 5 commits\n (merged to 'next' on 2017-11-27 at 2c35a2d831)\n + sha1_file: fast-path null sha1 as a missing object\n + everything_local: use \"quick\" object existence check\n + p5551: add a script to test fetch pack-dir rescans\n + t/perf/lib-pack: use fast-import checkpoint to create packs\n + p5550: factor out nonsense-pack creation\n\n Internaly we use 0{40} as a placeholder object name to signal the\n codepath that there is no such object (e.g. the fast-forward check\n while \"git fetch\" stores a new remote-tracking ref says \"we know\n there is no 'old' thing pointed at by the ref, as we are creating\n it anew\" by passing 0{40} for the 'old' side), and expect that a\n codepath to locate an in-core object to return NULL as a sign that\n the object does not exist. A look-up for an object that does not\n exist however is quite costly with a repository with large number\n of packfiles. This access pattern has been optimized.\n\n Will merge to 'master'.\n\n\n* jn/ssh-wrappers (2017-11-21) 9 commits\n (merged to 'next' on 2017-11-27 at 00a2bb7a3c)\n + connect: correct style of C-style comment\n + ssh: 'simple' variant does not support --port\n + ssh: 'simple' variant does not support -4/-6\n + ssh: 'auto' variant to select between 'ssh' and 'simple'\n + connect: split ssh option computation to its own function\n + connect: split ssh command line options into separate function\n + connect: split git:// setup into a separate function\n + connect: move no_fork fallback to git_tcp_connect\n + ssh test: make copy_ssh_wrapper_as clean up after itself\n (this branch uses bw/protocol-v1.)\n\n The ssh-variant 'simple' introduced earlier broke existing\n installations by not passing --port/-4/-6 and not diagnosing an\n attempt to pass these as an error. Instead, default to\n automatically detect how compatible the GIT_SSH/GIT_SSH_COMMAND is\n to OpenSSH convention and then error out an invocation to make it\n easier to diagnose connection errors.\n\n Will merge to 'master'.\n\n\n* hm/config-parse-expiry-date (2017-11-18) 1 commit\n (merged to 'next' on 2017-11-27 at 20014f5541)\n + config: add --expiry-date\n\n \"git config --expiry-date gc.reflogexpire\" can read \"2.weeks\" from\n the configuration and report it as a timestamp, just like \"--int\"\n would read \"1k\" and report 1024, to help consumption by scripts.\n\n Will merge to 'master'.\n\n\n* dj/runtime-prefix (2017-11-28) 4 commits\n . exec_cmd: RUNTIME_PREFIX on some POSIX systems\n . Makefile: add Perl runtime prefix support\n . Makefile: add support for \"perllibdir\"\n . Makefile: generate Perl header from template file\n\n A build-time option has been added to allow Git to be told to refer\n to its associated files relative to the main binary, in the same\n way that has been possible on Windows for quite some time, for\n Linux, BSDs and Darwin.\n\n Tentatively kicked out of 'next' to see how well another topic\n ab/simplify-perl-makefile that heavily conflicts with this fares.\n\n\n* ls/editor-waiting-message (2017-12-04) 2 commits\n - launch_editor(): indicate that Git waits for user input\n - refactor \"dumb\" terminal determination\n\n Git shows a message to tell the user that it is waiting for the\n user to finish editing when spawning an editor, in case the editor\n opens to a hidden window or somewhere obscure and the user gets\n lost.\n\n Still being bikeshed on, but hopefully the final is getting closer.\n This is at its v4.\n\n\n* tz/branch-doc-remove-set-upstream (2017-11-17) 1 commit\n (merged to 'next' on 2017-11-27 at 8fcad3f6e7)\n + branch doc: remove --set-upstream from synopsis\n\n \"git branch --set-upstream\" has been deprecated and (sort of)\n removed, as \"--set-upstream-to\" is the preferred one these days.\n The documentation still had \"--set-upstream\" listed on its\n synopsys section, which has been corrected.\n\n Will merge to 'master'.\n\n\n* en/rename-progress (2017-12-02) 5 commits\n (merged to 'next' on 2017-12-04 at 49b39d2297)\n + diffcore-rename: make diff-tree -l0 mean -l\n (merged to 'next' on 2017-11-20 at 77a2e0ddd9)\n + sequencer: show rename progress during cherry picks\n + diff: remove silent clamp of renameLimit\n + progress: fix progress meters when dealing with lots of work\n + sequencer: warn when internal merge may be suboptimal due to renameLimit\n\n Historically, the diff machinery for rename detection had a\n hardcoded limit of 32k paths; this is being lifted to allow users\n trade cycles with a (possibly) easier to read result.\n\n Will merge to 'master'.\n\n\n* ar/unconfuse-three-dots (2017-12-04) 7 commits\n - t4013: test new output from diff --abbrev --raw\n - diff: diff_aligned_abbrev: remove ellipsis after abbreviated SHA-1 value\n - t4013: prepare for upcoming \"diff --raw --abbrev\" output format change\n - checkout: describe_detached_head: remove ellipsis after committish\n - print_sha1_ellipsis: introduce helper\n - Documentation: user-manual: limit usage of ellipsis\n - Documentation: revisions: fix typo: \"three dot\" ---> \"three-dot\" (in line with \"two-dot\").\n\n Ancient part of codebase still shows dots after an abbreviated\n object name just to show that it is not a full object name, but\n these ellipses are confusing to people who newly discovered Git\n who are used to seeing abbreviated object names and find them\n confusing with the range syntax.\n\n Almost there.\n cf. \n\n\n* jh/object-filtering (2017-11-22) 6 commits\n (merged to 'next' on 2017-11-27 at e5008c3b28)\n + pack-objects: add list-objects filtering\n + rev-list: add list-objects filtering support\n + list-objects: filter objects in traverse_commit_list\n + oidset: add iterator methods to oidset\n + oidmap: add oidmap iterator methods\n + dir: allow exclusions from blob in addition to file\n (this branch is used by cc/object-filtering-typofix, jh/fsck-promisors and jh/partial-clone.)\n\n In preparation for implementing narrow/partial clone, the object\n walking machinery has been taught a way to tell it to \"filter\" some\n objects from enumeration.\n\n Expecting incremental updates for a finishing touch.\n\n\n* jh/fsck-promisors (2017-11-22) 10 commits\n - gc: do not repack promisor packfiles\n - rev-list: support termination at promisor objects\n - sha1_file: support lazily fetching missing objects\n - introduce fetch-object: fetch one promisor object\n - index-pack: refactor writing of .keep files\n - fsck: support promisor objects as CLI argument\n - fsck: support referenced promisor objects\n - fsck: support refs pointing to promisor objects\n - fsck: introduce partialclone extension\n - extension.partialclone: introduce partial clone extension\n (this branch is used by jh/partial-clone; uses jh/object-filtering; is tangled with cc/object-filtering-typofix.)\n\n In preparation for implementing narrow/partial clone, the machinery\n for checking object connectivity used by gc and fsck has been\n taught that a missing object is OK when it is referenced by a\n packfile specially marked as coming from trusted repository that\n promises to make them available on-demand and lazily.\n\n\n* jh/partial-clone (2017-11-22) 14 commits\n - fetch-pack: restore save_commit_buffer after use\n - unpack-trees: batch fetching of missing blobs\n - t5500: more tests for partial clone and fetch\n - t5601: test for partial clone\n - t5500: add fetch-pack tests for partial clone\n - fetch: add from_promisor and exclude-promisor-objects parameters\n - partial-clone: define partial clone settings in config\n - fetch-pack: test support excluding large blobs\n - pack-objects: test support for blob filtering\n - remote-curl: add object filtering for partial clone\n - fetch: add object filtering for partial fetch\n - fetch: refactor calculation of remote list\n - clone, fetch-pack, index-pack, transport: partial clone\n - upload-pack: add object filtering for partial clone\n (this branch uses jh/fsck-promisors and jh/object-filtering; is tangled with cc/object-filtering-typofix.)\n\n The machinery to clone & fetch, which in turn involves packing and\n unpacking objects, have been told how to omit certain objects using\n the filtering mechanism introduced by the jh/object-filtering\n topic, and also mark the resulting pack as a promisor pack to\n tolerate missing objects, taking advantage of the mechanism\n introduced by the jh/fsck-promisors topic.\n\n\n* sb/describe-blob (2017-11-16) 7 commits\n - builtin/describe.c: describe a blob\n - builtin/describe.c: factor out describe_commit\n - builtin/describe.c: print debug statements earlier\n - builtin/describe.c: rename `oid` to avoid variable shadowing\n - revision.h: introduce blob/tree walking in order of the commits\n - list-objects.c: factor out traverse_trees_and_blobs\n - t6120: fix typo in test name\n\n \"git describe\" was taught to dig trees deeper to find a\n : that refers to a given blob object.\n\n\n* pb/bisect-helper-2 (2017-10-28) 8 commits\n - t6030: make various test to pass GETTEXT_POISON tests\n - bisect--helper: `bisect_start` shell function partially in C\n - bisect--helper: `get_terms` & `bisect_terms` shell function in C\n - bisect--helper: `bisect_next_check` shell function in C\n - bisect--helper: `check_and_set_terms` shell function in C\n - wrapper: move is_empty_file() and rename it as is_empty_or_missing_file()\n - bisect--helper: `bisect_write` shell function in C\n - bisect--helper: `bisect_reset` shell function in C\n\n Expecting a reroll.\n cf. <0102015f5e5ee171-f30f4868-886f-47a1-a4e4-b4936afc545d-000000@eu-west-1.amazonses.com>\n\n\n* ot/mru-on-list (2017-10-01) 1 commit\n - mru: use double-linked list from list.h\n\n The first step to getting rid of mru API and using the\n doubly-linked list API directly instead.\n\n Is there any remaining issues on this one?\n\n\n* jm/svn-pushmergeinfo-fix (2017-09-17) 1 commit\n - git-svn: fix svn.pushmergeinfo handling of svn+ssh usernames.\n\n \"git svn dcommit\" did not take into account the fact that a\n svn+ssh:// URL with a username@ (typically used for pushing) refers\n to the same SVN repository without the username@ and failed when\n svn.pushmergeinfo option is set.\n\n Needs a test, perhaps as a follow-up patch.\n\n Will merge to 'next'.\n\n\n* bw/protocol-v1 (2017-10-17) 11 commits\n (merged to 'next' on 2017-11-27 at 55040d09ec)\n + Documentation: document Extra Parameters\n + ssh: introduce a 'simple' ssh variant\n + i5700: add interop test for protocol transition\n + http: tell server that the client understands v1\n + connect: tell server that the client understands v1\n + connect: teach client to recognize v1 server response\n + upload-pack, receive-pack: introduce protocol version 1\n + daemon: recognize hidden request arguments\n + protocol: introduce protocol extension mechanisms\n + pkt-line: add packet_write function\n + connect: in ref advertisement, shallows are last\n (this branch is used by jn/ssh-wrappers.)\n\n A new mechanism to upgrade the wire protocol in place is proposed\n and demonstrated that it works with the older versions of Git\n without harming them.\n\n Will merge to 'master'.\n\n\n* bc/hash-algo (2017-11-28) 5 commits\n (merged to 'next' on 2017-11-28 at 5c74632345)\n + repository: fix a sparse 'using integer as NULL pointer' warning\n (merged to 'next' on 2017-11-27 at 98cc9ef2a8)\n + Switch empty tree and blob lookups to use hash abstraction\n + Integrate hash algorithm support with repo setup\n + Add structure representing hash algorithm\n + setup: expose enumerated repo info\n\n An infrastructure to define what hash function is used in Git is\n introduced, and an effort to plumb that throughout various\n codepaths has been started.\n\n Will merge to 'master'.\n\n\n* sb/submodule-recursive-checkout-detach-head (2017-07-28) 2 commits\n (merged to 'next' on 2017-11-27 at 6584ec1a65)\n + Documentation/checkout: clarify submodule HEADs to be detached\n + recursive submodules: detach HEAD from new state\n\n \"git checkout --recursive\" may overwrite and rewind the history of\n the branch that happens to be checked out in submodule\n repositories, which might not be desirable. Detach the HEAD but\n still allow the recursive checkout to succeed in such a case.\n\n Will merge to 'master'.\n"},{"id":"334099","messageId":"65f9d6af-7f74-afab-4ee8-17077c3e3207@jeffhostetler.com","threadId":"47377","inReplyTo":"xmqqmv2ykvy4.fsf@gitster.mtv.corp.google.com","subject":"Re: What's cooking in git.git (Dec 2017, #01; Mon, 4)","fromName":"Jeff Hostetler","fromEmail":"git@jeffhostetler.com","sentAt":"2017-12-04T22:04:39Z","receivedAt":"2017-12-04T22:04:46Z","isPatch":false,"sender":{"key":"git@jeffhostetler.com","avatar":null},"body":"\n\nOn 12/4/2017 4:46 PM, Junio C Hamano wrote:\n> * cc/object-filtering-typofix (2017-12-04) 1 commit\n> - list-objects-filter-options: fix 'keword' typo in comment\n> (this branch uses jh/object-filtering; is tangled with jh/fsck-promisors and jh/partial-clone.)\n> \n> Typofix for a topic already in 'next'.\n> \n> JeffH said that jh/object-filtering needs further polishing a bit\n> before graduating to 'master', so it would be appreciated if this\n> can also be rolled into such an incremental update.\n\nI've pulled this into my branch and will include it in my V6 update.\nThanks\njeff\n\n"},{"id":"334104","messageId":"CABPp-BEoiCrMwKf9u7pdaOkYzE5Q6daJs7N2U05pGjERHzu2Gg@mail.gmail.com","threadId":"47377","inReplyTo":"xmqqmv2ykvy4.fsf@gitster.mtv.corp.google.com","subject":"Re: What's cooking in git.git (Dec 2017, #01; Mon, 4)","fromName":"Elijah Newren","fromEmail":"newren@gmail.com","sentAt":"2017-12-04T22:26:51Z","receivedAt":"2017-12-04T22:27:11Z","isPatch":false,"sender":{"key":"newren@gmail.com","avatar":"https://avatars.githubusercontent.com/u/5455730?v=4"},"body":"On Mon, Dec 4, 2017 at 1:46 PM, Junio C Hamano wrote:\n\n> * en/rename-directory-detection (2017-11-21) 33 commits\n\n> Rename detection logic in \"diff\" family that is used in \"merge\" has\n> learned to guess when all of x/a, x/b and x/c have moved to z/a,\n> z/b and z/c, it is likely that x/d added in the meantime would also\n> want to move to z/d by taking the hint that the entire directory\n> 'x' moved to 'z'.\n\nIt appears that you're still using V3 of my directory rename detection\nseries; I got some feedback from Stefan and Johannes on that series\n(including compilation failures on Windows) and believe I have\naddressed all of it in V4, which can be found here:\n https://public-inbox.org/git/20171129014237.32570-1-newren@gmail.com/\n"},{"id":"334108","messageId":"20171204154607.2e29acc6eb269ce4c96ee9a2@google.com","threadId":"47377","inReplyTo":"xmqqmv2ykvy4.fsf@gitster.mtv.corp.google.com","subject":"Re: What's cooking in git.git (Dec 2017, #01; Mon, 4)","fromName":"Jonathan Tan","fromEmail":"jonathantanmy@google.com","sentAt":"2017-12-04T23:46:07Z","receivedAt":"2017-12-04T23:46:17Z","isPatch":false,"sender":{"key":"jonathantanmy@fastmail.com","avatar":null},"body":"On Mon, 04 Dec 2017 13:46:43 -0800\nJunio C Hamano wrote:\n\n> * jt/diff-anchored-patience (2017-11-28) 1 commit\n> - diff: support anchoring line(s)\n> \n> \"git diff\" learned a variant of the \"--patience\" algorithm, to\n> which the user can specify which 'unique' line to be used as\n> anchoring points.\n\nIs there anything I can do to progress this? Johannes Schindelin wrote\nthat he is in favor of this feature [1]. He also remarked on the fact\nthat the anchors are not freed [2], but I have replied [3] that it seems\nthat diff arguments are not freed in general (so there would be no good\nplace to put the freeing statements).\n\n[1] https://public-inbox.org/git/alpine.DEB.2.21.1.1712011447550.98586@virtualbox/\n[2] https://public-inbox.org/git/alpine.DEB.2.21.1.1711300134560.6482@virtualbox/\n[3] https://public-inbox.org/git/20171130152605.1b775e9cc2ddd7f917424448@google.com/\n"},{"id":"334130","messageId":"xmqq8teikod0.fsf@gitster.mtv.corp.google.com","threadId":"47377","inReplyTo":"20171204154607.2e29acc6eb269ce4c96ee9a2@google.com","subject":"Re: What's cooking in git.git (Dec 2017, #01; Mon, 4)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2017-12-05T00:30:35Z","receivedAt":"2017-12-05T00:30:42Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Jonathan Tan writes:\n\n> On Mon, 04 Dec 2017 13:46:43 -0800\n> Junio C Hamano wrote:\n>\n>> * jt/diff-anchored-patience (2017-11-28) 1 commit\n>> - diff: support anchoring line(s)\n>> \n>> \"git diff\" learned a variant of the \"--patience\" algorithm, to\n>> which the user can specify which 'unique' line to be used as\n>> anchoring points.\n>\n> Is there anything I can do to progress this?\n\nI had an impression that there weren't any major outstanding issues.\nSuch topics will move at their own pace and the only thing somebody\ncould do to expedite is to somehow block other topics in flight so\nthat we have to worry about smaller number of topics ;-)\n\n"},{"id":"334247","messageId":"DA960DCE-0635-47CF-B3C4-8133021799F1@gmail.com","threadId":"47377","inReplyTo":"xmqqmv2ykvy4.fsf@gitster.mtv.corp.google.com","subject":"Re: What's cooking in git.git (Dec 2017, #01; Mon, 4)","fromName":"Lars Schneider","fromEmail":"larsxschneider@gmail.com","sentAt":"2017-12-06T15:14:41Z","receivedAt":"2017-12-06T15:14:53Z","isPatch":false,"sender":{"key":"larsxschneider@gmail.com","avatar":"https://avatars.githubusercontent.com/u/477434?v=4"},"body":"\n> On 04 Dec 2017, at 22:46, Junio C Hamano wrote:\n> \n> \n> * tb/check-crlf-for-safe-crlf (2017-11-27) 1 commit\n> - convert: tighten the safe autocrlf handling\n> \n> The \"safe crlf\" check incorrectly triggered for contents that does\n> not use CRLF as line endings, which has been corrected.\n> \n> Will merge to 'next'.\n> \n\nLooks like t0027-auto-crlf.sh is failing on Windows:\nhttps://travis-ci.org/git/git/jobs/312135514\n\nCould that patch be related?\n\n- Lars\n"},{"id":"334252","messageId":"0c2aa5cd-55d3-28ea-8577-b8b2839fd4c2@web.de","threadId":"47377","inReplyTo":"DA960DCE-0635-47CF-B3C4-8133021799F1@gmail.com","subject":"Re: What's cooking in git.git (Dec 2017, #01; Mon, 4)","fromName":"Torsten Bögershausen","fromEmail":"tboegi@web.de","sentAt":"2017-12-06T16:03:17Z","receivedAt":"2017-12-06T16:03:37Z","isPatch":false,"sender":{"key":"tboegi@web.de","avatar":"https://avatars.githubusercontent.com/u/7138363?v=4"},"body":"On 2017-12-06 16:14, Lars Schneider wrote:\n> \n>> On 04 Dec 2017, at 22:46, Junio C Hamano wrote:\n>>\n>>\n>> * tb/check-crlf-for-safe-crlf (2017-11-27) 1 commit\n>> - convert: tighten the safe autocrlf handling\n>>\n>> The \"safe crlf\" check incorrectly triggered for contents that does\n>> not use CRLF as line endings, which has been corrected.\n>>\n>> Will merge to 'next'.\n>>\n> \n> Looks like t0027-auto-crlf.sh is failing on Windows:\n> https://travis-ci.org/git/git/jobs/312135514\n> \n> Could that patch be related?\n> \n> - Lars\n> \nChances are high, I will have a look.\nThanks for noticing.\n\n\n"},{"id":"334256","messageId":"xmqqlgifhkfn.fsf@gitster.mtv.corp.google.com","threadId":"47377","inReplyTo":"0c2aa5cd-55d3-28ea-8577-b8b2839fd4c2@web.de","subject":"Re: What's cooking in git.git (Dec 2017, #01; Mon, 4)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2017-12-06T16:48:12Z","receivedAt":"2017-12-06T16:48:38Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Torsten Bögershausen writes:\n\n>> Looks like t0027-auto-crlf.sh is failing on Windows:\n>> https://travis-ci.org/git/git/jobs/312135514\n>> \n>> Could that patch be related?\n>> \n>> - Lars\n>> \n> Chances are high, I will have a look.\n> Thanks for noticing.\n\nThanks.\n"},{"id":"334339","messageId":"alpine.DEB.2.21.1.1712071643410.4318@virtualbox","threadId":"47377","inReplyTo":"DA960DCE-0635-47CF-B3C4-8133021799F1@gmail.com","subject":"Re: What's cooking in git.git (Dec 2017, #01; Mon, 4)","fromName":"Johannes Schindelin","fromEmail":"johannes.schindelin@gmx.de","sentAt":"2017-12-07T15:48:42Z","receivedAt":"2017-12-07T17:56:44Z","isPatch":false,"sender":{"key":"johannes.schindelin@gmx.de","avatar":"https://avatars.githubusercontent.com/u/127790?v=4"},"body":"Hi,\n\nOn Wed, 6 Dec 2017, Lars Schneider wrote:\n\n> \n> > On 04 Dec 2017, at 22:46, Junio C Hamano wrote:\n> > \n> > \n> > * tb/check-crlf-for-safe-crlf (2017-11-27) 1 commit\n> > - convert: tighten the safe autocrlf handling\n> > \n> > The \"safe crlf\" check incorrectly triggered for contents that does\n> > not use CRLF as line endings, which has been corrected.\n> > \n> > Will merge to 'next'.\n> > \n> \n> Looks like t0027-auto-crlf.sh is failing on Windows:\n> https://travis-ci.org/git/git/jobs/312135514\n> \n> Could that patch be related?\n\nThat is not the only thing going wrong:\n\n\thttps://travis-ci.org/git/git/builds/312551566\n\nIt would seem that t9001 is broken on Linux32, t5616 is broken on macOS,\nand something really kinky is going on with the GETTEXT_POISON text, as it\nseems to just abort while trying to run t6120.\n\nCiao,\nJohannes\n"},{"id":"334360","messageId":"20171207202201.GA3693@zaya.teonanacatl.net","threadId":"47377","inReplyTo":"alpine.DEB.2.21.1.1712071643410.4318@virtualbox","subject":"Re: What's cooking in git.git (Dec 2017, #01; Mon, 4)","fromName":"Todd Zullinger","fromEmail":"tmz@pobox.com","sentAt":"2017-12-07T20:22:02Z","receivedAt":"2017-12-07T20:22:22Z","isPatch":false,"sender":{"key":"tmz@pobox.com","avatar":"https://avatars.githubusercontent.com/u/806319?v=4"},"body":"Johannes Schindelin wrote:\n> That is not the only thing going wrong:\n>\n> \thttps://travis-ci.org/git/git/builds/312551566\n>\n> It would seem that t9001 is broken on Linux32, t5616 is broken on macOS,\n> and something really kinky is going on with the GETTEXT_POISON text, as it\n> seems to just abort while trying to run t6120.\n\nI thought the verbose logs from the test might be useful, but looking\nat the travis output for that job[1], there's an unrelated problem\npreventing the ci/print-test-failures.sh script from running properly:\n\n $ ci/print-test-failures.sh\n cat: t/test-results/t1304-default-acl.exit: Permission denied\n ------------------------------------------------------------------------\n t/test-results/t1304-default-acl.out...\n ------------------------------------------------------------------------\n cat: t/test-results/t1304-default-acl.out: Permission denied\n\n[1] https://travis-ci.org/git/git/jobs/312551595#L2185\n\nI didn't see the same failure for other build targets at a glance, so\nthe permission issue might only be a problem for the linux32 builds.\n\n-- \nTodd\n~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~\nA diplomat is a person who can tell you to go to Hell in such a way\nthat you actually look forward to the trip.\n -- Anonymous\n\n"},{"id":"334364","messageId":"xmqqlgiecleo.fsf@gitster.mtv.corp.google.com","threadId":"47377","inReplyTo":"20171207202201.GA3693@zaya.teonanacatl.net","subject":"Re: What's cooking in git.git (Dec 2017, #01; Mon, 4)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2017-12-07T20:50:39Z","receivedAt":"2017-12-07T20:52:41Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Todd Zullinger writes:\n\n> Johannes Schindelin wrote:\n>> That is not the only thing going wrong:\n>>\n>> \thttps://travis-ci.org/git/git/builds/312551566\n>>\n>> It would seem that t9001 is broken on Linux32, t5616 is broken on macOS,\n>> and something really kinky is going on with the GETTEXT_POISON text, as it\n>> seems to just abort while trying to run t6120.\n>\n> I thought the verbose logs from the test might be useful, but looking\n> at the travis output for that job[1], there's an unrelated problem\n> preventing the ci/print-test-failures.sh script from running properly:\n>\n> $ ci/print-test-failures.sh\n> cat: t/test-results/t1304-default-acl.exit: Permission denied\n> ------------------------------------------------------------------------\n> t/test-results/t1304-default-acl.out...\n> ------------------------------------------------------------------------\n> cat: t/test-results/t1304-default-acl.out: Permission denied\n>\n> [1] https://travis-ci.org/git/git/jobs/312551595#L2185\n>\n> I didn't see the same failure for other build targets at a glance, so\n> the permission issue might only be a problem for the linux32 builds.\n\nCurious. \n\nThe acl stuff hasn't changed for a long time and I do not think of a\nreason offhand why the test should behave differently between say\n'maint' and 'pu', yet 'maint' is passing while 'pu' is not...\n\nI wouldn't be surprised if Git::Error change is causing t9001\nfailure, as that can explain why 'pu' is different.\n"},{"id":"334367","messageId":"175f87bc-0270-fb18-fc14-24e8f59321d6@jeffhostetler.com","threadId":"47377","inReplyTo":"alpine.DEB.2.21.1.1712071643410.4318@virtualbox","subject":"Re: What's cooking in git.git (Dec 2017, #01; Mon, 4)","fromName":"Jeff Hostetler","fromEmail":"git@jeffhostetler.com","sentAt":"2017-12-07T21:08:02Z","receivedAt":"2017-12-07T21:08:19Z","isPatch":false,"sender":{"key":"git@jeffhostetler.com","avatar":null},"body":"\n\nOn 12/7/2017 10:48 AM, Johannes Schindelin wrote:\n> Hi,\n[...]\n> \n> That is not the only thing going wrong:\n> \n> \thttps://travis-ci.org/git/git/builds/312551566\n> \n> It would seem that t9001 is broken on Linux32, t5616 is broken on macOS,\n> and something really kinky is going on with the GETTEXT_POISON text, as it\n> seems to just abort while trying to run t6120.\n> \n> Ciao,\n> Johannes\n> \n\nI'm looking at t5616 now on my mac.\nLooks like the MAC doesn't like my line counting in the tests.\nI'll fix in my next version.\n\nJeff\n"},{"id":"334368","messageId":"71F9850F-5D7F-4CDE-8408-E000ADD48406@gmail.com","threadId":"47377","inReplyTo":"xmqqlgiecleo.fsf@gitster.mtv.corp.google.com","subject":"Re: What's cooking in git.git (Dec 2017, #01; Mon, 4)","fromName":"Lars Schneider","fromEmail":"larsxschneider@gmail.com","sentAt":"2017-12-07T21:08:02Z","receivedAt":"2017-12-07T21:08:36Z","isPatch":false,"sender":{"key":"larsxschneider@gmail.com","avatar":"https://avatars.githubusercontent.com/u/477434?v=4"},"body":"\n> On 07 Dec 2017, at 21:50, Junio C Hamano wrote:\n> \n> Todd Zullinger writes:\n> \n>> Johannes Schindelin wrote:\n>>> That is not the only thing going wrong:\n>>> \n>>> \thttps://travis-ci.org/git/git/builds/312551566\n>>> \n>>> It would seem that t9001 is broken on Linux32, t5616 is broken on macOS,\n>>> and something really kinky is going on with the GETTEXT_POISON text, as it\n>>> seems to just abort while trying to run t6120.\n>> \n>> I thought the verbose logs from the test might be useful, but looking\n>> at the travis output for that job[1], there's an unrelated problem\n>> preventing the ci/print-test-failures.sh script from running properly:\n>> \n>> $ ci/print-test-failures.sh\n>> cat: t/test-results/t1304-default-acl.exit: Permission denied\n>> ------------------------------------------------------------------------\n>> t/test-results/t1304-default-acl.out...\n>> ------------------------------------------------------------------------\n>> cat: t/test-results/t1304-default-acl.out: Permission denied\n>> \n>> [1] https://travis-ci.org/git/git/jobs/312551595#L2185\n>> \n>> I didn't see the same failure for other build targets at a glance, so\n>> the permission issue might only be a problem for the linux32 builds.\n> \n> Curious. \n> \n> The acl stuff hasn't changed for a long time and I do not think of a\n> reason offhand why the test should behave differently between say\n> 'maint' and 'pu', yet 'maint' is passing while 'pu' is not...\n\n\nMy recent 657343a602 (travis-ci: move Travis CI code into dedicated scripts, \n2017-09-10) change might have broken that somehow...\n\nSee this comment:\n\n# If this script runs inside a docker container, then all commands are\n# usually executed as root. Consequently, the host user might not be\n# able to access the test output files.\n# If a host user id is given, then create a user \"ci\" with the host user\n# id to make everything accessible to the host user.\n\nhttps://github.com/git/git/blob/95ec6b1b3393eb6e26da40c565520a8db9796e9f/ci/run-linux32-build.sh#L16-L20\n\n- Lars"},{"id":"334370","messageId":"xmqqh8t2ckgw.fsf@gitster.mtv.corp.google.com","threadId":"47377","inReplyTo":"175f87bc-0270-fb18-fc14-24e8f59321d6@jeffhostetler.com","subject":"Re: What's cooking in git.git (Dec 2017, #01; Mon, 4)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2017-12-07T21:10:55Z","receivedAt":"2017-12-07T21:11:45Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Jeff Hostetler writes:\n\n> I'm looking at t5616 now on my mac.\n> Looks like the MAC doesn't like my line counting in the tests.\n\nAh, of course, test \"$(wc -l)\" = number would not work over there\nwe have \"test_line_count\" helper exactly for that purose.\n\n"},{"id":"334375","messageId":"xmqqd13qcjz1.fsf@gitster.mtv.corp.google.com","threadId":"47377","inReplyTo":"71F9850F-5D7F-4CDE-8408-E000ADD48406@gmail.com","subject":"Re: What's cooking in git.git (Dec 2017, #01; Mon, 4)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2017-12-07T21:21:38Z","receivedAt":"2017-12-07T21:21:59Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Lars Schneider writes:\n\n>> The acl stuff hasn't changed for a long time and I do not think of a\n>> reason offhand why the test should behave differently between say\n>> 'maint' and 'pu', yet 'maint' is passing while 'pu' is not...\n>\n> My recent 657343a602 (travis-ci: move Travis CI code into dedicated scripts, \n> 2017-09-10) change might have broken that somehow...\n\nBut the puzzling thing is that commit is in 'maint' as well as 'pu'.\n"},{"id":"334378","messageId":"20171207213312.GB3693@zaya.teonanacatl.net","threadId":"47377","inReplyTo":"175f87bc-0270-fb18-fc14-24e8f59321d6@jeffhostetler.com","subject":"Re: What's cooking in git.git (Dec 2017, #01; Mon, 4)","fromName":"Todd Zullinger","fromEmail":"tmz@pobox.com","sentAt":"2017-12-07T21:33:12Z","receivedAt":"2017-12-07T21:33:33Z","isPatch":false,"sender":{"key":"tmz@pobox.com","avatar":"https://avatars.githubusercontent.com/u/806319?v=4"},"body":"Jeff Hostetler wrote:\n> I'm looking at t5616 now on my mac.\n> Looks like the MAC doesn't like my line counting in the tests.\n> I'll fix in my next version.\n\nPerhaps that's as simple as using the test_line_counts helper?\n\ndiff --git i/t/t5616-partial-clone.sh w/t/t5616-partial-clone.sh\nindex fa573f8cdb..6d673de90c 100755\n--- i/t/t5616-partial-clone.sh\n+++ w/t/t5616-partial-clone.sh\n@@ -19,7 +19,7 @@ test_expect_success 'setup normal src repo' '\n \t\tgit -C src ls-files -s file.$n.txt >>temp\n \tdone &&\n \tawk -f print_2.awk expect_1.oids &&\n-\ttest \"$(wc -l expect_2.oids &&\n-\ttest \"$(wc -l expect.blame\n '\n \n@@ -100,7 +100,7 @@ test_expect_success 'partial fetch inherits filter settings' '\n # it should be in a new packfile (since the promisor boundary is\n # currently a packfile, it should not get unpacked upon receipt.)\n test_expect_success 'verify diff causes dynamic object fetch' '\n-\ttest \"$(wc -l observed.oids &&\n \t\tcat observed.oids &&\n-\ttest \"$(wc -l Jeff Hostetler wrote:\n> >I'm looking at t5616 now on my mac.\n> >Looks like the MAC doesn't like my line counting in the tests.\n> >I'll fix in my next version.\n> \n[]\n> \t\t| sort >expect_2.oids &&\n> -\ttest \"$(wc -l +\ttest_line_count = 8 expect_2.oids &&\n> \tgit -C src blame master -- file.1.txt >expect.blame\n> '\n\n\nThe problem seems to be the '\"' around wc, this would work:\ntest $(wc -l ) {\n \t/^\\s*declare\\s+/ and err 'arrays/declare not portable';\n \t/^\\s*[^#]\\s*which\\s/ and err 'which is not portable (please use type)';\n \t/\\btest\\s+[^=]*==/ and err '\"test a == b\" is not portable (please use =)';\n+\t/\\bwc -l.*\"\\s*=/ and err '`\"$(wc -l)\"` is not portable, please use `$(wc -l)`';\n \t/\\bexport\\s+[A-Za-z0-9_]*=/ and err '\"export FOO=bar\" is not portable (please use FOO=bar && expo\n\n\n\n\n\n\n"},{"id":"334560","messageId":"20171210105013.3984-1-tboegi@web.de","threadId":"47377","inReplyTo":"xmqqh8t2ckgw.fsf@gitster.mtv.corp.google.com","subject":"[PATCH v1 1/1] check-non-portable-shell.pl: Quoted `wc -l` is not portable","fromName":"","fromEmail":"tboegi@web.de","sentAt":"2017-12-10T10:50:13Z","receivedAt":"2017-12-10T10:50:24Z","isPatch":true,"sender":{"key":"tboegi@web.de","avatar":"https://avatars.githubusercontent.com/u/7138363?v=4"},"body":"From: Torsten Bögershausen \n\nwc -l is used to count the number if lines in test scripts.\n$ wc -l Makefile\ngives a line like this:\n105 Makefile\nwhile Mac OS has 4 leading spaces:\n 105 Makefile\n\nAnd this means that shell expressions like\ntest \"$(wc -l On Thu, Dec 07, 2017 at 04:33:12PM -0500, Todd Zullinger wrote:\n> > Jeff Hostetler wrote:\n> > >I'm looking at t5616 now on my mac.\n> > >Looks like the MAC doesn't like my line counting in the tests.\n> > >I'll fix in my next version.\n> > \n> []\n> > \t\t| sort >expect_2.oids &&\n> > -\ttest \"$(wc -l > +\ttest_line_count = 8 expect_2.oids &&\n> > \tgit -C src blame master -- file.1.txt >expect.blame\n> > '\n> \n> \n> The problem seems to be the '\"' around wc, this would work:\n> test $(wc -l \n> \n> What do you guys think a about a lint test like this:\n> \n> diff --git a/t/check-non-portable-shell.pl b/t/check-non-portable-shell.pl\n> index 03dc9d2852..9ebf65c26f 100755\n> --- a/t/check-non-portable-shell.pl\n> +++ b/t/check-non-portable-shell.pl\n> @@ -21,6 +21,7 @@ while (<>) {\n> \t/^\\s*declare\\s+/ and err 'arrays/declare not portable';\n> \t/^\\s*[^#]\\s*which\\s/ and err 'which is not portable (please use type)';\n> \t/\\btest\\s+[^=]*==/ and err '\"test a == b\" is not portable (please use =)';\n> +\t/\\bwc -l.*\"\\s*=/ and err '`\"$(wc -l)\"` is not portable, please use `$(wc -l)`';\n> \t/\\bexport\\s+[A-Za-z0-9_]*=/ and err '\"export FOO=bar\" is not portable (please use FOO=bar && expo\n\nPlease suggest `test_line_count` instead.\n\nCiao,\nDscho"},{"id":"334564","messageId":"alpine.DEB.2.21.1.1712101458150.4318@virtualbox","threadId":"47377","inReplyTo":"20171210105013.3984-1-tboegi@web.de","subject":"Re: [PATCH v1 1/1] check-non-portable-shell.pl: Quoted `wc -l` is not portable","fromName":"Johannes Schindelin","fromEmail":"johannes.schindelin@gmx.de","sentAt":"2017-12-10T13:59:45Z","receivedAt":"2017-12-10T14:04:16Z","isPatch":true,"sender":{"key":"johannes.schindelin@gmx.de","avatar":"https://avatars.githubusercontent.com/u/127790?v=4"},"body":"Hi Torsten,\n\nOn Sun, 10 Dec 2017, tboegi@web.de wrote:\n\n> From: Torsten Bögershausen \n> \n> wc -l is used to count the number if lines in test scripts.\n> $ wc -l Makefile\n> gives a line like this:\n> 105 Makefile\n> while Mac OS has 4 leading spaces:\n> 105 Makefile\n> \n> And this means that shell expressions like\n> test \"$(wc -l \n> A portable way to use `wc -l` is to omit the '\"':\n> test $(wc -l \n> Add a check in check-non-portable-shell.pl to find '\"' between\n> `wc -l` and '='\n> \n> Signed-off-by: Torsten Bögershausen \n> ---\n> t/check-non-portable-shell.pl | 1 +\n> 1 file changed, 1 insertion(+)\n> \n> diff --git a/t/check-non-portable-shell.pl b/t/check-non-portable-shell.pl\n> index 03dc9d2852..9ebf65c26f 100755\n> --- a/t/check-non-portable-shell.pl\n> +++ b/t/check-non-portable-shell.pl\n> @@ -21,6 +21,7 @@ while (<>) {\n> \t/^\\s*declare\\s+/ and err 'arrays/declare not portable';\n> \t/^\\s*[^#]\\s*which\\s/ and err 'which is not portable (please use type)';\n> \t/\\btest\\s+[^=]*==/ and err '\"test a == b\" is not portable (please use =)';\n> +\t/\\bwc -l.*\"\\s*=/ and err '`\"$(wc -l)\"` is not portable, please use `$(wc -l)`';\n> \t/\\bexport\\s+[A-Za-z0-9_]*=/ and err '\"export FOO=bar\" is not portable (please use FOO=bar && export FOO)';\n> \t# this resets our $. for each file\n> \tclose ARGV if eof;\n\nAs noted elsewhere, this should suggest `test_line_count` instead. After\nall, that function is not only guaranteed to stay portable (even if we\nshould ever start supporting systems *without* `wc`), but it also has a\nsemantically-meaningful name worthy of the current century.\n\nCiao,\nDscho"},{"id":"334637","messageId":"xmqqvahc50eg.fsf@gitster.mtv.corp.google.com","threadId":"47377","inReplyTo":"20171210105013.3984-1-tboegi@web.de","subject":"Re: [PATCH v1 1/1] check-non-portable-shell.pl: Quoted `wc -l` is not portable","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2017-12-11T23:07:35Z","receivedAt":"2017-12-11T23:07:48Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"tboegi@web.de writes:\n\n> From: Torsten Bögershausen \n>\n> wc -l is used to count the number if lines in test scripts.\n\nHmmmm, \"is used\" -> \"should not be used\", probably.\n\nJust say \"Use test_line_count instead\" and leave the explanation why\nit is a better thing to use to fb3340a6 (\"test-lib: introduce\ntest_line_count to measure files\", 2010-10-31), perhaps?\n"},{"id":"334942","messageId":"20171216195246.10445-1-tboegi@web.de","threadId":"47377","inReplyTo":"xmqqh8t2ckgw.fsf@gitster.mtv.corp.google.com","subject":"[PATCH v2 1/1] check-non-portable-shell.pl: Quoted `wc -l` is not portable","fromName":"","fromEmail":"tboegi@web.de","sentAt":"2017-12-16T19:52:46Z","receivedAt":"2017-12-16T19:52:57Z","isPatch":true,"sender":{"key":"tboegi@web.de","avatar":"https://avatars.githubusercontent.com/u/7138363?v=4"},"body":"From: Torsten Bögershausen \n\nwc -l was used to count the number if lines in test scripts.\n$ wc -l Makefile\ngives a line like this:\n105 Makefile\nwhile Mac OS has 4 leading spaces:\n 105 Makefile\n\nAnd this means that shell expressions like\ntest \"$(wc -l wrote:\n> [...]\n> Add a check in check-non-portable-shell.pl to find '\"' between\n> `wc -l` and '=' and hint the user about test_line_count().\n>\n> Reviewed-by: Johannes Schindelin \n> Signed-off-by: Torsten Bögershausen \n> ---\n> diff --git a/t/check-non-portable-shell.pl b/t/check-non-portable-shell.pl\n> index 03dc9d2852..fadbb1e5a7 100755\n> --- a/t/check-non-portable-shell.pl\n> +++ b/t/check-non-portable-shell.pl\n> @@ -21,6 +21,7 @@ while (<>) {\n> /^\\s*declare\\s+/ and err 'arrays/declare not portable';\n> /^\\s*[^#]\\s*which\\s/ and err 'which is not portable (please use type)';\n> /\\btest\\s+[^=]*==/ and err '\"test a == b\" is not portable (please use =)';\n> + /\\bwc -l.*\"\\s*=/ and err '`\"$(wc -l)\"` is not portable, please use test_line_count';\n\nNit: Every other \"please use\" suggestion is parenthesized; for\nconsistency, this probably ought to be, as well.\n\n> /\\bexport\\s+[A-Za-z0-9_]*=/ and err '\"export FOO=bar\" is not portable (please use FOO=bar && export FOO)';\n"},{"id":"335153","messageId":"20171221212646.24461-1-tboegi@web.de","threadId":"47377","inReplyTo":"xmqqh8t2ckgw.fsf@gitster.mtv.corp.google.com","subject":"[PATCH v3 1/1] check-non-portable-shell.pl: Quoted `wc -l` is not portable","fromName":"","fromEmail":"tboegi@web.de","sentAt":"2017-12-21T21:26:46Z","receivedAt":"2017-12-21T21:26:55Z","isPatch":true,"sender":{"key":"tboegi@web.de","avatar":"https://avatars.githubusercontent.com/u/7138363?v=4"},"body":"From: Torsten Bögershausen \n\nwc -l was used to count the number if lines in test scripts.\n$ wc -l Makefile\ngives a line like this:\n105 Makefile\nwhile Mac OS has 4 leading spaces:\n 105 Makefile\n\nAnd this means that shell expressions like\ntest \"$(wc -l \n> Reviewed-by: Johannes Schindelin \n> Signed-off-by: Torsten Bögershausen \n\nI'll flip these and add a helped-by to credit Eric.\n\n check-non-portable-shell.pl: `wc -l` may have leading WS\n \n Test scripts count number of lines in an output and check it againt\n its expectation. fb3340a6 (\"test-lib: introduce test_line_count to\n measure files\", 2010-10-31) introduced a helper to show a failure in\n such a test in a more readable way than comparing `wc -l` output with\n a number.\n \n Besides, on some platforms, \"$(wc -l @@ -21,6 +21,7 @@ while (<>) {\n> \t/^\\s*declare\\s+/ and err 'arrays/declare not portable';\n> \t/^\\s*[^#]\\s*which\\s/ and err 'which is not portable (please use type)';\n> \t/\\btest\\s+[^=]*==/ and err '\"test a == b\" is not portable (please use =)';\n> +\t/\\bwc -l.*\"\\s*=/ and err '`\"$(wc -l)\"` is not portable (please use test_line_count)';\n> \t/\\bexport\\s+[A-Za-z0-9_]*=/ and err '\"export FOO=bar\" is not portable (please use FOO=bar && export FOO)';\n> \t# this resets our $. for each file\n> \tclose ARGV if eof;\n\nThanks. The dq before \\s*= is rather cute ;-)\n\n"},{"id":"335219","messageId":"20171223072736.GA30043@tor.lan","threadId":"47377","inReplyTo":"xmqq4loiqxn4.fsf@gitster.mtv.corp.google.com","subject":"Re: [PATCH v3 1/1] check-non-portable-shell.pl: Quoted `wc -l` is not portable","fromName":"Torsten Bögershausen","fromEmail":"tboegi@web.de","sentAt":"2017-12-23T07:27:36Z","receivedAt":"2017-12-23T07:27:46Z","isPatch":true,"sender":{"key":"tboegi@web.de","avatar":"https://avatars.githubusercontent.com/u/7138363?v=4"},"body":"On Fri, Dec 22, 2017 at 01:07:59PM -0800, Junio C Hamano wrote:\n> tboegi@web.de writes:\n> \n> >\n> > Reviewed-by: Johannes Schindelin \n> > Signed-off-by: Torsten Bögershausen \n> \n\n> I'll flip these and add a helped-by to credit Eric.\n...\n> Don't try to apply this patch to your tree yourself ;-)\n\nThanks so much for cleaning up my mess.\n"}]}