All of 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: 12+ 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-30  5:39             ` Andreas Ziegler
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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.