Mailing List Archive: syslinux: users

SYSLINUX 3.50-pre14 pushed out; adds INCLUDE support

Index | Next | Previous | View Threaded

hpa at zytor

May 22, 2007, 11:05 PM

Post #1 of 15 (1650 views)
Permalink

SYSLINUX 3.50-pre14 pushed out; adds INCLUDE support

It really turned out not to be all that hard to do, after all. I
definitely feel a bit silly, but there you have it.

Please test this out as 3.50 really is way overdue to get out the door.

-hpa

_______________________________________________
SYSLINUX mailing list
Submissions to SYSLINUX [at] zytor
Unsubscribe or set options at:
http://www.zytor.com/mailman/listinfo/syslinux
Please do not send private replies to mailing list traffic.

jbebel at google

May 23, 2007, 12:21 AM

Post #2 of 15 (1601 views)
Permalink

Re: SYSLINUX 3.50-pre14 pushed out; adds INCLUDE support [In reply to]

Thanks! I'll test it out tomorrow.

Joel

H. Peter Anvin wrote:
> It really turned out not to be all that hard to do, after all. I
> definitely feel a bit silly, but there you have it.
>
> Please test this out as 3.50 really is way overdue to get out the door.
>
> -hpa

_______________________________________________
SYSLINUX mailing list
Submissions to SYSLINUX [at] zytor
Unsubscribe or set options at:
http://www.zytor.com/mailman/listinfo/syslinux
Please do not send private replies to mailing list traffic.

buraphalinuxserver at gmail

May 23, 2007, 1:04 AM

Post #3 of 15 (1600 views)
Permalink

Re: SYSLINUX 3.50-pre14 pushed out; adds INCLUDE support [In reply to]

If you build as root, permissions of /dev/null change to 0777 from
0666 on my linux system.
I thought this was fixed?

On 5/23/07, Joel Ebel <jbebel [at] google> wrote:
> Thanks! I'll test it out tomorrow.
>
> Joel
>
> H. Peter Anvin wrote:
> > It really turned out not to be all that hard to do, after all. I
> > definitely feel a bit silly, but there you have it.
> >
> > Please test this out as 3.50 really is way overdue to get out the door.
> >
> > -hpa
>
> _______________________________________________
> SYSLINUX mailing list
> Submissions to SYSLINUX [at] zytor
> Unsubscribe or set options at:
> http://www.zytor.com/mailman/listinfo/syslinux
> Please do not send private replies to mailing list traffic.
>
>

_______________________________________________
SYSLINUX mailing list
Submissions to SYSLINUX [at] zytor
Unsubscribe or set options at:
http://www.zytor.com/mailman/listinfo/syslinux
Please do not send private replies to mailing list traffic.

hpa at zytor

May 23, 2007, 4:56 AM

Post #4 of 15 (1596 views)
Permalink

Re: SYSLINUX 3.50-pre14 pushed out; adds INCLUDE support [In reply to]

BuraphaLinux Server wrote:
> If you build as root, permissions of /dev/null change to 0777 from
> 0666 on my linux system.
> I thought this was fixed?

Well, 0666 are actually the correct permissions for /dev/null, but yes,
that shouldn't happen. (Of course, building as root is always a really
bad idea.)

-hpa

_______________________________________________
SYSLINUX mailing list
Submissions to SYSLINUX [at] zytor
Unsubscribe or set options at:
http://www.zytor.com/mailman/listinfo/syslinux
Please do not send private replies to mailing list traffic.

buraphalinuxserver at gmail

May 23, 2007, 5:24 AM

Post #5 of 15 (1598 views)
Permalink

Re: SYSLINUX 3.50-pre14 pushed out; adds INCLUDE support [In reply to]

My english is bad; normal perms are 0666, after a build as root of
syslinux 3.50-pre15 they were 0777. Why does the build do any chmod
on /dev/null anyway?

On 5/23/07, H. Peter Anvin <hpa [at] zytor> wrote:
> BuraphaLinux Server wrote:
> > If you build as root, permissions of /dev/null change to 0777 from
> > 0666 on my linux system.
> > I thought this was fixed?
>
> Well, 0666 are actually the correct permissions for /dev/null, but yes,
> that shouldn't happen. (Of course, building as root is always a really
> bad idea.)
>
> -hpa
>
> _______________________________________________
> SYSLINUX mailing list
> Submissions to SYSLINUX [at] zytor
> Unsubscribe or set options at:
> http://www.zytor.com/mailman/listinfo/syslinux
> Please do not send private replies to mailing list traffic.
>
>

_______________________________________________
SYSLINUX mailing list
Submissions to SYSLINUX [at] zytor
Unsubscribe or set options at:
http://www.zytor.com/mailman/listinfo/syslinux
Please do not send private replies to mailing list traffic.

genaldo at glinux

May 23, 2007, 5:50 AM

Post #6 of 15 (1599 views)
Permalink

Re: SYSLINUX 3.50-pre14 pushed out; adds INCLUDE support [In reply to]

