Hi,

On 12/09/13 01:53, David Kalnischkies wrote:
> On Wed, Sep 11, 2013 at 11:57 PM, Steven Chamberlain <ste...@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
ste...@pyro.eu.org


-- 
To UNSUBSCRIBE, email to debian-boot-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/5231163d.4090...@pyro.eu.org

Reply via email to