[PATCH] diff: Fixes shortstat number of files

2012-11-22 Thread Antoine Pelisse
There is a discrepancy between the last line of `git diff --stat` and `git diff --shortstat` in case of a merge. The unmerged files are actually counted twice, thus doubling the value of "file changed". In fact, while stat decrements number of files when seeing an unmerged file, shortstat doesn't.

New git.pot is generated for the start of git 1.8.1 l10n

2012-11-22 Thread Jiang Xin
Dear l10n team members, New "git.pot" is generated from v1.8.0-273-g2d242 in the master branch. l10n: Update git.pot (14 new, 3 removed messages) Generate po/git.pot from v1.8.0-273-g2d242, and there are 14 new and 3 removed messages. Signed-off-by: Jiang Xin This update is fo

[PATCH] gitk tag delete/rename support

2012-11-22 Thread Leon KUKOVEC
--- gitk-git/gitk | 154 ++ 1 file changed, 154 insertions(+) diff --git a/gitk-git/gitk b/gitk-git/gitk index 17ba10a..12a7139 100755 --- a/gitk-git/gitk +++ b/gitk-git/gitk @@ -1981,6 +1981,7 @@ proc makewindow {} { global h

Do you know this?

2012-11-22 Thread saritee11
Links of London in July 2006 was famous jewellery, watch and deserve to act the role of brand Folli Follie group acquisition. In August 2007, Andrew Marshall appointed executive director, Links of London. A parent company Folli Follie group support as a solid foundation,[url=http://www.linksoflondo

[PATCH 7/7] push: clarify rejection of update to non-commit-ish

2012-11-22 Thread Chris Rorvick
Pushes must already (by default) update to a commit-ish due the fast- forward check in set_ref_status_for_push(). But rejecting for not being a fast-forward suggests the situation can be resolved with a merge. Flag these updates (i.e., to a blob or a tree) as not forwardable so the user is present

[PATCH 6/7] push: require force for annotated tags

2012-11-22 Thread Chris Rorvick
Do not allow fast-forwarding of references that point to a tag object. This keeps the behavior consistent with lightweight tags. Additionally, allowing the reference to update could leave the old object dangling. Signed-off-by: Chris Rorvick --- Documentation/git-push.txt | 10 +- remot

[PATCH 5/7] push: require force for refs under refs/tags/

2012-11-22 Thread Chris Rorvick
References are allowed to update from one commit-ish to another if the former is an ancestor of the latter. This behavior is oriented to branches which are expected to move with commits. Tag references are expected to be static in a repository, though, thus an update to something under refs/tags/

[PATCH 4/7] push: flag updates that require force

2012-11-22 Thread Chris Rorvick
Add a flag for indicating an update to a reference requires force. Currently the nonfastforward flag of a ref is used for this when generating status the status message. A separate flag insulates the status logic from the details of set_ref_status_for_push(). Signed-off-by: Chris Rorvick --- ca

[PATCH 2/7] push: add advice for rejected tag reference

2012-11-22 Thread Chris Rorvick
Advising the user to fetch and merge only makes sense if the rejected reference is a branch. If none of the rejections are for branches, just tell the user the reference already exists. Signed-off-by: Chris Rorvick --- builtin/push.c | 11 +++ cache.h| 1 + remote.c | 10

[PATCH 3/7] push: flag updates

2012-11-22 Thread Chris Rorvick
If the reference exists on the remote and the update is not a delete, then mark as an update. This is in preparation for handling tags and branches differently when pushing. Signed-off-by: Chris Rorvick --- cache.h | 1 + remote.c | 18 +++--- 2 files changed, 12 insertions(+), 7

[PATCH 1/7] push: return reject reasons via a mask

2012-11-22 Thread Chris Rorvick
Pass all rejection reasons back from transport_push(). The logic is simpler and more flexible with regard to providing useful feedback. Signed-off-by: Chris Rorvick --- builtin/push.c | 13 - builtin/send-pack.c | 4 ++-- transport.c | 17 - transport.h

[PATCH v5 0/7] push: update remote tags only with force

2012-11-22 Thread Chris Rorvick
Incorporated Junio's feedback. Also, I broke the last patch of the previous series out into three to make the changes more clear. This patch set can be divided into two sets: 1. Provide useful advice for rejected tag references. push: return reject reasons via a mask push: add advic

Re: Remote hung up during `git fetch`

2012-11-22 Thread Andrew Ardill
On 23 November 2012 05:39, Yichao Yu wrote: > Hi everyone, > > I sent this email yesterday to the git mailing list but I cannot find > it in any archive so I decide to send it again. > Does anyone know what has happened to the mailing list? I haven't > receive any email from several kernel related

Re: Auto-repo-repair

2012-11-22 Thread Enrico Weigelt
Hi, > I still think that it would make the most sense to do the following > (if you insist on some sort of automated repair): > (1) Fetch a "good" clone (or clones) into a temporary directory; > (2) Cannibalize the objects from it (them); > (3) Re-run git fsck and check for still-missing / unreach

[PATCH 2/5] git-send-email: fix fallback code in extract_valid_address()

2012-11-22 Thread Krzysztof Mazur
In the fallback check, used when Email::Valid is not available, the extract_valid_address() uses $1 without checking for success of matching regex. The $1 variable may still hold the result of previous match, which is the address when email address was in '<>' or be undefined otherwise. Now if mat

Re: [RFC] Add a new email notification script to "contrib"

2012-11-22 Thread Michael Haggerty
On 11/08/2012 04:38 PM, Ævar Arnfjörð Bjarmason wrote: > On Thu, Nov 8, 2012 at 1:17 PM, Michael Haggerty wrote: >> On 11/08/2012 12:39 PM, Ævar Arnfjörð Bjarmason wrote: >>> But in addition to that we have our own custom E-Mail notification >>> scripts for: >>> >>> * People can subscribe to chan

Remote hung up during `git fetch`

2012-11-22 Thread Yichao Yu
Hi everyone, I want to build packages for snap shoot of different branches from different remote git repositories in the same local directory (so that I don't need to recompile everything everytime.) and I am using a combination of `git clone/checkout/reset/fetch` to do that. However, during git-f

Re: [RFC] Add a new email notification script to "contrib"

2012-11-22 Thread Michael Haggerty
On 11/08/2012 05:03 PM, Marc Branchaud wrote: > I look forward to trying out your updates. One thing I had to do to the > original script was override get_envelopesender() in GenericEnvironment to > use $USER if hooks.envelopesender is not set. (This is what the old > post-receive-email script do

[PATCH 3/5] git-send-email: remove invalid addresses earlier

2012-11-22 Thread Krzysztof Mazur
Some addresses are passed twice to unique_email_list() and invalid addresses may be reported twice per send_message. Now we warn about them earlier and we also remove invalid addresses. This also removes using of undefined values for string comparison for invalid addresses in cc list processing.

[PATCH 4/5] git-send-email: ask what to do with an invalid email address

2012-11-22 Thread Krzysztof Mazur
We used to warn about invalid emails and just drop them. Such warnings can be unnoticed by user or noticed after sending email when we are not giving the "final sanity check [Y/n]?" Now we quit by default. Signed-off-by: Krzysztof Mazur Suggested-by: Junio C Hamano --- git-send-email.perl | 12

[PATCH v2] Teach rm to remove submodules when given with a trailing '/'

2012-11-22 Thread Jens Lehmann
Doing a "git rm submod/" on a submodule results in an error: fatal: pathspec 'submod/' did not match any files This is really inconvenient as e.g. using TAB completion in a shell on a submodule automatically adds the trailing '/' when it completes the path of the submodule directory. The us

Re: Requirements for integrating a new git subcommand

2012-11-22 Thread Eric S. Raymond
Shawn Pearce : > [Lots of helpful stuff ended by] > > 4. How does "git help" work? That is, how is a subcommand expected > > to know when it is being called to export its help text? > > IIRC "git help foo" runs "man git-foo". OK, that makes sense. > > 5. I don't see any extensions written in Py

Re: [PATCH v5 05/15] Add new simplified git-remote-testgit

2012-11-22 Thread Felipe Contreras
On Wed, Nov 21, 2012 at 7:26 PM, Junio C Hamano wrote: > Felipe Contreras writes: > >> It's way simpler. It exerceises the same features of remote helpers. >> It's easy to read and understand. It doesn't depend on python. >> >> It does _not_ exercise the python remote helper framework; there's >>

Re: [PATCH v5 15/15] fast-export: don't handle uninteresting refs

2012-11-22 Thread Max Horn
On 21.11.2012, at 06:08, Junio C Hamano wrote: > Jonathan Nieder writes: > >> Never mind that others have said that that's not the current interface >> (I don't yet see why it would be a good interface after a transition, >> but maybe it would be). Still, hopefully that clarifies the intended >

[PATCH] makefile: hide stderr of curl-config test

2012-11-22 Thread Paul Gortmaker
Currently, if you don't have curl installed, you will get $ make distclean 2>&1 | grep curl /bin/sh: curl-config: not found /bin/sh: curl-config: not found /bin/sh: curl-config: not found /bin/sh: curl-config: not found /bin/sh: curl-config: not found $ The intent is n

Re: [PATCH v5 09/15] remote-testgit: exercise more features

2012-11-22 Thread Felipe Contreras
On Wed, Nov 21, 2012 at 7:26 PM, Junio C Hamano wrote: > Felipe Contreras writes: > >> Unfortunately they do not work. > > As far as I can tell, "more features" simply mean one, no? Perhaps > > remote-testgit: exercise non-default refspec feature It's the other way around, a good refspec wo

Re: gitk: Portuguese "Ignore space change" translation

2012-11-22 Thread Thiago Farina
Hi Joao, On Tue, Nov 20, 2012 at 8:03 AM, Joao Vitor P. Moraes wrote: > Inside gitk there's a checkbox which says: > Ignore space change > > It was translated to portuguese (pt-br) as: > Ignorar mudanças de caixa > > But that message in portuguese means: > Ignore case changes > > that checkbox do

Re: Remote hung up during `git fetch`

2012-11-22 Thread Yichao Yu
On Thu, Nov 22, 2012 at 2:52 PM, Shawn Pearce wrote: > On Thu, Nov 22, 2012 at 11:44 AM, Yichao Yu wrote: >>> US holiday today? The list traffic tends to be down during holidays. >> This silent? 0 email from the kernel mailing list for 10+ hours?.. >> anyway nvm... > > Check your spam fil

[PATCH] Completion script must sort before using uniq

2012-11-22 Thread Marc Khouzam
The uniq program only works with sorted input. The man page states "uniq prints the unique lines in a sorted file". When __git_refs use the guess heuristic employed by checkout for tracking branches it wants to consider remote branches but only if the branch name is unique. To do that, it calls

Re: [PATCH] Fix bash completion when `egrep` is aliased to `egrep --color=always`

2012-11-22 Thread Marc Khouzam
On Thu, Nov 22, 2012 at 10:41 AM, Adam Tkac wrote: > Hello all, > > attached patch fixes bash completion when `egrep` is aliased to `egrep > --color=always`. To avoid any aliases, it may be better to use \egrep This could be worthwhile for all utilities used by the script. Just a thought. Mar

Re: Remote hung up during `git fetch`

2012-11-22 Thread Shawn Pearce
On Thu, Nov 22, 2012 at 11:44 AM, Yichao Yu wrote: >> US holiday today? The list traffic tends to be down during holidays. > This silent? 0 email from the kernel mailing list for 10+ hours?.. > anyway nvm... Check your spam filters? I am having no trouble getting email for the Git list. T

Re: [PATCH v5 15/15] fast-export: don't handle uninteresting refs

2012-11-22 Thread Felipe Contreras
On Wed, Nov 21, 2012 at 8:48 PM, Jeff King wrote: > On Tue, Nov 20, 2012 at 09:08:36PM -0800, Junio C Hamano wrote: > >> With such a one-sided discussion, I've been having a hard time >> convincing myself if Felipe's effort is making the interface better, >> or just breaking it even more for exist

[PATCH] Completion must sort before using uniq

2012-11-22 Thread Marc Khouzam
The uniq program only works with sorted input. The man page states "uniq prints the unique lines in a sorted file". When __git_refs use the guess heuristic employed by checkout for tracking branches it wants to consider remote branches but only if the branch name is unique. To do that, it calls

Re: Remote hung up during `git fetch`

2012-11-22 Thread Yichao Yu
On Thu, Nov 22, 2012 at 2:01 PM, Shawn Pearce wrote: > On Thu, Nov 22, 2012 at 10:39 AM, Yichao Yu wrote: >> I sent this email yesterday to the git mailing list but I cannot find >> it in any archive so I decide to send it again. > > If it was HTML formatted it would have been silently dropped by

Re: git-fetch does not work from .git subdirectory

2012-11-22 Thread Patrik Gornicz
Just a hunch but your remote's location uses a relative path '../linux-2.6.git', perhaps git is messing up what the path is relative to. I assume this repo is beside the linux.cq-test repo? ie. perhaps they're located at /home/user/git/linux.cq-test and /home/user/git/linux-2.6.git. Then, f

Re: [PATCH v5 15/15] fast-export: don't handle uninteresting refs

2012-11-22 Thread Felipe Contreras
On Wed, Nov 21, 2012 at 11:30 PM, Max Horn wrote: > 2) Some are interfaces to foreign systems (bzr, hg, mediawiki, ...). They > cannot use sha1s and must use marks (at least that is how I understand > felipe's explanation). These tools use import combined with either export, or > push. Example

[PATCH v2 2/4] format-patch: stricter S-o-b detection

2012-11-22 Thread Nguyễn Thái Ngọc Duy
S-o-b in the middle of a sentence, at the beginning of the sentence but it is just because of text wrapping, or a full paragraph of valid S-o-b in the middle of the message. All these are not S-o-b, but detected as is. Fix them. Signed-off-by: Nguyễn Thái Ngọc Duy --- log-tree.c | 3

Re: Duplicate test numbers in pu.

2012-11-22 Thread Adam Spiers
On Wed, Nov 21, 2012 at 7:14 PM, Ramsay Jones wrote: > I noticed that the pu branch has two tests with number t0007, viz: > > $ cd t > $ make test-lint-duplicates > duplicate test numbers: t0007 > make: *** [test-lint-duplicates] Error 1 > $ > > In particular, t/t0007-git-var.s

[PATCH v2 0/4] nd/unify-appending-of-s-o-b

2012-11-22 Thread Nguyễn Thái Ngọc Duy
This round cherry-pick and commit only skip S-o-b if the last one is the same while format-patch skips it if the same one appears anywhere in S-o-b list. I don't have strong opinion about which way is more correct. Though if we decide to change format-patch behavior, the series may become a bit dif

[PATCH v2 1/4] t4014: more tests about appending s-o-b lines

2012-11-22 Thread Nguyễn Thái Ngọc Duy
Signed-off-by: Nguyễn Thái Ngọc Duy --- t/t4014-format-patch.sh | 145 1 file changed, 145 insertions(+) diff --git a/t/t4014-format-patch.sh b/t/t4014-format-patch.sh index 959aa26..c8d5d29 100755 --- a/t/t4014-format-patch.sh +++ b/t/t4014-form

Re: Requirements for integrating a new git subcommand

2012-11-22 Thread Shawn Pearce
On Wed, Nov 21, 2012 at 9:30 PM, Eric S. Raymond wrote: > I have completed work on git-weave (the tool I had called 'gitpacker' in some > previous postings). I want to submit a patch that integrates it into git; > in hopes of smoothing the process I have some technical and procedural > questions.

[PATCH v2 4/4] Unify appending signoff in format-patch, commit and sequencer

2012-11-22 Thread Nguyễn Thái Ngọc Duy
There are two implementations of append_signoff in log-tree.c and sequencer.c, which do more or less the same thing. This patch removes the sequencer.c's in favor of the format-patch's. Signed-off-by: Nguyễn Thái Ngọc Duy --- builtin/commit.c | 10 - log-tree.c| 14

[PATCH] emacs: make 'git-status' work with separate git dirs

2012-11-22 Thread Enrico Scholz
when trying 'M-x git-status' in a submodule created with recent (1.7.5+) git, the command fails with | ... is not a git working tree This is caused by creating submodules with '--separate-git-dir' but still checking for a working tree by testing for a '.git' directory. The patch fixes this by re

Re: [PATCH v4 2/4] diff: introduce diff.submodule configuration variable

2012-11-22 Thread Ramkumar Ramachandra
Ramkumar Ramachandra wrote: > Junio C Hamano wrote: >> Ramkumar Ramachandra writes: >>> +test_expect_success 'added submodule, set diff.submodule' " >> >> s/added/add/; > > I see that the topic is already in `next`. Do you want to fix it up there? I was simply following the example set by the pr

Requirements for integrating a new git subcommand

2012-11-22 Thread Eric S. Raymond
I have completed work on git-weave (the tool I had called 'gitpacker' in some previous postings). I want to submit a patch that integrates it into git; in hopes of smoothing the process I have some technical and procedural questions. First, however, let me present the git-weave documentation f

Re: [PATCH v5 08/15] remote-testgit: cleanup tests

2012-11-22 Thread Felipe Contreras
On Wed, Nov 21, 2012 at 7:28 PM, Junio C Hamano wrote: > Felipe Contreras writes: > >> We don't need a bare 'server' and an intermediary 'public'. The repos >> can talk to each other directly; that's what we want to exercise. > > The previous patch to remove the test (the one that covered a case

[PATCH] diff: Fixes shortstat number of files

2012-11-22 Thread Antoine Pelisse
There is a discrepancy between the last line of `git diff --stat` and `git diff --shortstat` in case of a merge. The unmerged files are actually counted twice, thus doubling the value of "file changed". In fact, while stat decrements number of files when seeing an unmerged file, shortstat doesn't.

Re: Remote hung up during `git fetch`

2012-11-22 Thread Shawn Pearce
On Thu, Nov 22, 2012 at 10:39 AM, Yichao Yu wrote: > I sent this email yesterday to the git mailing list but I cannot find > it in any archive so I decide to send it again. If it was HTML formatted it would have been silently dropped by the list. > Does anyone know what has happened to the maili

Re: Topics currently in the Stalled category

2012-11-22 Thread Nguyen Thai Ngoc Duy
On Wed, Nov 21, 2012 at 7:05 AM, Junio C Hamano wrote: > * nd/unify-appending-of-s-o-b (2012-11-15) 1 commit > - Unify appending signoff in format-patch, commit and sequencer > > I am not sure if the logic to refrain from adding a sign-off based > on the existing run of sign-offs is done correc

Fwd: Remote hung up during `git fetch`

2012-11-22 Thread Yichao Yu
Hi everyone, I sent this email yesterday to the git mailing list but I cannot find it in any archive so I decide to send it again. Does anyone know what has happened to the mailing list? I haven't receive any email from several kernel related busy mailing lists for several hours Yichao Yu --

[PATCH] Fix bash completion when `egrep` is aliased to `egrep --color=always`

2012-11-22 Thread Adam Tkac
Hello all, attached patch fixes bash completion when `egrep` is aliased to `egrep --color=always`. Comments are welcomed. Regards, Adam -- Adam Tkac, Red Hat, Inc. >From 2b62bd71af1158129492f74f0b77c9840a49507f Mon Sep 17 00:00:00 2001 From: Adam Tkac Date: Thu, 22 Nov 2012 16:34:58 +0100 Su

[PATCH 5/5] git-send-email: allow edit invalid email address

2012-11-22 Thread Krzysztof Mazur
In some cases the user may want to send email with "Cc:" line with email address we cannot extract. Now we allow user to extract such email address for us. Signed-off-by: Krzysztof Mazur --- git-send-email.perl | 9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/git-send

What's cooking in git.git (Nov 2012, #07; Wed, 21)

2012-11-22 Thread Junio C Hamano
What's cooking in git.git (Nov 2012, #07; Wed, 21) -- Here are the topics that have been cooking. Commits prefixed with '-' are only in 'pu' (proposed updates) while commits prefixed with '+' are in 'next'. Many topics have been merged to 'maint' i

[PATCH 1/5] git-send-email: remove garbage after email address

2012-11-22 Thread Krzysztof Mazur
In some cases it's very useful to add some additional information after email in Cc-list, for instance: "Cc: Stable kernel #v3.4 v3.5 v3.6" Currently the git refuses to add such invalid email to Cc-list, when the Email::Valid perl module is available or just uses whole line as the email address.

Re: [PATCH v4 2/4] diff: introduce diff.submodule configuration variable

2012-11-22 Thread Ramkumar Ramachandra
Junio C Hamano wrote: > Ramkumar Ramachandra writes: > >> diff --git a/t/t4041-diff-submodule-option.sh >> b/t/t4041-diff-submodule-option.sh >> index 6c01d0c..e401814 100755 >> --- a/t/t4041-diff-submodule-option.sh >> +++ b/t/t4041-diff-submodule-option.sh >> @@ -33,6 +33,7 @@ test_create_repo

[PATCH v2 3/4] format-patch: update append_signoff prototype

2012-11-22 Thread Nguyễn Thái Ngọc Duy
This is a preparation step for merging with append_signoff from sequencer.c Signed-off-by: Nguyễn Thái Ngọc Duy --- builtin/log.c | 13 + log-tree.c| 21 + revision.h| 2 +- 3 files changed, 15 insertions(+), 21 deletions(-) diff --git a/builtin/log.c b/