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

[avro-c] no absolute paths #21630

Merged
merged 1 commit into from
Nov 26, 2021
Merged

Conversation

autoantwort
Copy link
Contributor

For #20469

@JonLiu1993 JonLiu1993 self-assigned this Nov 24, 2021
@JonLiu1993 JonLiu1993 added the category:port-feature The issue is with a library, which is requesting new capabilities that didn’t exist label Nov 24, 2021
@JonLiu1993
Copy link
Member

@autoantwort ,Thanks for posting this issue, could you please take a look:

There should be no empty directories in D:\packages\avro-c_x86-windows
The following empty directories were found:

    D:/packages/avro-c_x86-windows/debug/lib/pkgconfig
    D:/packages/avro-c_x86-windows/lib/pkgconfig

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\pkgconfig" "${CURRENT_PACKAGES_DIR}\lib\pkgconfig")

@JonLiu1993 JonLiu1993 added info:reviewed Pull Request changes follow basic guidelines and removed requires:author-response labels Nov 25, 2021
@BillyONeal BillyONeal merged commit 08a0302 into microsoft:master Nov 26, 2021
@BillyONeal
Copy link
Member

Thanks for removing broken pkgconfigs!

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