git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: t5541: Bad file descriptor

From
Jeff King <peff@peff.net>
Date
May 5, 2011, 05:46 UTC
Message-ID
<20110505054611.GA29033@sigill.intra.peff.net>
In-Reply-To
<7voc3hbtgu.fsf@alter.siamese.dyndns.org>
On Wed, May 04, 2011 at 10:35:13PM -0700, Junio C Hamano wrote:
Show 10 quoted lines
> Brian Gernhardt <brian@gernhardtsoftware.com> writes:
> 
> > I haven't had a lot of time to track down it down until today, but I've
> > been getting failures in t5541-http-push-.sh.  Several tests fail with
> > the error "fatal: write error: Bad file descriptor".
> 
> A wild guess.
> 
> Does it help if you cherry-picked 1e41827 (http: clear POSTFIELDS when
> initializing a slot, 2011-04-26) on top of the faulty commit?

I think that 09c9957 (send-pack: avoid deadlock when pack-object dies early, 2011-04-25) is totally broken.

Looking back at my tests, I only tested the case where pack-objects fails. And it seems we totally broke the case where the push is supposed to succeed.

Still investigating...
-Peff
Previous: Junio C HamanoNext: Jeff King
Message 3 of 11 in “t5541: Bad file descriptor”
  1. Brian GernhardtMay 5, 2011
  2. Junio C HamanoMay 5, 2011
  3. Jeff KingMay 5, 2011
  4. Jeff KingMay 5, 2011
  5. Sverre RabbelierMay 5, 2011
  6. Avery PennarunMay 5, 2011
  7. Sverre RabbelierMay 5, 2011
  8. Jeff KingMay 9, 2011
  9. Jeff KingMay 5, 2011
  10. Brian GernhardtMay 5, 2011
  11. Johannes SixtMay 5, 2011

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.