Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Andreas Ziegler <br025@umbiko.net>
To: Alexander Sverdlin <alexander.sverdlin@gmail.com>
Cc: Thomas Petazzoni <thomas.petazzoni@bootlin.com>, buildroot@buildroot.org
Subject: Re: [Buildroot] [PATCH v2] boot/syslinux: Bump to 6.04-pre2
Date: Tue, 27 Oct 2026 04:54:45 +0000	[thread overview]
Message-ID: <45a821b584840d1cabe6df1dfe083fb8@umbiko.net> (raw)
In-Reply-To: <e34f4a3f8aaee192923e3ecdde4a287a1cfaabda.camel@gmail.com>

Hi Alexander, Thomas, Everyone,

On 2026-10-26 20:45, Alexander Sverdlin wrote:
> Hi Thomas,
> 
> On Sun, 2026-10-25 at 21:36 +0100, Thomas Petazzoni wrote:
>> On Sun, Oct 25, 2026 at 12:08:44AM +0200, Alexander Sverdlin wrote:
>> > The revision to build comes from Archlinux syslinux package, the archive
>> > from debian (to save time for git clone).
>> >
>> > Add patch fixing passing &load_error_buf to setjmp()/longjmp(), rejected
>> > since gcc 14 made -Wincompatible-pointer-types an error by default.
>> >
>> > Signed-off-by: Alexander Sverdlin <alexander.sverdlin@gmail.com>
>> 
>> Does it make sense to keep syslinux? Even Wikipedia says it's
>> "discontinued". Do you still use it?
> 
> I'll be fine having it downstream. I just thought, there is still an
> interest [1] in this update ;-)

Yes, there is still interest in this software, and I sadly mourn the 
absence of the don't-delete-host-syslinux patch. I did not reply 
earlier, because Thomas already did a deep dive.

Just some formal things:

You might take a look at the output of check-package:

$ utils/check-package boot/syslinux/*
boot/syslinux/0005-lzo-Use-the-host-toolchain-for-prepcore.patch:0: 
missing Upstream in the header 
(https://nightly.buildroot.org/#_additional_patch_documentation)
boot/syslinux/0011-extlinux-Use-the-host-toolchain-to-build.patch:0: 
missing Upstream in the header 
(https://nightly.buildroot.org/#_additional_patch_documentation)
boot/syslinux/0013-Fix-build-with-gnu-efi-version-3.0.9.patch:0: missing 
Upstream in the header 
(https://nightly.buildroot.org/#_additional_patch_documentation)
boot/syslinux/0014-Fix-build-with-binutils-note-gnu-property-section.patch:0: 
missing Upstream in the header 
(https://nightly.buildroot.org/#_additional_patch_documentation)
boot/syslinux/0016-Workaround-multiple-definition-of-symbol-errors.patch:0: 
missing Upstream in the header 
(https://nightly.buildroot.org/#_additional_patch_documentation)
boot/syslinux/0017-Replace-builtin-strlen-that-appears-to-get-optimized.patch:0: 
missing Upstream in the header 
(https://nightly.buildroot.org/#_additional_patch_documentation)
1137 lines processed
6 warnings generated

These should either be added to the .packageignore file or a "Upstream: 
N/A, dead" line, similar to 
0023-com32-chain-chain.h-use-proper-type-in-struct-data_a.patch:21.

A few of the existing patches should be rebased for 6.04, they do not 
apply cleanly:

>>> syslinux bf6db5b4 Patching

<...>

Applying 0011-extlinux-Use-the-host-toolchain-to-build.patch using 
patch:
patching file extlinux/Makefile
Hunk #2 succeeded at 53 (offset 1 line).

Applying 0013-Fix-build-with-gnu-efi-version-3.0.9.patch using patch:
patching file mk/lib.mk
Hunk #1 succeeded at 188 (offset 2 lines).
Hunk #2 succeeded at 198 (offset -5 lines).

<...>

Applying 0016-Workaround-multiple-definition-of-symbol-errors.patch 
using patch:
patching file com32/cmenu/Makefile
patching file com32/elflink/ldlinux/Makefile
patching file com32/gpllib/Makefile
patching file com32/hdt/Makefile
patching file core/Makefile
Hunk #1 succeeded at 158 (offset -6 lines).
patching file dos/Makefile
patching file efi/Makefile
Hunk #1 succeeded at 71 (offset 1 line).

<...>

Kind regards,
Andreas

> [1] https://lore.kernel.org/all/aq62x3ovAUoc_Lmn@windsurf/
> 
> --
> Alexander Sverdlin.
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

  reply	other threads:[~2026-10-27  4:54 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-10-24 22:08 [Buildroot] [PATCH v2] boot/syslinux: Bump to 6.04-pre2 Alexander Sverdlin
2026-10-25 20:36 ` Thomas Petazzoni via buildroot
2026-10-26 20:45   ` Alexander Sverdlin
2026-10-27  4:54     ` Andreas Ziegler [this message]
2026-10-27  5:57       ` Alexander Sverdlin
2026-10-27  8:20         ` Thomas Petazzoni via buildroot
2026-10-27 15:12           ` Lance Fredrickson
2026-10-27  8:19       ` Thomas Petazzoni via buildroot
2026-10-28  4:48         ` Andreas Ziegler
2026-10-29 20:30           ` Thomas Petazzoni via buildroot
2026-10-27  8:15     ` Thomas Petazzoni via buildroot

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=45a821b584840d1cabe6df1dfe083fb8@umbiko.net \
    --to=br025@umbiko.net \
    --cc=alexander.sverdlin@gmail.com \
    --cc=buildroot@buildroot.org \
    --cc=thomas.petazzoni@bootlin.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox