{"thread":{"id":"30935","subject":"bug: \"fatal: cannot pread pack file\", version 1.7.5.4","startedAt":"2012-07-02T19:03:13Z","lastAt":"2012-08-17T03:02:27Z","messageCount":15,"participants":["Lars Winterfeld","Junio C Hamano","Jeff King","Andreas Schwab","Nguyen Thai Ngoc Duy"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"194468","messageId":"5638dec6e156f83385a3baa01cf9ac4d@localhost","threadId":"30935","inReplyTo":null,"subject":"bug: \"fatal: cannot pread pack file\", version 1.7.5.4","fromName":"Lars Winterfeld","fromEmail":"lars.winterfeld@tu-ilmenau.de","sentAt":"2012-07-02T19:03:13Z","receivedAt":"2012-07-02T19:03:13Z","isPatch":false,"sender":{"key":"lars.winterfeld@tu-ilmenau.de","avatar":"https://gravatar.com/avatar/62c7f184072dc9656eedc7ede399a337fd2a566eada1dc9c740f4ae905e76ef5?d=mp&s=160"},"body":"Hi,\n\nI have used git for some time and regularly pushed changes to another \nserver (as some kind of backup), but today I got this error during `git \npush`:\n\nCounting objects: 1293, done.\nCompressing objects: 100% (1027/1027), done.\nWriting objects: 100% (1229/1229), 70.83 MiB | 364 KiB/s, done.\nTotal 1229 (delta 343), reused 0 (delta 0)\nfatal: cannot pread pack file: No such file or directory\nerror: unpack failed: index-pack abnormal exit\nTo user@host:~/path/\n ! [remote rejected] master -> master (n/a (unpacker error))\nerror: failed to push some refs to 'user@host:~/path/'\n\nI found similar bug reports suggesting to upgrade too version 1.7.x, \nbut I am already using git version 1.7.5.4. Can somebody explain to me \nwhat this error is about? What can I do to resolve it / is there a \nworkaround?\n\n(There is enough disk space on the other server.)\n\nLars\n"},{"id":"194477","messageId":"7v3959zvkx.fsf@alter.siamese.dyndns.org","threadId":"30935","inReplyTo":"5638dec6e156f83385a3baa01cf9ac4d@localhost","subject":"Re: bug: \"fatal: cannot pread pack file\", version 1.7.5.4","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2012-07-02T21:45:02Z","receivedAt":"2012-07-02T21:45:02Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Lars Winterfeld writes:\n\n> Hi,\n>\n> I have used git for some time and regularly pushed changes to another\n> server (as some kind of backup), but today I got this error during\n> git push`:\n>\n> Counting objects: 1293, done.\n> Compressing objects: 100% (1027/1027), done.\n> Writing objects: 100% (1229/1229), 70.83 MiB | 364 KiB/s, done.\n> Total 1229 (delta 343), reused 0 (delta 0)\n> fatal: cannot pread pack file: No such file or directory\n\nCurious. That is typically given for ENOENT but pread(2) would not\ngive that error according to\n\n http://pubs.opengroup.org/onlinepubs/9699919799/functions/pread.html\n\nYou didn't say what platform you are seeing the issue (the receiving\nend of the \"push\"), but are you using Git on a pread-deficient\nplatform with emulation perhaps?\n"},{"id":"194478","messageId":"20120702215736.GA5790@sigill.intra.peff.net","threadId":"30935","inReplyTo":"5638dec6e156f83385a3baa01cf9ac4d@localhost","subject":"Re: bug: \"fatal: cannot pread pack file\", version 1.7.5.4","fromName":"Jeff King","fromEmail":"peff@peff.net","sentAt":"2012-07-02T21:57:36Z","receivedAt":"2012-07-02T21:57:36Z","isPatch":false,"sender":{"key":"peff@peff.net","avatar":"https://avatars.githubusercontent.com/u/45925?v=4"},"body":"On Mon, Jul 02, 2012 at 12:03:13PM -0700, Lars Winterfeld wrote:\n\n> I have used git for some time and regularly pushed changes to another\n> server (as some kind of backup), but today I got this error during\n> `git push`:\n> \n> Counting objects: 1293, done.\n> Compressing objects: 100% (1027/1027), done.\n> Writing objects: 100% (1229/1229), 70.83 MiB | 364 KiB/s, done.\n> Total 1229 (delta 343), reused 0 (delta 0)\n> fatal: cannot pread pack file: No such file or directory\n> error: unpack failed: index-pack abnormal exit\n> To user@host:~/path/\n> ! [remote rejected] master -> master (n/a (unpacker error))\n> error: failed to push some refs to 'user@host:~/path/'\n> \n> I found similar bug reports suggesting to upgrade too version 1.7.x,\n> but I am already using git version 1.7.5.4. Can somebody explain to\n> me what this error is about? What can I do to resolve it / is there a\n> workaround?\n\nIt's very odd for pread to report ENOENT (since it is always operating\non an already-opened file descriptor). Older versions of git would\nsometimes sometimes report errno even when we get a return of 0 (when\nerrno is bogus, and they should say \"truncated file\" or similar), but\nthat was long ago fixed in fb74243 (Do not use errno when pread()\nreturns 0, 2008-10-06). There was also an actual bug fixed by a91d49c\n(index-pack: Loop over pread until data loading is complete.,\n2007-02-27) that I think could also generate this error.\n\nWhat OS is the server running? Is it using NFS? Are you sure that the\nserver version is also v1.7.5.4?\n\n-Peff\n"},{"id":"194501","messageId":"m2vci5n5r5.fsf@igel.home","threadId":"30935","inReplyTo":"20120702215736.GA5790@sigill.intra.peff.net","subject":"Re: bug: \"fatal: cannot pread pack file\", version 1.7.5.4","fromName":"Andreas Schwab","fromEmail":"schwab@linux-m68k.org","sentAt":"2012-07-02T22:43:42Z","receivedAt":"2012-07-02T22:43:42Z","isPatch":false,"sender":{"key":"schwab@linux-m68k.org","avatar":"https://avatars.githubusercontent.com/u/2175493?v=4"},"body":"Jeff King writes:\n\n> It's very odd for pread to report ENOENT (since it is always operating\n> on an already-opened file descriptor).\n\nIt doesn't, but gettext will clobber errno:\n\n\t\tn = pread(pack_fd, inbuf, n, from);\n\t\tif (n < 0)\n\t\t\tdie_errno(_(\"cannot pread pack file\"));\n\nThere is nothing that saves errno. This isn't limited to i18n though,\nany function call in the arguments may potentially clobber errno.\n\nAndreas.\n\n-- \nAndreas Schwab, schwab@linux-m68k.org\nGPG Key fingerprint = 58CA 54C7 6D53 942B 1756 01D3 44D5 214B 8276 4ED5\n\"And now for something completely different.\"\n"},{"id":"194503","messageId":"7v1uktyb7n.fsf@alter.siamese.dyndns.org","threadId":"30935","inReplyTo":"m2vci5n5r5.fsf@igel.home","subject":"Re: bug: \"fatal: cannot pread pack file\", version 1.7.5.4","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2012-07-02T23:50:20Z","receivedAt":"2012-07-02T23:50:20Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Andreas Schwab writes:\n\n> Jeff King writes:\n>\n>> It's very odd for pread to report ENOENT (since it is always operating\n>> on an already-opened file descriptor).\n>\n> It doesn't, but gettext will clobber errno:\n>\n> \t\tn = pread(pack_fd, inbuf, n, from);\n> \t\tif (n < 0)\n> \t\t\tdie_errno(_(\"cannot pread pack file\"));\n>\n> There is nothing that saves errno. This isn't limited to i18n though,\n> any function call in the arguments may potentially clobber errno.\n\nArgh...\n"},{"id":"194507","messageId":"20120703032345.GA7143@sigill.intra.peff.net","threadId":"30935","inReplyTo":"m2vci5n5r5.fsf@igel.home","subject":"Re: bug: \"fatal: cannot pread pack file\", version 1.7.5.4","fromName":"Jeff King","fromEmail":"peff@peff.net","sentAt":"2012-07-03T03:45:03Z","receivedAt":"2012-07-03T03:45:03Z","isPatch":false,"sender":{"key":"peff@peff.net","avatar":"https://avatars.githubusercontent.com/u/45925?v=4"},"body":"On Tue, Jul 03, 2012 at 12:43:42AM +0200, Andreas Schwab wrote:\n\n> Jeff King writes:\n> \n> > It's very odd for pread to report ENOENT (since it is always operating\n> > on an already-opened file descriptor).\n> \n> It doesn't, but gettext will clobber errno:\n> \n> \t\tn = pread(pack_fd, inbuf, n, from);\n> \t\tif (n < 0)\n> \t\t\tdie_errno(_(\"cannot pread pack file\"));\n> \n> There is nothing that saves errno. This isn't limited to i18n though,\n> any function call in the arguments may potentially clobber errno.\n\nThat's horribly lame of gettext. I don't expect arbitrary functions to\nsave errno, but when the entire purpose of a function is to be a\nnon-intrusive wrapper to massage messages to the user, it seems kind of\nevil to overwrite errno. Isn't the whole point of calling it \"_\" that\nyou don't want to or have to notice it?\n\nCan we do something like this to get around it?\n\ndiff --git a/gettext.h b/gettext.h\nindex 57ba8bb..b7c3ae5 100644\n--- a/gettext.h\n+++ b/gettext.h\n@@ -44,7 +44,10 @@ extern int use_gettext_poison(void);\n \n static inline FORMAT_PRESERVING(1) const char *_(const char *msgid)\n {\n-\treturn use_gettext_poison() ? \"# GETTEXT POISON #\" : gettext(msgid);\n+\tint saved_errno = errno;\n+\tconst char *r = use_gettext_poison() ? \"# GETTEXT POISON #\" : gettext(msgid);\n+\terrno = saved_errno;\n+\treturn r;\n }\n \n static inline FORMAT_PRESERVING(1) FORMAT_PRESERVING(2)\n"},{"id":"194509","messageId":"CACsJy8AmCo82YxHA_6+rkbuJOjOG-HqCgjoP2tz753SXH5w8_Q@mail.gmail.com","threadId":"30935","inReplyTo":"20120703032345.GA7143@sigill.intra.peff.net","subject":"Re: bug: \"fatal: cannot pread pack file\", version 1.7.5.4","fromName":"Nguyen Thai Ngoc Duy","fromEmail":"pclouds@gmail.com","sentAt":"2012-07-03T04:25:16Z","receivedAt":"2012-07-03T04:25:16Z","isPatch":false,"sender":{"key":"pclouds@gmail.com","avatar":"https://avatars.githubusercontent.com/u/720?v=4"},"body":"On Tue, Jul 3, 2012 at 10:45 AM, Jeff King wrote:\n> On Tue, Jul 03, 2012 at 12:43:42AM +0200, Andreas Schwab wrote:\n>\n>> Jeff King writes:\n>>\n>> > It's very odd for pread to report ENOENT (since it is always operating\n>> > on an already-opened file descriptor).\n>>\n>> It doesn't, but gettext will clobber errno:\n>>\n>>               n = pread(pack_fd, inbuf, n, from);\n>>               if (n < 0)\n>>                       die_errno(_(\"cannot pread pack file\"));\n>>\n>> There is nothing that saves errno.  This isn't limited to i18n though,\n>> any function call in the arguments may potentially clobber errno.\n>\n> That's horribly lame of gettext. I don't expect arbitrary functions to\n> save errno, but when the entire purpose of a function is to be a\n> non-intrusive wrapper to massage messages to the user, it seems kind of\n> evil to overwrite errno. Isn't the whole point of calling it \"_\" that\n> you don't want to or have to notice it?\n\nAgreed.\n\n> Can we do something like this to get around it?\n>\n> diff --git a/gettext.h b/gettext.h\n> index 57ba8bb..b7c3ae5 100644\n> --- a/gettext.h\n> +++ b/gettext.h\n> @@ -44,7 +44,10 @@ extern int use_gettext_poison(void);\n>\n>  static inline FORMAT_PRESERVING(1) const char *_(const char *msgid)\n>  {\n> -       return use_gettext_poison() ? \"# GETTEXT POISON #\" : gettext(msgid);\n> +       int saved_errno = errno;\n> +       const char *r = use_gettext_poison() ? \"# GETTEXT POISON #\" : gettext(msgid);\n> +       errno = saved_errno;\n> +       return r;\n>  }\n>\n>  static inline FORMAT_PRESERVING(1) FORMAT_PRESERVING(2)\n\nThe last line belongs to Q_(), which needs the same treatment.\n-- \nDuy\n"},{"id":"194510","messageId":"20120703051426.GA12965@sigill.intra.peff.net","threadId":"30935","inReplyTo":"CACsJy8AmCo82YxHA_6+rkbuJOjOG-HqCgjoP2tz753SXH5w8_Q@mail.gmail.com","subject":"Re: bug: \"fatal: cannot pread pack file\", version 1.7.5.4","fromName":"Jeff King","fromEmail":"peff@peff.net","sentAt":"2012-07-03T05:14:26Z","receivedAt":"2012-07-03T05:14:26Z","isPatch":false,"sender":{"key":"peff@peff.net","avatar":"https://avatars.githubusercontent.com/u/45925?v=4"},"body":"On Tue, Jul 03, 2012 at 11:25:16AM +0700, Nguyen Thai Ngoc Duy wrote:\n\n> On Tue, Jul 3, 2012 at 10:45 AM, Jeff King wrote:\n> > On Tue, Jul 03, 2012 at 12:43:42AM +0200, Andreas Schwab wrote:\n> >\n> >> Jeff King writes:\n> >>\n> >> > It's very odd for pread to report ENOENT (since it is always operating\n> >> > on an already-opened file descriptor).\n> >>\n> >> It doesn't, but gettext will clobber errno:\n> >>\n> >>               n = pread(pack_fd, inbuf, n, from);\n> >>               if (n < 0)\n> >>                       die_errno(_(\"cannot pread pack file\"));\n> >>\n> >> There is nothing that saves errno.  This isn't limited to i18n though,\n> >> any function call in the arguments may potentially clobber errno.\n> >\n> > That's horribly lame of gettext. I don't expect arbitrary functions to\n> > save errno, but when the entire purpose of a function is to be a\n> > non-intrusive wrapper to massage messages to the user, it seems kind of\n> > evil to overwrite errno. Isn't the whole point of calling it \"_\" that\n> > you don't want to or have to notice it?\n> \n> Agreed.\n\nHmm. According to gettext(3):\n\n ERRORS\n errno is not modified.\n\nAnd googling for \"gettext\" and \"errno\" reveals several bugfixes in GNU\ngettext to make sure that gettext preserves errno. I wonder if there are\nsystems where that is not the case, though; I don't know what non-GNU\ngettext implementations are in common use these days. I'd still be\ncurious to hear what platform the server is for this bug report.\n\n-Peff\n"},{"id":"194532","messageId":"7vobnwwyrl.fsf@alter.siamese.dyndns.org","threadId":"30935","inReplyTo":"20120703051426.GA12965@sigill.intra.peff.net","subject":"Re: bug: \"fatal: cannot pread pack file\", version 1.7.5.4","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2012-07-03T17:16:46Z","receivedAt":"2012-07-03T17:16:46Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Jeff King writes:\n\n> Hmm. According to gettext(3):\n>\n> ERRORS\n> errno is not modified.\n>\n> And googling for \"gettext\" and \"errno\" reveals several bugfixes in GNU\n> gettext to make sure that gettext preserves errno. I wonder if there are\n> systems where that is not the case, though; I don't know what non-GNU\n> gettext implementations are in common use these days. I'd still be\n> curious to hear what platform the server is for this bug report.\n\nYes, I am also curious.\n\nRegardless, we may need your \"save_errno\" patch if the bulk of it\nmay have to be made conditional on NEED_SAVE_ERRNO_IN_GETTEXT=YesPlease\nor something.\n\nThanks for digging.\n"},{"id":"197123","messageId":"7vvcgizesm.fsf@alter.siamese.dyndns.org","threadId":"30935","inReplyTo":"20120703051426.GA12965@sigill.intra.peff.net","subject":"Re: bug: \"fatal: cannot pread pack file\", version 1.7.5.4","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2012-08-16T21:51:21Z","receivedAt":"2012-08-16T21:51:21Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Jeff King writes:\n\n> On Tue, Jul 03, 2012 at 11:25:16AM +0700, Nguyen Thai Ngoc Duy wrote:\n>\n>> On Tue, Jul 3, 2012 at 10:45 AM, Jeff King wrote:\n>> > On Tue, Jul 03, 2012 at 12:43:42AM +0200, Andreas Schwab wrote:\n>> >\n>> >> Jeff King writes:\n>> >>\n>> >> > It's very odd for pread to report ENOENT (since it is always operating\n>> >> > on an already-opened file descriptor).\n>> >>\n>> >> It doesn't, but gettext will clobber errno:\n>> >>\n>> >> n = pread(pack_fd, inbuf, n, from);\n>> >> if (n < 0)\n>> >> die_errno(_(\"cannot pread pack file\"));\n>> >>\n>> >> There is nothing that saves errno. This isn't limited to i18n though,\n>> >> any function call in the arguments may potentially clobber errno.\n>> >\n>> > That's horribly lame of gettext. I don't expect arbitrary functions to\n>> > save errno, but when the entire purpose of a function is to be a\n>> > non-intrusive wrapper to massage messages to the user, it seems kind of\n>> > evil to overwrite errno. Isn't the whole point of calling it \"_\" that\n>> > you don't want to or have to notice it?\n>> \n>> Agreed.\n>\n> Hmm. According to gettext(3):\n>\n> ERRORS\n> errno is not modified.\n>\n> And googling for \"gettext\" and \"errno\" reveals several bugfixes in GNU\n> gettext to make sure that gettext preserves errno. I wonder if there are\n> systems where that is not the case, though; I don't know what non-GNU\n> gettext implementations are in common use these days. I'd still be\n> curious to hear what platform the server is for this bug report.\n\nHrm, has this ever been resolved?\n"},{"id":"197128","messageId":"fd74d7233b4b29fe06afad23fb87552a@localhost","threadId":"30935","inReplyTo":"7vvcgizesm.fsf@alter.siamese.dyndns.org","subject":"Re: bug: \"fatal: cannot pread pack file\", version 1.7.5.4","fromName":"Lars Winterfeld","fromEmail":"lars.winterfeld@tu-ilmenau.de","sentAt":"2012-08-16T22:10:42Z","receivedAt":"2012-08-16T22:10:42Z","isPatch":false,"sender":{"key":"lars.winterfeld@tu-ilmenau.de","avatar":"https://gravatar.com/avatar/62c7f184072dc9656eedc7ede399a337fd2a566eada1dc9c740f4ae905e76ef5?d=mp&s=160"},"body":"Junio C Hamano:\n> Jeff King writes:\n> \n>> On Tue, Jul 03, 2012 at 11:25:16AM +0700, Nguyen Thai Ngoc Duy wrote:\n>>\n>>> On Tue, Jul 3, 2012 at 10:45 AM, Jeff King wrote:\n>>> > On Tue, Jul 03, 2012 at 12:43:42AM +0200, Andreas Schwab wrote:\n>>> >\n>>> >> Jeff King writes:\n>>> >>\n>>> >> > It's very odd for pread to report ENOENT (since it is always operating\n>>> >> > on an already-opened file descriptor).\n>>> >>\n>>> >> It doesn't, but gettext will clobber errno:\n>>> >>\n>>> >> n = pread(pack_fd, inbuf, n, from);\n>>> >> if (n < 0)\n>>> >> die_errno(_(\"cannot pread pack file\"));\n>>> >>\n>>> >> There is nothing that saves errno. This isn't limited to i18n though,\n>>> >> any function call in the arguments may potentially clobber errno.\n>>> >\n>>> > That's horribly lame of gettext. I don't expect arbitrary functions to\n>>> > save errno, but when the entire purpose of a function is to be a\n>>> > non-intrusive wrapper to massage messages to the user, it seems kind of\n>>> > evil to overwrite errno. Isn't the whole point of calling it \"_\" that\n>>> > you don't want to or have to notice it?\n>>>\n>>> Agreed.\n>>\n>> Hmm. According to gettext(3):\n>>\n>> ERRORS\n>> errno is not modified.\n>>\n>> And googling for \"gettext\" and \"errno\" reveals several bugfixes in GNU\n>> gettext to make sure that gettext preserves errno. I wonder if there are\n>> systems where that is not the case, though; I don't know what non-GNU\n>> gettext implementations are in common use these days. I'd still be\n>> curious to hear what platform the server is for this bug report.\n> \n> Hrm, has this ever been resolved?\n\nThank you for asking.\nMy local git version is 1.7.5.4, the server version that I actually\nended up pushing the files to was however still some 1.6.x.\nSo it was a false alarm. Sorry about that.\n\nAs far as I can follow, there is no non-GNU gettext involved. Thanks\nfor digging that deep into the problem, but it was my fault, probably\nnot something about lost errno.\n"},{"id":"197133","messageId":"7v4no2zbd9.fsf@alter.siamese.dyndns.org","threadId":"30935","inReplyTo":"fd74d7233b4b29fe06afad23fb87552a@localhost","subject":"Re: bug: \"fatal: cannot pread pack file\", version 1.7.5.4","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2012-08-16T23:05:22Z","receivedAt":"2012-08-16T23:05:22Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Lars Winterfeld writes:\n\n>> Hrm, has this ever been resolved?\n>\n> Thank you for asking.\n> My local git version is 1.7.5.4, the server version that I actually\n> ended up pushing the files to was however still some 1.6.x.\n> So it was a false alarm. Sorry about that.\n\nNot really. Pushing or fetching by a new version of Git against old\nversion of Git *should* work; otherwise it is a bug.\n"},{"id":"197142","messageId":"a6d4278ab75e8d59436f214d8970a79b@localhost","threadId":"30935","inReplyTo":"7v4no2zbd9.fsf@alter.siamese.dyndns.org","subject":"Re: bug: \"fatal: cannot pread pack file\", version 1.7.5.4","fromName":"Lars Winterfeld","fromEmail":"lars.winterfeld@tu-ilmenau.de","sentAt":"2012-08-16T23:36:25Z","receivedAt":"2012-08-16T23:36:25Z","isPatch":false,"sender":{"key":"lars.winterfeld@tu-ilmenau.de","avatar":"https://gravatar.com/avatar/62c7f184072dc9656eedc7ede399a337fd2a566eada1dc9c740f4ae905e76ef5?d=mp&s=160"},"body":"Junio C Hamano schrieb:\n> Lars Winterfeld writes:\n> \n>>> Hrm, has this ever been resolved?\n>>\n>> Thank you for asking.\n>> My local git version is 1.7.5.4, the server version that I actually\n>> ended up pushing the files to was however still some 1.6.x.\n>> So it was a false alarm. Sorry about that.\n> \n> Not really. Pushing or fetching by a new version of Git against old\n> version of Git *should* work; otherwise it is a bug.\n\nWell, okay. The (outdated) server\nruns on openSUSE 11.1,\nhas kernel 2.6.27.39-0.2-default x86_64 GNU/Linux, \nuses git version 1.6.0.2\nand gettext (GNU gettext-runtime) 0.17 .\n\nAs I said earlier, `git push` worked in the beginning and just stopped\nworking one day.\nI can still reproduce this error, so let me know if you want more info.\n"},{"id":"197152","messageId":"20120817014526.GA447@sigill.intra.peff.net","threadId":"30935","inReplyTo":"7v4no2zbd9.fsf@alter.siamese.dyndns.org","subject":"Re: bug: \"fatal: cannot pread pack file\", version 1.7.5.4","fromName":"Jeff King","fromEmail":"peff@peff.net","sentAt":"2012-08-17T01:45:26Z","receivedAt":"2012-08-17T01:45:26Z","isPatch":false,"sender":{"key":"peff@peff.net","avatar":"https://avatars.githubusercontent.com/u/45925?v=4"},"body":"On Thu, Aug 16, 2012 at 04:05:22PM -0700, Junio C Hamano wrote:\n\n> Lars Winterfeld writes:\n> \n> >> Hrm, has this ever been resolved?\n> >\n> > Thank you for asking.\n> > My local git version is 1.7.5.4, the server version that I actually\n> > ended up pushing the files to was however still some 1.6.x.\n> > So it was a false alarm. Sorry about that.\n> \n> Not really. Pushing or fetching by a new version of Git against old\n> version of Git *should* work; otherwise it is a bug.\n\nYes, but there may be old bugs in the server version that have since\nbeen fixed. For example, the bug fixed by 838cd346 might be the cause of\nthis (that fix went into 1.6.0.3).\n\n-Peff\n"},{"id":"197154","messageId":"7vfw7mxlto.fsf@alter.siamese.dyndns.org","threadId":"30935","inReplyTo":"20120817014526.GA447@sigill.intra.peff.net","subject":"Re: bug: \"fatal: cannot pread pack file\", version 1.7.5.4","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2012-08-17T03:02:27Z","receivedAt":"2012-08-17T03:02:27Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Jeff King writes:\n\n> On Thu, Aug 16, 2012 at 04:05:22PM -0700, Junio C Hamano wrote:\n>\n>> Lars Winterfeld writes:\n>> \n>> >> Hrm, has this ever been resolved?\n>> >\n>> > Thank you for asking.\n>> > My local git version is 1.7.5.4, the server version that I actually\n>> > ended up pushing the files to was however still some 1.6.x.\n>> > So it was a false alarm. Sorry about that.\n>> \n>> Not really. Pushing or fetching by a new version of Git against old\n>> version of Git *should* work; otherwise it is a bug.\n>\n> Yes, but there may be old bugs in the server version that have since\n> been fixed. For example, the bug fixed by 838cd346 might be the cause of\n> this (that fix went into 1.6.0.3).\n\nYeah, Lars said 1.6.x above, but in the response it turns out the\nversion used is 1.6.0.2, so lack of that commit is a very likely\nreason.\n\nThanks.\n"}]}