Re: Building git 2.4.5 on AIX 6.1 problems

2015-07-09 Thread Junio C Hamano
John Norris writes: > But there are a few archive commands before I get to linking > credentials-store. Is there something special about credentials store? > And is the collect2 an AIX library or general library used by git? Wow, that's a blast from the past. My recollection is vague, as it has

Re: Building git 2.4.5 on AIX 6.1 problems

2015-07-09 Thread Jeff King
On Fri, Jul 10, 2015 at 04:32:37AM +, John Norris wrote: > But there are a few archive commands before I get to linking > credentials-store. Is there something special about credentials store? Not really. It is just the first program alphabetically, so it comes first in the Makefile. You can

Re: Building git 2.4.5 on AIX 6.1 problems

2015-07-09 Thread John Norris
Hi Junio, thank you for this - I will run make with V=1 as you suggested and try and talk to some IBM AIX people. But there are a few archive commands before I get to linking credentials-store. Is there something special about credentials store? And is the collect2 an AIX library or general lib

Re: [PATCH v7 2/8] cherry-pick: treat CHERRY_PICK_HEAD and REVERT_HEAD as refs

2015-07-09 Thread Michael Haggerty
On 07/10/2015 12:06 AM, Junio C Hamano wrote: > David Turner writes: > >> OK, here's my current best idea: >> >> 1. A "pseudoref" is an all-caps file in $GIT_DIR/ that always contains >> at least a SHA1. CHERRY_PICK_HEAD and REVERT_HEAD are examples. Because >> HEAD might be a symbolic ref, it i

Re: Merge after rename

2015-07-09 Thread Jacob Keller
On Thu, Jul 9, 2015 at 1:44 PM, Phil Susi wrote: > I'm trying to cherry pick an old change from an old branch onto the current > master, and since the old change, the directory structure was altered and > the modified files were moved. Instead of detecting the new location of the > file and apply

Re: [PATCH v2] refs: loosen restrictions on wildcard '*' refspecs

2015-07-09 Thread Jacob Keller
On Wed, Jul 8, 2015 at 6:00 AM, Jacob Keller wrote: > This patch updates the check_refname_component logic in order to allow for > a less strict refspec format in regards to REFNAME_REFSPEC_PATTERN. > Previously the '*' could only replace a single full component, and could > not replace arbitrary

[PATCH v4] notes: Allow treeish expressions as notes ref

2015-07-09 Thread Mike Hommey
init_notes() is the main point of entry to the notes API. It is an arbitrary restriction that all it allows as input is a strict ref name, when callers may want to give an arbitrary treeish. However, some operations that require updating the notes tree require a strict ref name, because they would

Re: [PATCH v3] notes: Allow committish expressions as notes ref

2015-07-09 Thread Johan Herland
On Thu, Jul 9, 2015 at 4:48 AM, Mike Hommey wrote: > init_notes() is the main point of entry to the notes API. It is an arbitrary > restriction that all it allows as input is a strict ref name, when callers > may want to give an arbitrary committish. > > However, some operations that require updat

Re: [PATCH v2 9/9] diffcore-pickaxe: support case insensitive match on non-ascii

2015-07-09 Thread Eric Sunshine
On Wed, Jul 8, 2015 at 6:38 AM, Nguyễn Thái Ngọc Duy wrote: > Similar to the "grep -F -i" case, we can't use kws on icase search > outside ascii range, quote we quote the string and pass it to regcomp s/quote we quote/so we quote/ (or something) > as a basic regexp and let regex engine deal wit

[PATCH v8 0/7] ref backend preamble

2015-07-09 Thread David Turner
The current state of the discussion on alternate ref backends is that we're going to continue to store pseudorefs (e.g. CHERRY_PICK_HEAD) as files in $GIT_DIR. So this re-roll of the refs backend preamble doesn't do anything to pseudorefs. It just does reflog stuff. In addition, this version rem

[PATCH v8 3/7] refs: new public ref function: safe_create_reflog

2015-07-09 Thread David Turner
The safe_create_reflog function creates a reflog, if it does not already exist. The log_ref_setup function becomes private and gains a force_create parameter to force the creation of a reflog even if log_all_ref_updates is false or the refname is not one of the special refnames. The new parameter

[PATCH v8 2/7] refs: Break out check for reflog autocreation

2015-07-09 Thread David Turner
This is just for clarity. Signed-off-by: David Turner --- refs.c | 16 +++- 1 file changed, 11 insertions(+), 5 deletions(-) diff --git a/refs.c b/refs.c index 03e7505..903b401 100644 --- a/refs.c +++ b/refs.c @@ -3118,6 +3118,16 @@ static int copy_msg(char *buf, const char *msg)

[PATCH v8 6/7] update-ref and tag: add --create-reflog arg

2015-07-09 Thread David Turner
Allow the creation of a ref (e.g. stash) with a reflog already in place. For most refs (e.g. those under refs/heads), this happens automatically, but for others, we need this option. Currently, git does this by pre-creating the reflog, but alternate ref backends might store reflogs somewhere other

[PATCH v8 4/7] git-reflog: add exists command

2015-07-09 Thread David Turner
Theis are necessary because alternate ref backends might store reflogs somewhere other than .git/logs. Code that now directly manipulates .git/logs should instead go through git-reflog. Signed-off-by: David Turner --- Documentation/git-reflog.txt | 4 builtin/reflog.c | 33 +++

[PATCH v8 7/7] git-stash: use update-ref --create-reflog instead of creating files

2015-07-09 Thread David Turner
This is in support of alternate ref backends which don't necessarily store reflogs as files. Signed-off-by: David Turner --- git-stash.sh | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/git-stash.sh b/git-stash.sh index 8e9e2cd..1d5ba7a 100755 --- a/git-stash.sh +++ b/g

[PATCH v8 5/7] refs: add REF_FORCE_CREATE_REFLOG flag

2015-07-09 Thread David Turner
Add a flag to allow forcing the creation of a reflog even if the ref name and core.logAllRefUpdates setting would not ordinarily cause ref creation. In a moment, we will use this to add options to git tag and git update-ref to force reflog creation. Signed-off-by: David Turner --- refs.c | 34 +

[PATCH v8 1/7] refs.c: add err arguments to reflog functions

2015-07-09 Thread David Turner
Add an err argument to log_ref_setup that can explain the reason for a failure. This then eliminates the need to manage errno through this function since we can just add strerror(errno) to the err string when meaningful. No callers relied on errno from this function for anything else than the error

Re: [PATCH] check_and_freshen_file: fix reversed success-check

2015-07-09 Thread Jeff King
On Thu, Jul 09, 2015 at 10:51:50PM +0200, Johannes Sixt wrote: > >Ah! That code is less than a year old. When I began to adopt a workflow > >requiring force-pushes lately, I wondered why I haven't seen these > >failures earlier, because I did do force pushes in the past, but not > >that frequently

Re: [PATCH] fast-import: Do less work when given "from" matches current branch head

2015-07-09 Thread Junio C Hamano
Junio C Hamano writes: > Mike Hommey writes: > >>> Does that mean the original code was doing a release that was >>> unnecessary? Or does it mean this patch changes what happens on >>> that codepath, namely (1) leaking resource, and/or (2) keeping a >>> tree of the original 'b' that does not ha

Re: [PATCH] fast-import: Do less work when given "from" matches current branch head

2015-07-09 Thread Junio C Hamano
Mike Hommey writes: >> Does that mean the original code was doing a release that was >> unnecessary? Or does it mean this patch changes what happens on >> that codepath, namely (1) leaking resource, and/or (2) keeping a >> tree of the original 'b' that does not have anything to do with the >> tr

Re: [PATCH] fast-import: Do less work when given "from" matches current branch head

2015-07-09 Thread Mike Hommey
On Thu, Jul 09, 2015 at 01:37:02PM -0700, Junio C Hamano wrote: > Mike Hommey writes: > > Cc'ed a few people who appear at the top of "shortlog --no-merges"; > I think the end result is not incorrect, but I want to hear second > opinions on this one. I do not know Shawn still remembers this > co

Re: [PATCH v7 2/8] cherry-pick: treat CHERRY_PICK_HEAD and REVERT_HEAD as refs

2015-07-09 Thread Junio C Hamano
David Turner writes: > OK, here's my current best idea: > > 1. A "pseudoref" is an all-caps file in $GIT_DIR/ that always contains > at least a SHA1. CHERRY_PICK_HEAD and REVERT_HEAD are examples. Because > HEAD might be a symbolic ref, it is not a pseudoref. > > Refs backends do not manage pse

