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

[boost-modular-build-helper] Fix issue that link incorrect dependent libraries #9224

Merged

Conversation

PhoebeHui
Copy link
Contributor

@PhoebeHui PhoebeHui commented Dec 5, 2019

Related issue #7559 and #8613

In current Jamroot.jam, it configured both release and debug libraries, but doesn’t distinguish them, when build release libraries, it try to find dependency port debug libraries, that cause build failures.

To fix this issue, configured the Jamroot.jam before build release libraries, and configured it with debug build info again when build debug libraries.

boost-modular-build-helper has no features that need to test locally.

@PhoebeHui PhoebeHui added the info:internal This PR or Issue was filed by the vcpkg team. label Dec 5, 2019
@JackBoosY
Copy link
Contributor

/azp run

@JackBoosY JackBoosY added the info:reviewed Pull Request changes follow basic guidelines label Dec 16, 2019
@JackBoosY JackBoosY changed the title [boost-modular-build-helper] Fix issue that link incorrect dependent libraries. [boost-modular-build-helper] Fix issue that link incorrect dependent libraries Dec 16, 2019
@JackBoosY
Copy link
Contributor

/azp run

@JackBoosY JackBoosY removed the info:reviewed Pull Request changes follow basic guidelines label Jan 16, 2020
@yurybura
Copy link
Contributor

Related to #9780

@JackBoosY
Copy link
Contributor

Please resolve conflicts.

Thanks.

@jgiannuzzi
Copy link
Contributor

Hi there, I rebased @PhoebeHui's branch on the latest master and tested successfully on Linux and macOS: https:/jgiannuzzi/vcpkg/tree/boost-modular-build-helper.

@PhoebeHui could you please update this PR? Otherwise I can create a separate PR, whichever is easier for you!

@JackBoosY
Copy link
Contributor

@jgiannuzzi Feel free to open a new PR, thanks.

@JackBoosY JackBoosY closed this May 6, 2020
@PhoebeHui PhoebeHui deleted the dev/Phoebe/boost-modular-build-helper branch July 14, 2020 06:19
@PhoebeHui PhoebeHui restored the dev/Phoebe/boost-modular-build-helper branch July 21, 2020 01:31
@PhoebeHui PhoebeHui reopened this Jul 21, 2020
@PhoebeHui PhoebeHui marked this pull request as draft July 21, 2020 05:53
@PhoebeHui PhoebeHui marked this pull request as ready for review July 23, 2020 01:22
@PhoebeHui PhoebeHui added the category:port-bug The issue is with a library, which is something the port should already support label Jul 23, 2020
@PhoebeHui
Copy link
Contributor Author

@ras0219-msft, could you help review this PR?

@strega-nil strega-nil merged commit 0a16cad into microsoft:master Aug 11, 2020
@PhoebeHui PhoebeHui deleted the dev/Phoebe/boost-modular-build-helper branch October 22, 2020 08:27
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.

8 participants