Re: GSoC 2016: applications open, deadline = Fri, 19/2

2016-02-11 Thread Matthieu Moy
Christian Couder writes: > Hi, > > On Wed, Feb 10, 2016 at 10:31 AM, Matthieu Moy > wrote: >> >> So, the first question is: are there volunteers to be GSoC mentors this >> year? > > I can co-mentor this year too, with you or someone else. > With you I think it will work out even if you have less

[PATCH] rev-parse: take prefix into account in --git-common-dir

2016-02-11 Thread Nguyễn Thái Ngọc Duy
Most of the time, get_git_common_dir() returns an absolute path so prefix is irrelevant. If it returns a relative path (e.g. from the main worktree) then prefixing is required. Noticed-by: Mike Hommey Signed-off-by: Nguyễn Thái Ngọc Duy --- On Fri, Feb 12, 2016 at 10:47 AM, Mike Hommey wrote:

`git rev-parse --git-common-dir` doesn't work in a subdirectory of the main work tree

2016-02-11 Thread Mike Hommey
Hi, In a worktree, git rev-parse --git-common-dir returns the non worktree-specific git directory. e.g. .git instead of .git/worktrees/name. The problem is that while it returns the right thing from a subdirectory of a worktree, it doesn't from a subdirectory of the "main" work tree. In the latter

[PATCHv10 2/7] submodule-config: drop check against NULL

2016-02-11 Thread Stefan Beller
Adhere to the common coding style of Git and not check explicitly for NULL throughout the file. There are still other occurrences in the code base but that is usually inside of conditions with side effects. Signed-off-by: Stefan Beller --- submodule-config.c | 6 +++--- 1 file changed, 3 inserti

[PATCHv10 0/7] Expose submodule parallelism to the user

2016-02-11 Thread Stefan Beller
This replaces origin/sb/submodule-parallel-update and is based on origin/master * broke out the patch for redirecting errors to stderr in "submodule update" (Thanks Jonathan, Jacob) * use git_config_int and manually check for less than 0. (Thanks Junio) * use an enum consistently now for subm

[PATCHv10 1/7] submodule-config: keep update strategy around

2016-02-11 Thread Stefan Beller
Currently submodule..update is only handled by git-submodule.sh. C code will start to need to make use of that value as more of the functionality of git-submodule.sh moves into library code in C. Add the update field to 'struct submodule' and populate it so it can be read as sm->update or from sm-

[PATCHv10 3/7] fetching submodules: respect `submodule.fetchJobs` config option

2016-02-11 Thread Stefan Beller
This allows to configure fetching and updating in parallel without having the command line option. This moved the responsibility to determine how many parallel processes to start from builtin/fetch to submodule.c as we need a way to communicate "The user did not specify the number of parallel proc

[PATCHv10 4/7] submodule update: direct error message to stderr

2016-02-11 Thread Stefan Beller
Reroute the error message for specified but initialized submodules to stderr instead of stdout. Signed-off-by: Stefan Beller --- git-submodule.sh | 4 ++-- t/t7400-submodule-basic.sh | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/git-submodule.sh b/git-submodu

[PATCHv10 6/7] submodule update: expose parallelism to the user

2016-02-11 Thread Stefan Beller
Expose possible parallelism either via the "--jobs" CLI parameter or the "submodule.fetchJobs" setting. By having the variable initialized to -1, we make sure 0 can be passed into the parallel processing machine, which will then pick as many parallel workers as there are CPUs. Signed-off-by: Stef

[PATCHv10 7/7] clone: allow an explicit argument for parallel submodule clones

2016-02-11 Thread Stefan Beller
Just pass it along to "git submodule update", which may pick reasonable defaults if you don't specify an explicit number. Signed-off-by: Stefan Beller --- Documentation/git-clone.txt | 6 +- builtin/clone.c | 19 +-- t/t7406-submodule-update.sh | 15 +

[PATCHv10 5/7] git submodule update: have a dedicated helper for cloning

2016-02-11 Thread Stefan Beller
This introduces a new helper function in git submodule--helper which takes care of cloning all submodules, which we want to parallelize eventually. Some tests (such as empty URL, update_mode=none) are required in the helper to make the decision for cloning. These checks have been moved into the C

[PATCH v2] Implement https public key pinning

2016-02-11 Thread Christoph Egger
Add the http.pinnedpubkey configuration option for public key pinning. It allows any string supported by libcurl -- base64(sha256(pubkey)) or filename of the full public key. Signed-off-by: Christoph Egger --- Documentation/config.txt | 6 ++ http.c | 11 +++ 2 fil

