Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[qtdeclarative] fails post build checks if qtsvg is installed? #38720

Open
Neumann-A opened this issue May 13, 2024 · 5 comments
Open

[qtdeclarative] fails post build checks if qtsvg is installed? #38720

Neumann-A opened this issue May 13, 2024 · 5 comments
Assignees
Labels
category:port-bug The issue is with a library, which is something the port should already support

Comments

@Neumann-A
Copy link
Contributor

Installing 9/41 qtdeclarative:x64-windows-release@6.7.0...
Building qtdeclarative:x64-windows-release@6.7.0...
E:\all\vcpkg\triplets\community\x64-windows-release.cmake: info: loaded community triplet from here. Community triplets are not built in the curated registry and are thus less likely to succeed.
-- Using cached qtdeclarative-everywhere-src-6.7.0.tar.xz.
-- Extracting source E:/vcpkg_cache/downloads/qtdeclarative-everywhere-src-6.7.0.tar.xz
-- Applying patch cb6b62f.diff
-- Using source at E:/bpy/qtdeclarative/src/here-src-6-a012fdeba0.clean
-- Found external ninja('1.11.0').
-- Configuring x64-windows-release
-- Building x64-windows-release-rel
-- Using cached msys2-mingw-w64-x86_64-pkgconf-1~2.2.0-1-any.pkg.tar.zst.
-- Using cached msys2-msys2-runtime-3.4.10-4-x86_64.pkg.tar.zst.
-- Using msys root at E:/vcpkg_cache/downloads/tools/msys2/d33df895d441a36d
-- Performing post-build validation
warning: The following EXEs were found in /bin or /debug/bin. EXEs are not valid distribution targets.

  E:\all\vcpkg\packages\qtdeclarative_x64-windows-release\bin\svgtoqml.exe

error: Found 1 post-build check problem(s). To submit these ports to curated catalogs, please first correct the portfile: E:\all\vcpkg\ports\qtdeclarative\portfile.cmake
@MonicaLiu0311 MonicaLiu0311 self-assigned this May 14, 2024
@Neumann-A
Copy link
Contributor Author

Neumann-A commented May 14, 2024

find_package(Qt6 ${PROJECT_VERSION} CONFIG REQUIRED COMPONENTS BuildInternals Core) # special case
find_package(Qt6 ${PROJECT_VERSION} QUIET CONFIG OPTIONAL_COMPONENTS Gui Network Widgets OpenGL OpenGLWidgets Sql Concurrent Test LanguageServerPrivate LinguistTools Svg)
qt_internal_project_setup()

@MonicaLiu0311 MonicaLiu0311 added the requires:repro The issue is not currently repro-able label May 14, 2024
@MonicaLiu0311
Copy link
Contributor

But I didn't reproduce it:

image

@Neumann-A
Copy link
Contributor Author

But I didn't reproduce it:

did you install qtsvg beforehand ?

@MonicaLiu0311
Copy link
Contributor

But I didn't reproduce it:

did you install qtsvg beforehand ?

I've reproduced it now and I'm fixing it, do you agree with #38738? Looking forward to your comments.

@MonicaLiu0311 MonicaLiu0311 added category:port-bug The issue is with a library, which is something the port should already support and removed requires:repro The issue is not currently repro-able labels May 15, 2024
@Neumann-A
Copy link
Contributor Author

#38682 already has a fix

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category:port-bug The issue is with a library, which is something the port should already support
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants