From: "Alexis Lothoré via buildroot" <buildroot@buildroot.org>
To: "Thomas Petazzoni" <thomas.petazzoni@bootlin.com>
Cc: buildroot@buildroot.org,
"Hervé Codina" <herve.codina@bootlin.com>,
"Nicolas Carrier" <nicolas.carrier@nav-timing.safrangroup.com>
Subject: Re: [Buildroot] [PATCH] package/php-pecl-dbus: fix build with PHP8
Date: Wed, 06 Aug 2025 12:23:59 +0200 [thread overview]
Message-ID: <DBV9S1S4XEEA.2DHT8AEV8PEL9@bootlin.com> (raw)
In-Reply-To: <20250806110836.387c867c@windsurf>
On Wed Aug 6, 2025 at 11:08 AM CEST, Thomas Petazzoni via buildroot wrote:
> Hello Alexis,
>
> On Wed, 06 Aug 2025 10:53:07 +0200
> Alexis Lothoré <alexis.lothore@bootlin.com> wrote:
>
>> The php-pecl-dbus encouters two build issues in buildroot 2025.02.x,
>> both related to PHP8. Bring the relevant patches to fix this build.
>
> Are the issues affecting master as well?
Yes, I have those issues on the master branch, on 2fd520c8d52b
("package/glm: bump version to 1.0.1")
>
> Is the issue
> https://autobuild.buildroot.net/results/e98/e9892bbefa781b403fd3d834b6c48527c8e078ba/build-end.log
> perhaps ? If so, please add a reference to the autobuilder failure.
Yes, that looks like the same issue ! Ok, I'll add a reference.
>
> For bug/build fixes, we like to understand since when the problem is
> occurring, i.e which commit introduced the regression. If this package
> was building on 2024.02 and no longer on 2025.02, I suspect it's one of
> the PHP version bump between those two releases:
The upstream commit in PHP project leading to this issue is actually the
same as for the php-lua package (commit 91ef4124e56a ("Refactor
zend_object_handlers API to pass zend_object* and zend_string* insted of
zval(s)."), so the issue has been introduced when the PHP packaged has been
bumped to 8.0.0, not later. And again, I can see the issue with a minimal
defconfig _when using bootlin bleeding-edge toolchain_. With stable
toolchain, we can spot the corresponding warning in php-dbus build logs,
but it is not turned into an actual error.
>
> befd90b7655804aa30eae494fae6640ff178217c package/php: security bump version to 8.3.19
> 32dd92d18edb3d83ff465e366ff211f741767222 package/php: bump version to 8.3.17
> 26a328f8ef178dab746b6cd766140661a5ee4c5d package/php: bump version to 8.3.16
> 3a20dfb4267f3dc6b11a3993a6a6dcc3e5c2ed88 package/php: backport upstream patch to fix/workaround xxhash issue
> bcaffa0a04af278d154e6e08b81a06a40a1b3ee9 package/php: bump version to 8.3.15
> e2f44b50602e51a3d279dbaa465093537a313638 package/php: fix 'deactivate' typo
> 6d81e3225672d146248ed3e36ba9ec33768d2d27 Merge branch 'next'
> 96d73b3091701651373c5b6659e437e7445a062b package/php: security bump version to 8.3.14
> b0760d4485c986cbddd5de73b7922dd720c8d3eb package/php: bump version to 8.3.13
> a2da88519fc1f96e517171dfb053aca8c00f7daf package/php: security bump version to 8.3.12
> 8331fb05925b33fbff1b08367470652e31996b39 package/php: bump version to 8.3.11
> 4f0243567d74323082e53c46e777b326476f5c2d package/php: fix build on arm
> ea301ef80c810791a6b673d46b4f16b6c2f62a29 package/php: bump version to 8.3.10
> 677c3f2995b4e6acaedcecc699b5f6ddf294f524 package/php: bump version to 8.3.9
> ecf060a2ff93626fb5fbe1c52b8d42b6ee0fbb5b package/php: security bump version to 8.3.8
> 932b52fad87d79d9f26a343edafe2981079de16e package/php: bump version to 8.3.7
> b5b927881daf3023ad38b4e910ab79ad081f7ee4 package/php: security bump version to 8.3.6
> 70031dd7fa09191566f2d1e895399ae578c62d1b package/php: fix build without threads
> e50460f9f12a8c5f82f2166733afe9cca3c8cd19 package/php: bump version to 8.3.4
> 2824aa8a23a765b660219c129d4980f1b7c7691c package/php: add libucontext optional dependency
>
> Most likely:
>
> e50460f9f12a8c5f82f2166733afe9cca3c8cd19 package/php: bump version to 8.3.4
>
> Which is a bump from 8.2.x to 8.3.x. The other bumps are within the
> 8.3.x branch, so I doubt they broke APIs there.
>
> Thomas
--
Alexis Lothoré, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
next prev parent reply other threads:[~2025-08-06 10:24 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-08-06 8:53 [Buildroot] [PATCH] package/php-pecl-dbus: fix build with PHP8 Alexis Lothoré via buildroot
2025-08-06 9:08 ` Thomas Petazzoni via buildroot
2025-08-06 10:23 ` Alexis Lothoré via buildroot [this message]
2025-08-06 10:34 ` Thomas Petazzoni via buildroot
2025-08-06 12:14 ` Alexis Lothoré via buildroot
2025-08-06 14:52 ` 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=DBV9S1S4XEEA.2DHT8AEV8PEL9@bootlin.com \
--to=buildroot@buildroot.org \
--cc=alexis.lothore@bootlin.com \
--cc=herve.codina@bootlin.com \
--cc=nicolas.carrier@nav-timing.safrangroup.com \
--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