Re: [PATCH] Implement https public key pinning

2016-02-11 Thread Christoph Egger
Daniel Stenberg writes: > On Thu, 11 Feb 2016, Christoph Egger wrote: >> +#if LIBCURL_VERSION_NUM >= 0x074400 > > That should probably be 0x072c00 ... This is, of course, right. I used 7.44 / 0x072c00 as base because it has robust support for this feature (including the sha256// variant). One co

Re: [PATCH v4 02/21] refs: add methods for misc ref operations

2016-02-11 Thread David Turner
On Thu, 2016-02-11 at 08:45 +0100, Michael Haggerty wrote: > On 02/05/2016 08:44 PM, David Turner wrote: > > From: Ronnie Sahlberg > > > > Add ref backend methods for: > > resolve_ref_unsafe, verify_refname_available, pack_refs, peel_ref, > > create_symref, resolve_gitlink_ref. > > > > Signed-of

Re: [PATCH v4 03/21] refs: add methods for the ref iterators

2016-02-11 Thread David Turner
On Thu, 2016-02-11 at 09:42 +0100, Michael Haggerty wrote: > On 02/05/2016 08:44 PM, David Turner wrote: > > From: Ronnie Sahlberg > > > > Signed-off-by: Ronnie Sahlberg > > Signed-off-by: David Turner > > --- > > refs.c | 54 > > ++

Re: Bug report: 'git commit --dry-run' corner case: returns error ("nothing to commit") when all conflicts resolved to HEAD

2016-02-11 Thread Stephen & Linda Smith
On Monday, February 08, 2016 06:55:17 PM Stephen & Linda Smith wrote: > > #!/bin/bash > > mkdir test-repository || exit 1 > > cd test-repository > > git init > > echo "Initial contents, unimportant" > test-file > > git add test-file > > git commit -m "Initial commit" > > echo "commit-1-state" > tes

Re: [PATCH v2 00/25] More flexibility in making shallow clones

2016-02-11 Thread Duy Nguyen
On Tue, Feb 9, 2016 at 4:45 AM, Junio C Hamano wrote: > Nguyễn Thái Ngọc Duy writes: > >> This series brings three new options to shallow clone/fetch, to let >> you specify cut point by time, or by excluding some refs, or to let >> you extend shallow boundary by commits. >> >> The series is now

Re: Git config depth option?

2016-02-11 Thread Duy Nguyen
On Thu, Feb 11, 2016 at 12:57 AM, Gary Mort wrote: > I checked the documentation and scanned through the source code for clone: > https://github.com/git/git/blob/master/builtin/clone.c > > And nothing jumps out at me as a way to specify a default depth using > a global git configuration file. I s

Re: RFC: Resumable clone based on hybrid "smart" and "dumb" HTTP

2016-02-11 Thread Duy Nguyen
On Thu, Feb 11, 2016 at 4:01 AM, Jonathan Nieder wrote: > > > I really like this design. I'm tempted to implement it (since it > lacks a bunch of the downsides of clone.bundle). A bit disappointed we still do not address resumable fetch. But I'm happy that it makes people excited and one of

Re: Gated Merge?

2016-02-11 Thread Stefan Beller
On Thu, Feb 11, 2016 at 2:42 PM, Andrew Ardill wrote: > On 12 February 2016 at 09:06, Junio C Hamano wrote: >> >> To realize this, there are low-hanging-fruit building blocks that >> are trivial: >> >> - Such an annotation can be made as a note attached to the commit >>in question; >> >> -

Re: [PATCH] Implement https public key pinning

2016-02-11 Thread Daniel Stenberg
On Thu, 11 Feb 2016, Christoph Egger wrote: +#if LIBCURL_VERSION_NUM >= 0x074400 That should probably be 0x072c00 ... -- / daniel.haxx.se -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vg

[PATCH] Implement https public key pinning

2016-02-11 Thread Christoph Egger
Add the http.pinnedpubkey configuration option for public key pinning. It allows any string supported by libcurl -- base64(sha256(pubkey)) or filename of the full public key. Signed-off-by: Christoph Egger --- For some more sensitive repositories I'd like to properly pin the public key of the

Re: Gated Merge?

2016-02-11 Thread Andrew Ardill
On 12 February 2016 at 09:06, Junio C Hamano wrote: > > To realize this, there are low-hanging-fruit building blocks that > are trivial: > > - Such an annotation can be made as a note attached to the commit >in question; > > - Such a check can be done in pre-commit hook; > > - Such a pre-co

