Ron <ron@debian.org> (2014-09-21): > On Sat, Sep 20, 2014 at 02:28:24PM -0700, Chris Kuehl wrote: > > Thanks for your work on this -- we're also experiencing this bug in > > both jessie and sid daily netboot images in our PXE boot > > environment. > > > > On Thu, Aug 07, 2014 at 01:46:06PM +0200, Cyril Brulebois wrote: > > > I could apply it blindly but it'd be nice if someone else would > > > confirm it works fine. I've got other things cooking, but I might > > > end up testing it myself it nobody steps up. > > > > I might be building the image wrong, > > No, it looks like you did everything right afaics :) > > That was a blind and untested patch, working backward from what I'd > hand-hacked in place to get it working for me, at the end of a long > day, and the first time I'd ever even looked at this code ... > > It looks like I read $(TEMP_NETBOOT_DIR)/$(BOOT_SCREEN_DIR) wrong in > the surrounding code and assumed BOOT_SCREEN_DIR only had the > "/boot-screens" part in it. Or at least that's the only thing I can > think I must have done, since now that you've pointed, and I'm looking > at it again, your version of that link definitely looks like the right > one to me. > > Indeed amd64/netboot.cfg has: > BOOT_SCREEN_DIR = $(NETBOOT_PATH)/boot-screens/ > > Only the ia64 version appears to define or use it how I used it there. > > Thanks for actually testing this and sending a fixed patch! Looks like I could just blindly apply both patches, squashing them and crediting both of you then? It was low-priority (especially since untested) and I wasn't too keen on spending time on it, but if we have success reports, merging looks like a good idea. Mraw, KiBi.
Attachment:
signature.asc
Description: Digital signature