[syslinux] Syslinux-5.10-pre1 looks for ldlinux.c32 in TFTP instead of HTTP

[Tobias Göbel]

Am Freitag, 08.03.2013 13:41, schrieb Matt Fleming:
> On Fri, 2013-03-08 at 13:37 +0100, Tobias Göbel wrote:
>> Am Freitag, 08.03.2013 13:19, schrieb Matt Fleming:
>>> On Fri, 2013-03-08 at 11:50 +0100, Tobias Göbel wrote:
>>>> Ok, i've tested with legacy pxelinux.0 and it's the same.
>>>> I also tested the different pre-releases, and the first version where it
>>>> happens is PRE-3.
>>> Syslinux-5.02-pre3? OK, that narrows it down. Do you have a development
>>> environment setup? The quickest way to track down what change broke your
>>> usecase is to do a git bisect between syslinux-5.02-pre2 and
>>> syslinux-5.02-pre3.
>>>
>>> Is that something you could do?
>>>
>> Never heared about bisect, but a quick googling makes me think i'm
>> possible to do it.
>> i'll give it a shot.. just figured out that i can checkout the taged
>> pre-releases with just checkout syslinux-5.02-pre2... this was driveing
>> me nuts bevore...
>> (previously i just downloaded the tar.gz files -.-)
>>
>> I'll report back with more info hopefully...
> Great! Shout if you get stuck.
>
Got it!

I think it's some sort of win-win situation. Learned something and found 
the commit that causes the error... I think git bisect will be my tool 
of the month :)

Now, back to bussiness:

Seems like the Commit which causes the error is: 
b208ba467f678ed8e73f8d11fc0609634120cb83

Here are the notes i took while testing, and the complete output from 
last git bisect good:

GOOD1 5.02 PRE2-16-gd4ce5b9
GOOD2 5.02 PRE2-24-g4e761f7
GOOD3 5.02 PRE2-28-g69c09a8 Commit -> 
69c09a88f6c46ff139cd5c0316d3eeae508e2b5c
  BAD1                Commit -> b208ba467f678ed8e73f8d11fc0609634120cb83
GOOD4 5.02 PRE2-29-gaa7dd29 Commit -> 
aa7dd29db684d73f044b520e8c148f7ddb8c38d5

# git bisect good
b208ba467f678ed8e73f8d11fc0609634120cb83 is the first bad commit
commit b208ba467f678ed8e73f8d11fc0609634120cb83
Author: Matt Fleming <matt.fleming at intel.com>
Date:   Tue Feb 19 21:09:55 2013 +0000

     pxe: Pass absolute path to pxe_chdir()

     We may have moved out of the root directory when calling get_prefix()
     and so need a way to create an absolute path.

     Historically, in versions prior to 5.xx we would call
     pxe_open_config() while we were still in the root directory ("::"),
     and so, we could construct a relative path to the config file.

     However, because we need to load ldlinux.c32 before opening the config
     file, which may involve us changing directory, we need a way of
     constructing an absolute path to the config file in get_prefix().

     Signed-off-by: Matt Fleming <matt.fleming at intel.com>

:040000 040000 a3fe656d40834a6379d166f9134acb574bbde05a 
d52b7d5448adcea31a5e55a29b48548fa6ee910b M      core

-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/pkcs7-signature
Size: 4477 bytes
Desc: S/MIME Kryptografische Unterschrift
URL: <http://www.zytor.com/pipermail/syslinux/attachments/20130308/fd911dc4/attachment.p7s>