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

Re: Debian installer build: failed or old builds



Hi,

On 12/09/13 01:53, David Kalnischkies wrote:
> On Wed, Sep 11, 2013 at 11:57 PM, Steven Chamberlain <steven@pyro.eu.org> wrote:
>> Does this change seem odd to you too?  I wondered if it could be causing
>> recent failures of APT to download required debs for d-i builds:
> 
> Looking at: http://d-i.debian.org/daily-images/amd64/daily/build_cdrom_isolinux.log
> The only think I can see is that APT completes successfully and
> whatever comes after that can't find cdebconf-newt-terminal even
> though in the output of APT it is seen that it was downloaded.

The acquire appeared successful in these cases at least - I guess we
need to know exactly what's in the archives dir on the buildds:

http://d-i.debian.org/daily-images/amd64/daily/build_cdrom_gtk.log
> Needed cdebconf-newt-terminal not found (looked in apt.udeb/cache/archives/, debugudebs/)

http://d-i.debian.org/daily-images/amd64/daily/build_netboot-gtk.log
> Needed speakup-modules-3.10-2-amd64-di not found (looked in apt.udeb/cache/archives/, debugudebs/)


But this one is particularly interesting - this is why I suspected an
APT bug such as StoreFilename being empty:

http://d-i.debian.org/daily-images/i386/daily/build_cdrom_gtk.log
> Err http://ftp.gr.debian.org/debian/ unstable/main/debian-installer xserver-xorg-video-fbdev-udeb i386 1:0.4.3-2
>   Could not open file /build/installer-yUDIZW/build/apt.udeb/cache/archives/partial/ - open (21: Is a directory)
> Err http://incoming.debian.org/debian/ unstable/main/debian-installer xserver-xorg-video-fbdev-udeb i386 1:0.4.3-2
>   Could not open file /build/installer-yUDIZW/build/apt.udeb/cache/archives/partial/ - open (21: Is a directory)
> Failed to fetch http://incoming.debian.org/debian/pool/main/x/xserver-xorg-video-fbdev/xserver-xorg-video-fbdev-udeb_0.4.3-2_i386.udeb  Could not open file /build/installer-yUDIZW/build/apt.udeb/cache/archives/partial/ - open (21: Is a directory)

And again on a completely different buildd, using a different mirror:

http://d-i.debian.org/daily-images/kfreebsd-amd64/daily/build_cdrom_gtk.log
> Err http://mirror-ubc.debian.org/debian/ unstable/main/debian-installer cdebconf-newt-udeb kfreebsd-amd64 0.185
>   Could not open file /build/installer-aXrvSw/checkout/build/apt.udeb/cache/archives/partial/ - open (21: Is a directory) [IP: 2607:f8f0:610:4000:6564:a62:ce0c:137e 80]
> Failed to fetch http://mirror-ubc.debian.org/debian/pool/main/c/cdebconf/cdebconf-newt-udeb_0.185_kfreebsd-amd64.udeb  Could not open file /build/installer-aXrvSw/checkout/build/apt.udeb/cache/archives/partial/ - open (21: Is a directory) [IP: 2607:f8f0:610:4000:6564:a62:ce0c:137e 80]


>> On 11/09/13 21:42, Steven Chamberlain wrote:
>>>>> -   for (; Vf.end() == false; Vf++)
>>>>> +   for (; Vf.end() == false; ++Vf)
>>> If that is some kind of iterator won't it now skip over the first
>>> element?  Otherwise how did it ever work before?
> 
> Did you recognize the first ; ?
> The value returned by the post-increase isn't stored or
> used for anything, so we can equally well use the pre-increment
> (as suggested by cppcheck), right?

Sorry, of course, the return value isn't what's used within the loop.

Regards,
-- 
Steven Chamberlain
steven@pyro.eu.org


Reply to: