Re: fix french translation

2013-05-26 Thread Antoine Pelisse
Please do not forget to "reply to all". On Mon, May 27, 2013 at 1:34 AM, 乙酸鋰 wrote: > git-gui/po/fr.po | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git b/git-gui/po/fr.po a/git-gui/po/fr.po > index 0aff186..40441db 100644 > --- b/git-gui/po/fr.po > +++ a/git-gui/po/fr.po >

contrib/git-normal-to-bare.sh

2013-05-26 Thread Zenaan Harkness
I needed this quite a bit in the last few days, basic script but serves my need. I think it would be useful for other beginners if in $git/contrib/ source dir. Just a start to a basic script. Needs more tests etc, but it's enough to get newbies (like me) off to a reasonable start. Handles multiple

Re: java zlib woes (was: Reading commit objects)

2013-05-26 Thread Andreas Krey
On Tue, 21 May 2013 22:56:21 +, Shawn Pearce wrote: ... > This was with JGit? A stack trace and JGit version (so we can > correlate line numbers) would be a much more useful bug report than > nothing at all. I now have a full test case (involving a generated repo just shy of 1GB) that will re

contrib/git-normal-to-bare.sh

2013-05-26 Thread Zenaan Harkness
This question comes up every now and then - how to convert from normal to bare, or vice versa. This is just a start to a basic script to go one way. Needs more tests etc, but it's enough to get newbies (like me) off to a reasonable start. PLEASE CC me, as I am not subscribed. Thanks, Zenaan gi

Re: [PATCH 4/6] build: be clearer about order-only prerequisites

2013-05-26 Thread Junio C Hamano
Felipe Contreras writes: > Signed-off-by: Felipe Contreras > --- This patch is wrong, I think. The canonical example of order-only is something like http://www.kolpackov.net/pipermail/notes/2004-January/01.html where you want to make sure an output directory already exists, but you do

Re: [PATCH] diff: add --ignore-blank-lines option

2013-05-26 Thread Johannes Sixt
Am 26.05.2013 19:58, schrieb Antoine Pelisse: > The goal of the patch is to introduce the GNU diff > -B/--ignore-blank-lines as closely as possible. The short option is not > available because it's already used for "break-rewrites". > > When this option is used, git-diff will not create hunks that

Re: [PATCH] fast-import: Remove redundant assignment of 'oe' to itself.

2013-05-26 Thread Stefan Beller
The line in question was not fixed up, but originally added in 85c6239. However I see your point, it may reintroduce warnings. Regards, Stefan On 05/26/2013 10:17 PM, Stefano Lattarini wrote: > On 05/26/2013 10:14 PM, Stefan Beller wrote: >> It's not done very often. >> > Of course; it's done onl

Re: [PATCH] fast-import: Remove redundant assignment of 'oe' to itself.

2013-05-26 Thread Stefano Lattarini
On 05/26/2013 10:14 PM, Stefan Beller wrote: > It's not done very often. > Of course; it's done only in those places where GCC gave spurious warnings. And it's done for no other reason that to silence said warnings. > [SNIP] rest of message Regards, Stefano -- To unsubscribe from this list: se

Re: [PATCH] fast-import: Remove redundant assignment of 'oe' to itself.

2013-05-26 Thread Stefan Beller
It's not done very often. For example at the very same place of the patch there is const char *p; not assigned. Well you could argue, that it can be detected by gcc as that variable p is assigned next line. So another counterexample, having the same pattern would be fast-import.c line 2992 in pars

Re: [PATCH] fast-import: Remove redundant assignment of 'oe' to itself.

2013-05-26 Thread Stefano Lattarini
On 05/26/2013 10:05 PM, Stefan Beller wrote: > Reported by cppcheck. > > Signed-off-by: Stefan Beller > --- > fast-import.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/fast-import.c b/fast-import.c > index 5f539d7..0142e3a 100644 > --- a/fast-import.c > +++ b/fast-i

[PATCH] fast-import: Remove redundant assignment of 'oe' to itself.

