Re: [PATCH 0/5] fix up 'jk/pack-bitmap' branch

2013-11-09 Thread Jeff King
On Sat, Nov 09, 2013 at 10:03:56PM +0100, Thomas Rast wrote: > > Of course. You write little endian and also read back little endian; > > that should work just fine, no? > > > > OTOH, if you write with Intel and read with PPC, then you should observe > > misbehavior with the above patch. > > Mayb

Re: [PATCH 0/5] fix up 'jk/pack-bitmap' branch

2013-11-09 Thread Thomas Rast
Johannes Sixt writes: > Am 09.11.2013 12:35, schrieb Torsten Bögershausen: >> >> If I do like this in compat/bswap.h >> >> # define ntohll(n) (n) >> # define htonll(n) (n) >> (on an Intel processor, little endian) >> >> then t5310 passes, even if the uint64_t words are written >> in little endi

Re: [PATCH] checkout: most of the time we have good leading directories

2013-11-09 Thread Junio C Hamano
Thomas Rast writes: > Junio C Hamano writes: > >> When "git checkout" wants to create a path, e.g. a/b/c/d/e, after >> seeing if the entire thing already exists (in which case we check if >> that is up-to-date and do not bother to check it out, or we unlink >> and recreate it), we validate that

Re: [PATCH 0/5] fix up 'jk/pack-bitmap' branch

2013-11-09 Thread Johannes Sixt
Am 09.11.2013 12:35, schrieb Torsten Bögershausen: > On 2013-11-08 23.29, Jeff King wrote: >> On Fri, Nov 08, 2013 at 06:10:30PM +0100, Torsten Bögershausen wrote: >> >>> Side question: >>> Do we have enough test coverage for htonll()/ntohll(), >>> or do we want do the "module test" which I send a

Re: [PATCH] l10n: de.po: improve error message when pushing to unknown upstream

2013-11-09 Thread Thomas Rast
Ralf Thielow writes: > Signed-off-by: Ralf Thielow > --- > po/de.po | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/po/de.po b/po/de.po > index a005dcc..8b824cc 100644 > --- a/po/de.po > +++ b/po/de.po > @@ -7923,7 +7923,7 @@ msgid "" > "to update which remote branch."

Re: [PATCH 00/86] replace prefixcmp() with has_prefix()

2013-11-09 Thread Thomas Rast
Christian Couder writes: > Christian Couder (86): > strbuf: add has_prefix() to be used instead of prefixcmp() > diff: replace prefixcmd() with has_prefix() > fast-import: replace prefixcmd() with has_prefix() [...] > builtin/update-ref: replace prefixcmd() with has_prefix() > builtin/u

Re: [PATCH] checkout: most of the time we have good leading directories

2013-11-09 Thread Thomas Rast
Junio C Hamano writes: > When "git checkout" wants to create a path, e.g. a/b/c/d/e, after > seeing if the entire thing already exists (in which case we check if > that is up-to-date and do not bother to check it out, or we unlink > and recreate it), we validate that the leading directory path is

Re: [PATCHv2] l10n: de.po: translate 68 new messages

2013-11-09 Thread Thomas Rast
Ralf Thielow writes: > #: remote.c:1837 > -#, fuzzy > msgid " (use \"git branch --unset-upstream\" to fixup)\n" > -msgstr "git branch --set-upstream-to %s\n" > +msgstr "(benutzen Sie \"git branch --unset-upstream\" zum Beheben)\n" Minor nit: the indent (it's simply a continuation inde

Re: [PATCH 0/5] fix up 'jk/pack-bitmap' branch

2013-11-09 Thread Torsten Bögershausen
On 2013-11-08 23.29, Jeff King wrote: > On Fri, Nov 08, 2013 at 06:10:30PM +0100, Torsten Bögershausen wrote: > >> Side question: >> Do we have enough test coverage for htonll()/ntohll(), >> or do we want do the "module test" which I send a couple of days before ? > > The series adds tests for bu

Re: [PATCH] git-remote-mediawiki: do not remove installed files in "clean" target

2013-11-09 Thread Matthieu Moy
Jonathan Nieder writes: > Running "make clean" after a successful "make install" should not > result in a broken mediawiki remote helper. Acked-by: Matthieu Moy Ideally, there could be a "make uninstall" removing the installed file. -- Matthieu Moy http://www-verimag.imag.fr/~moy/ -- To unsu

Re: [PATCH] po/TEAMS: update Thomas Rast's email address

2013-11-09 Thread Thomas Rast
Ralf Thielow writes: > Signed-off-by: Ralf Thielow > --- > po/TEAMS | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/po/TEAMS b/po/TEAMS > index 107aa59..f9a99ed 100644 > --- a/po/TEAMS > +++ b/po/TEAMS > @@ -8,7 +8,7 @@ Leader: Byrial Jensen > Language:

[PATCH] git-send-email: Added the ability to query the number of smtp password questions

2013-11-09 Thread silvio
From: Silvio F Signed-off-by: Silvio F --- Documentation/git-send-email.txt | 4 git-send-email.perl | 32 +--- 2 files changed, 25 insertions(+), 11 deletions(-) diff --git a/Documentation/git-send-email.txt b/Documentation/git-send-email.txt ind

Re: [PATCH 2/4] Emphasize options and force ASCIIDOC escaping of "--"

2013-11-09 Thread Johannes Sixt
Am 09.11.2013 01:48, schrieb Jason St. John: > -`--since=` limits to commits newer than ``, and using it > -with `--grep=` further limits to commits whose log message > +'\--since=' limits to commits newer than ``, and using it > +with '\--grep=' further limits to commits whose log message I don't