Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 928466 - dev-qt/qtwebengine-6.6.3 undeclared int32_t in video_transformation (musl)
Summary: dev-qt/qtwebengine-6.6.3 undeclared int32_t in video_transformation (musl)
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Qt Bug Alias
URL:
Whiteboard:
Keywords: PATCH
Depends on:
Blocks:
 
Reported: 2024-04-03 03:11 UTC by ktoupt
Modified: 2024-04-03 03:57 UTC (History)
1 user (show)

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


Attachments
emerge --info (info.txt,7.67 KB, text/plain)
2024-04-03 03:11 UTC, ktoupt
Details

Note You need to log in before you can comment on or make changes to this bug.
Description ktoupt 2024-04-03 03:11:41 UTC
Created attachment 889305 [details]
emerge --info

[ebuild   R   ] dev-qt/qtwebengine-6.6.3  USE="alsa jumbo-build opengl pulseaudio screencast system-icu vaapi vulkan widgets -accessibility -bindist -custom-cflags -designer -geolocation -kerberos -pdfium -qml -test"

Sorry, I forgot to copy the failed build.log... Anyway, the issue was
about int32_t being undefined or undeclared in video_transformation and
then something about no matches found. This is probably related to musl
headers being slightly different than glibc.

I built with the following patch, and it seems to work (tested with
anki):

diff --git qtwebengine-everywhere-src-6.6.3/src/3rdparty/chromium/media/base/video_transformation.h qtwebengine-everywhere-src-6.6.3/src/3rdparty/chromium/media/base/video_transformation.h
index ee80059..f8e243e 100644
--- qtwebengine-everywhere-src-6.6.3/src/3rdparty/chromium/media/base/video_transformation.h
+++ qtwebengine-everywhere-src-6.6.3/src/3rdparty/chromium/media/base/video_transformation.h
@@ -6,6 +6,7 @@
 #define MEDIA_BASE_VIDEO_TRANSFORMATION_H_
 
 #include <string>
+#include <stdint.h>
 
 #include "base/numerics/math_constants.h"
 #include "media/base/media_export.h"
Comment 1 Ionen Wolkens gentoo-dev 2024-04-03 03:16:32 UTC
It's fine don't super need the build.log for these.

I always test qtwebengine with clang+musl but not gcc+musl (given I do glibc for testing gcc), and the stdint.h issues only happen specifically with gcc+musl and this happened a few times.

Thanks for the patch, will save me the trouble of having to try it.
Comment 2 Larry the Git Cow gentoo-dev 2024-04-03 03:57:26 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=03a99428d35fcd93b2073cae67228807a23829c3

commit 03a99428d35fcd93b2073cae67228807a23829c3
Author:     Ionen Wolkens <ionen@gentoo.org>
AuthorDate: 2024-04-03 03:47:53 +0000
Commit:     Ionen Wolkens <ionen@gentoo.org>
CommitDate: 2024-04-03 03:56:31 +0000

    dev-qt/qtwebengine: update patchset for 6.6 and 6.7 branches
    
    6.6-9 -> 6.6-10 changes:
      Added:
      * clang18.patch (imported from files/)
      Updated:
      * cstdint.patch (for bug #928466)
    
    6.7-4 -> 6.7-5 changes:
      Added:
      * x11-header.patch (imported from files/)
      ...not adding clang18 given expect it to be fixed upstream soon
      Updated:
      * cstdint.patch (for bug #928466)
      ...gcc+musl still untested for 6.7.0, but updating what's known
    
    Closes: https://bugs.gentoo.org/928466
    Signed-off-by: Ionen Wolkens <ionen@gentoo.org>

 dev-qt/qtwebengine/Manifest                        |  3 ++-
 .../files/qtwebengine-6.7.0-x11-header.patch       | 22 ----------------------
 dev-qt/qtwebengine/qtwebengine-6.6.3.ebuild        |  3 +--
 dev-qt/qtwebengine/qtwebengine-6.7.0.ebuild        |  3 +--
 dev-qt/qtwebengine/qtwebengine-6.7.9999.ebuild     |  3 +--
 dev-qt/qtwebengine/qtwebengine-6.9999.ebuild       |  2 +-
 6 files changed, 6 insertions(+), 30 deletions(-)