m-ab-s / media-autobuild_suite

This Windows Batchscript helps setup a Mingw-w64 compiler environment for building ffmpeg and other media tools under Windows.
GNU General Public License v3.0
1.49k stars 256 forks source link

keeps failing at libmediainfo x64 build #2723

Open jamiesaun85 opened 1 week ago

jamiesaun85 commented 1 week ago
Paste output of the suite here

logs.zip

jamiesaun85 commented 1 week ago

┌ mediainfo git ......................................... [Updates found] ├ Running autogen... ├ Running configure... Likely error (tail of the failed operation logfile): checking if ccache g++ PIC flag -DDLL_EXPORT -DPIC works... yes checking if ccache g++ static flag -static works... yes checking if ccache g++ supports -c -o file.o... yes checking if ccache g++ supports -c -o file.o... (cached) yes checking whether the ccache g++ linker (C:/ab-suite/msys64/mingw64/x86_64-w64-mingw32/bin/ld.exe) supports shared libraries... yes checking dynamic linker characteristics... Win32 ld.exe checking how to hardcode library paths into programs... immediate ./configure: line 16929: libzen-config: command not found ./configure: line 17028: libmediainfo-config: command not found configure: error: libmediainfo configuration is not found configure failed. Check C:/ab-suite/build/mediainfo-git/Project/GNU/CLI/ab-suite.configure.log This is required for other packages, so this script will exit. Creating diagnostics file...

Attach C:\ab-suite\build\logs.zip to the GitHub issue. Make sure the suite is up-to-date before reporting an issue. It might've been fixed already.

Try running the build again at a later time.

logs.zip logs.zip

dbyoung720 commented 1 week ago

same with me . It seems to be related to zenlib

nero-0 commented 1 week ago

It happens to me too.

jamiesaun85 commented 1 week ago

so what do we do about it? Ive already deleted both folders and ran again, same thing happens. I guess not include them in the build maybe?

Not sure about you guys but I'm on the recent update for 23H2. The one from last week.

ls-milkyway commented 1 week ago

so what do we do about it? Ive already deleted both folders and ran again, same thing happens. I guess not include them in the build maybe?

Not sure about you guys but I'm on the recent update for 23H2. The one from last week.

nope nothing to do with 23H2...same output on my 22H2.

dbyoung720 commented 1 week ago

image

jamiesaun85 commented 1 week ago

Thanks man. Unfortunately I have absolutely no idea how to interpret any of that. Even less how to implement it. I just wanted to make one of my softwares use hardware acceleration, and somehow ended up here, thinking this build or suite or whatever it is would make it happen.

As far as coding, or communicating with a computer in any way, shit I may as well be Zoolander. I am really really goodlooking after all.

Thanks for the help though. Always appreciated

L4cache commented 1 week ago

It's the pkgconf can't detect existance of libmediainfo static library. Weird. error: libmediainfo configuration is not found is what ultimately caused the error


./configure: line 16929: libzen-config: command not found
./configure: line 17028: libmediainfo-config: command not found

These two messages are not important here because these commands are in a test command and a if statement, these are just kind of warnings.

L4cache commented 1 week ago

Pkgconf can't detect static libmediainfo because it can't find zstd required by libcurl required by libmediainfo.

But zstd also exists! (installed via pacman by the suite) ~Maybe just lack of .pc file for it.~ It just looking for zstd.pc instead of libzstd.pc

$ pkgconf --keep-system-cflags --static --print-errors --exists libmediainfo
Package zstd was not found in the pkg-config search path.
Perhaps you should add the directory containing `zstd.pc'
to the PKG_CONFIG_PATH environment variable
Package 'zstd', required by 'libcurl', not found
L4cache commented 1 week ago

Makeshift solution: copy libzstd.pc to zstd.pc. It's in /mingw64(or clang64)/lib/pkgconfig

L4cache commented 1 week ago

Maybe we should just disable curl-reader in libmediainfo like this https://github.com/m-ab-s/media-autobuild_suite/issues/2707#issuecomment-2145862504

diegocr commented 1 week ago

@L4cache worth a try at least, i can tell i just ran the suite, having seen the libzen update, and i got no issues at all getting mediainfo.exe compiled.

LigH-de commented 6 days ago

Pkgconf can't detect static libmediainfo because it can't find zstd required by libcurl required by libmediainfo.

But zstd also exists! (installed via pacman by the suite) ~Maybe just lack of .pc file for it.~ It just looking for zstd.pc instead of libzstd.pc

Your competence is seriously a win for this project! Thank you for contributing!

So this is mainly to be solved in the suite, no need to mention it upstream?

L4cache commented 6 days ago

Pkgconf can't detect static libmediainfo because it can't find zstd required by libcurl required by libmediainfo. But zstd also exists! (installed via pacman by the suite) ~Maybe just lack of .pc file for it.~ It just looking for zstd.pc instead of libzstd.pc

Your competence is seriously a win for this project! Thank you for contributing!

So this is mainly to be solved in the suite, no need to mention it upstream?

Thank you. Unfortunately I don't know the exact cause... It could be pkgconf changed behavior or zstd changed it's .pc file name, and all sorts of possibilities. But yeah, the workaround can be done just in the suite.

Reporting to upstream is, I don't know, maybe after narrowing the possibilities down?

L4cache commented 6 days ago

https://github.com/curl/curl/commit/f057de5a1a950a90d1920021db152a4b695f1a8a#diff-49473dca262eeab3b4a43002adb08b4db31020d190caaad1594b47f1d5daa810R1562

https://github.com/curl/curl/commit/f057de5a1a950a90d1920021db152a4b695f1a8a#diff-1e7de1ae2d059d21e1dd75d5812d5a34b0222cef273b7c3a2af62eb747f9d20aR606

Curl recently added Requires.private configuring, they use "libzstd" in cmake but "zstd" in autoconf.

Well, I guess, cmake vs. autoconf again? (not really)

LigH-de commented 6 days ago

Well, that was quick. Solved soon™ after MSYS2 updates packages?

jamiesaun85 commented 6 days ago

Soooo, its fixed now? do i just run it again?

hydra3333 commented 5 days ago

I deleted the folders for mediainfo, libmediainfo, curl, and re-ran it and ffmpeg built fine.

L4cache commented 5 days ago

Well, that was quick. Solved soon™ after MSYS2 updates packages?

Curl in MSYS2 packages is not (and will not be) affected, it uses release version and current release is before the addition of Requires.private, and in next release the fix is already there.

LigH-de commented 5 days ago

Confirming, compilation of MediaInfo passes.