[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: Fixing 131478 in debootstrap



On Sun, Mar 03, 2002 at 11:32:54PM +0000, Philip Blundell wrote:
> On Sun, 2002-03-03 at 21:54, Chris Tillman wrote:
> > Since everyone's mystified about the Malformed release bug, and we're 
> > NMUing debootstrap, how about applying the patch in 131478? I don't 
> > really understand the issues the bug writer is talking about, but it 
> > sure seems like races and such can be a source for mysterious bugs. 
> > AFAICT the patch doesn't change the functionality. Why not try it? 
> 
> It looks like the piece of code affected by the patch isn't executed
> until after the point at which the "Malformed.." error is generated.  So
> I don't think it will help here, though the patch seems benign enough in
> itself.
> 
> I must admit, I wasn't really aware that this problem was still with us.
> My first inclination would be to attack it by having debootstrap
> redirect wget's stderr to somewhere other than /dev/null, so that in the
> event of download problems the user can at least figure out what's going
> on.  Debootstrap itself probably won't be able to spot failed downloads
> of the Release file because its error checking relies on $PIPESTATUS
> which ash doesn't seem to support.

No _that_ sounds like a root cause.

-- 
*------v--------- Installing Debian GNU/Linux 3.0 --------v------*
|      <http://www.debian.org/releases/woody/installmanual>      |
|   debian-imac (potato): <http://debian-imac.sourceforge.net>   |
|            Chris Tillman        tillman@voicetrak.com          |
|                   May the Source be with you                   |
*----------------------------------------------------------------*



Reply to: