WebPDecodeRGBA link issue on Ubuntu-latest with static linkage (Issue #25786)
Blake-Madden
Blake-Madden created an issue (wxWidgets/wxWidgets#25786)Build System Used
CMake, building project with statically linking wxWidgets.
Description
When I build and get to the linkage phase, I get this error:
libwx_gtk3u_core-3.3.a(imagwebp.cpp.o): undefined reference to symbol 'WebPDecodeRGBA'
libwebp.so.7: error adding symbols: DSO missing from command line
Just starting happening today; I'm guessing it's related to f1463f2?
Platform and version information
ubuntu-latest (what is provided with GH actions)
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.
VZ
vadz left a comment (wxWidgets/wxWidgets#25786)@cjmayo Any idea about this by chance?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.
Blake-Madden
Blake-Madden left a comment (wxWidgets/wxWidgets#25786)I see this during the config steps:
wxUSE_LIBWEBP: sys (use libwebp (WebP file format))
in case that helps.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.
Maarten
MaartenBent left a comment (wxWidgets/wxWidgets#25786)It seems the WebP_LIBRARIES and WebP_INCLUDE_DIRS are empty when adding them to wxcore. Even though they are found and not empty in modules/FindWebP.cmake. This happens because these files are in different scopes.
I think for now we can add the following to the end of modules/FindWebP.cmake. I'll propose a better fix, that'll get rid of the duplicate find_package for other dependencies too, later:
set(WebP_LIBRARIES ${WebP_LIBRARIES} PARENT_SCOPE) set(WebP_INCLUDE_DIRS ${WebP_INCLUDE_DIRS} PARENT_SCOPE)
Can you commit this directly @vadz, or do you prefer a PR?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.
VZ
vadz left a comment (wxWidgets/wxWidgets#25786)Thanks Maarten!
@Blake-Madden Can you please confirm that this fixes the problem for you? If yes, I can push this directly, no need for the PR.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.
Blake-Madden
Blake-Madden left a comment (wxWidgets/wxWidgets#25786)Yes, that did indeed fix it.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.
VZ
Closed #25786 as completed via c6ddb6e.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.