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

[magnum] fix features #34980

Merged
merged 1 commit into from
Nov 9, 2023
Merged

Conversation

autoantwort
Copy link
Contributor

Fixes for example

warning: There should be no empty directories in G:\v\vcpkg3\packages\magnum_x64-windows. The following empty directories were found:

  G:\v\vcpkg3\packages\magnum_x64-windows\debug\lib\magnum-d\imageconverters
  G:\v\vcpkg3\packages\magnum_x64-windows\lib\magnum\imageconverters

warning: If a directory should be populated but is not, this might indicate an error in the portfile.
If the directories are not needed and their creation cannot be disabled, use something like this in the portfile to remove them:
    file(REMOVE_RECURSE "${CURRENT_PACKAGES_DIR}/debug/lib/magnum-d/imageconverters" "${CURRENT_PACKAGES_DIR}/lib/magnum/imageconverters")

Tested with test-features on x64-windows, x64-windows-static, x64-uwp

@MonicaLiu0311 MonicaLiu0311 added the category:port-feature The issue is with a library, which is requesting new capabilities that didn’t exist label Nov 9, 2023
@MonicaLiu0311 MonicaLiu0311 added the info:reviewed Pull Request changes follow basic guidelines label Nov 9, 2023
@JavierMatosD JavierMatosD merged commit a37c3c2 into microsoft:master Nov 9, 2023
15 checks passed
autoantwort added a commit to autoantwort/vcpkg that referenced this pull request Nov 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category:port-feature The issue is with a library, which is requesting new capabilities that didn’t exist info:reviewed Pull Request changes follow basic guidelines
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants