Re: Does git fast-import support deltas?

2013-04-19 Thread Jonathan Nieder
Hi Ilya, Ilya Basin wrote: 1) a created a git repo from a foreign source using git fast-import 2) new commits were added to the foreign source Can I create a fast-import input stream not containing the commits already existing in my git repo and import it? Yes, if the foreign source is

[RFD/PATCH 0/5] Improving the search for remote-tracking branches

2013-04-19 Thread Johan Herland
Hi, The following patch series has three uncontroversial patches (I hope) to improve the DWIM behavivor git checkout foo, followed by two more experimental patches meant to trigger discussion about how Git decides on what is a remote-tracking branch, and what is not: The first three patches

[RFD/PATCH 1/5] t2024: Add tests verifying current DWIM behavior of 'git checkout branch'

2013-04-19 Thread Johan Herland
The DWIM mode of checkout allows you to run git checkout foo when there is no existing local ref or path called foo, and there is exactly one remote with a remote-tracking branch called foo. Git will then automatically create a new local branch called foo using the remote-tracking foo as its

[RFD/PATCH 2/5] t2024: Show failure to use refspec when DWIMming remote branch names

2013-04-19 Thread Johan Herland
When using git checkout foo to DWIM the creation of local foo from some existing upstream foo, we assume conventional refspecs as created by git clone or git remote add, and fail to work correctly if the current refspecs do not follow the conventional refs/remotes/$remote/* pattern.

[RFD/PATCH 3/5] checkout: Use remote refspecs when DWIMming tracking branches

2013-04-19 Thread Johan Herland
The DWIM mode of checkout allows you to run git checkout foo when there is no existing local ref or path called foo, and there is exactly one remote with a remote-tracking branch called foo. Git will then automatically create a new local branch called foo using the remote-tracking foo as its

[RFD/PATCH 4/5] branch.c: Look up refspecs to validate tracking branches

2013-04-19 Thread Johan Herland
The current code for validating tracking branches (e.g. the argument to the -t/--track option) hardcodes refs/heads/* and refs/remotes/* as the potential locations for tracking branches. This works well with the conventional refspecs created by git clone or git remote add, but does not work if the

[RFD/PATCH 5/5] RFD: Disallow out-of-refspec refs within refs/remotes/* to be used as upstream

2013-04-19 Thread Johan Herland
The previous patch adds validation of upstream remote-tracking branches by parsing the configured refspecs, and making sure that the candidate upstream (if not already matching refs/heads/* or refs/remotes/*) is indeed a remote-tracking branch according to some remote's refspec. For a

[BUG] git reflog on a tag tries to do something unexpected

2013-04-19 Thread Yann Dirson
git reflog, when applied to a tag, should surely bail out saying there is reflog for the requested reference. However, instead it spits out a single line of abbreviated sha1's of an ancestry walk: $ git reflog --decorate v1.8.2.1 5bda18c (tag: v1.8.2.1) 6466fbb 2137ce0 (tag: v1.8.1.6) 4bbb830

Re: [PATCH] t3400 (rebase): add failing test for a peculiar rev spec

2013-04-19 Thread Ramkumar Ramachandra
Junio C Hamano wrote: peel_committish () { case $1 in :/*) peeltmp=$(git rev-parse --verify $1) git rev-parse --verify $peeltmp^0 ;; *)

Re: [PATCH v3 2/2] submodule: drop the top-level requirement

2013-04-19 Thread John Keeping
On Thu, Apr 18, 2013 at 03:40:41PM -0700, Junio C Hamano wrote: John Keeping j...@keeping.me.uk writes: +relative_path () +{ + local target curdir result + target=$1 + curdir=${2-$wt_prefix} + curdir=${curdir%/} + result= + + while test -n $curdir + do +

Re: Git crash in Ubuntu 12.04

2013-04-19 Thread Duy Nguyen
On Thu, Apr 18, 2013 at 9:32 PM, Sivaram Kannan siva.de...@gmail.com wrote: Hi, Probably not because there are no debugging symbols. Not sure how ubuntu packages these symbols.. Would recompiling the source packages and debugging would give different results? Yes. Please do. The reason is

Re: [PATCH v3 1/2] rev-parse: add --prefix option

2013-04-19 Thread Ramkumar Ramachandra
John Keeping wrote: Changes since v2: - Rewrite commit message - Add a new test for 'prefix ignored with HEAD:top' - Use '-\EOF' where appropriate in t1513 Thanks for the re-roll. In the previous iteration, I wasn't sure this was the right approach because I thought it would be

Re: [PATCH v3 1/2] rev-parse: add --prefix option

2013-04-19 Thread John Keeping
On Fri, Apr 19, 2013 at 03:23:37PM +0530, Ramkumar Ramachandra wrote: John Keeping wrote: Changes since v2: - Rewrite commit message - Add a new test for 'prefix ignored with HEAD:top' - Use '-\EOF' where appropriate in t1513 Thanks for the re-roll. In the previous

Re: [PATCH v3 1/2] rev-parse: add --prefix option

2013-04-19 Thread Ramkumar Ramachandra
John Keeping wrote: I'm not actually sure this is better. I'm more afraid of the condition for outputting files changing in the future than of passing output_prefix around, so I'd much rather keep that condition in one place. If you really feel strongly about it, I'd prefer to extract the

Re: Pushing/fetching from/into a shallow-cloned repository

2013-04-19 Thread Duy Nguyen
On Fri, Apr 19, 2013 at 6:27 AM, Philip Oakley philipoak...@iee.org wrote: From: Konstantin Khomoutov kostix+...@007spb.ru So I observe pushing/fetching works OK at least for a simple case like this one. Hence I'd like to ask: if the manual page is wrong or I'm observing some corner case?

Re: [PATCH v3 1/2] rev-parse: add --prefix option

2013-04-19 Thread John Keeping
On Fri, Apr 19, 2013 at 04:45:38PM +0530, Ramkumar Ramachandra wrote: John Keeping wrote: I'm not actually sure this is better. I'm more afraid of the condition for outputting files changing in the future than of passing output_prefix around, so I'd much rather keep that condition in one

Re: [PATCH v3 1/2] rev-parse: add --prefix option

2013-04-19 Thread Ramkumar Ramachandra
John Keeping wrote: It looks to me like setup_git_directory() returns NULL if we're at the top of the working tree so we do need that check. I suspect that's so that !prefix does the right thing. Ah, yes. My bad. I expected it to return / on the toplevel directory actually. -- To

Re: [BUG] git reflog on a tag tries to do something unexpected

2013-04-19 Thread Duy Nguyen
On Fri, Apr 19, 2013 at 5:08 PM, Yann Dirson dir...@bertin.fr wrote: git reflog, when applied to a tag, should surely bail out saying there is reflog for the requested reference. However, instead it spits out a single line of abbreviated sha1's of an ancestry walk: $ git reflog --decorate

Re: is git-p4 compatible with p4/linux?

2013-04-19 Thread Pete Wyckoff
a...@aivor.com wrote on Thu, 18 Apr 2013 20:34 -0500: Perhaps it is a configuration item on the server and/or client. It seems we are running the same version of p4. But just to be sure, check yours against mine: $ cksum $(which p4) 3254530484 2420552 /usr/bin/p4 If yours if

[PATCH] Teach git to change to a given directory using -C option

2013-04-19 Thread Nazri Ramliy
This is similar in spirit to to make -C dir ... and tar -C dir Signed-off-by: Nazri Ramliy ayieh...@gmail.com --- Often I find myself needing to find out quickly the status of a repository that is not in my currenct working directory, like this: $ (cd ~/foo; git log -1) With this

Re: [PATCH v2 1/8] Add new git-cc-cmd helper to contrib

2013-04-19 Thread Ramkumar Ramachandra
Felipe Contreras wrote: The code finds the changes of a commit, runs 'git blame' for each chunk to see which other commits are relevant, and then reports the author and signers. Finally, it calculates what percentage of the total relevant commits each person was involved in, and show only

Re: is git-p4 compatible with p4/linux?

2013-04-19 Thread Alex Tomlinson
I found some symlinks in our depot that have the newline and others that do not. It looks like p4 print just dumps whatever is in the depot file, which of course is what you would expect it to do. So the question (for me) is, how did some of my symlink targets end up without the newline? Note:

Re: Git over HTTPS with basic authentication

2013-04-19 Thread Jeff King
On Fri, Apr 19, 2013 at 10:56:06AM +0200, Sebastian Schmidt wrote: I tried switching to smart http now, but have the same error, I guess I did something wrong. It seems like it is getting 401, but I also noticed that info/refs is just an empty file. This is the config of my apache2: SetEnv

Re: Pushing/fetching from/into a shallow-cloned repository

2013-04-19 Thread Konstantin Khomoutov
On Thu, 18 Apr 2013 15:46:12 -0700 Junio C Hamano gits...@pobox.com wrote: Philip Oakley philipoak...@iee.org writes: So I observe pushing/fetching works OK at least for a simple case like this one. Hence I'd like to ask: if the manual page is wrong or I'm observing some corner case?

Re: [PATCH] Teach git to change to a given directory using -C option

2013-04-19 Thread Jeff King
On Fri, Apr 19, 2013 at 08:21:48PM +0800, Nazri Ramliy wrote: This is similar in spirit to to make -C dir ... and tar -C dir Signed-off-by: Nazri Ramliy ayieh...@gmail.com --- Often I find myself needing to find out quickly the status of a repository that is not in my currenct

Re: [PATCH] t3400 (rebase): add failing test for a peculiar rev spec

2013-04-19 Thread Junio C Hamano
Ramkumar Ramachandra artag...@gmail.com writes: I'm more interested in knowing what you think of my first point: is :/text fundamentally broken, as it can't be combined with other operators like the other rev specs can? If so, how do you think we should fix it? The question :/string wants

Re: [PATCH v2 1/8] Add new git-cc-cmd helper to contrib

2013-04-19 Thread Felipe Contreras
On Fri, Apr 19, 2013 at 8:26 AM, Ramkumar Ramachandra artag...@gmail.com wrote: Felipe Contreras wrote: The code finds the changes of a commit, runs 'git blame' for each chunk to see which other commits are relevant, and then reports the author and signers. Finally, it calculates what

Re: t6200: avoid path mangling issue on Windows

2013-04-19 Thread Junio C Hamano
Johannes Sixt j.s...@viscovery.net writes: Am 4/18/2013 19:05, schrieb Junio C Hamano: Johannes Sixt j.s...@viscovery.net writes: From: Johannes Sixt j...@kdbg.org MSYS bash interprets the slash in the argument core.commentchar=/ as root directory and mangles it into a Windows style path.

[PATCH 0/6] grep with textconv

2013-04-19 Thread Michael J Gruber
This series teaches show and grep to obey textconv: show by default (like diff), grep only on request (--textconv). We might switch the default for the latter also, of course. I'd actually like that. Compared to an earlier (historic) series this one comes with tests. Besides, it has been in use

[PATCH 1/6] t4030: demonstrate behavior of show with textconv

2013-04-19 Thread Michael J Gruber
git show commit obeys the textconc setting while git show blob does not. Demonstrate this in the test. Signed-off-by: Michael J Gruber g...@drmicha.warpmail.net --- t/t4030-diff-textconv.sh | 12 1 file changed, 12 insertions(+) diff --git a/t/t4030-diff-textconv.sh

[PATCH 2/6] show: obey --textconv for blobs

2013-04-19 Thread Michael J Gruber
Currently, diff and cat-file for blobs obey --textconv options (with the former defaulting to --textconv and the latter to --no-textconv) whereas show does not obey this option, even though it takes diff options. Make show on blobs behave like diff, i.e. obey --textconv by default and

[PATCH 3/6] cat-file: do not die on --textconv without textconv filters

2013-04-19 Thread Michael J Gruber
When a command is supposed to use textconv filters (by default or with --textconv) and none are configured then the blob is output without conversion; the only exception to this rule is cat-file --textconv. Make it behave like the rest of textconv aware commands. Signed-off-by: Michael J Gruber

[PATCH 4/6] t7008: demonstrate behavior of grep with textconv

2013-04-19 Thread Michael J Gruber
Currently, git grep does not invoke any textconv filters. Demonstrate this in the tests. Signed-off-by: Michael J Gruber g...@drmicha.warpmail.net --- t/t7008-grep-binary.sh | 19 +++ 1 file changed, 19 insertions(+) diff --git a/t/t7008-grep-binary.sh b/t/t7008-grep-binary.sh

[PATCH 5/6] grep: allow to use textconv filters

2013-04-19 Thread Michael J Gruber
From: Jeff King p...@peff.net Recently and not so recently, we made sure that log/grep type operations use textconv filters when a userfacing diff would do the same: ef90ab6 (pickaxe: use textconv for -S counting, 2012-10-28) b1c2f57 (diff_grep: use textconv buffers for add/deleted files,

Re: [PATCH v3 2/2] submodule: drop the top-level requirement

2013-04-19 Thread Junio C Hamano
John Keeping j...@keeping.me.uk writes: On Thu, Apr 18, 2013 at 03:40:41PM -0700, Junio C Hamano wrote: John Keeping j...@keeping.me.uk writes: +relative_path () +{ + local target curdir result + target=$1 + curdir=${2-$wt_prefix} + curdir=${curdir%/} + result= + +

Re: [PATCH v2 1/8] Add new git-cc-cmd helper to contrib

2013-04-19 Thread Junio C Hamano
Felipe Contreras felipe.contre...@gmail.com writes: The code finds the changes of a commit, runs 'git blame' for each chunk to see which other commits are relevant, and then reports the author and signers. In general, I am not all that interested in adding anything new to contrib/ as git.git

Re: [RFC/PATCH 0/7] Rework git core for native submodules

2013-04-19 Thread Jens Lehmann
Am 18.04.2013 01:17, schrieb Philip Oakley: Would it be possible to summarise the key points and proposals of where the subject is now? Here you go, time to post our third iteration of the comparison list, containing two updates: - easier coding was removed from the advantages - git

Find/prune local branches after upstream branch is deleted?

2013-04-19 Thread Jed Brown
Consider this workflow: $ git checkout -b my/branch hack, commit, ... $ git push -u origin my/branch The branch gets reviewed, merged, and eventually deleted upstream. The remote tracking branch gets pruned via 'git fetch --prune' or 'git remote prune', but that leaves my local branch with an

Re: What's cooking in git.git (Apr 2013, #05; Mon, 15)

2013-04-19 Thread Junio C Hamano
Jonathan Nieder jrnie...@gmail.com writes: Junio C Hamano wrote: You ran 'git add' with neither '-A (--all)' or '--no-all', whose behaviour will change in Git 2.0 with respect to paths you removed from your working tree. * 'git add --no-all pathspec', which is the current

Re: [PATCH] t3400 (rebase): add failing test for a peculiar rev spec

2013-04-19 Thread Junio C Hamano
Junio C Hamano gits...@pobox.com writes: That would make this :/!(a=Ramkumar)(s=move diff.wordRegex)~4 a way to find the fourth-generation parent of ... ... 0b830ac52137 (Documentation: move diff.wordRegex from config.txt to diff-config.txt, 2012-11-13) I had a log output on another

Re: [PATCH v2 1/8] Add new git-cc-cmd helper to contrib

2013-04-19 Thread Felipe Contreras
On Fri, Apr 19, 2013 at 12:08 PM, Junio C Hamano gits...@pobox.com wrote: Felipe Contreras felipe.contre...@gmail.com writes: The code finds the changes of a commit, runs 'git blame' for each chunk to see which other commits are relevant, and then reports the author and signers. In general,

Re: Find/prune local branches after upstream branch is deleted?

2013-04-19 Thread Jeff King
On Fri, Apr 19, 2013 at 12:11:38PM -0500, Jed Brown wrote: Consider this workflow: $ git checkout -b my/branch hack, commit, ... $ git push -u origin my/branch The branch gets reviewed, merged, and eventually deleted upstream. The remote tracking branch gets pruned via 'git fetch

Re: [PATCH] stash: tighten IS_STASH_LIKE logic

2013-04-19 Thread Junio C Hamano
Ramkumar Ramachandra artag...@gmail.com writes: My patch is not solving an end-user problem. Think of it as a source code comment: to answer the question what kind of commit does stash create make?, There already is sufficient comment that explains how a stash commit is constructed, isn't

Re: Find/prune local branches after upstream branch is deleted?

2013-04-19 Thread Jed Brown
Jeff King p...@peff.net writes: Try git branch --merged master to get a list of branches that have already been merged. That's what I use, but I was hoping for something more precise. For example, a branch that started at 'maint' would show up there, but its integration hasn't completed until

Re: [PATCH] t3400 (rebase): add failing test for a peculiar rev spec

2013-04-19 Thread Junio C Hamano
Ramkumar Ramachandra artag...@gmail.com writes: Junio C Hamano wrote: peel_committish () { case $1 in :/*) peeltmp=$(git rev-parse --verify $1) git rev-parse --verify $peeltmp^0

Re: [PATCH v2 5/8] contrib: cc-cmd: add option to parse from committish

2013-04-19 Thread Junio C Hamano
Felipe Contreras felipe.contre...@gmail.com writes: For example master..feature-a. Signed-off-by: Felipe Contreras felipe.contre...@gmail.com --- contrib/cc-cmd/git-cc-cmd | 36 ++-- 1 file changed, 34 insertions(+), 2 deletions(-) diff --git

Re: Find/prune local branches after upstream branch is deleted?

2013-04-19 Thread Jeff King
On Fri, Apr 19, 2013 at 12:50:48PM -0500, Jed Brown wrote: Jeff King p...@peff.net writes: Try git branch --merged master to get a list of branches that have already been merged. That's what I use, but I was hoping for something more precise. For example, a branch that started at

Re: [PATCH 3/6] cat-file: do not die on --textconv without textconv filters

2013-04-19 Thread Junio C Hamano
Michael J Gruber g...@drmicha.warpmail.net writes: When a command is supposed to use textconv filters (by default or with --textconv) and none are configured then the blob is output without conversion; the only exception to this rule is cat-file --textconv. I am of two minds. Because

Re: [PATCH] stash: tighten IS_STASH_LIKE logic

2013-04-19 Thread Ramkumar Ramachandra
Junio C Hamano wrote: The risk of actually closing the door for future developers. That is a downside of this patch, if we were to apply it. Okay, no issues. Drop it. Consider documenting the fact that IS_STASH_LIKE is merely a contextual synonym for IS_MERGE_COMMIT somewhere though. -- To

Re: [PATCH 0/6] grep with textconv

2013-04-19 Thread Junio C Hamano
Michael J Gruber g...@drmicha.warpmail.net writes: This series teaches show and grep to obey textconv: show by default (like diff), grep only on request (--textconv). We might switch the default for the latter also, of course. I'd actually like that. Compared to an earlier (historic)

Re: [PATCH v2 5/8] contrib: cc-cmd: add option to parse from committish

2013-04-19 Thread Felipe Contreras
On Fri, Apr 19, 2013 at 12:59 PM, Junio C Hamano gits...@pobox.com wrote: Felipe Contreras felipe.contre...@gmail.com writes: For example master..feature-a. Signed-off-by: Felipe Contreras felipe.contre...@gmail.com --- contrib/cc-cmd/git-cc-cmd | 36 ++--

Re: [PATCH] stash: tighten IS_STASH_LIKE logic

2013-04-19 Thread Junio C Hamano
Ramkumar Ramachandra artag...@gmail.com writes: Junio C Hamano wrote: The risk of actually closing the door for future developers. That is a downside of this patch, if we were to apply it. Okay, no issues. Drop it. Consider documenting the fact that IS_STASH_LIKE is merely a contextual

Re: [PATCH v3 2/2] submodule: drop the top-level requirement

2013-04-19 Thread Johannes Sixt
Am 19.04.2013 18:45, schrieb Junio C Hamano: John Keeping j...@keeping.me.uk writes: On Thu, Apr 18, 2013 at 03:40:41PM -0700, Junio C Hamano wrote: John Keeping j...@keeping.me.uk writes: +relative_path () +{ + local target curdir result + target=$1 + curdir=${2-$wt_prefix} +

Re: [PATCH v2 1/8] Add new git-cc-cmd helper to contrib

2013-04-19 Thread Junio C Hamano
Felipe Contreras felipe.contre...@gmail.com writes: On Fri, Apr 19, 2013 at 12:08 PM, Junio C Hamano gits...@pobox.com wrote: Felipe Contreras felipe.contre...@gmail.com writes: The code finds the changes of a commit, runs 'git blame' for each chunk to see which other commits are relevant,

[PATCH v3 00/11] Add new git-cc-cmd helper to contrib

2013-04-19 Thread Felipe Contreras
Hi, Here goes another try, I made some changes based on comments, and should be simpler now. Plus a few improvements. The interdiff will be on the patches, but only the first one changed substantially. This script allows you to get a list of relevant persons to Cc when sending a patch series.

[PATCH v3 02/11] contrib: cc-cmd: add option parsing

2013-04-19 Thread Felipe Contreras
Signed-off-by: Felipe Contreras felipe.contre...@gmail.com --- contrib/cc-cmd/git-cc-cmd | 17 + 1 file changed, 17 insertions(+) diff --git a/contrib/cc-cmd/git-cc-cmd b/contrib/cc-cmd/git-cc-cmd index aa83a1a..d78759d 100755 --- a/contrib/cc-cmd/git-cc-cmd +++

[PATCH v3 03/11] contrib: cc-cmd: add support for multiple patches

2013-04-19 Thread Felipe Contreras
Signed-off-by: Felipe Contreras felipe.contre...@gmail.com --- contrib/cc-cmd/git-cc-cmd | 34 ++ 1 file changed, 18 insertions(+), 16 deletions(-) diff --git a/contrib/cc-cmd/git-cc-cmd b/contrib/cc-cmd/git-cc-cmd index d78759d..3a9c70e 100755 ---

[PATCH v3 05/11] contrib: cc-cmd: add option to parse from committish

2013-04-19 Thread Felipe Contreras
For example master..feature-a. Signed-off-by: Felipe Contreras felipe.contre...@gmail.com --- contrib/cc-cmd/git-cc-cmd | 36 ++-- 1 file changed, 34 insertions(+), 2 deletions(-) diff --git a/contrib/cc-cmd/git-cc-cmd b/contrib/cc-cmd/git-cc-cmd index

[PATCH v3 08/11] contrib: cc-cmd: add option to fetch aliases

2013-04-19 Thread Felipe Contreras
Only the mutt format is supported for now. Signed-off-by: Felipe Contreras felipe.contre...@gmail.com --- contrib/cc-cmd/git-cc-cmd | 25 + 1 file changed, 25 insertions(+) diff --git a/contrib/cc-cmd/git-cc-cmd b/contrib/cc-cmd/git-cc-cmd index 4fad030..200da0d 100755

[PATCH v3 06/11] contrib: cc-cmd: parse committish like format-patch

2013-04-19 Thread Felipe Contreras
Signed-off-by: Felipe Contreras felipe.contre...@gmail.com --- contrib/cc-cmd/git-cc-cmd | 14 ++ 1 file changed, 14 insertions(+) diff --git a/contrib/cc-cmd/git-cc-cmd b/contrib/cc-cmd/git-cc-cmd index aecfa45..4bca7f1 100755 --- a/contrib/cc-cmd/git-cc-cmd +++

[PATCH v3 07/11] contrib: cc-cmd: fix parsing of rev-list args

2013-04-19 Thread Felipe Contreras
For example '-1'. Signed-off-by: Felipe Contreras felipe.contre...@gmail.com --- contrib/cc-cmd/git-cc-cmd | 9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/contrib/cc-cmd/git-cc-cmd b/contrib/cc-cmd/git-cc-cmd index 4bca7f1..4fad030 100755 ---

[PATCH v3 10/11] contrib: cc-cmd: sort by participation

2013-04-19 Thread Felipe Contreras
Signed-off-by: Felipe Contreras felipe.contre...@gmail.com --- contrib/cc-cmd/git-cc-cmd | 7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/contrib/cc-cmd/git-cc-cmd b/contrib/cc-cmd/git-cc-cmd index 67a276d..9a2d2fd 100755 --- a/contrib/cc-cmd/git-cc-cmd +++

[PATCH v3 11/11] contrib: cc-cmd: ignore chunks with no original lines

2013-04-19 Thread Felipe Contreras
Suggested-by: Junio C Hamano gits...@pobox.com Signed-off-by: Felipe Contreras felipe.contre...@gmail.com --- contrib/cc-cmd/git-cc-cmd | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/contrib/cc-cmd/git-cc-cmd b/contrib/cc-cmd/git-cc-cmd index 9a2d2fd..02841c4 100755 ---

[PATCH v3 09/11] contrib: cc-cmd: support multiple roles

2013-04-19 Thread Felipe Contreras
Currently only the roles of 'author' and 'signer' and handler, but now there's also 'reviewer' and 'acker'. Signed-off-by: Felipe Contreras felipe.contre...@gmail.com --- contrib/cc-cmd/git-cc-cmd | 29 - 1 file changed, 20 insertions(+), 9 deletions(-) diff --git

Re: [PATCH v2 1/8] Add new git-cc-cmd helper to contrib

2013-04-19 Thread Felipe Contreras
On Fri, Apr 19, 2013 at 2:24 PM, Junio C Hamano gits...@pobox.com wrote: Felipe Contreras felipe.contre...@gmail.com writes: On Fri, Apr 19, 2013 at 12:08 PM, Junio C Hamano gits...@pobox.com wrote: Felipe Contreras felipe.contre...@gmail.com writes: The code finds the changes of a commit,

Re: [PATCH v3 01/11] Add new git-cc-cmd helper to contrib

2013-04-19 Thread Felipe Contreras
On Fri, Apr 19, 2013 at 2:30 PM, Felipe Contreras felipe.contre...@gmail.com wrote: This script find people that might be interesting in a patch, by going back through the history for each single hunk modified, and finding people that reviewed, acknowledge, signed, or authored the code the

[PATCH v3 01/11] Add new git-cc-cmd helper to contrib

2013-04-19 Thread Felipe Contreras
This script find people that might be interesting in a patch, by going back through the history for each single hunk modified, and finding people that reviewed, acknowledge, signed, or authored the code the patch is modifying. It does this by running 'git blame' incrementally on each hunk, and

Re: [RFD/PATCH 3/5] checkout: Use remote refspecs when DWIMming tracking branches

2013-04-19 Thread Junio C Hamano
Johan Herland jo...@herland.net writes: The DWIM mode of checkout allows you to run git checkout foo when there is no existing local ref or path called foo, and there is exactly one remote with a remote-tracking branch called foo. Git will then automatically create a new local branch called

Re: t6200: avoid path mangling issue on Windows

2013-04-19 Thread Johannes Sixt
Am 19.04.2013 18:33, schrieb Junio C Hamano: Johannes Sixt j.s...@viscovery.net writes: Am 4/18/2013 19:05, schrieb Junio C Hamano: Johannes Sixt j.s...@viscovery.net writes: From: Johannes Sixt j...@kdbg.org MSYS bash interprets the slash in the argument core.commentchar=/ as root

Re: [RFD/PATCH 5/5] RFD: Disallow out-of-refspec refs within refs/remotes/* to be used as upstream

2013-04-19 Thread Junio C Hamano
Johan Herland jo...@herland.net writes: The previous patch adds validation of upstream remote-tracking branches by parsing the configured refspecs, and making sure that the candidate upstream (if not already matching refs/heads/* or refs/remotes/*) is indeed a remote-tracking branch according

Re: [PATCH v2 1/8] Add new git-cc-cmd helper to contrib

2013-04-19 Thread Junio C Hamano
Felipe Contreras felipe.contre...@gmail.com writes: If this were _only_ to be used within send-email (i.e. replacing the then send it out above with then use send-email to limit the usecase), git cc-cmd would be a reasonable name. But if that is the intended use case, it would even be more

Re: [PATCH v3 11/11] contrib: cc-cmd: ignore chunks with no original lines

2013-04-19 Thread Junio C Hamano
Felipe Contreras felipe.contre...@gmail.com writes: Suggested-by: Junio C Hamano gits...@pobox.com Signed-off-by: Felipe Contreras felipe.contre...@gmail.com --- contrib/cc-cmd/git-cc-cmd | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/contrib/cc-cmd/git-cc-cmd

Re: [PATCH v2 1/8] Add new git-cc-cmd helper to contrib

2013-04-19 Thread Johannes Sixt
Am 19.04.2013 21:24, schrieb Junio C Hamano: Felipe Contreras felipe.contre...@gmail.com writes: On Fri, Apr 19, 2013 at 12:08 PM, Junio C Hamano gits...@pobox.com wrote: Felipe Contreras felipe.contre...@gmail.com writes: The code finds the changes of a commit, runs 'git blame' for each

Re: [PATCH v2 1/8] Add new git-cc-cmd helper to contrib

2013-04-19 Thread Junio C Hamano
Felipe Contreras felipe.contre...@gmail.com writes: +File.open(file) do |f| + f.each do |line| +case line +when /^From (\h+) (.+)$/ + from = $1 +when /^---\s+(\S+)/ + source = $1 != '/dev/null' ? $1[2..-1] : nil This may need to be

Re: [PATCH v3 2/2] submodule: drop the top-level requirement

2013-04-19 Thread Junio C Hamano
Johannes Sixt j...@kdbg.org writes: Why not just replace the six-liner by this one-liner: target=${target#$curdir/} Simple enough ;-) -- 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

Re: What's cooking in git.git (Apr 2013, #05; Mon, 15)

2013-04-19 Thread Phil Hord
On Thu, Apr 18, 2013 at 7:48 PM, Felipe Contreras felipe.contre...@gmail.com wrote: On Thu, Apr 18, 2013 at 3:06 PM, Phil Hord phil.h...@gmail.com wrote: On Wed, Apr 17, 2013 at 2:50 PM, Felipe Contreras Yes please. Show me one of the instances where you hit a bisect with any of the remote-hg

Re: t6200: avoid path mangling issue on Windows

2013-04-19 Thread Junio C Hamano
Johannes Sixt j...@kdbg.org writes: A patch auther whose first instinct is to write 'foo=/' will never write 'foo=x', let alone 'foo=/'. Someone will have to discover the issue eventually and write a patch to fix it, and someone will have to apply it. That is a separate issue. Didn't I say

[PATCH] receive-pack: close sideband fd on early pack errors

2013-04-19 Thread Jeff King
Since commit a22e6f8 (receive-pack: send pack-processing stderr over sideband, 2012-09-21), receive-pack will start an async sideband thread to copy the stderr from our index-pack or unpack-objects child to the client. We hand the thread's input descriptor to unpack(), which puts it in the err

Re: What's cooking in git.git (Apr 2013, #05; Mon, 15)

2013-04-19 Thread Jeff King
On Fri, Apr 19, 2013 at 10:25:46AM -0700, Junio C Hamano wrote: Jonathan Nieder jrnie...@gmail.com writes: Junio C Hamano wrote: You ran 'git add' with neither '-A (--all)' or '--no-all', whose behaviour will change in Git 2.0 with respect to paths you removed from your

Re: What's cooking in git.git (Apr 2013, #05; Mon, 15)

2013-04-19 Thread Junio C Hamano
Jeff King p...@peff.net writes: On Fri, Apr 19, 2013 at 10:25:46AM -0700, Junio C Hamano wrote: Jonathan Nieder jrnie...@gmail.com writes: Junio C Hamano wrote: You ran 'git add' with neither '-A (--all)' or '--no-all', whose behaviour will change in Git 2.0 with respect to

Re: [PATCH v3 01/11] Add new git-cc-cmd helper to contrib

2013-04-19 Thread Junio C Hamano
Felipe Contreras felipe.contre...@gmail.com writes: This script find people that might be interesting in a patch, by going back through the history for each single hunk modified, and finding people that reviewed, acknowledge, signed, or authored the code the patch is modifying. It does this

Re: [PATCH] receive-pack: close sideband fd on early pack errors

2013-04-19 Thread Junio C Hamano
Jeff King p...@peff.net writes: This was triggered in the real world by attempting to push a ref from a corrupted repository. pack-objects dies on the local end, we get an eof on the receive-pack end without any data, notice that it's a bogus packfile, and hit the deadlock. The bug was

Re: [PATCH v3 00/11] Add new git-cc-cmd helper to contrib

2013-04-19 Thread Duy Nguyen
On Sat, Apr 20, 2013 at 5:30 AM, Felipe Contreras felipe.contre...@gmail.com wrote: This script allows you to get a list of relevant persons to Cc when sending a patch series. % git cc-cmd v1.8.1.6^^1..v1.8.1.6^^2 Nguyễn Thái Ngọc Duy pclo...@gmail.com (author: 30%, signer: 7%) Duy

Re: What's cooking in git.git (Apr 2013, #05; Mon, 15)

2013-04-19 Thread Felipe Contreras
On Fri, Apr 19, 2013 at 4:07 PM, Phil Hord phil.h...@gmail.com wrote: On Thu, Apr 18, 2013 at 7:48 PM, Felipe Contreras felipe.contre...@gmail.com wrote: If something is not hypothetical, it's real, which means it actually happened, but then you said you never made the claim that it did. So

Re: [PATCH v3 00/11] Add new git-cc-cmd helper to contrib

2013-04-19 Thread Felipe Contreras
On Fri, Apr 19, 2013 at 8:16 PM, Duy Nguyen pclo...@gmail.com wrote: On Sat, Apr 20, 2013 at 5:30 AM, Felipe Contreras felipe.contre...@gmail.com wrote: This script allows you to get a list of relevant persons to Cc when sending a patch series. % git cc-cmd v1.8.1.6^^1..v1.8.1.6^^2

Re: [PATCH] gitk: workaround Tcl/Tk Cmd-TAB behavior on OSX

2013-04-19 Thread Paul Mackerras
On Thu, Apr 11, 2013 at 01:02:48AM +0600, Tair Sabirgaliev wrote: On OSX Tcl/Tk application windows are created behind all the applications down the stack of windows. This is very annoying, because once a gitk window appears, it's the downmost window and switching to it is pain. The patch

Re: [PATCH 1/6] t4030: demonstrate behavior of show with textconv

2013-04-19 Thread Jeff King
On Fri, Apr 19, 2013 at 06:44:44PM +0200, Michael J Gruber wrote: git show commit obeys the textconc setting while git show blob does not. Demonstrate this in the test. s/textconc/textconv diff --git a/t/t4030-diff-textconv.sh b/t/t4030-diff-textconv.sh index 53ec330..f314ced 100755 ---

Re: [PATCH 3/6] cat-file: do not die on --textconv without textconv filters

2013-04-19 Thread Jeff King
On Fri, Apr 19, 2013 at 06:44:46PM +0200, Michael J Gruber wrote: - die(git cat-file --textconv: unable to run textconv on %s, - obj_name); - break; + if (textconv_object(obj_context.path, obj_context.mode, sha1, 1, buf,

Re: [PATCH 6/6] grep: obey --textconv for the case rev:path

2013-04-19 Thread Jeff King
On Fri, Apr 19, 2013 at 06:44:49PM +0200, Michael J Gruber wrote: @@ -820,12 +820,13 @@ int cmd_grep(int argc, const char **argv, const char *prefix) for (i = 0; i argc; i++) { const char *arg = argv[i]; unsigned char sha1[20]; + struct

Re: [PATCH 0/6] grep with textconv

2013-04-19 Thread Jeff King
On Fri, Apr 19, 2013 at 11:24:16AM -0700, Junio C Hamano wrote: grep: allow to use textconv filters This looked mostly sensible except for one minor eh, do we really need to assume textconv output is text, or wouldn't using the same codepath for raw blob and textconv result to make them

Re: [PATCH 5/6] grep: allow to use textconv filters

2013-04-19 Thread Jeff King
On Fri, Apr 19, 2013 at 06:44:48PM +0200, Michael J Gruber wrote: From: Jeff King p...@peff.net Recently and not so recently, we made sure that log/grep type operations use textconv filters when a userfacing diff would do the same: ef90ab6 (pickaxe: use textconv for -S counting,