Re: [PATCH v7 2/8] cherry-pick: treat CHERRY_PICK_HEAD and REVERT_HEAD as refs

2015-07-09 Thread David Turner
On Wed, 2015-07-08 at 22:55 -0700, Junio C Hamano wrote: > David Turner writes: > > > I didn't see this until after I had sent my previous message. I think > > the "multiple working trees" argument is strong enough that I will > > change the code (and tests). > > Not just code, but we probably

Re: [PATCH v4] clone: simplify string handling in guess_dir_name()

2015-07-09 Thread Sebastian Schuberth
On Thu, Jul 9, 2015 at 11:21 PM, Junio C Hamano wrote: >> - if (is_bare) { >> - struct strbuf result = STRBUF_INIT; >> - strbuf_addf(&result, "%.*s.git", (int)(end - start), start); >> - dir = strbuf_detach(&result, NULL); >> - } else >> + if (is_ba

Re: [PATCH v4] clone: simplify string handling in guess_dir_name()

2015-07-09 Thread Junio C Hamano
Sebastian Schuberth writes: > - if (is_bare) { > - struct strbuf result = STRBUF_INIT; > - strbuf_addf(&result, "%.*s.git", (int)(end - start), start); > - dir = strbuf_detach(&result, NULL); > - } else > + if (is_bare) > + dir = xstrfmt

Re: [PATCH] check_and_freshen_file: fix reversed success-check

2015-07-09 Thread Johannes Sixt
Am 08.07.2015 um 23:03 schrieb Johannes Sixt: Am 08.07.2015 um 20:33 schrieb Jeff King: ...or maybe in the utime() step there is actually a bug, and we report failure for no good reason. Ugh. Ah! That code is less than a year old. When I began to adopt a workflow requiring force-pushes lately,

Merge after rename

2015-07-09 Thread Phil Susi
I'm trying to cherry pick an old change from an old branch onto the current master, and since the old change, the directory structure was altered and the modified files were moved. Instead of detecting the new location of the file and applying the changes to it, git is re-adding the old file a

Re: [PATCH] fast-import: Do less work when given "from" matches current branch head

2015-07-09 Thread Junio C Hamano
Mike Hommey writes: Cc'ed a few people who appear at the top of "shortlog --no-merges"; I think the end result is not incorrect, but I want to hear second opinions on this one. I do not know Shawn still remembers this code, but what is under discussion seems to have come mostly from ea5e370a (fa

Re: [PATCH v5 1/4] implement submodule config API for lookup of .gitmodules values

2015-07-09 Thread Junio C Hamano
Jens Lehmann writes: >> How does git-submodule access this information? It looks like it just >> hits "git config -f .gitmodules" directly. Perhaps whatever interface is >> designed should be suitable for its use here (and if there really is no >> more interesting interface needed, then why is "g

Re: [PATCH v5 1/4] implement submodule config API for lookup of .gitmodules values

2015-07-09 Thread Jens Lehmann
Am 09.07.2015 um 17:49 schrieb Jeff King: On Thu, Jul 09, 2015 at 02:09:01PM +0200, Heiko Voigt wrote: Instead of test-submodule-config.c to test this new module, it could be useful to implement these as extensions to rev-parse: git rev-parse --submodule-name [:] git rev-parse --subm

[PATCH v4] clone: simplify string handling in guess_dir_name()

2015-07-09 Thread Sebastian Schuberth
Signed-off-by: Sebastian Schuberth --- builtin/clone.c | 17 + 1 file changed, 5 insertions(+), 12 deletions(-) diff --git a/builtin/clone.c b/builtin/clone.c index 00535d0..ebcb849 100644 --- a/builtin/clone.c +++ b/builtin/clone.c @@ -147,6 +147,7 @@ static char *get_repo_path(

[PATCH v3] clone: Simplify string handling in guess_dir_name()

2015-07-09 Thread Sebastian Schuberth
Signed-off-by: Sebastian Schuberth --- builtin/clone.c | 17 + 1 file changed, 5 insertions(+), 12 deletions(-) diff --git a/builtin/clone.c b/builtin/clone.c index 00535d0..ebcb849 100644 --- a/builtin/clone.c +++ b/builtin/clone.c @@ -147,6 +147,7 @@ static char *get_repo_path(

Re: [PATCH v2] clone: Simplify string handling in guess_dir_name()

2015-07-09 Thread Junio C Hamano
Sebastian Schuberth writes: > On Thu, Jul 9, 2015 at 8:05 PM, Junio C Hamano wrote: > >>> Content-Type: multipart/mixed; >>> boundary="=_Part_8_836493213.1436462597065" >> >> Please don't. > > This seems to come from submitgit, I've filed an issue about it: > > https://github.com/rtyley/su

Re: Building git 2.4.5 on AIX 6.1 problems

2015-07-09 Thread Junio C Hamano
John Norris writes: > I am trying to compile git 2.4.5 which I downloaded as a tar file on > AIX 6.1. The machine has gcc installed. > Having read through the INSTALL file, there appear to be several ways, > either using configure, make, make install or just make, make install > with prefix of wh

Re: [PATCH v2] clone: Simplify string handling in guess_dir_name()

2015-07-09 Thread Sebastian Schuberth
On Thu, Jul 9, 2015 at 8:05 PM, Junio C Hamano wrote: >> Subject: Re: [PATCH v2] clone: Simplify string handling in guess_dir_name() > > We seem not to capitalize the first word on the subject line. Will change that. >> Content-Type: multipart/mixed; >> boundary="=_Part_8_836493213.143646

Building git 2.4.5 on AIX 6.1 problems

2015-07-09 Thread John Norris
I am trying to compile git 2.4.5 which I downloaded as a tar file on AIX 6.1. The machine has gcc installed. Having read through the INSTALL file, there appear to be several ways, either using configure, make, make install or just make, make install with prefix of where to install. Anyway wh

Re: [PATCH v2] clone: Simplify string handling in guess_dir_name()

2015-07-09 Thread Junio C Hamano
Sebastian Schuberth writes: > Subject: Re: [PATCH v2] clone: Simplify string handling in guess_dir_name() We seem not to capitalize the first word on the subject line. > Content-Type: multipart/mixed; > boundary="=_Part_8_836493213.1436462597065" Please don't. > Signed-off-by: Sebastian

Re: [PATCH v4] log: add log.follow config option

2015-07-09 Thread Junio C Hamano
David Turner writes: > On Thu, 2015-07-09 at 10:23 -0700, Junio C Hamano wrote: > >> If I were David and sending this v4 patch, it would have looked like >> this. >> >> -- >8 -- >> >> From: David Turner >> Date: Tue, 7 Jul 2015 21:29:34 -0400 >> Subject: [PATCH v4] log: add "log.follow" config

Re: Reset sometimes updates mtime

2015-07-09 Thread Junio C Hamano
Dennis Kaarsemaker writes: > I'm seeing some behaviour with git reset that I find odd. Basically if I > do > > git fetch && \ > git reset --hard simple-tag-that-points-to-the-current-commit > > sometimes the reset will update the mtime of all files and directories > in the repo and sometimes it w

Re: [PATCH v4] log: add log.follow config option

2015-07-09 Thread David Turner
On Thu, 2015-07-09 at 10:23 -0700, Junio C Hamano wrote: > If I were David and sending this v4 patch, it would have looked like > this. > > -- >8 -- > > From: David Turner > Date: Tue, 7 Jul 2015 21:29:34 -0400 > Subject: [PATCH v4] log: add "log.follow" configuration variable > > People who wo

[PATCH v2] clone: Simplify string handling in guess_dir_name()

2015-07-09 Thread Sebastian Schuberth
Signed-off-by: Sebastian Schuberth --- builtin/clone.c | 16 +++- 1 file changed, 3 insertions(+), 13 deletions(-) diff --git a/builtin/clone.c b/builtin/clone.c index 00535d0..afdc004 100644 --- a/builtin/clone.c +++ b/builtin/clone.c @@ -147,6 +147,7 @@ static char *get_repo_path(c

Re: [PATCH v4] log: add log.follow config option

2015-07-09 Thread Junio C Hamano
Matthieu Moy writes: > David Turner writes: > >> This version uses tweak, and also includes Matthieu Moy's suggested >> whitespace fix. > > This comment should come below the --- after the commit message (right > before the diffstat). Otherwise, Junio will get this as the commit > message when a

Re: [PATCH] clone: Make use of the strip_suffix() helper method

2015-07-09 Thread Sebastian Schuberth
On Thu, Jul 9, 2015 at 7:00 PM, Jeff King wrote: > If you wanted to get really fancy, I think you could put a ternary > operator in the middle of the strip_suffix call. That makes it clear > that "len" is set in all code paths, but I think some people find > ternary operators unreadable. :) I li

Re: [PATCH] clone: Make use of the strip_suffix() helper method

2015-07-09 Thread Jeff King
On Thu, Jul 09, 2015 at 03:33:46PM +, Sebastian Schuberth wrote: > @@ -174,19 +175,17 @@ static char *guess_dir_name(const char *repo, int > is_bundle, int is_bare) >* Strip .{bundle,git}. >*/ > if (is_bundle) { > - if (end - start > 7 && !strncmp(end - 7, ".

Re: [PATCH v5 1/4] implement submodule config API for lookup of .gitmodules values

2015-07-09 Thread Jeff King
On Thu, Jul 09, 2015 at 02:09:01PM +0200, Heiko Voigt wrote: > > Instead of test-submodule-config.c to test this new module, it could > > be useful to implement these as extensions to rev-parse: > > > > git rev-parse --submodule-name [:] > > git rev-parse --submodule-path [:] > > git

[PATCH] clone: Make use of the strip_suffix() helper method

2015-07-09 Thread Sebastian Schuberth
Signed-off-by: Sebastian Schuberth --- builtin/clone.c | 11 +-- 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/builtin/clone.c b/builtin/clone.c index 00535d0..d35b2b9 100644 --- a/builtin/clone.c +++ b/builtin/clone.c @@ -147,6 +147,7 @@ static char *get_repo_path(const c

Reset sometimes updates mtime

2015-07-09 Thread Dennis Kaarsemaker
I'm seeing some behaviour with git reset that I find odd. Basically if I do git fetch && \ git reset --hard simple-tag-that-points-to-the-current-commit sometimes the reset will update the mtime of all files and directories in the repo and sometimes it will leave them alone. Changing it to git f

git svn timezone issue?

2015-07-09 Thread Stefan Tatschner
Hi, at work we use svn, so I used "git svn" to import that stuff to git. Now it seems that there are some timezone issues. "git log" shows + as timezone, while "git svn log" shows the correct timezone +0200. $ git log -1 commit ceb8a8647e257d6caf2ad0ecc2298f8b269c9727 Author: John Doe Date:

Re: git svn timezone issue?

2015-07-09 Thread Stefan Tatschner
On Do, 2015-07-09 at 15:28 +0200, Stefan Tatschner wrote: > Hi, > > at work we use svn, so I used "git svn" to import that stuff to git. > Now it seems that there are some timezone issues. "git log" shows > + > as timezone, while "git svn log" shows the correct timezone +0200. Ok, I just di

Re: [PATCH v2 08/10] tag.c: use 'ref-filter' APIs

2015-07-09 Thread Matthieu Moy
Karthik Nayak writes: > If anyone can help, this is what it's saying. > "[Net::SMTP::SSL] Connection closed at Perhaps your SMTP server thought you were sending too many emails to too many people and closed the connection thinking you were a spammer. If you're having this kind of issues, it may

Re: [PATCH v2 08/10] tag.c: use 'ref-filter' APIs

2015-07-09 Thread Matthieu Moy
Karthik Nayak writes: > + s->atom = parse_ref_filter_atom(arg, arg+len); Spaces around +. -- Matthieu Moy http://www-verimag.imag.fr/~moy/ -- 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://

Re: [PATCH v2 05/10] ref-filter: add option to match literal pattern

2015-07-09 Thread Matthieu Moy
Karthik Nayak writes: > Since 'ref-filter' only has an option to match path names > add an option for regular pattern matching. Here also, a hint on why this is needed would be welcome. > --- a/ref-filter.c > +++ b/ref-filter.c > @@ -956,6 +956,20 @@ static int commit_contains(struct ref_filter

Re: [PATCH v2 04/10] ref-filter: add support to sort by version

2015-07-09 Thread Matthieu Moy
Karthik Nayak writes: > Add support to sort by version using the "v:refname" and > "version:refname" option. This is achieved by using the > 'version_cmp()' function as the comparing function for qsort. You should elaborate on why you need this. Given the context, I can guess that you will need

Re: [PATCH v2 03/10] ref-filter: support printing N lines from tag annotation

2015-07-09 Thread Matthieu Moy
Karthik Nayak writes: > In 'tag.c' we can print N lines from the annotation of the tag > using the '-n' option. Not only annotation of the tag, but also from the commit message for lightweight tags. > --- a/builtin/tag.c > +++ b/builtin/tag.c > @@ -185,6 +185,10 @@ static enum contains_result c

Re: [PATCH v2 01/10] ref-filter: add %(refname:shortalign=X) option

2015-07-09 Thread Matthieu Moy
Karthik Nayak writes: > Add support for %(refname:shortalign=X) where X is a number. > This will print a shortened refname aligned to the left > followed by spaces for a total length of X characters. > If X is less than the shortened refname size, the entire > shortened refname is printed. Not r

Re: [PATCH v2 06/10] Documentation/tag: remove double occurance of ""

2015-07-09 Thread Karthik Nayak
On Thu, Jul 9, 2015 at 5:49 PM, Christian Couder wrote: > On Thu, Jul 9, 2015 at 12:27 PM, Karthik Nayak wrote: >> Mentored-by: Christian Couder >> Mentored-by: Matthieu Moy >> Signed-off-by: Karthik Nayak >> --- >> Documentation/git-tag.txt | 1 - >> 1 file changed, 1 deletion(-) >> >> diff

Re: [PATCH v2 08/10] tag.c: use 'ref-filter' APIs

2015-07-09 Thread Karthik Nayak
On Thu, Jul 9, 2015 at 6:18 PM, Matthieu Moy wrote: > Karthik Nayak writes: > >> Documentation/git-tag.txt | 16 ++- >> builtin/tag.c | 343 >> ++ >> t/t7004-tag.sh| 8 +- > > This patch was sent with Thunderbird unlike the ot

Re: [PATCH v2 08/10] tag.c: use 'ref-filter' APIs

2015-07-09 Thread Matthieu Moy
Karthik Nayak writes: > Documentation/git-tag.txt | 16 ++- > builtin/tag.c | 343 > ++ > t/t7004-tag.sh| 8 +- This patch was sent with Thunderbird unlike the others in the series. It has some line wrapping which make it una

Re: [PATCH v2 06/10] Documentation/tag: remove double occurance of ""

2015-07-09 Thread Christian Couder
On Thu, Jul 9, 2015 at 12:27 PM, Karthik Nayak wrote: > Mentored-by: Christian Couder > Mentored-by: Matthieu Moy > Signed-off-by: Karthik Nayak > --- > Documentation/git-tag.txt | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/Documentation/git-tag.txt b/Documentation/git-tag.txt > ind

Re: [PATCH v5 1/4] implement submodule config API for lookup of .gitmodules values

2015-07-09 Thread Heiko Voigt
On Wed, Jul 08, 2015 at 04:52:14PM -0400, Phil Hord wrote: > On Mon, Jun 15, 2015 at 5:06 PM, Heiko Voigt wrote: > > In a superproject some commands need to interact with submodules. They > > need to query values from the .gitmodules file either from the worktree > > of from certain revisions. At

[PATCH v2 10/10] tag.c: implement '--merged' and '--no-merged' options

2015-07-09 Thread Karthik Nayak
Using 'ref-filter' APIs implement the '--merged' and '--no-merged' options into 'tag.c'. The '--merged' option lets the user to only list tags merged into the named commit. The '--no-merged' option lets the user to only list tags not merged into the named commit. If no object is provided it assume

[PATCH v2 09/10] tag.c: implement '--format' option

2015-07-09 Thread Karthik Nayak
Implement the '--format' option provided by 'ref-filter'. This lets the user list tags as per desired format similar to the implementation in 'git for-each-ref'. Add tests and documentation for the same. Mentored-by: Christian Couder Mentored-by: Matthieu Moy Signed-off-by: Karthik Nayak ---

[PATCH v2 08/10] tag.c: use 'ref-filter' APIs

2015-07-09 Thread Karthik Nayak
Make 'tag.c' use 'ref-filter' APIs for iterating through refs sorting and printing of refs. This removes most of the code used in 'tag.c' replacing it with calls to the 'ref-filter' library. Make 'tag.c' use the 'filter_refs()' function provided by 'ref-filter' to filter out tags based on the opt

[PATCH v2 08/10] tag.c: use 'ref-filter' APIs

2015-07-09 Thread Karthik Nayak
Make 'tag.c' use 'ref-filter' APIs for iterating through refs sorting and printing of refs. This removes most of the code used in 'tag.c' replacing it with calls to the 'ref-filter' library. Make 'tag.c' use the 'filter_refs()' function provided by 'ref-filter' to filter out tags based on the opt

[PATCH v2 06/10] Documentation/tag: remove double occurance of ""

2015-07-09 Thread Karthik Nayak
Mentored-by: Christian Couder Mentored-by: Matthieu Moy Signed-off-by: Karthik Nayak --- Documentation/git-tag.txt | 1 - 1 file changed, 1 deletion(-) diff --git a/Documentation/git-tag.txt b/Documentation/git-tag.txt index 034d10d..4b04c2b 100644 --- a/Documentation/git-tag.txt +++ b/Documen

[PATCH v2 04/10] ref-filter: add support to sort by version

2015-07-09 Thread Karthik Nayak
Add support to sort by version using the "v:refname" and "version:refname" option. This is achieved by using the 'version_cmp()' function as the comparing function for qsort. Mentored-by: Christian Couder Mentored-by: Matthieu Moy Signed-off-by: Karthik Nayak --- ref-filter.c | 20

[PATCH v2 03/10] ref-filter: support printing N lines from tag annotation

2015-07-09 Thread Karthik Nayak
In 'tag.c' we can print N lines from the annotation of the tag using the '-n' option. Copy code from 'tag.c' to 'ref-filter' and modify 'ref-filter' to support printing of N lines from the annotation of tags. Mentored-by: Christian Couder Mentored-by: Matthieu Moy Signed-off-by: Karthik Nayak -

[PATCH v2 05/10] ref-filter: add option to match literal pattern

2015-07-09 Thread Karthik Nayak
Since 'ref-filter' only has an option to match path names add an option for regular pattern matching. Mentored-by: Christian Couder Mentored-by: Matthieu Moy Signed-off-by: Karthik Nayak --- builtin/for-each-ref.c | 1 + ref-filter.c | 25 - ref-filter.h

[PATCH v2 02/10] ref-filter: add option to filter only tags

2015-07-09 Thread Karthik Nayak
Add an option in 'filter_refs()' to use 'for_each_tag_ref()' and filter refs. This type checking is done by adding a 'FILTER_REFS_TAGS' in 'ref-filter.h' Mentored-by: Christian Couder Mentored-by: Matthieu Moy Signed-off-by: Karthik Nayak --- ref-filter.c | 2 ++ ref-filter.h | 1 + 2 files ch

[PATCH v2 07/10] tag.c: use 'ref-filter' data structures

2015-07-09 Thread Karthik Nayak
Make 'tag.c' use 'ref-filter' data structures and make changes to support the new data structures. This is a part of the process of porting 'tag.c' to use 'ref-filter' APIs. This is a temporary step before porting 'tag.c' to use 'ref-filter' completely. As this is a temporary step, most of the cod

[PATCH v2 01/10] ref-filter: add %(refname:shortalign=X) option

2015-07-09 Thread Karthik Nayak
Add support for %(refname:shortalign=X) where X is a number. This will print a shortened refname aligned to the left followed by spaces for a total length of X characters. If X is less than the shortened refname size, the entire shortened refname is printed. Mentored-by: Christian Couder Mentored

[PATCH v2 00/10] Port tag.c to use ref-filter APIs

2015-07-09 Thread Karthik Nayak
This is part of my GSoC project to unify git tag -l, git branch -l, git for-each-ref This patch series is continued from: http://article.gmane.org/gmane.comp.version-control.git/273569 The previous RFC version is here: http://thread.gmane.org/gmane.comp.version-control.git/272654 Changes in this

Re: [PATCH v4] log: add log.follow config option

2015-07-09 Thread Matthieu Moy
David Turner writes: > This version uses tweak, and also includes Matthieu Moy's suggested > whitespace fix. This comment should come below the --- after the commit message (right before the diffstat). Otherwise, Junio will get this as the commit message when applying, and your actual commit mes