Skip to content

[vcpkg baseline][libfabric] Only support dynamic build#11647

Merged
dan-shaw merged 1 commit intomicrosoft:masterfrom
JackBoosY:dev/jack/libfabric_ony_support_dynamic
Jun 3, 2020
Merged

[vcpkg baseline][libfabric] Only support dynamic build#11647
dan-shaw merged 1 commit intomicrosoft:masterfrom
JackBoosY:dev/jack/libfabric_ony_support_dynamic

Conversation

@JackBoosY
Copy link
Contributor

There is no configuration information for building a static library in libfabric.vcxproj, so static building is disabled.

Related: #11637.

@JackBoosY JackBoosY added category:port-bug The issue is with a library, which is something the port should already support info:internal labels May 29, 2020
@JackBoosY JackBoosY changed the title [libfabric] Only support dynamic build [vcpkg baseline][libfabric] Only support dynamic build May 29, 2020
@JackBoosY JackBoosY marked this pull request as ready for review May 29, 2020 11:35
@JackBoosY JackBoosY requested a review from PhoebeHui May 29, 2020 11:35
@@ -746,6 +746,7 @@ libfabric:x64-linux=fail
libfabric:x64-osx=fail
libfabric:x64-uwp=fail
libfabric:x64-windows=ignore
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's a bit strange to have fail and ignore for the same port?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't know...Maybe this port has file conflicts with other ports.

@dan-shaw
Copy link
Contributor

dan-shaw commented Jun 2, 2020

Please tag 'reviewed' when ready to merge

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

@dan-shaw Added.

@dan-shaw dan-shaw merged commit 5be0414 into microsoft:master Jun 3, 2020
@JackBoosY JackBoosY deleted the dev/jack/libfabric_ony_support_dynamic branch June 3, 2020 06:43
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: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