RE: [PATCH 1/2] Support for setitimer() on platforms lacking it

2012-09-03 Thread Joachim Schmitz
From: Junio C Hamano [mailto:gits...@pobox.com] Sent: Sunday, September 02, 2012 10:44 PM To: Joachim Schmitz Cc: git@vger.kernel.org; Johannes Sixt Subject: Re: [PATCH 1/2] Support for setitimer() on platforms lacking it Joachim Schmitz j...@schmitz-digital.de writes: Should we

Re: [PATCH v4] Thunderbird: fix appp.sh format problems

2012-09-03 Thread Marco Stornelli
2012/9/2 Junio C Hamano gits...@pobox.com: Marco Stornelli marco.storne...@gmail.com writes: Il 01/09/2012 15:59, Johannes Sixt ha scritto: Look how you write: perl -e '... $ENV{'PATCHTMP'} ...' That is, perl actually sees this script: ... $ENV{PATCHTMP} ... (no quotes around

Clone to an SSH destination

2012-09-03 Thread Mark Hills
How do I clone a repo _to_ a new repo over SSH? I tried: cd xx git clone --bare . gitserver:/scm/xx.git git clone --bare . ssh://gitserver/scm/xx.git This does not have the expected result, and instead a local path of the given name is created (eg. a 'gitserver:' directory) This seems to

[PATCH] fetch: align new ref summary printout in UTF-8 locales

2012-09-03 Thread Nguyễn Thái Ngọc Duy
fetch does printf(%-*s, width, foo) where foo can be an utf-8 string, but width is bytes, not letters. This results in misaligned ref summary table. Introduce gettext_length() function that returns the string length in letters. Make the code use TRANSPORT_SUMMARY(x) where the length is

Re: Clone to an SSH destination

2012-09-03 Thread Konstantin Khomoutov
On Mon, 3 Sep 2012 11:21:43 +0100 (BST) Mark Hills mark.hi...@framestore.com wrote: How do I clone a repo _to_ a new repo over SSH? I tried: cd xx git clone --bare . gitserver:/scm/xx.git git clone --bare . ssh://gitserver/scm/xx.git This does not have the expected result, and

Re: Clone to an SSH destination

2012-09-03 Thread Mark Hills
On Mon, 3 Sep 2012, Sitaram Chamarty wrote: On Mon, Sep 3, 2012 at 5:17 PM, Konstantin Khomoutov flatw...@users.sourceforge.net wrote: On Mon, 3 Sep 2012 11:21:43 +0100 (BST) Mark Hills mark.hi...@framestore.com wrote: [snip] This is quite cumbersome; we have a large team of devs who

Re: Clone to an SSH destination

2012-09-03 Thread Sitaram Chamarty
On Mon, Sep 3, 2012 at 6:45 PM, Mark Hills mark.hi...@framestore.com wrote: On Mon, 3 Sep 2012, Sitaram Chamarty wrote: On Mon, Sep 3, 2012 at 5:17 PM, Konstantin Khomoutov flatw...@users.sourceforge.net wrote: On Mon, 3 Sep 2012 11:21:43 +0100 (BST) Mark Hills mark.hi...@framestore.com

Re: checkout extra files

2012-09-03 Thread Carlos Martín Nieto
Angelo Borsotti angelo.borso...@gmail.com writes: Hello, the man page of git checkout states: git checkout [-p|--patch] [tree-ish] [--] pathspec... It updates the named paths in the working tree from the index file or from a named tree-ish ... This means that for each file denoted by

Re: checkout extra files

2012-09-03 Thread Nguyen Thai Ngoc Duy
On Mon, Sep 3, 2012 at 8:42 PM, Angelo Borsotti angelo.borso...@gmail.com wrote: $ git checkout 94d8 -- * $ ls f1 Note that the work directory is empty when the checkout is done, and that the checkout restores f1 in it, a file that is not denoted by the * pathspec. I think in this case '*'

Re: Clone to an SSH destination

2012-09-03 Thread Konstantin Khomoutov
On Mon, 3 Sep 2012 14:07:48 +0100 (BST) Mark Hills mark.hi...@framestore.com wrote: [...] But I'm actually more curious about why you need this in the first place, there's a bunch of devs where I work as well, but they never have the need to create new repos on some NFS drive in this

Re: Clone to an SSH destination

2012-09-03 Thread Sitaram Chamarty
On Mon, Sep 3, 2012 at 7:38 PM, Konstantin Khomoutov flatw...@users.sourceforge.net wrote: On Mon, 3 Sep 2012 14:07:48 +0100 (BST) Mark Hills mark.hi...@framestore.com wrote: [...] But I'm actually more curious about why you need this in the first place, there's a bunch of devs where I

Re: checkout extra files

2012-09-03 Thread Carlos Martín Nieto
Angelo Borsotti angelo.borso...@gmail.com writes: [please keep it in the list] Hi Carlos, the behavior is quite clear, but the man pages do not describe it properly. The man pages state: It updates the named paths in the working tree from the index file or from a named tree-ish

Re: [PATCH v4] Thunderbird: fix appp.sh format problems

2012-09-03 Thread Marco Stornelli
Il 02/09/2012 22:42, Junio C Hamano ha scritto: Marco Stornelli marco.storne...@gmail.com writes: Il 01/09/2012 15:59, Johannes Sixt ha scritto: Look how you write: perl -e '... $ENV{'PATCHTMP'} ...' That is, perl actually sees this script: ... $ENV{PATCHTMP} ... (no quotes

[PATCH v6] Thunderbird: fix appp.sh format problems

2012-09-03 Thread Marco Stornelli
The current script has got the following problems: 1) It doesn't work if the language used by Thunderbird is not English; 2) The field To: filled by format-patch is not evaluated; 3) The field Cc: is loaded from Cc used in the commit message instead of using the Cc field filled by format-patch in

Re: warning: There are too many unreachable loose objects; run 'git prune' to remove them.

2012-09-03 Thread Antony Male
On 29/08/2012 22:16, Dun Peal wrote: Hi, I am getting this error every time I pull. All the following have been executed, but failed to remove this warning: git prune git prune --expire now git gc git gc --aggressive What should I do? Was the error prefixed by 'remote:' (i.e. was it an

Re: [PATCH 1/2] Support for setitimer() on platforms lacking it

2012-09-03 Thread Junio C Hamano
Johannes Sixt j...@kdbg.org writes: Am 03.09.2012 11:31, schrieb Joachim Schmitz: Hmm, I see that there the errors are handled differently, like this: if (ovalue != NULL) return errno = EINVAL, error(setitimer param 3 != NULL not

Re: [PATCH 1/2] Support for setitimer() on platforms lacking it

2012-09-03 Thread Junio C Hamano
Joachim Schmitz j...@schmitz-digital.de writes: if (!value ) { Style: space before ')'? Will fix. errno = EFAULT; return -1; EFAULT is good ;-) That's what 'man setitimer()' on Linux says to happen if invalid value is found. The emulation in

Re: [PATCH] fetch: align new ref summary printout in UTF-8 locales

2012-09-03 Thread Junio C Hamano
Nguyễn Thái Ngọc Duy pclo...@gmail.com writes: fetch does printf(%-*s, width, foo) where foo can be an utf-8 string, but width is bytes, not letters. This results in misaligned ref summary table. but width is bytes, not letters is a misleading statement. Be careful about three different

Re: checkout extra files

2012-09-03 Thread Junio C Hamano
Nguyen Thai Ngoc Duy pclo...@gmail.com writes: On Mon, Sep 3, 2012 at 8:42 PM, Angelo Borsotti angelo.borso...@gmail.com wrote: $ git checkout 94d8 -- * $ ls f1 Note that the work directory is empty when the checkout is done, and that the checkout restores f1 in it, a file that is not

Re: [PATCH] fetch: align new ref summary printout in UTF-8 locales

2012-09-03 Thread Johannes Sixt
Am 03.09.2012 21:26, schrieb Junio C Hamano: Nguyễn Thái Ngọc Duy pclo...@gmail.com writes: fetch does printf(%-*s, width, foo) where foo can be an utf-8 string, but width is bytes, not letters. This results in misaligned ref summary table. but width is bytes, not letters is a misleading

RE: [PATCH 1/2] Support for setitimer() on platforms lacking it

2012-09-03 Thread Joachim Schmitz
From: Junio C Hamano [mailto:gits...@pobox.com] Sent: Monday, September 03, 2012 9:03 PM To: Joachim Schmitz Cc: git@vger.kernel.org; 'Johannes Sixt' Subject: Re: [PATCH 1/2] Support for setitimer() on platforms lacking it Joachim Schmitz j...@schmitz-digital.de writes: if (!value )

Re: [PATCH v4] Thunderbird: fix appp.sh format problems

2012-09-03 Thread Junio C Hamano
Marco Stornelli marco.storne...@gmail.com writes: I tried the Johannes's script, but it seems it doesn't work well with the pattern of format-patch (To: mail1,\n mail2,\n mailN). The multilines are not well managed. I am guessing that the reason why Jonahhes's copy our headers out with

Re: [PATCH] fetch: align new ref summary printout in UTF-8 locales

2012-09-03 Thread Junio C Hamano
Johannes Sixt j...@kdbg.org writes: Am 03.09.2012 21:26, schrieb Junio C Hamano: Nguyễn Thái Ngọc Duy pclo...@gmail.com writes: fetch does printf(%-*s, width, foo) where foo can be an utf-8 string, but width is bytes, not letters. This results in misaligned ref summary table. but width

Re: diff/merge tool that ignores whitespace changes

2012-09-03 Thread Jonas Fonseca
On Sun, Sep 2, 2012 at 5:07 PM, Enrico Weigelt enrico.weig...@vnc.biz wrote: Hi, Would that help ? git help diff [snip] --ignore-space-at-eol Ignore changes in whitespace at EOL. -b, --ignore-space-change Ignore changes in amount of whitespace. This

Automatic repacking when cloning - Only seems to work via file but not ssh or git

2012-09-03 Thread haasip satang
Hi all, I already posted this question in the git users groups (https://groups.google.com/forum/?fromgroups=#!topic/git-users/dguTJFrw5MI) but was advised to better ask the experts in this list. So here we go: The question is regarding the packs and the repacking of those during cloning. I'm

Re: checkout extra files

2012-09-03 Thread Nguyen Thai Ngoc Duy
On Tue, Sep 4, 2012 at 2:36 AM, Junio C Hamano gits...@pobox.com wrote: It's confusing but I don't think there's much we can do about it. The user can, by telling the shell to expand '*' that does not match to nothing, though. It works 99% the time, I don't think any users would bother

Re: checkout extra files

2012-09-03 Thread Carlos Martín Nieto
Keep it in the list. Angelo Borsotti angelo.borso...@gmail.com writes: Hi Carlos, That grouping is not what it's saying. It doesn't update the files that exist in the working tree matching some glob. It updates the files in the working tree from either the index or a treeish. The pathspec

Re: checkout extra files

2012-09-03 Thread Junio C Hamano
Nguyen Thai Ngoc Duy pclo...@gmail.com writes: For this particular scenario, I do not see anything offhand that is unclear about the behaviour of Git in the documentation, even though as you pointed out, if the user is unaware that the shell passes globs unmodified when they do not match, it