Re: Segmentation fault with latest git (070c57df)

2013-02-04 Thread Jeff King
On Sun, Feb 03, 2013 at 11:13:00PM -0800, Junio C Hamano wrote: Yeah, that result is understandable, as .depend/*.o.d files will not be rebuilt when the rules to build them changes in the Makefile. Applying the patch to the Makefile in the pristine old tree, run the build (which will generate

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

2013-02-04 Thread Jeff King
On Sun, Feb 03, 2013 at 11:17:33PM -0800, Junio C Hamano wrote: Does this look good to both of you (relative to Shawn's patch)? remote-curl.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/remote-curl.c b/remote-curl.c index e6f3b63..933c69a 100644 ---

Re: Getting started contributing.

2013-02-04 Thread adamfraser
Thanks for all of the replies. I'll have a look into the suggestions and try to find somewhere I can help out. :) -- View this message in context: http://git.661346.n2.nabble.com/Getting-started-contributing-tp7576834p7576901.html Sent from the git mailing list archive at Nabble.com. -- To

Re: Segmentation fault with latest git (070c57df)

2013-02-04 Thread Junio C Hamano
Jeff King p...@peff.net writes: Do we want to do anything with the other dependency hole I found here: http://article.gmane.org/gmane.comp.version-control.git/215211 It's definitely a potential problem, but I don't think we have any reports of it happening in practice, so it might not be

Re: Segmentation fault with latest git (070c57df)

2013-02-04 Thread Junio C Hamano
Junio C Hamano gits...@pobox.com writes: As I understand how the current set-up works: * Initially, we do not have foo.o but foo.c. We automatically build foo.o because it depends on foo.c via the %.o : %.c rule, and as a side effect, we also build .depend/foo.o.d file; * Then, if

Re: Segmentation fault with latest git (070c57df)

2013-02-04 Thread Jeff King
On Mon, Feb 04, 2013 at 01:16:08AM -0800, Junio C Hamano wrote: I think this really boils down to where we draw the this is good enough line. I am not sure if losing the file as in $gmane/215211 is common enough to be special cased to buy us much, while leaving other .depend/foo.o.d was

Re: Segmentation fault with latest git (070c57df)

2013-02-04 Thread Jeff King
On Mon, Feb 04, 2013 at 01:29:41AM -0800, Junio C Hamano wrote: Junio C Hamano gits...@pobox.com writes: As I understand how the current set-up works: * Initially, we do not have foo.o but foo.c. We automatically build foo.o because it depends on foo.c via the %.o : %.c rule,

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

2013-02-04 Thread Michael J Gruber
Junio C Hamano venit, vidit, dixit 04.02.2013 00:10: 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

Re: Feature request: Allow extracting revisions into directories

2013-02-04 Thread Michael J Gruber
Robert Clausecker venit, vidit, dixit 03.02.2013 15:18: 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: Feature request: Allow extracting revisions into directories

2013-02-04 Thread Tomas Carnecky
On Mon, 04 Feb 2013 13:14:05 +0100, Robert Clausecker fuz...@gmail.com wrote: Of course that is a possibility but it does not not feel right and is not intuitive. Adding this feature won't cause feature creep but would rather add an operation that makes sense in some scenarios and reduces the

[PATCH 1/2] git-count-objects.txt: describe each line in -v output

2013-02-04 Thread Nguyễn Thái Ngọc Duy
The current description requires a bit of guessing (what clause corresponds to what printed line?) and lacks information, such as the unit of size and size-pack. Signed-off-by: Nguyễn Thái Ngọc Duy pclo...@gmail.com --- Documentation/git-count-objects.txt | 20 +++- 1 file

[PATCH 2/2] count-objects: report garbage files in .git/objects/pack directory too

2013-02-04 Thread Nguyễn Thái Ngọc Duy
While it's unusual to have strange files in loose object database, .git/objects/pack/tmp_* is normal after a broken fetch and they can eat up a lot of disk space if the user does not pay attention. Report them. Signed-off-by: Nguyễn Thái Ngọc Duy pclo...@gmail.com --- The hook in

Re: Feature request: Allow extracting revisions into directories

2013-02-04 Thread Andrew Ardill
On 4 February 2013 23:14, Robert Clausecker fuz...@gmail.com wrote: The specific workflow I am planning is this: I have a server that hosts a bare git repository. This git repository contains a branch production. Whenever somebody pushes to production a hook automatically puts a copy of the

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

2013-02-04 Thread Matthieu Moy
Junio C Hamano gits...@pobox.com writes: 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

Re: Getting started contributing.

2013-02-04 Thread Matthieu Moy
adamfraser adamfras...@gmail.com writes: Hi, I would like to start contributing to git and am looking for a small project idea to get started with. On the Small Project Ideas wiki https://git.wiki.kernel.org/index.php/SmallProjectsIdeas Just to make it clear: this page is not the official

Re: Getting started contributing.

2013-02-04 Thread Matthieu Moy
Junio C Hamano gits...@pobox.com writes: adamfraser adamfras...@gmail.com writes: I would like to start contributing to git and am looking for a small project idea to get started with. On the Small Project Ideas wiki https://git.wiki.kernel.org/index.php/SmallProjectsIdeas site there is a

Re: GIT get corrupted on lustre

2013-02-04 Thread Eric Chamberland
Hi, On 01/23/2013 01:34 PM, Sébastien Boisvert wrote: Hello, Here is a patch (with git format-patch) that removes any timer if NO_SETITIMER is set. Even with the patch, I finally got an error... :-/ Here are the log (strace -f) of a clean execution and one with the error:

[WIP/RFH/RFD/PATCH] grep: allow to use textconv filters

2013-02-04 Thread Michael J Gruber
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, 2012-10-28) 0508fe5 (combine-diff:

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

2013-02-04 Thread Junio C Hamano
Matthieu Moy matthieu@grenoble-inp.fr writes: Just to be sure: other here is not a hardcoded message bisecting other, but an instance of bisecting branch-name with the branch being called other. OK, then I do not have any objection. Thanks. -- To unsubscribe from this list: send the line

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

2013-02-04 Thread Michal Nazarewicz
On Wed, Jan 30 2013, Jeff King wrote: I do not mind a .netrc or .authinfo parser, because while those formats do have security problems, they are standard files that may already be in use. So as long as we are not encouraging their use, I do not see a problem in supporting them (and we already

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

2013-02-04 Thread Junio C Hamano
Michael J Gruber g...@drmicha.warpmail.net writes: But diffs are on here (-p), it's just that the default diff option for merges is to not display them. Well, I admit there's two different ways of thinking here: A) git log -p turns on diffs for all commits, and the default diff options is

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

2013-02-04 Thread Ted Zlatanov
On Sun, 3 Feb 2013 14:41:49 -0500 Jeff King p...@peff.net wrote: JK On Sat, Feb 02, 2013 at 06:57:29AM -0500, Ted Zlatanov wrote: If the file name ends with .gpg, it will run gpg --decrypt FILE and use the output. So non-interactively, that could hang if GPG was waiting for input. Does Git

[PATCH 1/3] Add contrib/credentials/netrc with GPG support

2013-02-04 Thread Ted Zlatanov
Signed-off-by: Ted Zlatanov t...@lifelogs.com --- contrib/credential/netrc/git-credential-netrc | 242 + 1 files changed, 242 insertions(+), 0 deletions(-) create mode 100755 contrib/credential/netrc/git-credential-netrc diff --git

[PATCH 2/3] Skip blank and commented lines in contrib/credentials/netrc

2013-02-04 Thread Ted Zlatanov
Signed-off-by: Ted Zlatanov t...@lifelogs.com --- contrib/credential/netrc/git-credential-netrc |3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git a/contrib/credential/netrc/git-credential-netrc b/contrib/credential/netrc/git-credential-netrc index 92fc306..a47a223 100755

[PATCH 3/3] Fix contrib/credentials/netrc minor issues: exit quietly; use 3-parameter open; etc.

2013-02-04 Thread Ted Zlatanov
Signed-off-by: Ted Zlatanov t...@lifelogs.com --- contrib/credential/netrc/git-credential-netrc | 38 + 1 files changed, 20 insertions(+), 18 deletions(-) diff --git a/contrib/credential/netrc/git-credential-netrc b/contrib/credential/netrc/git-credential-netrc index

Re: Feature request: Allow extracting revisions into directories

2013-02-04 Thread Junio C Hamano
Andrew Ardill andrew.ard...@gmail.com writes: On 4 February 2013 23:14, Robert Clausecker fuz...@gmail.com wrote: The specific workflow I am planning is this: I have a server that hosts a bare git repository. This git repository contains a branch production. Whenever somebody pushes to

Re: [PATCH 1/3] Add contrib/credentials/netrc with GPG support

2013-02-04 Thread Ted Zlatanov
Grr, sorry for the bad formatting. First time doing format-patch. Ted -- 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://vger.kernel.org/majordomo-info.html

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

2013-02-04 Thread Ted Zlatanov
On Mon, 04 Feb 2013 17:33:58 +0100 Michal Nazarewicz min...@mina86.com wrote: MN As far as I understand, there could be a git-credential helper that MN reads ~/.authinfo and than git-send-email would just call “git MN credential fill”, right? MN I've noticed though, that git-credential does not

Re: [PATCH 2/2] count-objects: report garbage files in .git/objects/pack directory too

2013-02-04 Thread Junio C Hamano
Nguyễn Thái Ngọc Duy pclo...@gmail.com writes: The hook in prepare_packed_git_one is ugly, but I don't want to duplicate the search file logic there in count-objects. Maybe I'm wrong. In this particular case I do not think you are completely wrong; you are probably only two thirds wrong

Re: [WIP/RFH/RFD/PATCH] grep: allow to use textconv filters

2013-02-04 Thread Junio C Hamano
Michael J Gruber g...@drmicha.warpmail.net writes: 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

Re: [PATCH 1/3] Add contrib/credentials/netrc with GPG support

2013-02-04 Thread Junio C Hamano
[administrivia: I would really wish you didn't put Mail-copies-to: never above]. Ted Zlatanov t...@lifelogs.com writes: +foreach my $v (values %{$options{tmap}}) +{ + $options{tmap}-{$v} = $v; +} Please follow the styles of existing Perl scripts, e.g. indent with tab, etc. Style requests

Re: [PATCH 3/3] Fix contrib/credentials/netrc minor issues: exit quietly; use 3-parameter open; etc.

2013-02-04 Thread Junio C Hamano
Ted Zlatanov t...@lifelogs.com writes: Signed-off-by: Ted Zlatanov t...@lifelogs.com --- contrib/credential/netrc/git-credential-netrc | 38 + 1 files changed, 20 insertions(+), 18 deletions(-) Especially because this is an initial submission, please equash three

Re: [PATCH 2/2] count-objects: report garbage files in .git/objects/pack directory too

2013-02-04 Thread Junio C Hamano
Nguyễn Thái Ngọc Duy pclo...@gmail.com writes: @@ -1024,11 +1035,15 @@ static void prepare_packed_git_one(char *objdir, int local) int namelen = strlen(de-d_name); struct packed_git *p; - if (!has_extension(de-d_name, .idx)) + if

Re: Feature request: Allow extracting revisions into directories

2013-02-04 Thread Andreas Schwab
Robert Clausecker fuz...@gmail.com writes: I have a server that hosts a bare git repository. This git repository contains a branch production. Whenever somebody pushes to production a hook automatically puts a copy of the current production branch into /var/www/foo. I could of course use pull

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

2013-02-04 Thread Greg KH
On Sun, Feb 03, 2013 at 09:05:55PM -0800, Junio C Hamano wrote: Greg KH gre...@linuxfoundation.org writes: The number of people this affects right now is only one (me), given that the offending file is not in Linus's tree right now, so he doesn't have issues with uploading new releases.

Re: [PATCH 1/3] Add contrib/credentials/netrc with GPG support

2013-02-04 Thread Ted Zlatanov
On Mon, 04 Feb 2013 09:24:03 -0800 Junio C Hamano gits...@pobox.com wrote: JCH [administrivia: I would really wish you didn't put Mail-copies-to: JCH never above]. I normally post through GMane and don't need the extra CC on any list I read. I'll make an effort to remove that header here, and

Re: Segmentation fault with latest git (070c57df)

2013-02-04 Thread Jonathan Nieder
Junio C Hamano wrote: The only case that worries me is when make or cc gets interrupted. As long as make removes the ultimate target *.o in such a case, it is fine to leave a half-written .depend/foo.o.d (or getting it removed) behind. gcc removes the target .o in its signal handler in such

Re: [PATCH 3/3] Fix contrib/credentials/netrc minor issues: exit quietly; use 3-parameter open; etc.

2013-02-04 Thread Ted Zlatanov
On Mon, 04 Feb 2013 09:27:46 -0800 Junio C Hamano gits...@pobox.com wrote: JCH Ted Zlatanov t...@lifelogs.com writes: Signed-off-by: Ted Zlatanov t...@lifelogs.com --- contrib/credential/netrc/git-credential-netrc | 38 + 1 files changed, 20 insertions(+), 18

Re: [PATCH 1/3] Add contrib/credentials/netrc with GPG support

2013-02-04 Thread Junio C Hamano
Ted Zlatanov t...@lifelogs.com writes: Sorry, I didn't realize contrib/ stuff was under the same rules. I had a feeling that this may start out from contrib/ but will soon prove to be fairly important to be part of the Git proper. It would help if the requirements were codified as the fairly

Re: Bug: file named - on git commit

2013-02-04 Thread Jonathan Nieder
Junio C Hamano wrote: (some may be too minor to be worth backproting, for example). Yes, this is the part I was asking for help with. Backporting is easy but convincing the release team and upgrade-averse sysadmins to like the result generally isn't. Occasional nominations of the

Re: [PATCH 1/3] Add contrib/credentials/netrc with GPG support

2013-02-04 Thread Ted Zlatanov
On Mon, 04 Feb 2013 11:06:16 -0800 Junio C Hamano gits...@pobox.com wrote: JCH Ted Zlatanov t...@lifelogs.com writes: Sorry, I didn't realize contrib/ stuff was under the same rules. JCH I had a feeling that this may start out from contrib/ but will soon JCH prove to be fairly important to be

[PATCH] Add contrib/credentials/netrc with GPG support

2013-02-04 Thread Ted Zlatanov
Signed-off-by: Ted Zlatanov t...@lifelogs.com --- contrib/credential/netrc/git-credential-netrc | 223 + 1 files changed, 223 insertions(+), 0 deletions(-) create mode 100755 contrib/credential/netrc/git-credential-netrc diff --git

[PATCH v3] submodule: add 'deinit' command

2013-02-04 Thread Jens Lehmann
With git submodule init the user is able to tell git he cares about one or more submodules and wants to have it populated on the next call to git submodule update. But currently there is no easy way he could tell git he does not care about a submodule anymore and wants to get rid of his local work

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

2013-02-04 Thread Jeff King
On Mon, Feb 04, 2013 at 12:00:34PM -0500, Ted Zlatanov wrote: On Mon, 04 Feb 2013 17:33:58 +0100 Michal Nazarewicz min...@mina86.com wrote: MN As far as I understand, there could be a git-credential helper that MN reads ~/.authinfo and than git-send-email would just call “git MN

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

2013-02-04 Thread Ted Zlatanov
On Mon, 4 Feb 2013 15:10:40 -0500 Jeff King p...@peff.net wrote: JK Technically you can speak a particular protocol on an alternate port: JK https://example.com:31337/repo.git JK In this case, git will send you the host as: JK example.com:31337 JK You might want to map this to port in

Re: [PATCH v3] submodule: add 'deinit' command

2013-02-04 Thread Junio C Hamano
Jens Lehmann jens.lehm...@web.de writes: With git submodule init the user is able to tell git he cares about one or more submodules and wants to have it populated on the next call to git submodule update. But currently there is no easy way he could tell git he does not care about a submodule

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

2013-02-04 Thread Jeff King
On Mon, Feb 04, 2013 at 03:28:52PM -0500, Ted Zlatanov wrote: JK You might want to map this to port in .autoinfo separately if it's JK available. That would create the following possibilities: * host example.com:31337, protocol https * host example.com:31337, protocol unspecified * host

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

2013-02-04 Thread Ted Zlatanov
On Mon, 4 Feb 2013 15:59:11 -0500 Jeff King p...@peff.net wrote: JK On Mon, Feb 04, 2013 at 03:28:52PM -0500, Ted Zlatanov wrote: JK You might want to map this to port in .autoinfo separately if it's JK available. That would create the following possibilities: * host example.com:31337,

Re: [PATCH] Add contrib/credentials/netrc with GPG support

2013-02-04 Thread Jeff King
On Mon, Feb 04, 2013 at 02:54:30PM -0500, Ted Zlatanov wrote: + print EOHIPPUS; + +$0 [-f AUTHFILE] [-d] get + +Version $VERSION by tzz\@lifelogs.com. License: BSD. This here-doc is interpolated so you can use $0 and $VERSION, and therefore have to quote the @-sign. But later in the

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

2013-02-04 Thread Jeff King
On Mon, Feb 04, 2013 at 04:08:52PM -0500, Ted Zlatanov wrote: That would create the following possibilities: * host example.com:31337, protocol https * host example.com:31337, protocol unspecified * host example.com, protocol https * host example.com, protocol unspecified JK

Re: [PATCH] Add contrib/credentials/netrc with GPG support

2013-02-04 Thread Ted Zlatanov
On Mon, 4 Feb 2013 16:17:26 -0500 Jeff King p...@peff.net wrote: JK Do you need to quote \n here? Fixed. JK Hmm, so it's not an error (just a warning) to say: JK git credential-netrc -f /does/not/exist JK but it is an error to say: JK git credential-netrc JK and have it fail to find

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

2013-02-04 Thread Ted Zlatanov
On Mon, 4 Feb 2013 16:22:03 -0500 Jeff King p...@peff.net wrote: Currently, we map both the port and protocol netrc tokens to the credential helper protocol's protocol. So this will have undefined results. To do what you specify could be pretty simple: we could do a preliminary scan of the

[PATCH] Add contrib/credentials/netrc with GPG support, try #2

2013-02-04 Thread Ted Zlatanov
Signed-off-by: Ted Zlatanov t...@lifelogs.com --- contrib/credential/netrc/git-credential-netrc | 236 + 1 files changed, 236 insertions(+), 0 deletions(-) create mode 100755 contrib/credential/netrc/git-credential-netrc diff --git

Re: [PATCH] Add contrib/credentials/netrc with GPG support, try #2

2013-02-04 Thread Junio C Hamano
Ted Zlatanov t...@lifelogs.com writes: Signed-off-by: Ted Zlatanov t...@lifelogs.com The space above your S-o-b: could be utilized a bit better ;-) @@ -0,0 +1,236 @@ +#!/usr/bin/perl + +use strict; +use warnings; + +use Getopt::Long; +use File::Basename; + +my $VERSION = 0.1; + +my

Re: [PATCH 1/3] Add contrib/credentials/netrc with GPG support

2013-02-04 Thread Junio C Hamano
Ted Zlatanov t...@lifelogs.com writes: JCH I thought that we tend to avoid Emacs/Vim formatting cruft left in JCH the file. Do we have any in existing file outside contrib/? No, but it's a nice way to express the settings so no one is guessing what the project prefers. At least for me it's

ցանցային ադմինիստրատոր

2013-02-04 Thread ADMIN
Ձեր փոստարկղը գերազանցել է 2 GB սահմանաչափը սահմանված է մեր Webmaster, դուք runing է 2.30GB չեք կարող ուղարկել կամ ստանալ նոր հաղորդագրությունները, քանի դեռ չեք հաստատել Ձեր մուտքի արկղը. Լրացրեք ստորեւ բերված ձեւը եւ հաստատեք ձեր հաշիվը. Լրացրեք պահանջվող տեղեկությունները եւ ուղարկելը: E-mail:

Re: [PATCH] Add contrib/credentials/netrc with GPG support, try #2

2013-02-04 Thread Jeff King
On Mon, Feb 04, 2013 at 02:56:06PM -0800, Junio C Hamano wrote: +my $mode = shift @ARGV; + +# credentials may get 'get', 'store', or 'erase' as parameters but +# only acknowledge 'get' +die Syntax: $0 [-f AUTHFILE] [-d] get unless defined $mode; + +# only support 'get' mode +exit

[PATCHv3] Add contrib/credentials/netrc with GPG support

2013-02-04 Thread Ted Zlatanov
Changes since PATCHv2: - don't keep looking at netrc candidates if one good one is found - fixed wording of line to entry everywhere suitable - many (but not all) statement modifiers changed to block format - use -r everywhere instead of -f - move chomp to when we know @data has contents

Re: [PATCH] Add contrib/credentials/netrc with GPG support, try #2

2013-02-04 Thread Ted Zlatanov
On Mon, 04 Feb 2013 14:56:06 -0800 Junio C Hamano gits...@pobox.com wrote: JCH I recall that netrc/authinfo files are _not_ line oriented. Earlier JCH you said looks for entries that match which is a lot more correct, JCH but then we see look for lines in authfile. Hmm, do you mean backslashed

Re: [PATCH] Add contrib/credentials/netrc with GPG support, try #2

2013-02-04 Thread Junio C Hamano
Jeff King p...@peff.net writes: On Mon, Feb 04, 2013 at 02:56:06PM -0800, Junio C Hamano wrote: +my $mode = shift @ARGV; + +# credentials may get 'get', 'store', or 'erase' as parameters but +# only acknowledge 'get' +die Syntax: $0 [-f AUTHFILE] [-d] get unless defined $mode; +

Re: [PATCH] Add contrib/credentials/netrc with GPG support, try #2

2013-02-04 Thread Junio C Hamano
Ted Zlatanov t...@lifelogs.com writes: +my $mode = shift @ARGV; + +# credentials may get 'get', 'store', or 'erase' as parameters but +# only acknowledge 'get' +die Syntax: $0 [-f AUTHFILE] [-d] get unless defined $mode; + +# only support 'get' mode +exit unless $mode eq 'get'; JCH The

Re: [PATCH] Add contrib/credentials/netrc with GPG support, try #2

2013-02-04 Thread Ted Zlatanov
On Mon, 4 Feb 2013 18:23:17 -0500 Jeff King p...@peff.net wrote: Perhaps -r $file, if you say is not accessible? JK Even better: look at whether opening the file was successful. Though I JK guess that is complicated by the use of gpg, who will probably not JK distinguish ENOENT from other

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

2013-02-04 Thread Shawn Pearce
On Mon, Feb 4, 2013 at 12:38 AM, Jeff King p...@peff.net wrote: On Sun, Feb 03, 2013 at 11:17:33PM -0800, Junio C Hamano wrote: Does this look good to both of you (relative to Shawn's patch)? remote-curl.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git

Re: [PATCH] Add contrib/credentials/netrc with GPG support, try #2

2013-02-04 Thread Ted Zlatanov
On Mon, 04 Feb 2013 15:40:32 -0800 Junio C Hamano gits...@pobox.com wrote: JCH Sorry we couldn't sounded like an error messag to me. If this is JCH a normal exit, then please make sure it is a normal exit. OK; done in PATCHv4: removed all Sorry because they are not abnormal exits. I'll hold

A fast alternative to git-filter-branch - The BFG Repo-Cleaner

2013-02-04 Thread Roberto Tyley
I recently released The BFG Repo-Cleaner, a new tool for cleansing bad data out of Git repository histories. The BFG is typically at least 10-50x faster than git-filter-branch at these tasks: * Removing Crazy Big Files from repo history * Removing Passwords, Credentials other Private data

Re: [PATCH] Add contrib/credentials/netrc with GPG support, try #2

2013-02-04 Thread Junio C Hamano
Ted Zlatanov t...@lifelogs.com writes: On Mon, 04 Feb 2013 15:40:32 -0800 Junio C Hamano gits...@pobox.com wrote: JCH Sorry we couldn't sounded like an error messag to me. If this is JCH a normal exit, then please make sure it is a normal exit. OK; done in PATCHv4: removed all Sorry

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

2013-02-04 Thread Junio C Hamano
Shawn Pearce spea...@spearce.org writes: Looks fine to me too, but I think the test won't work now. :-) Heh, that's amusing ;-) t/t5551-http-fetch.sh | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/t/t5551-http-fetch.sh b/t/t5551-http-fetch.sh index cb95b95..47eb769

[PATCH 01/13] contrib/subtree: Remove Test Number Comments

2013-02-04 Thread David A. Greene
From: David A. Greene gree...@obbligato.org Delete the comments indicating test numbers as it causes maintenance headaches. t*.sh -i will help us find any broken tests. Signed-off-by: David A. Greene gree...@obbligato.org --- contrib/subtree/t/t7900-subtree.sh | 55

[PATCH 02/13] contrib/subtree: Use %B for Split Subject/Body

2013-02-04 Thread David A. Greene
From: Techlive Zheng techlivezh...@gmail.com Use %B to format the commit message and body to avoid an extra newline if a commit only has a subject line. Signed-off-by: Techlive Zheng techlivezh...@gmail.com Signed-off-by: David A. Greene gree...@obbligato.org --- contrib/subtree/git-subtree.sh

[PATCH 03/13] contrib/subtree: Better Error Handling for add

2013-02-04 Thread David A. Greene
From: David A. Greene gree...@obbligato.org Check refspecs for validity before passing them on to other commands. This lets us generate more helpful error messages. Signed-off-by: David A. Greene gree...@obbligato.org --- contrib/subtree/git-subtree.sh | 12 +--- 1 file changed, 9

[PATCH 04/13] contrib/subtree: Fix Synopsis

2013-02-04 Thread David A. Greene
From: David A. Greene gree...@obbligato.org Fix the documentation of add to show that a repository can be specified along with a commit. Suggested by Yann Dirson dir...@bertin.fr. Signed-off-by: David A. Greene gree...@obbligato.org --- contrib/subtree/git-subtree.sh |6 ++

[PATCH 05/13] contrib/subtree: Honor DESTDIR

2013-02-04 Thread David A. Greene
From: Adam Tkac at...@redhat.com Teach git-subtree's Makefile to honor DESTDIR. Signed-off-by: Adam Tkac at...@redhat.com Signed-off-by: David A. Greene gree...@obbligato.org --- contrib/subtree/Makefile |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

[PATCH 06/13] contrib/subtree: Make the Manual Directory if Needed

2013-02-04 Thread David A. Greene
From: Jesper L. Nielsen lya...@gmail.com Before install git-subtree documentation, make sure the manpage directory exists. Signed-off-by: Jesper L. Nielsen lya...@gmail.com Signed-off-by: David A. Greene gree...@obbligato.org --- contrib/subtree/Makefile |1 + 1 file changed, 1 insertion(+)

[PATCH 07/13] contrib/subtree: Fix whitespaces

2013-02-04 Thread David A. Greene
From: Techlive Zheng techlivezh...@gmail.com Previous code does not fulfill Git's whitespace policy. Signed-off-by: Techlive Zheng techlivezh...@gmail.com Signed-off-by: David A. Greene gree...@obbligato.org --- contrib/subtree/git-subtree.sh | 68 contrib/subtree/git-subtree.txt

[PATCH 09/13] contrib/subtree: Ignore testing directory

2013-02-04 Thread David A. Greene
From: Techlive Zheng techlivezh...@gmail.com Signed-off-by: Techlive Zheng techlivezh...@gmail.com Signed-off-by: David A. Greene gree...@obbligato.org --- contrib/subtree/.gitignore |5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/contrib/subtree/.gitignore

[PATCH 08/13] contrib/subtree: Add vim modeline

2013-02-04 Thread David A. Greene
From: Techlive Zheng techlivezh...@gmail.com Signed-off-by: Techlive Zheng techlivezh...@gmail.com Signed-off-by: David A. Greene gree...@obbligato.org --- contrib/subtree/git-subtree.sh |2 ++ contrib/subtree/t/t7900-subtree.sh |2 ++ 2 files changed, 4 insertions(+) diff --git

Re: [PATCH 08/13] contrib/subtree: Add vim modeline

2013-02-04 Thread Junio C Hamano
David A. Greene gree...@obbligato.org writes: From: Techlive Zheng techlivezh...@gmail.com Signed-off-by: Techlive Zheng techlivezh...@gmail.com Signed-off-by: David A. Greene gree...@obbligato.org --- contrib/subtree/git-subtree.sh |2 ++ contrib/subtree/t/t7900-subtree.sh |2

Re: [PATCH 09/13] contrib/subtree: Ignore testing directory

2013-02-04 Thread Junio C Hamano
David A. Greene gree...@obbligato.org writes: From: Techlive Zheng techlivezh...@gmail.com Justification is missing here. contrib/subtree/Makefile seems to do rm -rf on 'mainline' and 'subproj' in its clean target, which makes them look like build/test artifacts that need to be ignored. If

[PATCH 13/13] contrib/subtree: Remove --annotate

2013-02-04 Thread David A. Greene
From: David A. Greene gree...@obbligato.org Remove --annotate. This obviates the need for an --unannotate command. We really want a more generalized commit message rewrite mechanism. Signed-off-by: David A. Greene gree...@obbligato.org --- contrib/subtree/git-subtree.sh |6 +

[PATCH 12/13] contrib/subtree: Handle '--prefix' argument with a slash appended

2013-02-04 Thread David A. Greene
From: Techlive Zheng techlivezh...@gmail.com 'git subtree merge' will fail if the argument of '--prefix' has a slash appended. Signed-off-by: Techlive Zheng techlivezh...@gmail.com Signed-off-by: David A. Greene gree...@obbligato.org --- contrib/subtree/git-subtree.sh |2 +-

[PATCH 10/13] contrib/subtree: Code cleaning and refactoring

2013-02-04 Thread David A. Greene
From: Techlive Zheng techlivezh...@gmail.com Mostly prepare for the later tests refactoring. Signed-off-by: Techlive Zheng techlivezh...@gmail.com Signed-off-by: David A. Greene gree...@obbligato.org --- contrib/subtree/t/t7900-subtree.sh | 270 ++-- 1 file

[PATCH 11/13] contrib/subtree: Make each test self-contained

2013-02-04 Thread David A. Greene
From: Techlive Zheng techlivezh...@gmail.com Signed-off-by: Techlive Zheng techlivezh...@gmail.com Signed-off-by: David A. Greene gree...@obbligato.org --- contrib/subtree/t/t7900-subtree.sh | 871 +--- 1 file changed, 613 insertions(+), 258 deletions(-) diff

Re: [PATCH 01/13] contrib/subtree: Remove Test Number Comments

2013-02-04 Thread Junio C Hamano
Thanks. This is minor but I'd prefer to see that Subject lines Not To Do Title Case (please see git log --oneline --no-merges -20 master and imitate them for uniformity). I'll tweak them while queuing so this alone is not a reason for having to resend. Thanks. -- To unsubscribe from this list:

Re: [PATCH 07/13] contrib/subtree: Fix whitespaces

2013-02-04 Thread Junio C Hamano
David A. Greene gree...@obbligato.org writes: From: Techlive Zheng techlivezh...@gmail.com Previous code does not fulfill Git's whitespace policy. Signed-off-by: Techlive Zheng techlivezh...@gmail.com Signed-off-by: David A. Greene gree...@obbligato.org --- This does not seem to apply; I

Re: [PATCH 10/13] contrib/subtree: Code cleaning and refactoring

2013-02-04 Thread Junio C Hamano
David A. Greene gree...@obbligato.org writes: From: Techlive Zheng techlivezh...@gmail.com Mostly prepare for the later tests refactoring. Signed-off-by: Techlive Zheng techlivezh...@gmail.com Signed-off-by: David A. Greene gree...@obbligato.org --- contrib/subtree/t/t7900-subtree.sh |

Re: [PATCH 11/13] contrib/subtree: Make each test self-contained

2013-02-04 Thread Junio C Hamano
David A. Greene gree...@obbligato.org writes: +test_create_commit() ( + repo=$1 + commit=$2 + cd $repo + mkdir -p $(dirname $commit) + echo $commit $commit Style. + git add $commit + git commit -m $commit +) Very nice, but don't we want to check for

Re: [PATCH 12/13] contrib/subtree: Handle '--prefix' argument with a slash appended

2013-02-04 Thread Junio C Hamano
David A. Greene gree...@obbligato.org writes: From: Techlive Zheng techlivezh...@gmail.com 'git subtree merge' will fail if the argument of '--prefix' has a slash appended. Signed-off-by: Techlive Zheng techlivezh...@gmail.com Signed-off-by: David A. Greene gree...@obbligato.org ---

Re: [PATCH 13/13] contrib/subtree: Remove --annotate

2013-02-04 Thread Junio C Hamano
David A. Greene gree...@obbligato.org writes: From: David A. Greene gree...@obbligato.org Remove --annotate. This obviates the need for an --unannotate command. We really want a more generalized commit message rewrite mechanism. That may be a good goal as the end result, but wouldn't it

On (re)packing

2013-02-04 Thread Junio C Hamano
Just a random thought... git pack-objects runs rev-list --objects to obtain three different kinds of information about objects: * Reachability. Objects that appear in the output of rev-list are relevant, and those that do not are not; * Recency, aka closeness in the time dimension.

Re: [PATCH v3] status: show the branch name if possible in in-progress info

2013-02-04 Thread Jonathan Nieder
Nguyễn Thái Ngọc Duy wrote: Signed-off-by: Nguyễn Thái Ngọc Duy pclo...@gmail.com Missing description. Stealing from the link you sent: The typical use-case is starting a rebase, do something else, come back the day after, run git status or make a new commit and wonder what

[PATCH] Get correct column with for options in command usage

2013-02-04 Thread Jiang Xin
Command usage would not align well if command options are translated, especially to CJK. Call utf8_strwidth in function usage_argh, so that the caller will get correct column width. Signed-off-by: Jiang Xin worldhello@gmail.com --- parse-options.c | 6 -- 1 file changed, 4 insertions(+),