Re: [PATCH 5/5] list-objects: pass full pathname to callbacks

2016-02-11 Thread Jeff King
On Thu, Feb 11, 2016 at 05:28:36PM -0500, Jeff King wrote: > +void show_object_with_name(FILE *out, struct object *obj, const char *name) > [...] > fprintf(out, "%s ", oid_to_hex(&obj->oid)); > for (p = name; *p && *p != '\n'; p++) > fputc(*p, out); > fputc('\n', ou

Re: [PATCH 00/22] add the FORMATPRINTF macro to declare the gcc function

2016-02-11 Thread Jeff King
On Thu, Feb 11, 2016 at 09:59:21AM -0800, Junio C Hamano wrote: > Elia Pinto writes: > > > Add the FORMATPRINTF macro for declaring the gcc function attribute 'format > > printf' > > for code style consistency with similar macro that git already use for > > other gcc > > attributes. And use it

[PATCH 5/5] list-objects: pass full pathname to callbacks

2016-02-11 Thread Jeff King
When we find a blob at "a/b/c", we currently pass this to our show_object_fn callbacks as two components: "a/b/" and "c". Callbacks which want the full value then call path_name(), which concatenates the two. But this is an inefficient interface; the path is a strbuf, and we could simply append "c"

[PATCH 3/5] list-objects: convert name_path to a strbuf

2016-02-11 Thread Jeff King
The "struct name_path" data is examined in only two places: we generate it in process_tree(), and we convert it to a single string in path_name(). Everyone else just passes it through to those functions. We can further note that process_tree() already keeps a single strbuf with the leading tree pa

[PATCH 4/5] list-objects: drop name_path entirely

2016-02-11 Thread Jeff King
In the previous commit, we left name_path as a thin wrapper around a strbuf. This patch drops it entirely. As a result, every show_object_fn callback needs to be adjusted. However, none of their code needs to be changed at all, because the only use was to pass it to path_name(), which now handles t

[PATCH 2/5] show_object_with_name: simplify by using path_name()

2016-02-11 Thread Jeff King
When "git rev-list" shows an object with its associated path name, it does so by walking the name_path linked list and printing each component (stopping at any embedded NULs or newlines). We'd like to eventually get rid of name_path entirely in favor of a single buffer, and dropping this custom pr

[PATCH 1/5] http-push: stop using name_path

2016-02-11 Thread Jeff King
The graph traversal code here passes along a name_path to build up the pathname at which we find each blob. But we never actually do anything with the resulting names, making it a waste of code and memory. This usage came in aa1dbc9 (Update http-push functionality, 2006-03-07), and originally the

[PATCH 0/5] drop "struct name_path" and path_name()

2016-02-11 Thread Jeff King
The graph traversal code in list-objects.c uses "struct name_path" to build a linked list of path components, which it then feeds to the callbacks. This is meant to be efficient, because we keep pointers into the actual tree data for each name. However, there are two things that work against this:

Gated Merge?

2016-02-11 Thread Junio C Hamano
I wish there were a way to let me annotate a commit with a statement like this: * When merging a branch that does not have me in it into a trunk that already has me, do this extra check on the result of the merge before allowing it to be recorded. For example, suppose there is a topic that

Re: [PATCH 03/22] argv-array.h: use the FORMATPRINTF macro to declare the gcc function attribute 'format printf'

2016-02-11 Thread Junio C Hamano
Junio C Hamano writes: > OK, this may answer my previous question. > > It might be pleasing to the eyes when these two macros appearing > together in all CAPS, making them look similar and consistent, > perhaps with two more tweaks: > > - Imitate LAST_ARG_MUST_BE_NULL and spell it FORMAT_PRINTF;

Re: RFC: Resumable clone based on hybrid "smart" and "dumb" HTTP

2016-02-11 Thread Jeff King
On Thu, Feb 11, 2016 at 01:32:22PM -0800, Junio C Hamano wrote: > > ... One > > alternative would be to amend the bundle format so that rather than a > > single file, you get a bundle header whose end says "...and my matching > > packfile is 1234-abcd". And then the client knows that they can fetc

Re: RFC: Resumable clone based on hybrid "smart" and "dumb" HTTP

2016-02-11 Thread Junio C Hamano
Jeff King writes: > ... One > alternative would be to amend the bundle format so that rather than a > single file, you get a bundle header whose end says "...and my matching > packfile is 1234-abcd". And then the client knows that they can fetch > that separately from the same source. I would im

Re: [PATCH v4 12/21] refs: allow log-only updates

2016-02-11 Thread David Turner
On Thu, 2016-02-11 at 11:03 +0100, Michael Haggerty wrote: > On 02/05/2016 08:44 PM, David Turner wrote: > > The refs infrastructure learns about log-only ref updates, which > > only > > update the reflog. Later, we will use this to separate symbolic > > reference resolution from ref updating. >

Re: [PATCH v4 20/21] refs: add LMDB refs storage backend

2016-02-11 Thread David Turner
On Thu, 2016-02-11 at 09:48 +0100, Michael Haggerty wrote: > On 02/05/2016 08:44 PM, David Turner wrote: > > Add a database backend for refs using LMDB. This backend runs git > > for-each-ref about 30% faster than the files backend with fully > > -packed > > refs on a repo with ~120k refs. It's a

Re: [PATCH v4 08/21] refs: add methods to init refs db

2016-02-11 Thread David Turner
On Thu, 2016-02-11 at 09:54 +0100, Michael Haggerty wrote: > On 02/05/2016 08:44 PM, David Turner wrote: > > Alternate refs backends might not need the refs/heads directory and > > so > > on, so we make ref db initialization part of the backend. > > > > Signed-off-by: David Turner > > --- > > bu

Re: [PATCH] mergetool: reorder vim/gvim buffers in three-way diffs

2016-02-11 Thread David Aguilar
On Thu, Feb 11, 2016 at 08:03:57AM -0800, Junio C Hamano wrote: > Michael J Gruber writes: > > >> Does this mean that I should warn in the release notes that some > >> existing users might get their expectation broken but we are going > >> ahead anyway because we think most people read left to ri

Re: [PATCH v4 09/21] refs: add method to rename refs

2016-02-11 Thread David Turner
On Thu, 2016-02-11 at 10:00 +0100, Michael Haggerty wrote: > On 02/05/2016 08:44 PM, David Turner wrote: > > Signed-off-by: David Turner > > --- > > refs.c | 5 + > > refs/files-backend.c | 4 +++- > > refs/refs-internal.h | 9 + > > 3 files changed, 17 insertions(+), 1

Re: [PATCH 03/22] argv-array.h: use the FORMATPRINTF macro to declare the gcc function attribute 'format printf'

2016-02-11 Thread Junio C Hamano
Elia Pinto writes: > Use the new FORMATPRINTF macro (in git-compat-util.h) to declare the gcc > function > attribute 'format printf' > > Signed-off-by: Elia Pinto > --- > argv-array.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/argv-array.h b/argv-array.h > index a

Re: [PATCHv9 0/6] Expose submodule parallelism to the user

2016-02-11 Thread Stefan Beller
On Thu, Feb 11, 2016 at 12:23 PM, Junio C Hamano wrote: > Junio C Hamano writes: > >> Stefan Beller writes: >> > * This seems to clash with 00/20] refs backend. >> Applied this on top of a merge between the current 'master' and >> 'sb/submodule-parallel-update' topic to untangle the

