Re: [PATCH] bundle, fast-import: detect write failure

!MAILaRCHIVE_VOTE_RePLACE
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: Jim Meyering <jim@...>
Cc: git list <git@...>
Date: Thursday, January 10, 2008 - 8:37 am

Hi,

On Thu, 10 Jan 2008, Jim Meyering wrote:


Are you aware what this code does?  It writes a ".keep" file.  Whose 
purpose is to _exist_, and whose purpose is fulfilled, even if the write 
or the push-back did not succeed.

I could not care less what the manual says.  What is important is if the 
defensive programming is done mindlessly, and therefore can fail so not 
gracefully.

Ciao,
Dscho
-
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[PATCH] bundle, fast-import: detect write failure, Jim Meyering, (Thu Jan 10, 4:54 am)
Re: [PATCH] bundle, fast-import: detect write failure, Johannes Schindelin, (Thu Jan 10, 8:05 am)
Re: [PATCH] bundle, fast-import: detect write failure, Jim Meyering, (Thu Jan 10, 8:26 am)
Re: [PATCH] bundle, fast-import: detect write failure, Johannes Schindelin, (Thu Jan 10, 8:37 am)
Re: [PATCH] bundle, fast-import: detect write failure, Jim Meyering, (Thu Jan 10, 9:00 am)
Re: [PATCH] bundle, fast-import: detect write failure, Junio C Hamano, (Fri Jan 11, 3:36 am)
Re: [PATCH] bundle, fast-import: detect write failure, Johannes Schindelin, (Fri Jan 11, 7:39 am)
Re: [PATCH] bundle, fast-import: detect write failure, David Tweed, (Fri Jan 11, 5:37 am)
Re: [PATCH] bundle, fast-import: detect write failure, Pierre Habouzit, (Thu Jan 10, 12:25 pm)
Re: [PATCH] bundle, fast-import: detect write failure, Jim Meyering, (Thu Jan 10, 2:05 pm)
Re: [PATCH] bundle, fast-import: detect write failure, Pierre Habouzit, (Thu Jan 10, 2:18 pm)
Re: [PATCH] bundle, fast-import: detect write failure, Pierre Habouzit, (Thu Jan 10, 5:17 am)