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

[bde] Remove from the curated registry. #32601

Closed
wants to merge 1 commit into from

Conversation

BillyONeal
Copy link
Member

bde has been broken in vcpkg for a long time, as when we updated to a distro where python meant python3, it failed. #31961 , merged a couple of weeks ago, fixed this problem by updating to a more recent copy of bde. Unfortunately, the new copy of bde is link-incompatible with other vcpkg ports, as they vendor ryu and pcre2 with changes.

Given that this was broken for most customers for a long time anyway, I think we should deindex for now and bring it back if/when we have an understanding with upstream as to how this is to integrate with the rest of vcpkg' ecosystem.

bde has been broken in vcpkg for a long time, as when we updated to a distro where python meant python3, it failed. microsoft#31961 , merged a couple of weeks ago, fixed this problem by updating to a more recent copy of bde. Unfortunately, the new copy of bde is link-incompatible with other vcpkg ports, as they vendor ryu and pcre2 with changes.

Given that this was broken for most customers for a long time anyway, I think we should deindex for now and bring it back if/when we have an understanding with upstream as to how this is to integrate with the rest of vcpkg' ecosystem.
@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 Jul 17, 2023
@jimwang118 jimwang118 self-assigned this Jul 18, 2023
@jimwang118 jimwang118 added the info:reviewed Pull Request changes follow basic guidelines label Jul 18, 2023
@BillyONeal
Copy link
Member Author

@adamncasey you did a lot of work to update this version. Do you have ideas on what we could do instead? For example, if bde hides all these symbols and doesn't try to install patched headers that could work for dynamic releases...

@BillyONeal BillyONeal added requires:vcpkg-team-review This PR or issue requires someone on the vcpkg team to take a further look. and removed info:reviewed Pull Request changes follow basic guidelines labels Jul 18, 2023
@jimwang118 jimwang118 marked this pull request as draft July 19, 2023 02:07
@adamncasey
Copy link
Contributor

Thanks for raising - this is being looked into. There is a workable way forward here for ryu/pcre2 but suspect we won't be able to address this in the next few days.

@BillyONeal
Copy link
Member Author

Thanks for raising - this is being looked into. There is a workable way forward here for ryu/pcre2 but suspect we won't be able to address this in the next few days.

It doesn't need to be "next few days" as long as it is being worked on. We can "skip" it in our CI testing temporarily if you're actually working on it like we just did for openvino. How about we skip for, say, 30 days, and check back afterwards?

BillyONeal added a commit to BillyONeal/vcpkg that referenced this pull request Jul 19, 2023
Replaces microsoft#32601 ; will proceed with deindex only after some time for bde maintainers to fix this.
@BillyONeal BillyONeal closed this Jul 19, 2023
BillyONeal added a commit that referenced this pull request Jul 19, 2023
Replaces #32601 ; will proceed with deindex only after some time for bde maintainers to fix this.
@BillyONeal BillyONeal deleted the deindex-bde branch July 28, 2023 19:21
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. requires:vcpkg-team-review This PR or issue requires someone on the vcpkg team to take a further look.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants