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

[libosip2, talib] Don't install include/makefile.am. #34481

Merged
merged 2 commits into from
Oct 14, 2023

Conversation

BillyONeal
Copy link
Member

Detected by https://dev.azure.com/vcpkg/public/_build/results?buildId=95376

error: The following files are already installed in D:/installed/x86-windows and are in conflict with talib:x86-windows
Installed by libosip2:x86-windows
include/Makefile.am

Stored binaries in 1 destinations in 446 ms.
Elapsed time to handle talib:x86-windows: 40 s

Broken by #34316 which adds talib

Detected by https://dev.azure.com/vcpkg/public/_build/results?buildId=95376

```
error: The following files are already installed in D:/installed/x86-windows and are in conflict with talib:x86-windows
Installed by libosip2:x86-windows
include/Makefile.am

Stored binaries in 1 destinations in 446 ms.
Elapsed time to handle talib:x86-windows: 40 s
```

Broken by microsoft#34316 which adds talib
@BillyONeal BillyONeal added category:port-bug The issue is with a library, which is something the port should already support info:internal This PR or Issue was filed by the vcpkg team. labels Oct 13, 2023
@BillyONeal BillyONeal merged commit 486a464 into microsoft:master Oct 14, 2023
@BillyONeal BillyONeal deleted the include-makefile-am branch October 14, 2023 02:43
@dg0yt
Copy link
Contributor

dg0yt commented Oct 14, 2023

A case for "restricted headers"?

@BillyONeal
Copy link
Member Author

A case for "restricted headers"?

Yes

@BillyONeal
Copy link
Member Author

A case for "restricted headers"?

microsoft/vcpkg-tool#1246

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 info:internal This PR or Issue was filed by the vcpkg team.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants