Re: [PATCH] clone: allow initial sparse checkouts

2014-02-22 Thread Robin H. Johnson
On Sun, Feb 23, 2014 at 09:52:16AM +0700, Duy Nguyen wrote: > On Sun, Feb 23, 2014 at 9:31 AM, Robin H. Johnson wrote: > > Presently if you want to perform a sparse checkout, you must either do a > > full clone and then recheckout, or do a git init, manually set up > > sparse, and then fetch and

Re: [PATCH] clone: allow initial sparse checkouts

2014-02-22 Thread Duy Nguyen
On Sun, Feb 23, 2014 at 9:31 AM, Robin H. Johnson wrote: > Presently if you want to perform a sparse checkout, you must either do a > full clone and then recheckout, or do a git init, manually set up > sparse, and then fetch and checkout. I think you could do "clone -n" (no checkout), set up spar

Re: [PATCH v3 0/8] Hiding refs

2014-02-22 Thread Duy Nguyen
(Digging up an old thread about initial refs listing in git protocol) On Thu, Feb 7, 2013 at 7:12 AM, Junio C Hamano wrote: > Ævar Arnfjörð Bjarmason writes: > >> I think there's a simpler way to do this, which is that: >> >> * New clients supporting v2 of the protocol send some piece of data >

[PATCH] clone: allow initial sparse checkouts

2014-02-22 Thread Robin H. Johnson
Presently if you want to perform a sparse checkout, you must either do a full clone and then recheckout, or do a git init, manually set up sparse, and then fetch and checkout. This patch implements easily accessible sparse checkouts during clone, in the --sparse-checkout option. $ git clone REPO

Re: [PATCH] i18n: mark all progress lines for translation

2014-02-22 Thread Peter Krefting
Nguy?n Thái Ng?c Duy: I can't think of any case that progress lines are for machine. It started because of the only local untranslated line in "git clone". But I think we should do all at once. Indeed, a good move. Anything that is to be displayed to the end user should be subject to localiz

Re: Fwd: git-reviewed: linking commits to review discussion in git

2014-02-22 Thread Thomas Rast
Jeff King writes: > On Mon, Feb 17, 2014 at 03:12:48PM -0500, Murtuza Mukadam wrote: > >> We have linked peer review discussions on >> git@vger.kernel.org to their respective commits within the main >> git.git repository. You can view the linked reviews from 2012 >> until present in the GitHub re

Re: [PATCH] builtin/blame.c::find_copy_in_blob: no need to scan for line end

2014-02-22 Thread David Kastrup
David Kastrup writes: Oh, good grief. Forgot to sign off, and on rereading, the commit message is clearly subfabulous. So forget this one. The content is fine, though. -- David Kastrup -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majord...@vge

[PATCH] builtin/blame.c::find_copy_in_blob: no need to scan for region end

2014-02-22 Thread David Kastrup
The region end can be looked up just like its beginning. Signed-off-by: David Kastrup --- builtin/blame.c | 9 + 1 file changed, 1 insertion(+), 8 deletions(-) diff --git a/builtin/blame.c b/builtin/blame.c index e44a6bb..96716dd 100644 --- a/builtin/blame.c +++ b/builtin/blame.c @@ -93

[PATCH] builtin/blame.c::find_copy_in_blob: no need to scan for line end

