From mboxrd@z Thu Jan  1 00:00:00 1970
From: Nick Peirson <nick@netcentral.co.uk>
Date: Fri, 31 Oct 2008 09:50:39 +0000
Subject: [Buildroot] Patch review
Message-ID: <490AD4EF.1010106@netcentral.co.uk>
List-Id: <buildroot.busybox.net>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: buildroot@busybox.net

Would it be possible for the patch against bug 
http://bugs.uclibc.org/view.php?id=5674 to be reviewed and, hopefully, 
commited to svn?

The patch (attached) bumps the syslinux version and updates the patches 
required for it to compile.

-- 
Thanks

Nick Peirson

--
Internet Central Ltd is a limited company registered in England. 
Registered number: 3079542. Registered office: Ivy House Foundry, 
Hanley, Stoke-on-Trent ST1 3NR.

This message is private and confidential. If you have received this 
message in error, please notify us and remove it from your system.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: syslinux-3.72.patch
Type: text/x-patch
Size: 3521 bytes
Desc: not available
Url : http://busybox.net/lists/buildroot/attachments/20081031/ad373d52/attachment.bin 

From mboxrd@z Thu Jan  1 00:00:00 1970
From: Bernhard Reutner-Fischer <rep.dot.nop@gmail.com>
Date: Fri, 31 Oct 2008 11:43:15 +0100
Subject: [Buildroot] Patch review
In-Reply-To: <490AD4EF.1010106@netcentral.co.uk>
References: <490AD4EF.1010106@netcentral.co.uk>
Message-ID: <20081031104315.GJ26468@mx.loc>
List-Id: <buildroot.busybox.net>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: buildroot@busybox.net

On Fri, Oct 31, 2008 at 09:50:39AM +0000, Nick Peirson wrote:
> Would it be possible for the patch against bug  
> http://bugs.uclibc.org/view.php?id=5674 to be reviewed and, hopefully,  
> commited to svn?
>
> The patch (attached) bumps the syslinux version and updates the patches  
> required for it to compile.

I did not need any patches to syslinux itself but had to add nasm
support for both the host and the target?

Did you send the non-makefile patch upstream? If you didn't already then
please do now..

From mboxrd@z Thu Jan  1 00:00:00 1970
From: Nick Peirson <nick@netcentral.co.uk>
Date: Fri, 31 Oct 2008 10:57:15 +0000
Subject: [Buildroot] Patch review
In-Reply-To: <20081031104315.GJ26468@mx.loc>
References: <490AD4EF.1010106@netcentral.co.uk> <20081031104315.GJ26468@mx.loc>
Message-ID: <490AE48B.20306@netcentral.co.uk>
List-Id: <buildroot.busybox.net>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: buildroot@busybox.net

Bernhard Reutner-Fischer wrote:
> I did not need any patches to syslinux itself but had to add nasm
> support for both the host and the target?

I added nasm on the host, but didn't make any changes to the target. The 
error I was getting was due to syslinux makefile specifying "-Wall 
-Werror" and printf throwing a warning when called without a format string.

> Did you send the non-makefile patch upstream? If you didn't already then
> please do now..

I wasn't sure if the bug was related to buildroot/uclibc, which I 
mentioned in the bug notes, but I'll send it upstream and see what's 
made of it.

Thanks
Nick

From mboxrd@z Thu Jan  1 00:00:00 1970
From: Nick Peirson <nick@netcentral.co.uk>
Date: Fri, 31 Oct 2008 15:26:43 +0000
Subject: [Buildroot] Patch review
In-Reply-To: <490AE48B.20306@netcentral.co.uk>
References: <490AD4EF.1010106@netcentral.co.uk> <20081031104315.GJ26468@mx.loc>
	<490AE48B.20306@netcentral.co.uk>
Message-ID: <490B23B3.60801@netcentral.co.uk>
List-Id: <buildroot.busybox.net>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: buildroot@busybox.net

Nick Peirson wrote:
> Bernhard Reutner-Fischer wrote:
>> Did you send the non-makefile patch upstream? If you didn't already then
>> please do now..
> 
> I wasn't sure if the bug was related to buildroot/uclibc, which I
> mentioned in the bug notes, but I'll send it upstream and see what's
> made of it.
> 

Posted the patch on the syslinux list, but apparently there's already a 
fix upstream of syslinux.

Can the patch I attached be applied to the buildroot svn? Currently the 
syslinux.mk points to a version of syslinux that isn't available and 
will break the build.

I can create a patch to move to 3.63 if that's preferred, as there 
aren't as many changes, although going to 3.72 seems preferable to me?

-- 
Regards

Nick Peirson

--
Internet Central Ltd is a limited company registered in England. 
Registered number: 3079542. Registered office: Ivy House Foundry, 
Hanley, Stoke-on-Trent ST1 3NR.

This message is private and confidential. If you have received this 
message in error, please notify us and remove it from your system.

From mboxrd@z Thu Jan  1 00:00:00 1970
From: Peter Korsgaard <jacmet@uclibc.org>
Date: Sun, 02 Nov 2008 19:33:46 +0100
Subject: [Buildroot] Patch review
In-Reply-To: <490B23B3.60801@netcentral.co.uk> (Nick Peirson's message of
	"Fri\, 31 Oct 2008 15\:26\:43 +0000")
References: <490AD4EF.1010106@netcentral.co.uk>
	<20081031104315.GJ26468@mx.loc> <490AE48B.20306@netcentral.co.uk>
	<490B23B3.60801@netcentral.co.uk>
Message-ID: <87iqr6vuyd.fsf@macbook.be.48ers.dk>
List-Id: <buildroot.busybox.net>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: buildroot@busybox.net

>>>>> "Nick" == Nick Peirson <nick@netcentral.co.uk> writes:

Hi,

 Nick> Can the patch I attached be applied to the buildroot svn? Currently the 
 Nick> syslinux.mk points to a version of syslinux that isn't available and 
 Nick> will break the build.

 Nick> I can create a patch to move to 3.63 if that's preferred, as there 
 Nick> aren't as many changes, although going to 3.72 seems preferable to me?

