Re: Getting started contributing.

2013-02-03 Thread Duy Nguyen
On Sun, Feb 3, 2013 at 2:49 PM, Junio C Hamano gits...@pobox.com wrote: On the other hand, there probably still are many loose ends. A few other things - Mark more strings for translation (not as easy as it sounds, some strings can't be translated) - Color more in the output where it makes

Re: What's cooking in git.git (Feb 2013, #01; Fri, 1)

2013-02-03 Thread David Aguilar
On Sat, Feb 2, 2013 at 9:44 PM, Junio C Hamano gits...@pobox.com wrote: Junio C Hamano gits...@pobox.com writes: Regarding these two topics * da/mergetool-docs (2013-01-30) 7 commits - doc: generate a list of valid merge tools - mergetool--lib: list user configured tools in

Re: What's cooking in git.git (Feb 2013, #01; Fri, 1)

2013-02-03 Thread John Keeping
On Sun, Feb 03, 2013 at 04:13:22AM -0800, David Aguilar wrote: On Sat, Feb 2, 2013 at 9:44 PM, Junio C Hamano gits...@pobox.com wrote: Junio C Hamano gits...@pobox.com writes: Regarding these two topics * da/mergetool-docs (2013-01-30) 7 commits - doc: generate a list of valid

Re: Getting started contributing.

2013-02-03 Thread Philip Oakley
From: Junio C Hamano gits...@pobox.com Sent: Sunday, February 03, 2013 7:49 AM adamfraser adamfras...@gmail.com writes: I've done a little searching and haven't been able to find an official bug tracker for git is there somewhere I can find some bugs to help fix? You came to the right

Feature request: Allow extracting revisions into directories

2013-02-03 Thread Robert Clausecker
Hello! git currently has the archive command that allows to save an arbitrary revision into a tar or zip file. Sometimes it is useful to not save this revision into an archive but to directly put all files into an arbitrary directory. Currently this seems to be not possible to archive directly;

[PATCH 0/3] Make Git compile warning-free with Clang

2013-02-03 Thread John Keeping
The first two patches here were sent to the list before but seem to have got lost in the noise [1][2]. The final one is new but was prompted by discussion in the same thread. After applying all of these patches, I don't see any warnings compiling Git with Clang 3.2. [1]

[PATCH 1/3] fix clang -Wtautological-compare with unsigned enum

2013-02-03 Thread John Keeping
From: Antoine Pelisse apeli...@gmail.com Create a GREP_HEADER_FIELD_MIN so we can check that the field value is sane and silent the clang warning. Clang warning happens because the enum is unsigned (this is implementation-defined, and there is no negative fields) and the check is then

[PATCH 2/3] combine-diff: suppress a clang warning

2013-02-03 Thread John Keeping
When compiling combine-diff.c, clang 3.2 says: combine-diff.c:1006:19: warning: adding 'int' to a string does not append to the string [-Wstring-plus-int] prefix = COLONS + offset; ~~~^~~~ combine-diff.c:1006:19: note: use array

[PATCH 3/3] builtin/apply: tighten (dis)similarity index parsing

2013-02-03 Thread John Keeping
This was prompted by an incorrect warning issued by clang [1], and a suggestion by Linus to restrict the range to check for values greater than INT_MAX since these will give bogus output after casting to int. In fact the (dis)similarity index is a percentage, so reject values greater than 100.

[PATCH] send-email: ignore files ending with ~

2013-02-03 Thread Alexandre Courbot
It certainly happened to a lot of people already: you carefully prepare your set of patches, export them using format-patch --cover-letter, write your cover letter, and send the set like this: $ git send-email --to=somerenowneddeveloper --to=myfutureemployer --cc=thismailinglistiwanttoimpress

Re: [RFC/PATCH v2] CodingGuidelines: add Python coding guidelines

2013-02-03 Thread Pete Wyckoff
j...@keeping.me.uk wrote on Fri, 01 Feb 2013 11:16 +: On Fri, Feb 01, 2013 at 09:39:39AM +0100, Michael Haggerty wrote: On 01/30/2013 09:31 PM, John Keeping wrote: On Wed, Jan 30, 2013 at 11:05:10AM +0100, Michael Haggerty wrote: [...] maybe we should establish a small Python library

Re: Feature request: Allow extracting revisions into directories

2013-02-03 Thread Sitaram Chamarty
On 02/03/2013 07:48 PM, Robert Clausecker wrote: Hello! git currently has the archive command that allows to save an arbitrary revision into a tar or zip file. Sometimes it is useful to not save this revision into an archive but to directly put all files into an arbitrary directory.

Re: [PATCH 0/3] Make Git compile warning-free with Clang

2013-02-03 Thread Antoine Pelisse
Thanks John, I couldn't find any time to send that sum-up series. On Sun, Feb 3, 2013 at 3:37 PM, John Keeping j...@keeping.me.uk wrote: The first two patches here were sent to the list before but seem to have got lost in the noise [1][2]. The final one is new but was prompted by discussion

Re: [PATCH] send-email: ignore files ending with ~

2013-02-03 Thread Antoine Pelisse
On Sun, Feb 3, 2013 at 3:55 PM, Alexandre Courbot gnu...@gmail.com wrote: It certainly happened to a lot of people already: you carefully prepare your set of patches, export them using format-patch --cover-letter, write your cover letter, and send the set like this: $ git send-email

Re: Feature request: Allow extracting revisions into directories

2013-02-03 Thread Robert Clausecker
Am Sonntag, den 03.02.2013, 21:55 +0530 schrieb Sitaram Chamarty: Could you help me understand why piping it to tar (actually 'tar -C /dest/dir -x') is not sufficient to achieve what you want? Piping the output of git archive into tar is of course a possible solution; I just don't like the

Re: [PATCH 2/3] combine-diff: suppress a clang warning

2013-02-03 Thread Tay Ray Chuan
On Sun, Feb 3, 2013 at 10:37 PM, John Keeping j...@keeping.me.uk wrote: When compiling combine-diff.c, clang 3.2 says: combine-diff.c:1006:19: warning: adding 'int' to a string does not append to the string [-Wstring-plus-int] prefix = COLONS + offset;

Re: [PATCH] gitk: Display the date of a tag in a human friendly way.

2013-02-03 Thread Anand Kumria
Hi Pau, I've not been able to find the canonical location of your gitk repository. I've tried kernel.org, samba.org and ozlabs.org; none of them to have it - nor does any amount of google searching I do reveal the location. I realise you've probably had a busy month with linux.conf.au but it

Re: [PATCH 2/3] combine-diff: suppress a clang warning

2013-02-03 Thread John Keeping
On Mon, Feb 04, 2013 at 02:20:06AM +0800, Tay Ray Chuan wrote: On Sun, Feb 3, 2013 at 10:37 PM, John Keeping j...@keeping.me.uk wrote: When compiling combine-diff.c, clang 3.2 says: combine-diff.c:1006:19: warning: adding 'int' to a string does not append to the string

Re: [PATCH 1/3] fix clang -Wtautological-compare with unsigned enum

2013-02-03 Thread Jonathan Nieder
John Keeping wrote: From: Antoine Pelisse apeli...@gmail.com Create a GREP_HEADER_FIELD_MIN so we can check that the field value is sane and silent the clang warning. Thanks. Looks good to me. [...] --- a/grep.c +++ b/grep.c @@ -625,7 +625,8 @@ static struct grep_expr

Re: [PATCH] git-send-email: add ~/.authinfo parsing

2013-02-03 Thread Jeff King
On Sat, Feb 02, 2013 at 06:57:29AM -0500, Ted Zlatanov wrote: I wrote a Perl credential helper for netrc parsing which is pretty robust, has built-in docs with -h, and doesn't depend on external modules. The netrc parser regex was stolen from Net::Netrc. It will by default use

Re: What's cooking in git.git (Feb 2013, #01; Fri, 1)

2013-02-03 Thread Junio C Hamano
John Keeping j...@keeping.me.uk writes: On Sun, Feb 03, 2013 at 04:13:22AM -0800, David Aguilar wrote: It looks good to go. The additional || : in the makefile is a nice touchup that made it more robust too. Looks good to me as well. Thanks, both. -- To unsubscribe from this list: send the

Re: Getting started contributing.

2013-02-03 Thread Junio C Hamano
Duy Nguyen pclo...@gmail.com writes: On Sun, Feb 3, 2013 at 2:49 PM, Junio C Hamano gits...@pobox.com wrote: On the other hand, there probably still are many loose ends. A few other things - Mark more strings for translation (not as easy as it sounds, some strings can't be translated)

Re: [PATCH 2/3] combine-diff: suppress a clang warning

2013-02-03 Thread Junio C Hamano
John Keeping j...@keeping.me.uk writes: When compiling combine-diff.c, clang 3.2 says: combine-diff.c:1006:19: warning: adding 'int' to a string does not append to the string [-Wstring-plus-int] prefix = COLONS + offset; ~~~^~~~

Re: [PATCH] Handle path completion and colon for tcsh script

2013-02-03 Thread Manlio Perillo
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 Il 02/02/2013 21:10, Junio C Hamano ha scritto: Marc Khouzam marc.khou...@ericsson.com writes: Recent enhancements to git-completion.bash provide intelligent path completion for git commands. Such completions do not add the '/' at the end of

Re: [PATCH] gitk: Display the date of a tag in a human friendly way.

2013-02-03 Thread Jonathan Nieder
Hi Anand, Anand Kumria wrote: I've not been able to find the canonical location of your gitk repository. Here's how I find it: $ git clone git://repo.or.cz/git.git [...] $ cd git $ git log -1 --oneline -- gitk-git ec3ae6ec Merge git://ozlabs.org/~paulus/gitk

Re: [PATCH] gitk: Display the date of a tag in a human friendly way.

2013-02-03 Thread Junio C Hamano
Jonathan Nieder jrnie...@gmail.com writes: Anand Kumria wrote: I've not been able to find the canonical location of your gitk repository. Here's how I find it: $ git clone git://repo.or.cz/git.git [...] $ cd git $ git log -1 --oneline -- gitk-git ec3ae6ec Merge

Re: [PATCH 2/3] combine-diff: suppress a clang warning

2013-02-03 Thread John Keeping
On Sun, Feb 03, 2013 at 11:58:15AM -0800, Junio C Hamano wrote: John Keeping j...@keeping.me.uk writes: When compiling combine-diff.c, clang 3.2 says: combine-diff.c:1006:19: warning: adding 'int' to a string does not append to the string [-Wstring-plus-int]

Re: [PATCH] Handle path completion and colon for tcsh script

2013-02-03 Thread Junio C Hamano
Manlio Perillo manlio.peri...@gmail.com writes: The problem is that when using the new `compopt -o filenames` command, Bash assumes COMPREPLY contains a list of filenames, and when it detects a directory name, it adds a slash. The problem is, if the directory name *already* has a slash, Bash

Re: [PATCH 3/3] builtin/apply: tighten (dis)similarity index parsing

2013-02-03 Thread Junio C Hamano
John Keeping j...@keeping.me.uk writes: diff --git a/builtin/apply.c b/builtin/apply.c index 6c11e8b..4745e75 100644 --- a/builtin/apply.c +++ b/builtin/apply.c @@ -1041,15 +1041,17 @@ static int gitdiff_renamedst(const char *line, struct patch *patch) static int

Re: [PATCH 2/3] combine-diff: suppress a clang warning

2013-02-03 Thread Junio C Hamano
John Keeping j...@keeping.me.uk writes: A quick search turned up the original thread where this feature was added to Clang [1]. It seems that it does find genuine bugs where people try to log values by doing: log(failed to handle error: + errno); To be perfectly honest, anybody who

Re: [PATCH v2] branch: show rebase/bisect info when possible instead of (no branch)

2013-02-03 Thread Matthieu Moy
Nguyễn Thái Ngọc Duy pclo...@gmail.com writes: --- a/t/t6030-bisect-porcelain.sh +++ b/t/t6030-bisect-porcelain.sh @@ -164,7 +164,7 @@ test_expect_success 'bisect start: existing .git/BISECT_START not modified if cp .git/BISECT_START saved test_must_fail git bisect start

Re: [PATCH v2] branch: show rebase/bisect info when possible instead of (no branch)

2013-02-03 Thread Junio C Hamano
Matthieu Moy matthieu@grenoble-inp.fr writes: Nguyễn Thái Ngọc Duy pclo...@gmail.com writes: --- a/t/t6030-bisect-porcelain.sh +++ b/t/t6030-bisect-porcelain.sh @@ -164,7 +164,7 @@ test_expect_success 'bisect start: existing .git/BISECT_START not modified if cp .git/BISECT_START

Re: [PATCH v2] branch: show rebase/bisect info when possible instead of (no branch)

2013-02-03 Thread Junio C Hamano
Junio C Hamano gits...@pobox.com writes: Yeah, I do not think bisecting other alone makes much sense. What does other refer to when you start your bisection at a detached head? I personally think other has _any_ value in that s/_any_/_no_/; obviously ;-) message, because (no branch,

[New Feature] git-submodule-move - Easily move submodules

2013-02-03 Thread TJ
I've recently had need to re-arrange more than ten submodules within a project and discovered there is apparently no easy way to do it. Using some suggestions I found on Stack Overflow I eventually figured out the steps required. Because the steps can be complex I thought it would be handy to

Re: [PATCH] git p4: chdir resolves symlinks only for relative paths

2013-02-03 Thread Pete Wyckoff
mfaze...@szemafor.com wrote on Tue, 29 Jan 2013 09:37 +0100: If a p4 client is configured to /p/foo which is a symlink to /vol/bar/projects/foo, then resolving symlink, which is done by git-p4's chdir will confuse p4: Path /vol/bar/projects/foo/... is not under client root /p/foo While

[RFC] Should log --cc imply log --cc -p?

2013-02-03 Thread Junio C Hamano
I think a natural way to ask reviewing the recent merges while showing tricky ones would be to say: $ git log --first-parent --cc master..pu But this does not to show what I expect to see, which is an output of: $ git log --first-parent --cc -p master..pu This is only a minor

Re: [PATCH 2/3] combine-diff: suppress a clang warning

2013-02-03 Thread John Keeping
On Sun, Feb 03, 2013 at 01:07:48PM -0800, Junio C Hamano wrote: John Keeping j...@keeping.me.uk writes: A quick search turned up the original thread where this feature was added to Clang [1]. It seems that it does find genuine bugs where people try to log values by doing:

Re: Feature request: Allow extracting revisions into directories

2013-02-03 Thread Konstantin Khomoutov
On Sun, Feb 03, 2013 at 03:18:05PM +0100, Robert Clausecker wrote: git currently has the archive command that allows to save an arbitrary revision into a tar or zip file. Sometimes it is useful to not save this revision into an archive but to directly put all files into an arbitrary

Re: Moving (renaming) submodules, recipe/script

2013-02-03 Thread W. Trevor King
On Sun, Feb 03, 2013 at 10:36:17PM +, TJ wrote: I've recently had need to re-arrange more than ten submodules within a project and discovered there is apparently no easy way to do it. I ran into a similar problem last month, and wrote a similar script [1] ;). There are a few other related

Re: [PATCH 2/3] combine-diff: suppress a clang warning

2013-02-03 Thread Junio C Hamano
John Keeping j...@keeping.me.uk writes: If we were to be touching that area of code, I'd rather see a change to make it more robust against such a corner case. If it results in squelching misguided clang warnings against programmers who should not be writing in C, that is a nice side effect,

Re: Feature request: Allow extracting revisions into directories

2013-02-03 Thread Sitaram Chamarty
On 02/03/2013 11:41 PM, Robert Clausecker wrote: Am Sonntag, den 03.02.2013, 21:55 +0530 schrieb Sitaram Chamarty: Could you help me understand why piping it to tar (actually 'tar -C /dest/dir -x') is not sufficient to achieve what you want? Piping the output of git archive into tar is of

Re: Re: Re: Re: Re: Segmentation fault with latest git (070c57df)

2013-02-03 Thread Jongman Heo
Jonathan Nieder wrote: Jongman Heo wrote: But it doesn't stimulate any prerequisites in make, which is weird. What's in builtin/.depend/fetch.o.d? [...] please see below~. $ cat builtin/.depend/fetch.o.d fetch.o: builtin/fetch.c cache.h git-compat-util.h compat/bswap.h \

Re: [PATCH] send-email: ignore files ending with ~

2013-02-03 Thread Alexandre Courbot
On Mon, Feb 4, 2013 at 2:16 AM, Antoine Pelisse apeli...@gmail.com wrote: On Sun, Feb 3, 2013 at 3:55 PM, Alexandre Courbot gnu...@gmail.com wrote: It certainly happened to a lot of people already: you carefully prepare your set of patches, export them using format-patch --cover-letter, write

RE: [PATCH] Handle path completion and colon for tcsh script

2013-02-03 Thread Marc Khouzam
The problem is, if the directory name *already* has a slash, Bash adds another slash! So bash users do see the trailing slash because bash adds one to what we compute and return, which we do strip the trailing slash exactly because we know bash will add one. The problem is slightly

Re: [PATCH] Handle path completion and colon for tcsh script

2013-02-03 Thread Junio C Hamano
Thanks for a detailed explanation. The two examples illustrating different interpretation of the same word were really good. Will replace and requeue. -- 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: git archve --format=tar output changed from 1.8.1 to 1.8.2.1

2013-02-03 Thread Greg KH
On Thu, Jan 31, 2013 at 10:16:27AM -0800, Junio C Hamano wrote: Greg KH gre...@linuxfoundation.org writes: On Thu, Jan 31, 2013 at 09:32:12AM -0800, Junio C Hamano wrote: How about fixing kup to teach the let's cheat and let the other end run 'git archive', if the resulting archive and

Re: git archve --format=tar output changed from 1.8.1 to 1.8.2.1

2013-02-03 Thread Greg KH
On Thu, Jan 31, 2013 at 07:33:52PM +0100, René Scharfe wrote: Am 31.01.2013 18:28, schrieb Greg KH: I tracked this down to commit 22f0dcd9634a818a0c83f23ea1a48f2d620c0546 (archive-tar: split long paths more carefully). The diff of a hex dump of the tar archives shows the following

Re: git archve --format=tar output changed from 1.8.1 to 1.8.2.1

2013-02-03 Thread Greg KH
On Thu, Jan 31, 2013 at 12:52:15PM -0500, Konstantin Ryabitsev wrote: On 31/01/13 12:41 PM, Greg KH wrote: Ugh, uploading a 431Mb file, over a flaky wireless connection (I end up doing lots of kernel releases while traveling), would be a horrible change. I'd rather just keep using the same

Re: Segmentation fault with latest git (070c57df)

2013-02-03 Thread Jonathan Nieder
Jongman Heo wrote: Unfortunately, the patch didn't help to me. Thanks for testing. Did you apply the patch to the older version of git that generates builtin/.depend/fetch.o.d or the newer version that consumes it? Curious, Jonathan -- To unsubscribe from this list: send the line unsubscribe

Re: git archve --format=tar output changed from 1.8.1 to 1.8.2.1

2013-02-03 Thread Junio C Hamano
Greg KH gre...@linuxfoundation.org writes: Then how about fixing kup to try both versions of Git? There will be people who run different versions of Git anyway, and kup should not be preventing Git from helping people on other platforms, or improving its output in general. I think the

pandora charms initiated to provide a couple transfer internet business on 1982

2013-02-03 Thread peteriner
This is often ones own all sorts of things to grasp around tutorial to help you Pandora expensive jewelry, as a result of Pandora bracelets and even pandora charms with the Pandora bangles and even Pandora rings including some past. [url=http://www.genuinepandoracharmssale.co.uk/] pandora

Re: [PATCH v4 1/2] for-each-repo: new command used for multi-repo operations

2013-02-03 Thread Junio C Hamano
Jens Lehmann jens.lehm...@web.de writes: Am 28.01.2013 21:34, schrieb Junio C Hamano: ... I was imagining that foreach --untracked could go something like this: * If you are inside an existing git repository, read its index to learn the gitlinks in the directory and its subdirectories.

Re: Re: Segmentation fault with latest git (070c57df)

2013-02-03 Thread Jongman Heo
Jonathan Nieder wrote: Jongman Heo wrote: Unfortunately, the patch didn't help to me. Thanks for testing. Did you apply the patch to the older version of git that generates builtin/.depend/fetch.o.d or the newer version that consumes it? Curious, Jonathan Hi, Jonathan, I applied the

Re: Segmentation fault with latest git (070c57df)

2013-02-03 Thread Junio C Hamano
Jongman Heo jongman@samsung.com writes: Jonathan Nieder wrote: Jongman Heo wrote: Unfortunately, the patch didn't help to me. Thanks for testing. Did you apply the patch to the older version of git that generates builtin/.depend/fetch.o.d or the newer version that consumes it?

Re: [PATCH v2] branch: show rebase/bisect info when possible instead of (no branch)

2013-02-03 Thread Duy Nguyen
On Mon, Feb 4, 2013 at 4:23 AM, Matthieu Moy matthieu@grenoble-inp.fr wrote: Nguyễn Thái Ngọc Duy pclo...@gmail.com writes: --- a/t/t6030-bisect-porcelain.sh +++ b/t/t6030-bisect-porcelain.sh @@ -164,7 +164,7 @@ test_expect_success 'bisect start: existing .git/BISECT_START not modified

Re: [PATCH] Verify Content-Type from smart HTTP servers

2013-02-03 Thread Junio C Hamano
Jeff King p...@peff.net writes: I was specifically thinking of this (on top of your patch): diff --git a/remote-curl.c b/remote-curl.c index e6f3b63..63680a8 100644 --- a/remote-curl.c +++ b/remote-curl.c @@ -134,14 +134,12 @@ static struct discovery* discover_refs(const char *service)

Re: [PATCH v2] branch: show rebase/bisect info when possible instead of (no branch)

2013-02-03 Thread Duy Nguyen
On Mon, Feb 4, 2013 at 2:14 PM, Duy Nguyen pclo...@gmail.com wrote: * foo (rebasing) Well, this one does not make sense (or causes more confusion). -- Duy -- 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

pandora charms happen to be an exquisite individual variant within the charisma accessory

2013-02-03 Thread nutionerp
pandora charms http://www.pandoracheapcanada.com is not really an individual's early days charisma accessory, however , it's quite as exceptional. On earth do you take into account the main charisma accessory everyone used the as the young lady? Which you were which means that proud of every

tiffany uk: fashionables

2013-02-03 Thread butyist
As one of the numerous agreed on earrings, tiffany uk http://www.cheapstiffanyandcoclub.co.uk . adornment is getting when the allow to be aware of any cool gal in any curve within the society. Yet, any Tiffany Earrings Adornment originates out when using the fasted velocity when the fast