2014-02-22 Thread David Kastrup
--- builtin/blame.c | 9 + 1 file changed, 1 insertion(+), 8 deletions(-) diff --git a/builtin/blame.c b/builtin/blame.c index e44a6bb..96716dd 100644 --- a/builtin/blame.c +++ b/builtin/blame.c @@ -939,7 +939,6 @@ static void find_copy_in_blob(struct scoreboard *sb,

Re: [PATCH] sha1_file: fix delta_stack memory leak in unpack_entry

2014-02-22 Thread Thomas Rast
Junio C Hamano writes: > Jeff King writes: > >> On Fri, Feb 21, 2014 at 06:47:47AM +0700, Nguyễn Thái Ngọc Duy wrote: >> >>> This delta_stack array can grow to any length depending on the actual >>> delta chain, but we forget to free it. Normally it does not matter >>> because we use small_delta

Re: git gc --aggressive led to about 40 times slower "git log --raw"

2014-02-22 Thread Duy Nguyen
On Sat, Feb 22, 2014 at 4:14 PM, Duy Nguyen wrote: > On Sat, Feb 22, 2014 at 3:53 PM, David Kastrup wrote: >> David Kastrup writes: >> >>> Duy Nguyen writes: >>> OK with git://git.savannah.gnu.org/emacs.git we have - a 209MB pack with --aggressive - 1.3GB with --depth=50 >

Re: Still waiting for your response

2014-02-22 Thread Mrs. Supini Thrunkul
Hello, I am Mrs. Supini Thrunkul from Tai Yau Bank Hong Kong, I need your cooperation to transfer $ 47.3 million US Dollars to any trusted account within your control. Contact me for more details. Mrs. Supini Thrunkul Tel: +85 2580 848 65 -- To unsubscribe from this list: send the line "u

Re: [PATCH] diff: do not reuse_worktree_file for submodules

2014-02-22 Thread Thomas Rast
Junio C Hamano writes: > Thomas Rast writes: > >> @@ -2845,8 +2845,9 @@ static struct diff_tempfile *prepare_temp_file(const >> char *name, >> remove_tempfile_installed = 1; >> } >> >> -if (!one->sha1_valid || >> -reuse_worktree_file(name, one->sha1, 1)) { >> +

Re: git gc --aggressive led to about 40 times slower "git log --raw"

2014-02-22 Thread Andreas Schwab
David Kastrup writes: > That does look strange: Emacs has a history of more than 30 years. But > the Git mirror is quite younger. Maybe one needs to make sure to use > the author date rather than the commit date here? There is no difference between commit and author date in the Emacs git mirro

[PATCH v2 2/8] merge-recursive: internal flag to avoid touching the worktree

2014-02-22 Thread Thomas Rast
From: Thomas Rast o->call_depth has a double function: a nonzero call_depth means we want to construct virtual merge bases, but it also means we want to avoid touching the worktree. Introduce a new flag o->no_worktree to trigger only the latter. Signed-off-by: Thomas Rast Signed-off-by: Junio

[PATCH v2 3/8] merge-recursive: -Xindex-only to leave worktree unchanged

2014-02-22 Thread Thomas Rast
From: Thomas Rast Using the new no_worktree flag from the previous commit, we can teach merge-recursive to leave the worktree untouched. Expose this with a new strategy option so that scripts can use it. Signed-off-by: Junio C Hamano --- Documentation/merge-strategies.txt | 4 merge-rec

[PATCH v2 4/8] combine-diff: do not pass revs->dense_combined_merges redundantly

2014-02-22 Thread Thomas Rast
The existing code passed revs->dense_combined_merges along revs itself into the combine-diff functions, which is rather redundant. Remove the 'dense' argument until much further down the callchain to simplify callers. Note that while the caller in submodule.c needs to do extra work now, the next

[PATCH v2 1/8] merge-recursive: remove dead conditional in update_stages()

2014-02-22 Thread Thomas Rast
From: Thomas Rast 650467c (merge-recursive: Consolidate different update_stages functions, 2011-08-11) changed the former argument 'clear' to always be true. Remove the useless conditional. Signed-off-by: Thomas Rast Signed-off-by: Junio C Hamano --- merge-recursive.c | 6 ++ 1 file chan

[PATCH v2 6/8] merge-recursive: allow storing conflict hunks in index

2014-02-22 Thread Thomas Rast
Add a --conflicts-in-index option to merge-recursive, which instructs it to always store the 3-way merged result in the index. (Normally it only does so in recursive invocations, but not for the final result.) This serves as a building block for the "remerge diff" feature coming up in a subsequen

[PATCH v2 5/8] Fold all merge diff variants into an enum

2014-02-22 Thread Thomas Rast
The four ways of displaying merge diffs, * none: no diff * -m: against each parent * -c: combined * --cc: combined-condensed were encoded in three flag bits in struct rev_info. Fold them all into a single enum field that captures the variants. This makes it easier to add new merge diff variants

[PATCH v2 8/8] log --remerge-diff: show what the conflict resolution changed

2014-02-22 Thread Thomas Rast
Git has --cc as a very fast inspection tool that shows a brief summary of what a conflicted merge "looks like", and -c/-m as "give me the full information" data dumps. But --cc actually loses information: if the merge lost(!) some changes from one side, that hunk would fully agree with the other s

[PATCH v2 7/8] name-hash: allow dir hashing even when !ignore_case

2014-02-22 Thread Thomas Rast
The directory hash (for fast checks if the index already has a directory) was only used in ignore_case mode and so depended on that flag. Make it generally available on request. Signed-off-by: Thomas Rast --- cache.h | 2 ++ name-hash.c | 19 --- 2 files changed, 14 inserti

[PATCH v2 0/8] log --remerge-diff

2014-02-22 Thread Thomas Rast
This is the second iteration of http://thread.gmane.org/gmane.comp.version-control.git/241565 Changes since the last version: * Dropped patches 4 and 5 (log --merge-bases) * Implemented the "full-file conflict" scheme explained in the previous cover letter: if the conflicted version is lack

Re: git gc --aggressive led to about 40 times slower "git log --raw"

2014-02-22 Thread Duy Nguyen
On Sat, Feb 22, 2014 at 3:53 PM, David Kastrup wrote: > David Kastrup writes: > >> Duy Nguyen writes: >> >>> OK with git://git.savannah.gnu.org/emacs.git we have >>> >>> - a 209MB pack with --aggressive >>> - 1.3GB with --depth=50 >>> - 1.3GB with --window=4000 --depth=32 >>> - 1.3GB with --

Re: [PATCH] tag: support --sort=version

2014-02-22 Thread Duy Nguyen
On Sat, Feb 22, 2014 at 2:59 PM, Jeff King wrote: > On Fri, Feb 21, 2014 at 06:58:16PM +0700, Duy Nguyen wrote: > >> > --sort=[-][comparison:]field >> [...] >> Why not reversed order? So its syntax could be >> >> [ "-" ] FIELD [ ":" [ "version" | "v" ] ] >> >> It fits better to current f-e-r syn

Re: git gc --aggressive led to about 40 times slower "git log --raw"

2014-02-22 Thread David Kastrup
David Kastrup writes: > Duy Nguyen writes: > >> OK with git://git.savannah.gnu.org/emacs.git we have >> >> - a 209MB pack with --aggressive >> - 1.3GB with --depth=50 >> - 1.3GB with --window=4000 --depth=32 >> - 1.3GB with --depth=20 >> - 821MB with --depth=250 for commits --before=2.years

Re: git gc --aggressive led to about 40 times slower "git log --raw"

2014-02-22 Thread David Kastrup
Duy Nguyen writes: > OK with git://git.savannah.gnu.org/emacs.git we have > > - a 209MB pack with --aggressive > - 1.3GB with --depth=50 > - 1.3GB with --window=4000 --depth=32 > - 1.3GB with --depth=20 > - 821MB with --depth=250 for commits --before=2.years.ago, --depth=50 > for the rest >

Re: [PATCH 4/4] config: teach "git config --file -" to read from the standard input

2014-02-22 Thread Jeff King
On Wed, Feb 19, 2014 at 12:58:55AM +0200, Kirill A. Shutemov wrote: > The patch extends git config --file interface to allow read config from > stdin. > > Editing stdin or setting value in stdin is an error. > > Include by absolute path is allowed in stdin config, but not by relative > path. >

Re: [PATCH] demonstrate git-commit --dry-run exit code behaviour

2014-02-22 Thread Jeff King
On Fri, Feb 21, 2014 at 12:21:13PM -0800, Junio C Hamano wrote: > Tay Ray Chuan writes: > > > In particular, show that --short and --porcelain, while implying > > --dry-run, do not return the same exit code as --dry-run. This is due to > > the wt_status.commitable flag being set only when a long

Re: [PATCH 3/3] make commit --verbose work with --no-status

2014-02-22 Thread Jeff King
On Sat, Feb 22, 2014 at 03:09:22AM +0800, Tay Ray Chuan wrote: > @@ -1141,7 +1146,12 @@ static int parse_and_validate_options(int argc, const > char *argv[], > if (all && argc > 0) > die(_("Paths with -a does not make sense.")); > > - if (status_format != STATUS_FORMAT_D

Re: [PATCH 1/3] rename STATUS_FORMAT_NONE to STATUS_FORMAT_DEFAULT

2014-02-22 Thread Jeff King
On Sat, Feb 22, 2014 at 03:09:20AM +0800, Tay Ray Chuan wrote: > In f3f47a1 (status: add --long output format option), STATUS_FORMAT_NONE > was introduced, meaning "the user did not specify anything". Rename this > to *_DEFAULT to better indicate its meaning. Hmm. We later introduced STATUS_FORMA

Re: [PATCH] sha1_file: fix delta_stack memory leak in unpack_entry

2014-02-22 Thread Jeff King
On Fri, Feb 21, 2014 at 10:09:33AM -0800, Junio C Hamano wrote: > >> This delta_stack array can grow to any length depending on the actual > >> delta chain, but we forget to free it. Normally it does not matter > >> because we use small_delta_stack[] from stack and small_delta_stack > >> can hold

Re: [PATCH v2] tag: support --sort=

2014-02-22 Thread Jeff King
On Sat, Feb 22, 2014 at 10:29:22AM +0700, Nguyễn Thái Ngọc Duy wrote: > Signed-off-by: Nguyễn Thái Ngọc Duy > --- > The new prereq GNULINUX is an ugly workaround until people provide > strverscmp compat implementation. I hope that will happen soon as > strverscmp.c does not look very complex.

Re: [PATCH] tag: support --sort=version

2014-02-22 Thread Jeff King
On Fri, Feb 21, 2014 at 06:58:16PM +0700, Duy Nguyen wrote: > > --sort=[-][comparison:]field > [...] > Why not reversed order? So its syntax could be > > [ "-" ] FIELD [ ":" [ "version" | "v" ] ] > > It fits better to current f-e-r syntax where modifiers are after the > colon. And it avoids th