BuraphaLinux Server wrote:
> My english is bad; normal perms are 0666, after a build as root of
> syslinux 3.50-pre15 they were 0777. Why does the build do any chmod
> on /dev/null anyway?
>
>

not only it changed the perm in /dev/null but also turns it into a normal
file, not a character device anymore.

Genaldo

_______________________________________________
SYSLINUX mailing list
Submissions to SYSLINUX [at] zytor
Unsubscribe or set options at:
http://www.zytor.com/mailman/listinfo/syslinux
Please do not send private replies to mailing list traffic.

genaldo at glinux

May 23, 2007, 5:55 AM

Post #7 of 15 (1600 views)
Permalink

Re: SYSLINUX 3.50-pre14 pushed out; adds INCLUDE support [In reply to]

It is a bug in gcc:

gcc -o /dev/null dummy.c

changed the perms in /dev/null

Genaldo

_______________________________________________
SYSLINUX mailing list
Submissions to SYSLINUX [at] zytor
Unsubscribe or set options at:
http://www.zytor.com/mailman/listinfo/syslinux
Please do not send private replies to mailing list traffic.

hpa at zytor

May 23, 2007, 6:21 AM

Post #8 of 15 (1602 views)
Permalink

Re: SYSLINUX 3.50-pre14 pushed out; adds INCLUDE support [In reply to]

BuraphaLinux Server wrote:
> My english is bad; normal perms are 0666, after a build as root of
> syslinux 3.50-pre15 they were 0777. Why does the build do any chmod
> on /dev/null anyway?