2013-05-26 Thread Stefan Beller
Reported by cppcheck. Signed-off-by: Stefan Beller --- fast-import.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/fast-import.c b/fast-import.c index 5f539d7..0142e3a 100644 --- a/fast-import.c +++ b/fast-import.c @@ -2914,7 +2914,7 @@ static void cat_blob(struct object_en

[PATCH] daemon.c:handle: Remove unneeded check for null pointer.

2013-05-26 Thread Stefan Beller
The handle function has the signature static void handle(int incoming, struct sockaddr *addr, socklen_t addrlen); so the addr pointer may be passed as null syntactically. However there is just one place in daemon.c, which calls the handle function in function service_loop in daemon.c:1044: handle(

[PATCH] archive-zip:write_zip_entry: Remove second reset of size variable to zero.

2013-05-26 Thread Stefan Beller
It is set to zero just 3 lines before. Reported by cppcheck. Signed-off-by: Stefan Beller --- archive-zip.c | 1 - 1 file changed, 1 deletion(-) diff --git a/archive-zip.c b/archive-zip.c index b2c4fe0..4bde019 100644 --- a/archive-zip.c +++ b/archive-zip.c @@ -232,7 +232,6 @@ static int write_

Re: git clone does not understand insteadOf URLs

2013-05-26 Thread Gioele Barabucci
Il 26/05/2013 21:00, John Keeping ha scritto: Regardless of the implementation of the commands, if I do mkdir projectA cd projectA git init . git remote add origin remote-repo/projectA.git git pull origin master I get a working repository. If I do git clone

Re: git clone does not understand insteadOf URLs

2013-05-26 Thread John Keeping
On Sun, May 26, 2013 at 08:21:45PM +0200, Gioele Barabucci wrote: > Il 26/05/2013 20:14, John Keeping ha scritto: > > On Sun, May 26, 2013 at 08:09:56PM +0200, Gioele Barabucci wrote: > >> Il 26/05/2013 20:00, Andreas Schwab ha scritto: > Simple, I keep all my projects on the same server, so I

Re: git clone does not understand insteadOf URLs

2013-05-26 Thread Gioele Barabucci
Il 26/05/2013 20:14, John Keeping ha scritto: On Sun, May 26, 2013 at 08:09:56PM +0200, Gioele Barabucci wrote: Il 26/05/2013 20:00, Andreas Schwab ha scritto: Simple, I keep all my projects on the same server, so I would like to refer to that server + path using 'remote-repo'. "git+ssh://git.

Re: git clone does not understand insteadOf URLs

2013-05-26 Thread John Keeping
On Sun, May 26, 2013 at 08:09:56PM +0200, Gioele Barabucci wrote: > Il 26/05/2013 20:00, Andreas Schwab ha scritto: > >> Simple, I keep all my projects on the same server, so I would like to > >> refer to that server + path using 'remote-repo'. > >> > >> "git+ssh://git.example.org//users/gioele/pro

Re: git clone does not understand insteadOf URLs

2013-05-26 Thread Gioele Barabucci
Il 26/05/2013 20:00, Andreas Schwab ha scritto: Simple, I keep all my projects on the same server, so I would like to refer to that server + path using 'remote-repo'. "git+ssh://git.example.org//users/gioele/projects" insteadOf "remote-repo" You can use "remote-repo:" instead. Do you mean I

[PATCH] diff: add --ignore-blank-lines option

2013-05-26 Thread Antoine Pelisse
The goal of the patch is to introduce the GNU diff -B/--ignore-blank-lines as closely as possible. The short option is not available because it's already used for "break-rewrites". When this option is used, git-diff will not create hunks that simply adds or removes empty lines, but will still show

Re: git clone does not understand insteadOf URLs

2013-05-26 Thread Andreas Schwab
Gioele Barabucci writes: > Simple, I keep all my projects on the same server, so I would like to > refer to that server + path using 'remote-repo'. > > "git+ssh://git.example.org//users/gioele/projects" insteadOf "remote-repo" You can use "remote-repo:" instead. Andreas. -- Andreas Schwab, sc

Re: git clone does not understand insteadOf URLs

2013-05-26 Thread Gioele Barabucci
Il 26/05/2013 17:13, Elia Pinto ha scritto: >> $ git clone remote-repo/ProjectA.git >> fatal repository 'remote-repo/ProjectA.git' does not exist > Why someone should be do something like this ? What is the use case ? Simple, I keep all my projects on the same server, so I would lik

Re: fix french translation

2013-05-26 Thread Antoine Pelisse
Hi, Patch should be inlined, please have a look at `Documentation/SubmittingPatches`. Also, how is "copy" (an english word) better than "copie" (the literal french translation) ? On Sun, May 26, 2013 at 5:27 PM, 乙酸鋰 wrote: > see patch -- To unsubscribe from this list: send the line "unsubscribe

Re: [PATCH] difftool --dir-diff: copy back all files matching the working tree

2013-05-26 Thread John Keeping
On Mon, May 27, 2013 at 12:00:46AM +0900, Kenichi Saita wrote: > After running the user's diff tool, "git difftool --dir-dif --no-symlink" > currently copied back a temporary file to working tree only when a file > contains unstaged changes in the working tree. > > Change this behavior so that tem

fix french translation

2013-05-26 Thread 乙酸鋰
see patch git-gui-french.patch Description: Binary data

Re: git clone does not understand insteadOf URLs

2013-05-26 Thread Elia Pinto
Why someone should be do something like this ? What is the use case ? Best regards 2013/5/25, Gioele Barabucci : > Hello, > > it seems that `git clone` does not understand keywords used `insteadOf` > longer URLs. > > $ git clone remote-repo/ProjectA.git > fatal repository 'remote-repo/P

[PATCH] difftool --dir-diff: copy back all files matching the working tree

2013-05-26 Thread Kenichi Saita
After running the user's diff tool, "git difftool --dir-dif --no-symlink" currently copied back a temporary file to working tree only when a file contains unstaged changes in the working tree. Change this behavior so that temporary files are copied back to working tree whenever the right-hand side

[PATCH 6/6] t/send-email: test suppress-cc=self with non-ascii

2013-05-26 Thread Michael S. Tsirkin
test suppress-cc=self when sender is non-acsii Signed-off-by: Michael S. Tsirkin --- t/t9001-send-email.sh | 4 1 file changed, 4 insertions(+) diff --git a/t/t9001-send-email.sh b/t/t9001-send-email.sh index 0ab4056..66ebb1e 100755 --- a/t/t9001-send-email.sh +++ b/t/t9001-send-email.sh @

[PATCH 5/6] t/send-email: add test with quoted sender

2013-05-26 Thread Michael S. Tsirkin
add test where sender address needs to be quoted. Make sure --suppress-cc=self works well in this case. Signed-off-by: Michael S. Tsirkin --- t/t9001-send-email.sh | 4 1 file changed, 4 insertions(+) diff --git a/t/t9001-send-email.sh b/t/t9001-send-email.sh index 8aa55f8..0ab4056 100755

[PATCH 3/6] t/send-email: test suppress-cc=self on cccmd

2013-05-26 Thread Michael S. Tsirkin
Check that suppress-cc=self works when applied to output of cccmd. Signed-off-by: Michael S. Tsirkin --- t/t9001-send-email.sh | 50 +- 1 file changed, 33 insertions(+), 17 deletions(-) diff --git a/t/t9001-send-email.sh b/t/t9001-send-email.sh in

[PATCH 4/6] send-email: make --suppress-cc=self sanitize input

2013-05-26 Thread Michael S. Tsirkin
--suppress-cc=self fails to filter sender address in many cases where it needs to be sanitized in some way, for example quoted: "A U. Thor" To fix, make send-email sanitize both sender and the address it is compared against. Signed-off-by: Michael S. Tsirkin --- git-send-email.perl | 18 +++

[PATCH 2/6] send-email: fix suppress-cc=self on cccmd

2013-05-26 Thread Michael S. Tsirkin
When cccmd is used, old-style suppress-from filter is applied by the newer suppress-cc=self isn't. Fix this up. Signed-off-by: Michael S. Tsirkin --- git-send-email.perl | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/git-send-email.perl b/git-send-email.perl index bd13cc8..a

[PATCH 1/6] t/send-email.sh: add test for suppress self

2013-05-26 Thread Michael S. Tsirkin
Signed-off-by: Michael S. Tsirkin --- t/t9001-send-email.sh | 41 + 1 file changed, 41 insertions(+) diff --git a/t/t9001-send-email.sh b/t/t9001-send-email.sh index ebd5c5d..36ecf73 100755 --- a/t/t9001-send-email.sh +++ b/t/t9001-send-email.sh @@ -171,6

[PATCH 0/6] git send-email suppress-cc=self fixes

2013-05-26 Thread Michael S. Tsirkin
This includes bugfixes related to handling of --suppress-cc=self flag. Tests are also included. -- MST -- 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: GIT Architecture

2013-05-26 Thread Duy Nguyen
On Sun, May 26, 2013 at 12:36 PM, Rafael Abraão wrote: > Hello. I´m postgraduate student in distributed computing and I'm searching on > git. > > I would like to know is there any website where I can find the main > components and connectors of the architecture of GIT. Is there any website > wh