Skip to content

[python3] Modernize port + added --enable-shared#9978

Merged
strega-nil merged 1 commit intomicrosoft:masterfrom
theblackunknown:integration-python3
Mar 10, 2020
Merged

[python3] Modernize port + added --enable-shared#9978
strega-nil merged 1 commit intomicrosoft:masterfrom
theblackunknown:integration-python3

Conversation

@theblackunknown
Copy link
Contributor

Modernize port + added Feature to request to build shared libraries

  • What does your PR fix?
    No logged issue, but the problem arose when trying to use this port to build Python extensions on Linux : shared library are needed, even when we are using VCPKG_LIBRARY_LINKAGE as static

  • Which triplets are supported/not supported? Have you updated the CI baseline?
    No CI update, works at least with windows & linux

  • Does your PR follow the maintainer guide?
    It should !

@JackBoosY
Copy link
Contributor

Will rerun ci test after #10051 merged.

@theblackunknown
Copy link
Contributor Author

Thank @JackBoosY !

@NancyLi1013
Copy link
Contributor

/azp run

@JackBoosY JackBoosY added the info:needs-maintainer-attention Lets the current 'on rotation' vcpkg maintainer know they need to look at this. label Feb 20, 2020
@JackBoosY
Copy link
Contributor

@strega-nil Could you please review and merge this PR?

@strega-nil
Copy link
Contributor

Thanks @theblackunknown :D

@strega-nil strega-nil merged commit e48fe56 into microsoft:master Mar 10, 2020
@theblackunknown theblackunknown deleted the integration-python3 branch March 10, 2020 22:00
Toxe pushed a commit to Toxe/vcpkg that referenced this pull request Mar 11, 2020
Toxe added a commit to Toxe/vcpkg that referenced this pull request Mar 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

info:needs-maintainer-attention Lets the current 'on rotation' vcpkg maintainer know they need to look at this.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants