Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 944285 - media-gfx/oxipng-9.1.2-r1 fails to compile: warning: libdeflate-sys@1.20.0: common_defs.h:513:1: error: inlining failed in call to always_inline get_unaligned_le64: target specific option mismatch
Summary: media-gfx/oxipng-9.1.2-r1 fails to compile: warning: libdeflate-sys@1.20.0: c...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal
Assignee: unlord
URL:
Whiteboard:
Keywords: PullRequest
Depends on:
Blocks:
 
Reported: 2024-11-21 09:12 UTC by Agostino Sarubbo
Modified: 2024-12-01 14:56 UTC (History)
2 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
build.log (build.log,133.88 KB, text/plain)
2024-11-21 09:12 UTC, Agostino Sarubbo
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Agostino Sarubbo gentoo-dev 2024-11-21 09:12:36 UTC
https://blogs.gentoo.org/ago/2020/07/04/gentoo-tinderbox/

Issue: media-gfx/oxipng-9.1.2-r1 fails to compile.
Discovered on: arm64 (internal ref: tinderbox_arm64)
System: GCC-15-SYSTEM (https://wiki.gentoo.org/wiki/Project:Tinderbox/Common_Issues_Helper#GCC-15)

Info about the issue:
https://wiki.gentoo.org/wiki/Project:Tinderbox/Common_Issues_Helper#CF0014
Comment 1 Agostino Sarubbo gentoo-dev 2024-11-21 09:12:37 UTC
Created attachment 910982 [details]
build.log

build log and emerge --info
Comment 2 YOSHIOKA Takuma 2024-12-01 12:22:26 UTC
error: inlining failed in call to 'always_inline' in some builds · Issue #369 · ebiggers/libdeflate · GitHub
https://github.com/ebiggers/libdeflate/issues/369

lib/arm: don't use explicit armv8.2-a on gcc 13.2 and later by ebiggers · Pull Request #370 · ebiggers/libdeflate · GitHub
https://github.com/ebiggers/libdeflate/pull/370

Looks like this, which has been fixed since libdeflate v1.21.
Comment 3 Sam James archtester Gentoo Infrastructure gentoo-dev Security 2024-12-01 12:23:26 UTC
Can we have it always use a system copy? Or does the crate not support that?
Comment 4 YOSHIOKA Takuma 2024-12-01 12:36:29 UTC
https://github.com/adamkewley/libdeflater/blob/9bd20a39c33cdeed66ea515b983ae694de370be0/libdeflate-sys/build.rs#L8-L17

libdeflate has `dynamic` flag to use system library if available, but it falls back to the bundled version if the corresponding version of the system library is not available. It provides no means to force using system library.
(And additionally, `oxipng` does not have feature flag to enable such internal flags.)

oxipng and libdeflate_sys can be patched from ebuild, but such patches may need to be updated for every new releases.
Comment 5 YOSHIOKA Takuma 2024-12-01 13:53:52 UTC
Submitted <https://github.com/gentoo/gentoo/pull/39539>.
This new revision of oxipng always use system app-arch/libdeflate (instead of the bundled one), so this should fix the bug.

Could anyone test this (media-gfx/oxipng-9.1.2-r2) on arm64?
Comment 6 Larry the Git Cow gentoo-dev 2024-12-01 14:56:25 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=33b61c0ada2a5ec8dd041c73bb5744f9777e4e7d

commit 33b61c0ada2a5ec8dd041c73bb5744f9777e4e7d
Author:     YOSHIOKA Takuma <lo48576@hard-wi.red>
AuthorDate: 2024-12-01 13:17:49 +0000
Commit:     Sam James <sam@gentoo.org>
CommitDate: 2024-12-01 14:55:49 +0000

    media-gfx/oxipng: use app-arch/libdeflate instead of bundled one
    
    Closes: https://bugs.gentoo.org/944285
    Signed-off-by: YOSHIOKA Takuma <lo48576@hard-wi.red>
    Signed-off-by: Sam James <sam@gentoo.org>

 media-gfx/oxipng/Manifest                          |   1 +
 .../files/oxipng-9.1.2-use-system-libdeflate.patch |  38 ++++++
 media-gfx/oxipng/oxipng-9.1.2-r2.ebuild            | 145 +++++++++++++++++++++
 3 files changed, 184 insertions(+)