Re: [PATCHv9 0/6] Expose submodule parallelism to the user

2016-02-11 Thread Junio C Hamano
Junio C Hamano writes: > Having said that, at least for this round, I do not think there is > nothing to do at this point on your end;... I obviously meant "I think there is nothing" here. Sorry. -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to major

Re: [PATCHv9 0/6] Expose submodule parallelism to the user

2016-02-11 Thread Junio C Hamano
Junio C Hamano writes: > Stefan Beller writes: > * This seems to clash with 00/20] refs backend. > Applied this on top of a merge between the current 'master' and > 'sb/submodule-parallel-update' topic to untangle the dependency; > otherwise there is no way for this topic to mak

Re: [PATCH] mergetool: reorder vim/gvim buffers in three-way diffs

2016-02-11 Thread Dickson Wong
On 11 February 2016 at 11:31, Junio C Hamano wrote: > So it should suffice to mention it in the release notes as one > bullet item that describes one positive change, among all other > updates described in a simlar way. And there is no special > "warnings for existing users" necessary, I would sa

Re: [PATCHv9 1/6] submodule-config: keep update strategy around

2016-02-11 Thread Junio C Hamano
Junio C Hamano writes: > Stefan Beller writes: > >> +else { >> +submodule->update_command = NULL; >> +if (!strcmp(value, "none")) >> +submodule->update = SM_UPDATE_NONE; >> +else if (!strcmp(value

Re: [PATCH] mergetool: reorder vim/gvim buffers in three-way diffs

2016-02-11 Thread Junio C Hamano
Dickson Wong writes: > Thanks for looking into the patch. No. Thank _you_ for contributing ;-) > Vim buffers are not reorderable and windows are always numbered from > top-left to bottom-right. However, I can see someone who is familiar with > the old behavior issue a "CTRL-W_K" command to mov

Re: [PATCH] mergetool: reorder vim/gvim buffers in three-way diffs

2016-02-11 Thread Dickson Wong
Thanks for looking into the patch. > Thanks. I, being a non-user of vim, was wondering if people who had > their own user-defined commands (macros? and possibly short-cut keys > to invoke them) built around the old (and odd) numbering need to > adjust--in which case we may need to forewarn. > Vim

Re: [PATCH 1/3] git reset --hard gives clean working tree

2016-02-11 Thread Junio C Hamano
tbo...@web.de writes: > From: Torsten Bögershausen > > We define the working tree file is clean if either: > > * the result of running convert_to_git() on the working tree > contents matches what is in the index (because that would mean > doing another "git add" on the path is a no-op);

Re: RFC: Resumable clone based on hybrid "smart" and "dumb" HTTP

2016-02-11 Thread Shawn Pearce
On Wed, Feb 10, 2016 at 7:43 PM, Junio C Hamano wrote: > Jonathan Nieder writes: > >> I really like this design. I'm tempted to implement it (since it >> lacks a bunch of the downsides of clone.bundle). > > Just to see people are not stepping on each others toe, implementing > slightly different

Re: [PATCH 00/22] add the FORMATPRINTF macro to declare the gcc function

2016-02-11 Thread Junio C Hamano
Elia Pinto writes: > Add the FORMATPRINTF macro for declaring the gcc function attribute 'format > printf' > for code style consistency with similar macro that git already use for other > gcc > attributes. And use it where necessary. > > Elia Pinto (22): > git-compat-util.h: add the FORMATPRI

[PATCH 2/3] Factor out convert_cmp_checkout() into convert.c

2016-02-11 Thread tboegi
From: Torsten Bögershausen Factor out the code from read_cache.c retrieveing the blob, converting to worktree format and comparing with the file on disc into convert.c. Move static compare_with_fd() from read_cache.c to convert.c as well. Add a shortcut in convert_cmp_checkout(): When no conve

[PATCH 1/3] git reset --hard gives clean working tree

2016-02-11 Thread tboegi
From: Torsten Bögershausen We define the working tree file is clean if either: * the result of running convert_to_git() on the working tree contents matches what is in the index (because that would mean doing another "git add" on the path is a no-op); OR * the result of running conv

[PATCH 3/3] convert.c: Optimize convert_cmp_checkout() for changed file len

2016-02-11 Thread tboegi
From: Torsten Bögershausen Whenever the size of data from convert_to_working_tree() is different from the length of the file in the working tree, the must be different and compare_with_fd() can be skipped. Signed-off-by: Torsten Bögershausen --- convert.c | 34 +

Re: assert failed in pathspec.c

2016-02-11 Thread Junio C Hamano
Dingyuan Wang writes: > Hi all, > > I've got an assertion error when `git add .` > > git: pathspec.c:317: prefix_pathspec: Assertion `item->nowildcard_len <= > item->len && item->prefix <= item->len' failed. > Aborted (core dumped) Sounds like http://article.gmane.org/gmane.comp.version-control.

Re: [PATCH] mergetool: reorder vim/gvim buffers in three-way diffs

2016-02-11 Thread Junio C Hamano
Michael J Gruber writes: >> Does this mean that I should warn in the release notes that some >> existing users might get their expectation broken but we are going >> ahead anyway because we think most people read left to right and >> then top down? I am OK with saying that--I just wanted to make

Re: [PATCH] mergetool: reorder vim/gvim buffers in three-way diffs

2016-02-11 Thread Michael J Gruber
Junio C Hamano venit, vidit, dixit 10.02.2016 18:45: > Michael J Gruber writes: > >>> Second call for help. Any comments on this from anybody other than >>> the author that I missed to support this change? >> >> OK, applied it (on top of next), looks sane and improves the situation >> for the ma

RE: git 2.7.0 crashes when top-down memory allocation preference is set

2016-02-11 Thread Johannes Schindelin
Hi, On Fri, 5 Feb 2016, Johannes Schindelin wrote: > On Thu, 4 Feb 2016, Klinger, Xia wrote: > > > Thanks for confirming it. I hope a fix is available soon. I am using a > > very old version of Git at the moment to work around this issue, which > > doesn't comply to the requirement of our Stash

[PATCH 07/22] builtin/upload-archive.c: use the FORMATPRINTF macro to declare the gcc function attribute 'format printf'

2016-02-11 Thread Elia Pinto
Use the new FORMATPRINTF macro (in git-compat-util.h) to declare the gcc function attribute 'format printf' Signed-off-by: Elia Pinto --- builtin/upload-archive.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/builtin/upload-archive.c b/builtin/upload-archive.c index dbfe14

[PATCH 02/22] advice.h: use the FORMATPRINTF macro to declare the gcc function attribute 'format printf'

2016-02-11 Thread Elia Pinto
Use the new FORMATPRINTF macro (in git-compat-util.h) to declare the gcc function attribute 'format printf' Signed-off-by: Elia Pinto --- advice.h | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/advice.h b/advice.h index b341a55..eef82e1 100644 --- a/advice.h +++ b/advice.

[PATCH 05/22] builtin/receive-pack.c: use the FORMATPRINTF macro to declare the gcc function attribute 'format printf'

2016-02-11 Thread Elia Pinto
Use the new FORMATPRINTF macro (in git-compat-util.h) to declare the gcc function attribute 'format printf' Signed-off-by: Elia Pinto --- builtin/receive-pack.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/builtin/receive-pack.c b/builtin/receive-pack.c index f2d6761.

[PATCH 04/22] builtin/index-pack.c: use the FORMATPRINTF macro to declare the gcc function attribute 'format printf'

2016-02-11 Thread Elia Pinto
Use the new FORMATPRINTF macro (in git-compat-util.h) to declare the gcc function attribute 'format printf' Signed-off-by: Elia Pinto --- builtin/index-pack.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/builtin/index-pack.c b/builtin/index-pack.c index 6a01509..5941963 1

[PATCH 01/22] git-compat-util.h: add the FORMATPRINTF macro

2016-02-11 Thread Elia Pinto
Add the FORMATPRINTF macro for declaring the gcc function attribute 'format printf' for code style consistency with similar macro that git already use for other gcc attributes. Signed-off-by: Elia Pinto --- git-compat-util.h | 14 -- 1 file changed, 8 insertions(+), 6 deletions(-)

[PATCH 16/22] pkt-line.h: use the FORMATPRINTF macro to declare the gcc function attribute 'format printf'

2016-02-11 Thread Elia Pinto
Use the new FORMATPRINTF macro (in git-compat-util.h) to declare the gcc function attribute 'format printf' Signed-off-by: Elia Pinto --- pkt-line.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/pkt-line.h b/pkt-line.h index 3cb9d91..d3ba3be 100644 --- a/pkt-line.h +++

[PATCH 11/22] daemon.c: use the FORMATPRINTF macro to declare the gcc function attribute 'format printf'

2016-02-11 Thread Elia Pinto
Use the new FORMATPRINTF macro (in git-compat-util.h) to declare the gcc function attribute 'format printf' Signed-off-by: Elia Pinto --- daemon.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/daemon.c b/daemon.c index 46b411c..b4a9207 100644 --- a/daemon.c +++ b/daemo

[PATCH 13/22] http-backend.c: use the FORMATPRINTF macro to declare the gcc function attribute 'format printf'

2016-02-11 Thread Elia Pinto
Use the new FORMATPRINTF macro (in git-compat-util.h) to declare the gcc function attribute 'format printf' Signed-off-by: Elia Pinto --- http-backend.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/http-backend.c b/http-backend.c index 8870a26..c05e99f 100644 --- a/

[PATCH 18/22] strbuf.h: use the FORMATPRINTF macro to declare the gcc function attribute 'format printf'

2016-02-11 Thread Elia Pinto
Use the new FORMATPRINTF macro (in git-compat-util.h) to declare the gcc function attribute 'format printf' Signed-off-by: Elia Pinto --- strbuf.h | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/strbuf.h b/strbuf.h index f72fd14..c887dab 100644 --- a/strbuf.h

[PATCH 14/22] imap-send.c: use the FORMATPRINTF macro to declare the gcc function attribute 'format printf'

2016-02-11 Thread Elia Pinto
Use the new FORMATPRINTF macro (in git-compat-util.h) to declare the gcc function attribute 'format printf' Signed-off-by: Elia Pinto --- imap-send.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/imap-send.c b/imap-send.c index 4d3b773..6739a7c 100644 --- a/imap

[PATCH 09/22] color.h: use the FORMATPRINTF macro to declare the gcc function attribute 'format printf'

2016-02-11 Thread Elia Pinto
Use the new FORMATPRINTF macro (in git-compat-util.h) to declare the gcc function attribute 'format printf' Signed-off-by: Elia Pinto --- color.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/color.h b/color.h index e155d13..8f763d7 100644 --- a/color.h +++ b/color.h @

[PATCH 15/22] merge-recursive.c: use the FORMATPRINTF macro to declare the gcc function attribute 'format printf'

2016-02-11 Thread Elia Pinto
Use the new FORMATPRINTF macro (in git-compat-util.h) to declare the gcc function attribute 'format printf' Signed-off-by: Elia Pinto --- merge-recursive.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/merge-recursive.c b/merge-recursive.c index 8eabde2..891d909 100644 ---

[PATCH 17/22] remote.c: use the FORMATPRINTF macro to declare the gcc function attribute 'format printf'

2016-02-11 Thread Elia Pinto
Use the new FORMATPRINTF macro (in git-compat-util.h) to declare the gcc function attribute 'format printf' Signed-off-by: Elia Pinto --- remote.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/remote.c b/remote.c index 35940a5..579cfcb 100644 --- a/remote.c +++ b/remote.c

[PATCH 03/22] argv-array.h: use the FORMATPRINTF macro to declare the gcc function attribute 'format printf'

2016-02-11 Thread Elia Pinto
Use the new FORMATPRINTF macro (in git-compat-util.h) to declare the gcc function attribute 'format printf' Signed-off-by: Elia Pinto --- argv-array.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/argv-array.h b/argv-array.h index a2fa0aa..e303dfd 100644 --- a/argv-array.h

[PATCH 20/22] transport-helper.c: use the FORMATPRINTF macro to declare the gcc function attribute 'format printf'

2016-02-11 Thread Elia Pinto
Use the new FORMATPRINTF macro (in git-compat-util.h) to declare the gcc function attribute 'format printf' Signed-off-by: Elia Pinto --- transport-helper.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/transport-helper.c b/transport-helper.c index a6bff8b..94c7aab 100644

[PATCH 21/22] utf8.h: use the FORMATPRINTF macro to declare the gcc function attribute 'format printf'

2016-02-11 Thread Elia Pinto
Use the new FORMATPRINTF macro (in git-compat-util.h) to declare the gcc function attribute 'format printf' Signed-off-by: Elia Pinto --- utf8.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/utf8.h b/utf8.h index 7930b44..3dc301c 100644 --- a/utf8.h +++ b/utf8.h @@ -10,7 +

[PATCH 19/22] trace.h: use the FORMATPRINTF macro to declare the gcc function attribute 'format printf'

2016-02-11 Thread Elia Pinto
Use the new FORMATPRINTF macro (in git-compat-util.h) to declare the gcc function attribute 'format printf' Signed-off-by: Elia Pinto --- trace.h | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/trace.h b/trace.h index 179b249..14df130 100644 --- a/trace.h ++

[PATCH 12/22] fsck.c: use the FORMATPRINTF macro to declare the gcc function attribute 'format printf'

2016-02-11 Thread Elia Pinto
Use the new FORMATPRINTF macro (in git-compat-util.h) to declare the gcc function attribute 'format printf' Signed-off-by: Elia Pinto --- fsck.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/fsck.c b/fsck.c index c637f66..953b8e4 100644 --- a/fsck.c +++ b/fsck.c @@ -264,7

[PATCH 22/22] wt-status.h: use the FORMATPRINTF macro to declare the gcc function attribute 'format printf'

2016-02-11 Thread Elia Pinto
Use the new FORMATPRINTF macro (in git-compat-util.h) to declare the gcc function attribute 'format printf' Signed-off-by: Elia Pinto --- wt-status.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/wt-status.h b/wt-status.h index c9b3b74..f8b403a 100644 --- a/wt-status.h

[PATCH 10/22] config.c: use the FORMATPRINTF macro to declare the gcc function attribute 'format printf'

2016-02-11 Thread Elia Pinto
Use the new FORMATPRINTF macro (in git-compat-util.h) to declare the gcc function attribute 'format printf' Signed-off-by: Elia Pinto --- config.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/config.c b/config.c index 86a5eb2..671bfd3 100644 --- a/config.c +++ b/config.c

[PATCH 06/22] builtin/update-index.c: use the FORMATPRINTF macro to declare the gcc function attribute 'format printf'

2016-02-11 Thread Elia Pinto
Use the new FORMATPRINTF macro (in git-compat-util.h) to declare the gcc function attribute 'format printf' Signed-off-by: Elia Pinto --- builtin/update-index.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/builtin/update-index.c b/builtin/update-index.c index 7c5c143..cc6

[PATCH 08/22] cache.h: use the FORMATPRINTF macro to declare the gcc function attribute 'format printf'

2016-02-11 Thread Elia Pinto
Use the new FORMATPRINTF macro (in git-compat-util.h) to declare the gcc function attribute 'format printf' Signed-off-by: Elia Pinto --- cache.h | 22 +++--- 1 file changed, 11 insertions(+), 11 deletions(-) diff --git a/cache.h b/cache.h index 553b04b..fb29154 100644 --- a/ca

[PATCH 00/22] add the FORMATPRINTF macro to declare the gcc function

2016-02-11 Thread Elia Pinto
Add the FORMATPRINTF macro for declaring the gcc function attribute 'format printf' for code style consistency with similar macro that git already use for other gcc attributes. And use it where necessary. Elia Pinto (22): git-compat-util.h: add the FORMATPRINTF macro advice.h: use the FORMATP

Re: [PATCH v4 12/21] refs: allow log-only updates

2016-02-11 Thread Michael Haggerty
On 02/05/2016 08:44 PM, David Turner wrote: > The refs infrastructure learns about log-only ref updates, which only > update the reflog. Later, we will use this to separate symbolic > reference resolution from ref updating. This looks good. I assume it will get some testing later in the series.

Re: [PATCH v4 08/21] refs: add methods to init refs db

2016-02-11 Thread Michael Haggerty
On 02/05/2016 08:44 PM, David Turner wrote: > Alternate refs backends might not need the refs/heads directory and so > on, so we make ref db initialization part of the backend. > > Signed-off-by: David Turner > --- > builtin/init-db.c| 20 ++-- > refs.c | 5 +++

Re: [PATCH v4 09/21] refs: add method to rename refs

2016-02-11 Thread Michael Haggerty
On 02/05/2016 08:44 PM, David Turner wrote: > Signed-off-by: David Turner > --- > refs.c | 5 + > refs/files-backend.c | 4 +++- > refs/refs-internal.h | 9 + > 3 files changed, 17 insertions(+), 1 deletion(-) > > [...] > diff --git a/refs/refs-internal.h b/refs/refs-in

Re: [PATCH v4 20/21] refs: add LMDB refs storage backend

2016-02-11 Thread Michael Haggerty
On 02/05/2016 08:44 PM, David Turner wrote: > Add a database backend for refs using LMDB. This backend runs git > for-each-ref about 30% faster than the files backend with fully-packed > refs on a repo with ~120k refs. It's also about 4x faster than using > fully-unpacked refs. In addition, and

Re: assert failed in pathspec.c

2016-02-11 Thread Dingyuan Wang
git version 2.7.0.GIT (latest git master) Sorry, I have found out that this problem is exactly the same as the previous post I mentioned. When I created the repo, scripts/ was a subrepo, and then I moved scripts/.git out. If I add files in the scripts/ dir, SIGABRT. 在 2016年02月11日 16:24, Torsten

Re: [PATCH v4 03/21] refs: add methods for the ref iterators

2016-02-11 Thread Michael Haggerty
On 02/05/2016 08:44 PM, David Turner wrote: > From: Ronnie Sahlberg > > Signed-off-by: Ronnie Sahlberg > Signed-off-by: David Turner > --- > refs.c | 54 > > refs/files-backend.c | 41 +++ >

Re: GSoC 2016: applications open, deadline = Fri, 19/2

2016-02-11 Thread Christian Couder
Hi, On Wed, Feb 10, 2016 at 10:31 AM, Matthieu Moy wrote: > > So, the first question is: are there volunteers to be GSoC mentors this > year? I can co-mentor this year too, with you or someone else. With you I think it will work out even if you have less time than last year. Best, Christian. --

Re: assert failed in pathspec.c

2016-02-11 Thread Torsten Bögershausen
On 02/11/2016 04:59 AM, Dingyuan Wang wrote: [] Thanks for the bug report. What does git --version give you ? And what happens, if you run the latest version of Git ? -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majord...@vger.kernel.org More ma