Probably because /dev/null is being invoked as -o /dev/null somewhere
(and because you're building as root.)

-hpa

_______________________________________________
SYSLINUX mailing list
Submissions to SYSLINUX [at] zytor
Unsubscribe or set options at:
http://www.zytor.com/mailman/listinfo/syslinux
Please do not send private replies to mailing list traffic.

hpa at zytor

May 23, 2007, 6:50 AM

Post #9 of 15 (1600 views)
Permalink

Re: SYSLINUX 3.50-pre14 pushed out; adds INCLUDE support [In reply to]

H. Peter Anvin wrote:
> BuraphaLinux Server wrote:
>> My english is bad; normal perms are 0666, after a build as root of
>> syslinux 3.50-pre15 they were 0777. Why does the build do any chmod
>> on /dev/null anyway?
>
> Probably because /dev/null is being invoked as -o /dev/null somewhere
> (and because you're building as root.)

Ah, yes. It's the bloody gcc_ok macro which does it, and any attempt I
have made to fix it fails for unexplained reasons.

I don't have time to dig into this. Help would be appreciated; the
Makefiles are a mess.

-hpa

_______________________________________________
SYSLINUX mailing list
Submissions to SYSLINUX [at] zytor
Unsubscribe or set options at:
http://www.zytor.com/mailman/listinfo/syslinux
Please do not send private replies to mailing list traffic.

strange at nsk

May 23, 2007, 8:49 AM

Post #10 of 15 (1609 views)
Permalink

Re: SYSLINUX 3.50-pre14 pushed out; adds INCLUDE support [In reply to]

On Wed, May 23, 2007 at 06:50:59AM -0700, H. Peter Anvin wrote:
> H. Peter Anvin wrote:
> > BuraphaLinux Server wrote:
> >> My english is bad; normal perms are 0666, after a build as root of
> >> syslinux 3.50-pre15 they were 0777. Why does the build do any chmod
> >> on /dev/null anyway?
> >
> > Probably because /dev/null is being invoked as -o /dev/null somewhere
> > (and because you're building as root.)
>
> Ah, yes. It's the bloody gcc_ok macro which does it, and any attempt I
> have made to fix it fails for unexplained reasons.
>
> I don't have time to dig into this. Help would be appreciated; the
> Makefiles are a mess.

perl -pi -e 's!-o /dev/null!-o /dev/stderr!g' `find . -name Makefile -o
-name MCONFIG` did it for me.

But the previous version only changed the permissions of /dev/null, not
the file type as Genaldo reported. So the safest version, but specific
to linux, would be to change to /proc/self/fd/2 instead.

--
lfr
0/0

hpa at zytor

May 23, 2007, 10:57 AM

Post #11 of 15 (1602 views)
Permalink

Re: SYSLINUX 3.50-pre14 pushed out; adds INCLUDE support [In reply to]

Luciano Rocha wrote:
>
> perl -pi -e 's!-o /dev/null!-o /dev/stderr!g' `find . -name Makefile -o
> -name MCONFIG` did it for me.
>
> But the previous version only changed the permissions of /dev/null, not
> the file type as Genaldo reported. So the safest version, but specific
> to linux, would be to change to /proc/self/fd/2 instead.
>

There are better ways to fix this, writing and deleting a temp file is
the right thing to do, but for some unfanthomable reason it just doesn't
work when I try to implement it.

-hpa

_______________________________________________
SYSLINUX mailing list
Submissions to SYSLINUX [at] zytor
Unsubscribe or set options at:
http://www.zytor.com/mailman/listinfo/syslinux
Please do not send private replies to mailing list traffic.

strange at nsk

May 23, 2007, 1:50 PM

Post #12 of 15 (1599 views)
Permalink

Re: SYSLINUX 3.50-pre14 pushed out; adds INCLUDE support [In reply to]

On Wed, May 23, 2007 at 10:57:16AM -0700, H. Peter Anvin wrote:
> Luciano Rocha wrote:
> >
> > perl -pi -e 's!-o /dev/null!-o /dev/stderr!g' `find . -name Makefile -o
> > -name MCONFIG` did it for me.
> >
> > But the previous version only changed the permissions of /dev/null, not
> > the file type as Genaldo reported. So the safest version, but specific
> > to linux, would be to change to /proc/self/fd/2 instead.
> >
>
> There are better ways to fix this, writing and deleting a temp file is
> the right thing to do, but for some unfanthomable reason it just doesn't
> work when I try to implement it.

The attached patch works for me.

--
lfr
0/0

Attachments:

syslinux_gcc_ok_tmpfile.patch (7.38 KB)

jbebel at google

May 23, 2007, 6:21 PM

Post #13 of 15 (1603 views)
Permalink

Re: SYSLINUX 3.50-pre14 pushed out; adds INCLUDE support [In reply to]

This works great! Thanks for the quick addition of a very useful
feature. It was a little tricky dealing with including entire
configuration files because I picked up all the unwanted settings too,
like MENU TITLE, or MENU DEFAULT. I don't have a great method of
dealing with the MENU DEFAULT when including other configs, but the MENU
TITLE I dealt with by creating a separate header file for each submenu
that contained the correct title, and then created a file with only
labels, which I included both in the header for that submenu, and in the
top level pxelinux.cfg. This is working quite well, and greatly
simplifies my configuration.

Thanks again,
Joel

Joel Ebel wrote:
> Thanks! I'll test it out tomorrow.
>
> Joel
>
> H. Peter Anvin wrote:
>> It really turned out not to be all that hard to do, after all. I
>> definitely feel a bit silly, but there you have it.
>>
>> Please test this out as 3.50 really is way overdue to get out the door.
>>
>> -hpa
>
> _______________________________________________
> SYSLINUX mailing list
> Submissions to SYSLINUX [at] zytor
> Unsubscribe or set options at:
> http://www.zytor.com/mailman/listinfo/syslinux
> Please do not send private replies to mailing list traffic.
>

_______________________________________________
SYSLINUX mailing list
Submissions to SYSLINUX [at] zytor
Unsubscribe or set options at:
http://www.zytor.com/mailman/listinfo/syslinux
Please do not send private replies to mailing list traffic.

stappers at stappers

May 24, 2007, 9:42 AM

Post #14 of 15 (1598 views)
Permalink

Re: SYSLINUX 3.50-pre14 pushed out; adds INCLUDE support [In reply to]

Op 23-05-2007 om 18:21 schreef Joel Ebel:
> This works great! Thanks for the quick addition of a very useful
> feature. It was a little tricky dealing with including entire
> configuration files because I picked up all the unwanted settings too,
> like MENU TITLE, or MENU DEFAULT. I don't have a great method of
> dealing with the MENU DEFAULT when including other configs, but the MENU
> TITLE I dealt with by creating a separate header file for each submenu
> that contained the correct title, and then created a file with only
> labels, which I included both in the header for that submenu, and in the
> top level pxelinux.cfg. This is working quite well, and greatly
> simplifies my configuration.

Sound like something worth to be added
on http://syslinux.zytor.com/wiki

Cheers
Geert Stappers

_______________________________________________
SYSLINUX mailing list
Submissions to SYSLINUX [at] zytor
Unsubscribe or set options at:
http://www.zytor.com/mailman/listinfo/syslinux
Please do not send private replies to mailing list traffic.

hpa at zytor

May 25, 2007, 3:15 PM

Post #15 of 15 (1593 views)
Permalink

Re: SYSLINUX 3.50-pre14 pushed out; adds INCLUDE support [In reply to]

Luciano Rocha wrote:
> On Wed, May 23, 2007 at 10:57:16AM -0700, H. Peter Anvin wrote:
>> Luciano Rocha wrote:
>>> perl -pi -e 's!-o /dev/null!-o /dev/stderr!g' `find . -name Makefile -o
>>> -name MCONFIG` did it for me.
>>>
>>> But the previous version only changed the permissions of /dev/null, not
>>> the file type as Genaldo reported. So the safest version, but specific
>>> to linux, would be to change to /proc/self/fd/2 instead.
>>>
>> There are better ways to fix this, writing and deleting a temp file is
>> the right thing to do, but for some unfanthomable reason it just doesn't
>> work when I try to implement it.
>
> The attached patch works for me.
>

Thanks. Applied.

-hpa

_______________________________________________
SYSLINUX mailing list
Submissions to SYSLINUX [at] zytor
Unsubscribe or set options at:
http://www.zytor.com/mailman/listinfo/syslinux
Please do not send private replies to mailing list traffic.

Index | Next | Previous | View Threaded

Interested in having your list archived? Contact Gossamer Threads