* What's cooking in git.git (Mar 2011, #02; Sun, 20) @ 2011-03-20 19:49 Junio C Hamano 2011-03-20 20:43 ` Jens Lehmann ` (2 more replies) 0 siblings, 3 replies; 15+ messages in thread From: Junio C Hamano @ 2011-03-20 19:49 UTC (permalink / raw) To: git Here are the topics that have been cooking. Commits prefixed with '-' are only in 'pu' while commits prefixed with '+' are in 'next'. The very basic of i18n is now in 'master'. I'll be moving the rest of the series forward shortly. -------------------------------------------------- [Graduated to "master"] * ab/i18n-basic (2011-02-25) 9 commits (merged to 'next' on 2011-03-10 at ab70d57) + i18n: "make distclean" should clean up after "make pot" + i18n: Makefile: "pot" target to extract messages marked for translation + i18n: add stub Q_() wrapper for ngettext + i18n: do not poison translations unless GIT_GETTEXT_POISON envvar is set + i18n: add GETTEXT_POISON to simulate unfriendly translator + i18n: add no-op _() and N_() wrappers + commit, status: use status_printf{,_ln,_more} helpers + commit: refer to commit template as s->fp + wt-status: add helpers for printing wt-status lines (this branch is used by ab/i18n-st.) * fk/maint-cvsimport-early-failure (2011-01-31) 1 commit (merged to 'next' on 2011-03-09 at b4dee9e) + git-cvsimport.perl: Bail out right away when reading from the server fails * jc/checkout-orphan-warning (2011-02-18) 1 commit (merged to 'next' on 2011-03-09 at 765cf75) + commit: give final warning when reattaching HEAD to leave commits behind * jc/maint-apply-no-double-patch (2011-03-04) 1 commit (merged to 'next' on 2011-03-09 at 6ec48e7) + apply: do not patch lines that were already patched * jc/maint-apply-report-offset (2011-03-04) 1 commit (merged to 'next' on 2011-03-09 at 1bb74ef) + apply -v: show offset count when patch did not apply exactly * jh/maint-do-not-track-non-branches (2011-02-17) 1 commit (merged to 'next' on 2011-03-09 at 13a9328) + branch/checkout --track: Ensure that upstream branch is indeed a branch * jk/format-patch-multiline-header (2011-02-23) 3 commits (merged to 'next' on 2011-03-09 at f970fe3) + format-patch: rfc2047-encode newlines in headers + format-patch: wrap long header lines + strbuf: add fixed-length version of add_wrapped_text * jk/merge-rename-ux (2011-02-20) 6 commits (merged to 'next' on 2011-03-09 at df1943a) + pull: propagate --progress to merge + merge: enable progress reporting for rename detection + add inexact rename detection progress infrastructure + commit: stop setting rename limit + bump rename limit defaults (again) + merge: improve inexact rename limit warning * jk/strbuf-vaddf (2011-03-08) 3 commits (merged to 'next' on 2011-03-09 at 9651bc5) + compat: fall back on __va_copy if available + strbuf: add strbuf_vaddf + compat: provide a fallback va_copy definition (this branch is used by ab/i18n-basic, ab/i18n-st, jk/trace-sifter and jn/status-translatable.) * jk/trace-sifter (2011-02-24) 6 commits (merged to 'next' on 2011-03-09 at 07841db) + trace: give repo_setup trace its own key + add packet tracing debug code + trace: add trace_strbuf + trace: factor out "do we want to trace" logic + trace: refactor to support multiple env variables + trace: add trace_vprintf * jn/maint-instaweb-plack-fix (2011-02-26) 1 commit (merged to 'next' on 2011-03-09 at 3c9e14b) + git-instaweb: Change how gitweb.psgi is made runnable as standalone app * jn/status-translatable (2011-02-25) 3 commits (merged to 'next' on 2011-03-09 at d00d23d) + commit, status: use status_printf{,_ln,_more} helpers + commit: refer to commit template as s->fp + wt-status: add helpers for printing wt-status lines * js/rerere-forget-always-take-pathspec (2011-03-01) 1 commit (merged to 'next' on 2011-03-10 at 2d1ded3) + rerere forget: deprecate invocation without pathspec * mg/rev-list-n-reverse-doc (2011-03-08) 2 commits (merged to 'next' on 2011-03-10 at 62d3161) + git-log.txt,rev-list-options.txt: put option blocks in proper order + git-log.txt,rev-list-options.txt: -n/--max-count is commit limiting * mm/maint-log-n-with-diff-filtering (2011-03-09) 1 commit (merged to 'next' on 2011-03-10 at 33af1cd) + log: fix --max-count when used together with -S or -G * mm/push-default-advice (2011-03-02) 2 commits (merged to 'next' on 2011-03-09 at 160095b) + push: better error message when no remote configured + push: better error messages when push.default = tracking * mr/hpux (2011-03-07) 2 commits (merged to 'next' on 2011-03-09 at 8a5fe11) + git-compat-util.h: Honor HP C's noreturn attribute + Makefile: add NO_FNMATCH_CASEFOLD to HP-UX section * sp/maint-fd-limit (2011-03-02) 3 commits (merged to 'next' on 2011-03-09 at 389e97d) + sha1_file.c: Don't retain open fds on small packs + mingw: add minimum getrlimit() compatibility stub + Limit file descriptors used by packs * ss/mergetool--lib (2011-02-27) 2 commits (merged to 'next' on 2011-03-09 at 004fb9c) + mergetool--lib: Add Beyond Compare 3 as a tool + mergetool--lib: Sort tools alphabetically for easier lookup * uk/ls-remote-in-get-remote-url (2011-03-01) 2 commits (merged to 'next' on 2011-03-09 at 09b570b) + git-request-pull: open-code the only invocation of get_remote_url + get_remote_url(): use the same data source as ls-remote to get remote urls -------------------------------------------------- [New Topics] * jc/maint-fetch-alt (2011-03-11) 2 commits (merged to 'next' on 2011-03-17 at c61ec4f) + fetch-pack: objects in our alternates are available to us + refs_from_alternate: helper to use refs from alternates * jc/maint-rerere-in-workdir (2011-03-10) 1 commit - rerere: make sure it works even in a workdir attached to a young repository Do not merge; needs more thought around relative symlinks. * lt/default-abbrev (2010-10-28) 1 commit (merged to 'next' on 2011-03-17 at 1d7e406) + Make the default abbrev length configurable The configuration roughly corresponds to --abbrev command line option, so I should rename it back to core.abbrev as Linus originally named it. * pk/stash-apply-status-relative (2011-03-12) 1 commit - git stash: show status relative to current directory Will merge to 'next'. * sg/complete-symmetric-diff (2011-03-10) 2 commits (merged to 'next' on 2011-03-17 at adb0618) + bash: complete 'git diff ...branc<TAB>' + bash: fix misindented esac statement in __git_complete_file() * jc/maint-rev-list-culled-boundary (2011-03-14) 1 commit (merged to 'next' on 2011-03-17 at 001b546) + list-objects.c: don't add an unparsed NULL as a pending tree * jn/test-sanitize-git-env (2011-03-15) 5 commits (merged to 'next' on 2011-03-17 at 80a3678) + tests: scrub environment of GIT_* variables + config: drop support for GIT_CONFIG_NOGLOBAL + gitattributes: drop support for GIT_ATTR_NOGLOBAL + tests: suppress system gitattributes + tests: stop worrying about obsolete environment variables * js/maint-stash-index-copy (2011-03-16) 2 commits (merged to 'next' on 2011-03-17 at 9f74cc8) + stash: copy the index using --index-output instead of cp -p + stash: fix incorrect quoting in cleanup of temporary files * sp/maint-fetch-pack-stop-early (2011-03-14) 2 commits (merged to 'next' on 2011-03-17 at 2580068) + fetch-pack: Implement no-done capability + fetch-pack: Finish negotation if remote replies "ACK %s ready" (this branch is used by jc/fetch-progressive-stride.) * sp/maint-upload-pack-stop-early (2011-03-14) 2 commits (merged to 'next' on 2011-03-17 at e717a84) + upload-pack: Implement no-done capability + upload-pack: More aggressively send 'ACK %s ready' * mg/doc-bisect-tweak-worktree (2011-03-15) 2 commits (merged to 'next' on 2011-03-17 at e0acf52) + git-bisect.txt: example for bisecting with hot-fix + git-bisect.txt: streamline run presentation * tl/p4 (2011-03-15) 3 commits (merged to 'next' on 2011-03-17 at 3d7f69f) + git-p4: Fix error message crash in P4Sync.commit. + Teach git-p4 to ignore case in perforce filenames if configured. + git-p4: Teach gitConfig method about arguments. * cn/system-path-tweak (2011-03-17) 1 commit - system_path: use a static buffer * jc/fetch-progressive-stride (2011-03-18) 3 commits - fetch-pack: use smaller handshake window for initial request - fetch-pack: progressively use larger handshake windows - fetch-pack: factor out hardcoded handshake window size (this branch uses sp/maint-fetch-pack-stop-early.) The top two need to be cleaned up with correct author name. * jc/maint-diff-q-filter (2011-03-16) 1 commit - diff --quiet: disable optimization when --diff-filter=X is used Will merge to 'next'. * jn/fortify-source-workaround (2011-03-16) 1 commit - run-command: prettify -D_FORTIFY_SOURCE workaround Will merge to 'next'. * jn/maint-c99-format (2011-03-16) 2 commits - unbreak and eliminate NO_C99_FORMAT - mktag: avoid %td in format string Will merge to 'next'. * kc/gitweb-pathinfo-w-anchor (2011-03-18) 1 commit - gitweb: fix #patchNN anchors when path_info is enabled Will merge to 'next'. * pw/p4 (2011-03-16) 2 commits - git-p4: test sync new branch - git-p4: fix sync new branch regression Will merge to 'next'. -------------------------------------------------- [Stalled] * jh/merge-sans-branch (2011-02-10) 4 commits . merge: add support for merging from upstream by default - merge: introduce per-branch-configuration helper function - merge: introduce setup_merge_commit helper function - merge: update the usage information to be more modern There was an objection to the tip one that determines the upstream in a wrong way? I've been wanting to move this forward for quite some time but also hoping somebody else would send in a final version. * mz/rebase (2011-02-28) 34 commits - rebase: define options in OPTIONS_SPEC (merged to 'next' on 2011-02-25 at 52caa7a) + Makefile: do not install sourced rebase scripts (merged to 'next' on 2011-02-22 at 3219155) + rebase: use @{upstream} if no upstream specified + rebase -i: remove unnecessary state rebase-root + rebase -i: don't read unused variable preserve_merges + git-rebase--am: remove unnecessary --3way option + rebase -m: don't print exit code 2 when merge fails + rebase -m: remember allow_rerere_autoupdate option + rebase: remember strategy and strategy options + rebase: remember verbose option + rebase: extract code for writing basic state + rebase: factor out sub command handling + rebase: make -v a tiny bit more verbose + rebase -i: align variable names + rebase: show consistent conflict resolution hint + rebase: extract am code to new source file + rebase: extract merge code to new source file + rebase: remove $branch as synonym for $orig_head + rebase -i: support --stat + rebase: factor out call to pre-rebase hook + rebase: factor out clean work tree check + rebase: factor out reference parsing + rebase: reorder validation steps + rebase -i: remove now unnecessary directory checks + rebase: factor out command line option processing + rebase: align variable content + rebase: align variable names + rebase: stricter check of standalone sub command + rebase: act on command line outside parsing loop + rebase: improve detection of rebase in progress + rebase: remove unused rebase state 'prev_head' + rebase: read state outside loop + rebase: refactor reading of state + rebase: clearer names for directory variables I think the tip one is a response to J6t's regression concerns; it looked Ok to me. * jc/diff-irreversible-delete (2011-02-28) 1 commit - git diff -D: omit the preimage of deletes "diff -B -D" should omit the deleting half of a broken pair from the output. Need documentation and tests. * jc/index-pack (2011-02-25) 5 commits - index-pack --verify: read anomalous offsets from v2 idx file - write_idx_file: need_large_offset() helper function - index-pack: --verify - write_idx_file: introduce a struct to hold idx customization options - index-pack: group the delta-base array entries also by type Still a WIP. Need to put histogram output into index-pack --verify to really kill verify-pack. * gr/cvsimport-alternative-cvspass-location (2011-02-18) 1 commit - Look for password in both CVS and CVSNT password files. It seems that we need separate parsers for these two formats in order not to regress the users of the original cvs. * jk/tag-contains (2010-07-05) 4 commits - Why is "git tag --contains" so slow? - default core.clockskew variable to one day - limit "contains" traversals based on commit timestamp - tag: speed up --contains calculation The idea of the bottom one is probably Ok, except that the use of object flags needs to be rethought, or at least the helper needs to be moved to builtin/tag.c to make it clear that it should not be used outside the current usage context. * jk/edit-notes-in-commit-log (2011-03-07) 2 commits - [wip] commit: allow editing notes in commit message editor - notes: make expand_notes_ref globally accessible * jc/rename-degrade-cc-to-c (2011-01-06) 3 commits . diffcore-rename: fall back to -C when -C -C busts the rename limit . diffcore-rename: record filepair for rename src . diffcore-rename: refactor "too many candidates" logic Will take another look. * mg/grep-full-tree (2011-03-01) 2 commits - grep: make --full-tree work with pathspecs - grep: --full-tree Do not merge; it would be preferable to use ":/" or whatever magic pathspec that is relative to the root of the working tree. * nd/rfc-add-u-full-tree (2011-02-07) 1 commit - add: make "add -u" update full tree without pathspec A three-phase migration that: (1) first only warns when we would deviate from traditional behaviour and suggest adding "." if the user wants to limit it to cwd, but doesn't actually add full-tree, in the next release; (2) warn that we have changed the behaviour and suggest adding "." if the user wants to limit it to cwd, and actually do full-tree, in 1.8.0; (3) remove the warning, in 1.8.X (for X at least 2 or more). would be a good way forward. -------------------------------------------------- [Cooking] * en/merge-recursive (2011-03-17) 4 commits (merged to 'next' on 2011-03-18 at a32016b) + merge-recursive: tweak magic band-aid (merged to 'next' on 2011-03-09 at 3762932) + merge-recursive: When we detect we can skip an update, actually skip it + t6022: New test checking for unnecessary updates of files in D/F conflicts + t6022: New test checking for unnecessary updates of renamed+modified files I am not happy with these magic band aids. * nd/init-gitdir (2011-03-19) 2 commits - init, clone: support --separate-git-dir for .git file - git-init.txt: move description section up * jl/submodule-fetch-on-demand (2011-03-06) 7 commits - fetch/pull: Describe --recurse-submodule restrictions in the BUGS section - submodule update: Don't fetch when the submodule commit is already present - fetch/pull: Don't recurse into a submodule when commits are already present - Submodules: Add 'on-demand' value for the 'fetchRecurseSubmodule' option - config: teach the fetch.recurseSubmodules option the 'on-demand' value - fetch/pull: Add the 'on-demand' value to the --recurse-submodules option - fetch/pull: recurse into submodules when necessary Is this ready to move forward? * ss/git-gui-mergetool (2011-02-26) 2 commits (merged to 'next' on 2011-03-17 at a8fac18) + mergetool--lib: Add Beyond Compare 3 as a tool + mergetool--lib: Sort tools alphabetically for easier lookup The CLI side (ss/mergetool--lib topic) will be in the next release, so I'll ping Pat to take this into gut-gui tree. * nd/index-doc (2010-09-06) 1 commit - doc: technical details about the index file format * ab/i18n-st (2011-02-22) 69 commits - i18n: git-shortlog basic messages - i18n: git-revert split up "could not revert/apply" message - i18n: git-revert literal "me" messages - i18n: git-revert "Your local changes" message - i18n: git-revert basic messages - i18n: git-notes GIT_NOTES_REWRITE_MODE error message - i18n: git-notes basic commands - i18n: git-gc "Auto packing the repository" message - i18n: git-gc basic messages - i18n: git-describe basic messages - i18n: git-clean clean.requireForce messages - i18n: git-clean basic messages - i18n: git-bundle basic messages - i18n: git-archive basic messages - i18n: git-status "renamed: " message - i18n: git-status "Initial commit" message - i18n: git-status "Changes to be committed" message - i18n: git-status shortstatus messages - i18n: git-status "nothing to commit" messages - i18n: git-status basic messages - i18n: git-push "prevent you from losing" message - i18n: git-push basic messages - i18n: git-tag tag_template message - i18n: git-tag basic messages - i18n: git-reset "Unstaged changes after reset" message - i18n: git-reset reset_type_names messages - i18n: git-reset basic messages - i18n: git-rm basic messages - i18n: git-mv "bad" messages - i18n: git-mv basic messages - i18n: git-merge "Wonderful" message - i18n: git-merge "You have not concluded your merge" messages - i18n: git-merge "Updating %s..%s" message - i18n: git-merge basic messages - i18n: git-log "--OPT does not make sense" messages - i18n: git-log basic messages - i18n: git-grep "--open-files-in-pager" message - i18n: git-grep basic messages - i18n: git-fetch split up "(non-fast-forward)" message - i18n: git-fetch update_local_ref messages - i18n: git-fetch formatting messages - i18n: git-fetch basic messages - i18n: git-diff basic messages - i18n: git-commit advice messages - i18n: git-commit "enter the commit message" message - i18n: git-commit print_summary messages - i18n: git-commit formatting messages - i18n: git-commit "middle of a merge" message - i18n: git-commit basic messages - i18n: git-checkout "Switched to a .. branch" message - i18n: git-checkout "HEAD is now at" message - i18n: git-checkout describe_detached_head messages - i18n: git-checkout: our/their version message - i18n: git-checkout basic messages - i18n: git-branch "(no branch)" message - i18n: git-branch "git branch -v" messages - i18n: git-branch "Deleted branch [...]" message - i18n: git-branch "remote branch '%s' not found" message - i18n: git-branch basic messages - i18n: git-add "Unstaged changes" message - i18n: git-add "remove '%s'" message - i18n: git-add "did not match any files" message - i18n: git-add "The following paths are ignored" message - i18n: git-add basic messages - i18n: git-clone "Cloning into" message - i18n: git-clone "Cloning into" message - i18n: git-clone basic messages - i18n: git-init "Initialized [...] repository" message - i18n: git-init basic messages * mg/rev-list-one-side-only (2011-03-10) 10 commits (merged to 'next' on 2011-03-10 at 2a69869) + git-log: put space after commit mark + t6007: test rev-list --cherry + log --cherry: a synonym + rev-list: documentation and test for --cherry-mark + revision.c: introduce --cherry-mark + rev-list/log: factor out revision mark generation + rev-list: --left/right-only are mutually exclusive + rev-list: documentation and test for --left/right-only + t6007: Make sure we test --cherry-pick + revlist.c: introduce --left/right-only for unsymmetric picking Looked sensible, including the change of the output. ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: What's cooking in git.git (Mar 2011, #02; Sun, 20) 2011-03-20 19:49 What's cooking in git.git (Mar 2011, #02; Sun, 20) Junio C Hamano @ 2011-03-20 20:43 ` Jens Lehmann 2011-03-20 21:06 ` Junio C Hamano 2011-03-22 10:57 ` What's cooking in git.git (Mar 2011, #02; Sun, 20) Ævar Arnfjörð Bjarmason 2 siblings, 0 replies; 15+ messages in thread From: Jens Lehmann @ 2011-03-20 20:43 UTC (permalink / raw) To: Junio C Hamano; +Cc: git Am 20.03.2011 20:49, schrieb Junio C Hamano: > * jl/submodule-fetch-on-demand (2011-03-06) 7 commits > - fetch/pull: Describe --recurse-submodule restrictions in the BUGS section > - submodule update: Don't fetch when the submodule commit is already present > - fetch/pull: Don't recurse into a submodule when commits are already present > - Submodules: Add 'on-demand' value for the 'fetchRecurseSubmodule' option > - config: teach the fetch.recurseSubmodules option the 'on-demand' value > - fetch/pull: Add the 'on-demand' value to the --recurse-submodules option > - fetch/pull: recurse into submodules when necessary > > Is this ready to move forward? I'm not aware of any issues with that series, so I'd say yes. (The change in behavior and that we support disconnected operation for submodules way better than before should of course be mentioned in the release notes) ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: What's cooking in git.git (Mar 2011, #02; Sun, 20) 2011-03-20 19:49 What's cooking in git.git (Mar 2011, #02; Sun, 20) Junio C Hamano 2011-03-20 20:43 ` Jens Lehmann @ 2011-03-20 21:06 ` Junio C Hamano 2011-03-20 21:22 ` Junio C Hamano 2011-03-22 10:57 ` What's cooking in git.git (Mar 2011, #02; Sun, 20) Ævar Arnfjörð Bjarmason 2 siblings, 1 reply; 15+ messages in thread From: Junio C Hamano @ 2011-03-20 21:06 UTC (permalink / raw) To: git; +Cc: Nguyễn Thái Ngọc Duy Junio C Hamano <gitster@pobox.com> writes: > * nd/rfc-add-u-full-tree (2011-02-07) 1 commit > - add: make "add -u" update full tree without pathspec > > A three-phase migration that: > > (1) first only warns when we would deviate from traditional behaviour and > suggest adding "." if the user wants to limit it to cwd, but doesn't > actually add full-tree, in the next release; > > (2) warn that we have changed the behaviour and suggest adding "." if the > user wants to limit it to cwd, and actually do full-tree, in 1.8.0; > > (3) remove the warning, in 1.8.X (for X at least 2 or more). > > would be a good way forward. ... and here is how the first step would look like. -- >8 -- Subject: [PATCH] add -u: warn when ran without pathspec from a subdirectory We will be changing the behaviour of "add -u" without any pathspec to be tree-wide in future releases; warn the users and train them to say "add -u ." to help smoother migration. Signed-off-by: Junio C Hamano <gitster@pobox.com> --- builtin/add.c | 18 ++++++++++++++++++ 1 files changed, 18 insertions(+), 0 deletions(-) diff --git a/builtin/add.c b/builtin/add.c index e127d5a..443d3bb 100644 --- a/builtin/add.c +++ b/builtin/add.c @@ -359,6 +359,21 @@ static int add_files(struct dir_struct *dir, int flags) return exit_status; } +static const char *warn_add_u_without_pathspec_msg[] = { + "In release 1.8.0, running 'git add -u' from a subdirectory", + "without giving any pathspec WILL take effect on the whole", + "working tree, not just the part under the current directory.", + "Please make it a habit to say 'git add -u .' when you mean", + "to only add paths under the current directory." +}; + +static void warn_add_u_without_pathspec(void) +{ + int i; + for (i = 0; i < ARRAY_SIZE(warn_add_u_without_pathspec_msg); i++) + warning("%s", warn_add_u_without_pathspec_msg[i]); +} + int cmd_add(int argc, const char **argv, const char *prefix) { int exit_status = 0; @@ -390,6 +405,9 @@ int cmd_add(int argc, const char **argv, const char *prefix) die("Option --ignore-missing can only be used together with --dry-run"); if ((addremove || take_worktree_changes) && !argc) { static const char *here[2] = { ".", NULL }; + + if (take_worktree_changes && prefix) + warn_add_u_without_pathspec(); argc = 1; argv = here; } -- 1.7.4.1.514.ga171c ^ permalink raw reply related [flat|nested] 15+ messages in thread
* Re: What's cooking in git.git (Mar 2011, #02; Sun, 20) 2011-03-20 21:06 ` Junio C Hamano @ 2011-03-20 21:22 ` Junio C Hamano 2011-03-20 21:36 ` Matthieu Moy 0 siblings, 1 reply; 15+ messages in thread From: Junio C Hamano @ 2011-03-20 21:22 UTC (permalink / raw) To: Nguyễn Thái Ngọc Duy; +Cc: git Junio C Hamano <gitster@pobox.com> writes: >> * nd/rfc-add-u-full-tree (2011-02-07) 1 commit >> - add: make "add -u" update full tree without pathspec >> >> A three-phase migration that: >> >> (1) first only warns when we would deviate from traditional behaviour and >> suggest adding "." if the user wants to limit it to cwd, but doesn't >> actually add full-tree, in the next release; >> >> (2) warn that we have changed the behaviour and suggest adding "." if the >> user wants to limit it to cwd, and actually do full-tree, in 1.8.0; >> >> (3) remove the warning, in 1.8.X (for X at least 2 or more). >> >> would be a good way forward. > > ... and here is how the first step would look like. ... and here is the second step meant for 1.8.0 release. The third step for 1.8.2 or later should be trivially obvious, so I wouldn't bother sending it out. -- >8 -- Subject: [PATCH] add -u: tree-wide without pathspec The warning still remains in this version, but has to stay here to help people who skipped late 1.7.X releases. Unfortunately we do not have a good way to let people who have already heard about the transition to squelch it yet. That has to wait until we have a magic ":/" pathspec that would mean to match from the top of the working tree no matter where $cwd is. At that point, people can say "git add -u :/" to squelch the warning. Signed-off-by: Junio C Hamano <gitster@pobox.com> --- builtin/add.c | 18 ++++++++++-------- 1 files changed, 10 insertions(+), 8 deletions(-) diff --git a/builtin/add.c b/builtin/add.c index 443d3bb..0ece4b1 100644 --- a/builtin/add.c +++ b/builtin/add.c @@ -360,11 +360,11 @@ static int add_files(struct dir_struct *dir, int flags) } static const char *warn_add_u_without_pathspec_msg[] = { - "In release 1.8.0, running 'git add -u' from a subdirectory", - "without giving any pathspec WILL take effect on the whole", - "working tree, not just the part under the current directory.", - "Please make it a habit to say 'git add -u .' when you mean", - "to only add paths under the current directory." + "Since release 1.8.0, running 'git add -u' from a subdirectory", + "without giving any pathspec takes effect on the whole working", + "tree, not just the part under the current directory.", + "Please say 'git add -u .' if you want to only add paths under", + "the current directory." }; static void warn_add_u_without_pathspec(void) @@ -403,11 +403,13 @@ int cmd_add(int argc, const char **argv, const char *prefix) die("-A and -u are mutually incompatible"); if (!show_only && ignore_missing) die("Option --ignore-missing can only be used together with --dry-run"); - if ((addremove || take_worktree_changes) && !argc) { + + if (take_worktree_changes && !argc && prefix) + warn_add_u_without_pathspec(); + + if (addremove && !argc) { static const char *here[2] = { ".", NULL }; - if (take_worktree_changes && prefix) - warn_add_u_without_pathspec(); argc = 1; argv = here; } -- 1.7.4.1.514.ga171c ^ permalink raw reply related [flat|nested] 15+ messages in thread
* Re: What's cooking in git.git (Mar 2011, #02; Sun, 20) 2011-03-20 21:22 ` Junio C Hamano @ 2011-03-20 21:36 ` Matthieu Moy 2011-03-21 0:06 ` Junio C Hamano 0 siblings, 1 reply; 15+ messages in thread From: Matthieu Moy @ 2011-03-20 21:36 UTC (permalink / raw) To: Junio C Hamano; +Cc: Nguyễn Thái Ngọc Duy, git Junio C Hamano <gitster@pobox.com> writes: > Unfortunately > we do not have a good way to let people who have already heard > about the transition to squelch it yet. How about an advice.addWithoutPath config variable to disable it? -- Matthieu Moy http://www-verimag.imag.fr/~moy/ ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: What's cooking in git.git (Mar 2011, #02; Sun, 20) 2011-03-20 21:36 ` Matthieu Moy @ 2011-03-21 0:06 ` Junio C Hamano 2011-03-21 11:16 ` Jeff King 0 siblings, 1 reply; 15+ messages in thread From: Junio C Hamano @ 2011-03-21 0:06 UTC (permalink / raw) To: Matthieu Moy; +Cc: Nguyễn Thái Ngọc Duy, git Matthieu Moy <Matthieu.Moy@grenoble-inp.fr> writes: > Junio C Hamano <gitster@pobox.com> writes: > >> Unfortunately >> we do not have a good way to let people who have already heard >> about the transition to squelch it yet. > > How about an advice.addWithoutPath config variable to disable it? I don't think it is worth it; I'd rather see :/ magic implemented earlier. Another possibility is to error out on "git add -u" without pathspec in a subdirectory as an error in 1.8.0, and then make it full-tree in 1.8.X in a distant future, but that wouldn't be as pleasant. ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: What's cooking in git.git (Mar 2011, #02; Sun, 20) 2011-03-21 0:06 ` Junio C Hamano @ 2011-03-21 11:16 ` Jeff King 2011-03-21 14:28 ` Junio C Hamano 2011-03-23 0:02 ` [PATCH] add: make "add -u/-A" update full tree without pathspec Junio C Hamano 0 siblings, 2 replies; 15+ messages in thread From: Jeff King @ 2011-03-21 11:16 UTC (permalink / raw) To: Junio C Hamano; +Cc: Matthieu Moy, Nguyễn Thái Ngọc Duy, git On Sun, Mar 20, 2011 at 05:06:32PM -0700, Junio C Hamano wrote: > Matthieu Moy <Matthieu.Moy@grenoble-inp.fr> writes: > > > Junio C Hamano <gitster@pobox.com> writes: > > > >> Unfortunately > >> we do not have a good way to let people who have already heard > >> about the transition to squelch it yet. > > > > How about an advice.addWithoutPath config variable to disable it? > > I don't think it is worth it; I'd rather see :/ magic implemented earlier. > > Another possibility is to error out on "git add -u" without pathspec in a > subdirectory as an error in 1.8.0, and then make it full-tree in 1.8.X in > a distant future, but that wouldn't be as pleasant. I thought we cared about giving fans of the current behavior a way to keep it (without having to use additional syntax). Script-writers already need to use "./" to be portable to old versions of git, so it's not hurting them (unless you want to count N years into the future, when we can discount such old versions). -Peff ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: What's cooking in git.git (Mar 2011, #02; Sun, 20) 2011-03-21 11:16 ` Jeff King @ 2011-03-21 14:28 ` Junio C Hamano 2011-03-23 0:02 ` [PATCH] add: make "add -u/-A" update full tree without pathspec Junio C Hamano 1 sibling, 0 replies; 15+ messages in thread From: Junio C Hamano @ 2011-03-21 14:28 UTC (permalink / raw) To: Jeff King Cc: Junio C Hamano, Matthieu Moy, Nguyễn Thái Ngọc Duy, git Jeff King <peff@peff.net> writes: >> Matthieu Moy <Matthieu.Moy@grenoble-inp.fr> writes: >> >> > Junio C Hamano <gitster@pobox.com> writes: >> > >> >> Unfortunately >> >> we do not have a good way to let people who have already heard >> >> about the transition to squelch it yet. >> > >> > How about an advice.addWithoutPath config variable to disable it? >> >> I don't think it is worth it; I'd rather see :/ magic implemented earlier. >> >> Another possibility is to error out on "git add -u" without pathspec in a >> subdirectory as an error in 1.8.0, and then make it full-tree in 1.8.X in >> a distant future, but that wouldn't be as pleasant. > > I thought we cared about giving fans of the current behavior a way to > keep it (without having to use additional syntax). Script-writers > already need to use "./" to be portable to old versions of git, so it's > not hurting them (unless you want to count N years into the future, when > we can discount such old versions). So you would rather have "addu.relativetocwd" configuration variable that will warn when unset but the behaviour defaults to true for now, and later flip to false? I'd be in favor of that (the variable name can be painted in any color). The patch series was an illustration of the logical conclusion of Duy's patch that punted on the migration path, and if it showed it would not be pleasant, it served its purpose ;-). ^ permalink raw reply [flat|nested] 15+ messages in thread
* [PATCH] add: make "add -u/-A" update full tree without pathspec 2011-03-21 11:16 ` Jeff King 2011-03-21 14:28 ` Junio C Hamano @ 2011-03-23 0:02 ` Junio C Hamano 2011-03-23 5:17 ` [PATCH] add: make "add -u/-A" update full tree without pathspec (step 2) Junio C Hamano 1 sibling, 1 reply; 15+ messages in thread From: Junio C Hamano @ 2011-03-23 0:02 UTC (permalink / raw) To: Jeff King; +Cc: Nguyễn Thái Ngọc Duy, git When -u was introduced in dfdac5d (git-add -u: match the index with working tree., 2007-04-20), "add -u" (without pathspec) added everything. Shortly after, 2ed2c22 (git-add -u paths... now works from subdirectory, 2007-08-16) broke it while fixing something related. This makes -u and -A inconsistent with some other options, namely -p. It's been four years since the unintentional breakage and people are probably used to "git add -u" updating only current directory. Let's plan in 1.8.0 to change its behaviour in such a way that does not hurt existing users too badly during the transition period. - A new add.treewideupdate configuration variable can be set to "true" to make "add -u/-A" that is ran without any pathspec from a subdirectory to affect the whole tree. When the variable is set to "false", the operation is limited to the current working directory. - Missing configuration variable means the same thing as setting it to "false" for now, but the user will be given a warning about the transition plan, and an advise to either set the variable or to say "." - In 1.8.0, the warning message needs to be rephrased, the added test needs to be updated, and the default value for the variable needs to be flipped to "true". In a few releases after that, we would remove the warning message. Signed-off-by: Junio C Hamano <gitster@pobox.com> --- * So how about an alternative with a bit saner migration path? The message would need rewording, though. builtin/add.c | 47 ++++++++++++++++++++++++++++++++++++++++++----- t/t2200-add-update.sh | 49 +++++++++++++++++++++++++++++++++++++++++++++++++ 2 files changed, 91 insertions(+), 5 deletions(-) diff --git a/builtin/add.c b/builtin/add.c index e127d5a..595f5cc 100644 --- a/builtin/add.c +++ b/builtin/add.c @@ -310,6 +310,7 @@ static const char ignore_error[] = static int verbose = 0, show_only = 0, ignored_too = 0, refresh_only = 0; static int ignore_add_errors, addremove, intent_to_add, ignore_missing = 0; +static int default_tree_wide_update = -1; static struct option builtin_add_options[] = { OPT__DRY_RUN(&show_only, "dry run"), @@ -335,6 +336,10 @@ static int add_config(const char *var, const char *value, void *cb) ignore_add_errors = git_config_bool(var, value); return 0; } + if (!strcasecmp(var, "add.treewideupdate")) { + default_tree_wide_update = git_config_bool(var, value); + return 0; + } return git_default_config(var, value, cb); } @@ -359,6 +364,29 @@ static int add_files(struct dir_struct *dir, int flags) return exit_status; } +static const char *warn_add_uA_180_migration_msg[] = { + "In release 1.8.0, running 'git add -u' (or 'git add -A') from", + "a subdirectory without giving any pathspec WILL take effect", + "on the whole working tree, not just the part under the current", + "directory. You can set add.treewideupdate configuration variable", + "to 'false' to keep the current behaviour.", + "You can set the configuration variable to 'true' to make the", + "'git add -u/-A' command without pathspec take effect on the whole", + "working tree now. If you do so, you can use '.' at the end of", + "the command, e.g. 'git add -u .' when you want to limit the", + "operation to the current directory.", + "This warning will be issued until you set the configuration variable", + "to either 'true' or 'false'." +}; + +static int warn_180_migration(void) +{ + int i; + for (i = 0; i < ARRAY_SIZE(warn_add_uA_180_migration_msg); i++) + warning("%s", warn_add_uA_180_migration_msg[i]); + return 0; /* default to "no" (not tree-wide, i.e. local) */ +} + int cmd_add(int argc, const char **argv, const char *prefix) { int exit_status = 0; @@ -368,6 +396,7 @@ int cmd_add(int argc, const char **argv, const char *prefix) int flags; int add_new_files; int require_pathspec; + int whole_tree_add = 0; char *seen = NULL; git_config(add_config, NULL); @@ -389,9 +418,13 @@ int cmd_add(int argc, const char **argv, const char *prefix) if (!show_only && ignore_missing) die("Option --ignore-missing can only be used together with --dry-run"); if ((addremove || take_worktree_changes) && !argc) { - static const char *here[2] = { ".", NULL }; - argc = 1; - argv = here; + whole_tree_add = 1; + if (prefix) { + if (default_tree_wide_update < 0) + default_tree_wide_update = warn_180_migration(); + if (!default_tree_wide_update) + whole_tree_add = 0; + } } add_new_files = !take_worktree_changes && !refresh_only; @@ -406,12 +439,16 @@ int cmd_add(int argc, const char **argv, const char *prefix) (!(addremove || take_worktree_changes) ? ADD_CACHE_IGNORE_REMOVAL : 0)); - if (require_pathspec && argc == 0) { + if (require_pathspec && !(argc || whole_tree_add)) { fprintf(stderr, "Nothing specified, nothing added.\n"); fprintf(stderr, "Maybe you wanted to say 'git add .'?\n"); return 0; } - pathspec = validate_pathspec(argc, argv, prefix); + + if (whole_tree_add) + pathspec = NULL; + else + pathspec = validate_pathspec(argc, argv, prefix); if (read_cache() < 0) die("index file corrupt"); diff --git a/t/t2200-add-update.sh b/t/t2200-add-update.sh index 0692427..7ac8b70 100755 --- a/t/t2200-add-update.sh +++ b/t/t2200-add-update.sh @@ -80,6 +80,55 @@ test_expect_success 'change gets noticed' ' ' +test_expect_success 'update from a subdirectory without pathspec (no config)' ' + # This test needs to be updated to expect the whole tree + # update after 1.8.0 migration. + test_might_fail git config --remove add.treewideupdate && + test_might_fail git reset check dir1 && + echo changed >check && + ( + cd dir1 && + echo even more >sub2 && + git add -u 2>../expect.warning + ) && + git diff-files --name-only dir1 check >actual && + echo check >expect && + test_cmp expect actual && + grep warning expect.warning +' + +test_expect_success 'update from a subdirectory without pathspec (local)' ' + test_when_finished "git config --remove add.treewideupdate; :" && + git config add.treewideupdate false && + test_might_fail git reset check dir1 && + echo changed >check && + ( + cd dir1 && + echo even more >sub2 && + git add -u 2>../expect.warning + ) && + git diff-files --name-only dir1 check >actual && + echo check >expect && + test_cmp expect actual && + ! grep warning expect.warning +' + +test_expect_success 'update from a subdirectory without pathspec (global)' ' + test_when_finished "git config --remove add.treewideupdate; :" && + git config add.treewideupdate true && + test_might_fail git reset check dir1 && + echo changed >check && + ( + cd dir1 && + echo even more >sub2 && + git add -u 2>../expect.warning + ) && + git diff-files --name-only dir1 check >actual && + : >expect && + test_cmp expect actual && + ! grep warning expect.warning +' + test_expect_success SYMLINKS 'replace a file with a symlink' ' rm foo && -- 1.7.4.1.559.ga7cf60f ^ permalink raw reply related [flat|nested] 15+ messages in thread
* [PATCH] add: make "add -u/-A" update full tree without pathspec (step 2) 2011-03-23 0:02 ` [PATCH] add: make "add -u/-A" update full tree without pathspec Junio C Hamano @ 2011-03-23 5:17 ` Junio C Hamano 2011-03-23 5:19 ` [PATCH] add: make "add -u/-A" update full tree without pathspec (step 3) Junio C Hamano 0 siblings, 1 reply; 15+ messages in thread From: Junio C Hamano @ 2011-03-23 5:17 UTC (permalink / raw) To: Jeff King; +Cc: Nguyễn Thái Ngọc Duy, git Flip the default behaviour when "git add -u/-A" is run without a pathspec from a subdirectory to tree-wide, and reword the advice message. We will need to keep the advice message for a while to help people who skipped the 1.8.0 boundary. Signed-off-by: Junio C Hamano <gitster@pobox.com> --- * This is how 1.8.0 "flip the default" would look like. builtin/add.c | 20 +++++++++----------- t/t2200-add-update.sh | 5 ++--- 2 files changed, 11 insertions(+), 14 deletions(-) diff --git a/builtin/add.c b/builtin/add.c index 595f5cc..1d76d55 100644 --- a/builtin/add.c +++ b/builtin/add.c @@ -365,17 +365,15 @@ static int add_files(struct dir_struct *dir, int flags) } static const char *warn_add_uA_180_migration_msg[] = { - "In release 1.8.0, running 'git add -u' (or 'git add -A') from", - "a subdirectory without giving any pathspec WILL take effect", + "Since release 1.8.0, running 'git add -u' (or 'git add -A')", + "from a subdirectory without giving any pathspec takes effect", "on the whole working tree, not just the part under the current", - "directory. You can set add.treewideupdate configuration variable", - "to 'false' to keep the current behaviour.", - "You can set the configuration variable to 'true' to make the", - "'git add -u/-A' command without pathspec take effect on the whole", - "working tree now. If you do so, you can use '.' at the end of", - "the command, e.g. 'git add -u .' when you want to limit the", - "operation to the current directory.", - "This warning will be issued until you set the configuration variable", + "directory. If you want to limit the operation to the current", + "directory, please add '.' at the end of the command, like", + "'git add -u .'", + "Alternatively, you can set add.treewideupdate configuration", + "variable to 'false' to keep using the old behaviour.", + "This warning will be issued until you set the variable", "to either 'true' or 'false'." }; @@ -384,7 +382,7 @@ static int warn_180_migration(void) int i; for (i = 0; i < ARRAY_SIZE(warn_add_uA_180_migration_msg); i++) warning("%s", warn_add_uA_180_migration_msg[i]); - return 0; /* default to "no" (not tree-wide, i.e. local) */ + return 1; /* default to "true" (tree-wide, i.e. not local) */ } int cmd_add(int argc, const char **argv, const char *prefix) diff --git a/t/t2200-add-update.sh b/t/t2200-add-update.sh index 7ac8b70..97478cd 100755 --- a/t/t2200-add-update.sh +++ b/t/t2200-add-update.sh @@ -81,8 +81,7 @@ test_expect_success 'change gets noticed' ' ' test_expect_success 'update from a subdirectory without pathspec (no config)' ' - # This test needs to be updated to expect the whole tree - # update after 1.8.0 migration. + # This test needs to be adjusted when warning message is removed test_might_fail git config --remove add.treewideupdate && test_might_fail git reset check dir1 && echo changed >check && @@ -92,7 +91,7 @@ test_expect_success 'update from a subdirectory without pathspec (no config)' ' git add -u 2>../expect.warning ) && git diff-files --name-only dir1 check >actual && - echo check >expect && + : >expect && test_cmp expect actual && grep warning expect.warning ' -- 1.7.4.1.559.ga7cf60f ^ permalink raw reply related [flat|nested] 15+ messages in thread
* [PATCH] add: make "add -u/-A" update full tree without pathspec (step 3) 2011-03-23 5:17 ` [PATCH] add: make "add -u/-A" update full tree without pathspec (step 2) Junio C Hamano @ 2011-03-23 5:19 ` Junio C Hamano 2011-03-23 14:02 ` Nguyen Thai Ngoc Duy 0 siblings, 1 reply; 15+ messages in thread From: Junio C Hamano @ 2011-03-23 5:19 UTC (permalink / raw) To: Jeff King; +Cc: Nguyễn Thái Ngọc Duy, git Now long after 1.8.0 happened, people should have got used to the new default behaviour and it is no longer necessary to give the migration advice anymore. Signed-off-by: Junio C Hamano <gitster@pobox.com> --- * And this concludes the two-year long series from the future ;-) builtin/add.c | 23 +---------------------- t/t2200-add-update.sh | 3 +-- 2 files changed, 2 insertions(+), 24 deletions(-) diff --git a/builtin/add.c b/builtin/add.c index 1d76d55..71f3aa5 100644 --- a/builtin/add.c +++ b/builtin/add.c @@ -364,27 +364,6 @@ static int add_files(struct dir_struct *dir, int flags) return exit_status; } -static const char *warn_add_uA_180_migration_msg[] = { - "Since release 1.8.0, running 'git add -u' (or 'git add -A')", - "from a subdirectory without giving any pathspec takes effect", - "on the whole working tree, not just the part under the current", - "directory. If you want to limit the operation to the current", - "directory, please add '.' at the end of the command, like", - "'git add -u .'", - "Alternatively, you can set add.treewideupdate configuration", - "variable to 'false' to keep using the old behaviour.", - "This warning will be issued until you set the variable", - "to either 'true' or 'false'." -}; - -static int warn_180_migration(void) -{ - int i; - for (i = 0; i < ARRAY_SIZE(warn_add_uA_180_migration_msg); i++) - warning("%s", warn_add_uA_180_migration_msg[i]); - return 1; /* default to "true" (tree-wide, i.e. not local) */ -} - int cmd_add(int argc, const char **argv, const char *prefix) { int exit_status = 0; @@ -419,7 +398,7 @@ int cmd_add(int argc, const char **argv, const char *prefix) whole_tree_add = 1; if (prefix) { if (default_tree_wide_update < 0) - default_tree_wide_update = warn_180_migration(); + default_tree_wide_update = 1; if (!default_tree_wide_update) whole_tree_add = 0; } diff --git a/t/t2200-add-update.sh b/t/t2200-add-update.sh index 97478cd..8479ff2 100755 --- a/t/t2200-add-update.sh +++ b/t/t2200-add-update.sh @@ -81,7 +81,6 @@ test_expect_success 'change gets noticed' ' ' test_expect_success 'update from a subdirectory without pathspec (no config)' ' - # This test needs to be adjusted when warning message is removed test_might_fail git config --remove add.treewideupdate && test_might_fail git reset check dir1 && echo changed >check && @@ -93,7 +92,7 @@ test_expect_success 'update from a subdirectory without pathspec (no config)' ' git diff-files --name-only dir1 check >actual && : >expect && test_cmp expect actual && - grep warning expect.warning + ! grep warning expect.warning ' test_expect_success 'update from a subdirectory without pathspec (local)' ' -- 1.7.4.1.586.g183369 ^ permalink raw reply related [flat|nested] 15+ messages in thread
* Re: [PATCH] add: make "add -u/-A" update full tree without pathspec (step 3) 2011-03-23 5:19 ` [PATCH] add: make "add -u/-A" update full tree without pathspec (step 3) Junio C Hamano @ 2011-03-23 14:02 ` Nguyen Thai Ngoc Duy 2011-03-23 15:54 ` Junio C Hamano 0 siblings, 1 reply; 15+ messages in thread From: Nguyen Thai Ngoc Duy @ 2011-03-23 14:02 UTC (permalink / raw) To: Junio C Hamano; +Cc: Jeff King, git 2011/3/23 Junio C Hamano <gitster@pobox.com>: > Now long after 1.8.0 happened, people should have got used to the > new default behaviour and it is no longer necessary to give the > migration advice anymore. > > Signed-off-by: Junio C Hamano <gitster@pobox.com> > --- > > * And this concludes the two-year long series from the future ;-) Beautiful. I'd expect you kill add.treewideupdate too. Say one year after this patch, we can start to annoy users asking them to remove add.treewideupdate and switch to new behavior. Another year goes by, we refuse to work with those who still keep add.treewideupdate config. A consistent default behavior is a good thing. If I help a git user and later realize "git add -u" on their machine does not work as I expect it to be, it'd be less pleasant. -- Duy ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH] add: make "add -u/-A" update full tree without pathspec (step 3) 2011-03-23 14:02 ` Nguyen Thai Ngoc Duy @ 2011-03-23 15:54 ` Junio C Hamano 0 siblings, 0 replies; 15+ messages in thread From: Junio C Hamano @ 2011-03-23 15:54 UTC (permalink / raw) To: Nguyen Thai Ngoc Duy; +Cc: Jeff King, git Nguyen Thai Ngoc Duy <pclouds@gmail.com> writes: > I'd expect you kill add.treewideupdate too. Of course not. That move would be _only_ to break and irritate users who are content with the behaviour they have chosen to use. What is your justification? I wouldn't be defending such a change in the flamewar after a release of such a version, so you'd better start preparing one for your forked git project if you plan to release one with such a misfeature ;-). ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: What's cooking in git.git (Mar 2011, #02; Sun, 20) 2011-03-20 19:49 What's cooking in git.git (Mar 2011, #02; Sun, 20) Junio C Hamano 2011-03-20 20:43 ` Jens Lehmann 2011-03-20 21:06 ` Junio C Hamano @ 2011-03-22 10:57 ` Ævar Arnfjörð Bjarmason 2011-03-22 18:00 ` Junio C Hamano 2 siblings, 1 reply; 15+ messages in thread From: Ævar Arnfjörð Bjarmason @ 2011-03-22 10:57 UTC (permalink / raw) To: Junio C Hamano; +Cc: git On Sun, Mar 20, 2011 at 20:49, Junio C Hamano <gitster@pobox.com> wrote: > [Graduated to "master"] > > * ab/i18n-basic (2011-02-25) 9 commits > (merged to 'next' on 2011-03-10 at ab70d57) > + i18n: "make distclean" should clean up after "make pot" > + i18n: Makefile: "pot" target to extract messages marked for translation > + i18n: add stub Q_() wrapper for ngettext > + i18n: do not poison translations unless GIT_GETTEXT_POISON envvar is set > + i18n: add GETTEXT_POISON to simulate unfriendly translator > + i18n: add no-op _() and N_() wrappers > + commit, status: use status_printf{,_ln,_more} helpers > + commit: refer to commit template as s->fp > + wt-status: add helpers for printing wt-status lines > (this branch is used by ab/i18n-st.) Thanks a lot for merging this down. > * ab/i18n-st (2011-02-22) 69 commits > - i18n: git-shortlog basic messages > - i18n: git-revert split up "could not revert/apply" message > - i18n: git-revert literal "me" messages > - i18n: git-revert "Your local changes" message > - i18n: git-revert basic messages > - i18n: git-notes GIT_NOTES_REWRITE_MODE error message > - i18n: git-notes basic commands > - i18n: git-gc "Auto packing the repository" message > - i18n: git-gc basic messages > - i18n: git-describe basic messages > - i18n: git-clean clean.requireForce messages > - i18n: git-clean basic messages > - i18n: git-bundle basic messages > - i18n: git-archive basic messages > - i18n: git-status "renamed: " message > - i18n: git-status "Initial commit" message > - i18n: git-status "Changes to be committed" message > - i18n: git-status shortstatus messages > - i18n: git-status "nothing to commit" messages > - i18n: git-status basic messages > - i18n: git-push "prevent you from losing" message > - i18n: git-push basic messages > - i18n: git-tag tag_template message > - i18n: git-tag basic messages > - i18n: git-reset "Unstaged changes after reset" message > - i18n: git-reset reset_type_names messages > - i18n: git-reset basic messages > - i18n: git-rm basic messages > - i18n: git-mv "bad" messages > - i18n: git-mv basic messages > - i18n: git-merge "Wonderful" message > - i18n: git-merge "You have not concluded your merge" messages > - i18n: git-merge "Updating %s..%s" message > - i18n: git-merge basic messages > - i18n: git-log "--OPT does not make sense" messages > - i18n: git-log basic messages > - i18n: git-grep "--open-files-in-pager" message > - i18n: git-grep basic messages > - i18n: git-fetch split up "(non-fast-forward)" message > - i18n: git-fetch update_local_ref messages > - i18n: git-fetch formatting messages > - i18n: git-fetch basic messages > - i18n: git-diff basic messages > - i18n: git-commit advice messages > - i18n: git-commit "enter the commit message" message > - i18n: git-commit print_summary messages > - i18n: git-commit formatting messages > - i18n: git-commit "middle of a merge" message > - i18n: git-commit basic messages > - i18n: git-checkout "Switched to a .. branch" message > - i18n: git-checkout "HEAD is now at" message > - i18n: git-checkout describe_detached_head messages > - i18n: git-checkout: our/their version message > - i18n: git-checkout basic messages > - i18n: git-branch "(no branch)" message > - i18n: git-branch "git branch -v" messages > - i18n: git-branch "Deleted branch [...]" message > - i18n: git-branch "remote branch '%s' not found" message > - i18n: git-branch basic messages > - i18n: git-add "Unstaged changes" message > - i18n: git-add "remove '%s'" message > - i18n: git-add "did not match any files" message > - i18n: git-add "The following paths are ignored" message > - i18n: git-add basic messages > - i18n: git-clone "Cloning into" message > - i18n: git-clone "Cloning into" message > - i18n: git-clone basic messages > - i18n: git-init "Initialized [...] repository" message > - i18n: git-init basic messages What's your plan for merging this to master? Once this is in I'll rebase my ab/i18n on master, sort out that eval_gettext() issue you raised (maybe we don't need git-sh-i18n--envsubst after all), and submit the shell script i18n translations. ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: What's cooking in git.git (Mar 2011, #02; Sun, 20) 2011-03-22 10:57 ` What's cooking in git.git (Mar 2011, #02; Sun, 20) Ævar Arnfjörð Bjarmason @ 2011-03-22 18:00 ` Junio C Hamano 0 siblings, 0 replies; 15+ messages in thread From: Junio C Hamano @ 2011-03-22 18:00 UTC (permalink / raw) To: Ævar Arnfjörð Bjarmason; +Cc: Junio C Hamano, git Ævar Arnfjörð Bjarmason <avarab@gmail.com> writes: > On Sun, Mar 20, 2011 at 20:49, Junio C Hamano <gitster@pobox.com> wrote: > >> [Graduated to "master"] >> >> * ab/i18n-basic (2011-02-25) 9 commits >> (merged to 'next' on 2011-03-10 at ab70d57) >> + i18n: "make distclean" should clean up after "make pot" >> + i18n: Makefile: "pot" target to extract messages marked for translation >> + i18n: add stub Q_() wrapper for ngettext >> + i18n: do not poison translations unless GIT_GETTEXT_POISON envvar is set >> + i18n: add GETTEXT_POISON to simulate unfriendly translator >> + i18n: add no-op _() and N_() wrappers >> + commit, status: use status_printf{,_ln,_more} helpers >> + commit: refer to commit template as s->fp >> + wt-status: add helpers for printing wt-status lines >> (this branch is used by ab/i18n-st.) > > Thanks a lot for merging this down. Heh, thanks a lot for starting it. Do I have to thank you every time part of this huge series advances??? >> * ab/i18n-st (2011-02-22) 69 commits >> ... >> - i18n: git-init basic messages > > What's your plan for merging this to master? A handful of them down to 'next' first, and then trickle them all reasonably quickly, but probably not before 1.7.5 happens. ^ permalink raw reply [flat|nested] 15+ messages in thread
end of thread, other threads:[~2011-03-23 15:54 UTC | newest] Thread overview: 15+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2011-03-20 19:49 What's cooking in git.git (Mar 2011, #02; Sun, 20) Junio C Hamano 2011-03-20 20:43 ` Jens Lehmann 2011-03-20 21:06 ` Junio C Hamano 2011-03-20 21:22 ` Junio C Hamano 2011-03-20 21:36 ` Matthieu Moy 2011-03-21 0:06 ` Junio C Hamano 2011-03-21 11:16 ` Jeff King 2011-03-21 14:28 ` Junio C Hamano 2011-03-23 0:02 ` [PATCH] add: make "add -u/-A" update full tree without pathspec Junio C Hamano 2011-03-23 5:17 ` [PATCH] add: make "add -u/-A" update full tree without pathspec (step 2) Junio C Hamano 2011-03-23 5:19 ` [PATCH] add: make "add -u/-A" update full tree without pathspec (step 3) Junio C Hamano 2011-03-23 14:02 ` Nguyen Thai Ngoc Duy 2011-03-23 15:54 ` Junio C Hamano 2011-03-22 10:57 ` What's cooking in git.git (Mar 2011, #02; Sun, 20) Ævar Arnfjörð Bjarmason 2011-03-22 18:00 ` Junio C Hamano
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).