Skip to content

[vcpkg baseline] Disable paraview build on OSX#12757

Closed
JackBoosY wants to merge 1 commit intomicrosoft:masterfrom
JackBoosY:dev/jack/disable_paraview_osx_temp
Closed

[vcpkg baseline] Disable paraview build on OSX#12757
JackBoosY wants to merge 1 commit intomicrosoft:masterfrom
JackBoosY:dev/jack/disable_paraview_osx_temp

Conversation

@JackBoosY
Copy link
Contributor

@JackBoosY JackBoosY commented Aug 5, 2020

paraview:x64-osx regression will be fixed in #12434.
Error log:

ninja: error: '/Users/vagrant/Data/installed/x64-osx/lib/libmath.a', needed by 'bin/pvrenderserver', missing and no known rule to make it

Disable paraview build on OSX for unblcoking PR #12405 #12559.

@JackBoosY JackBoosY added category:vcpkg-bug The issue is with the vcpkg system (including helper scripts in `scripts/cmake/`) info:internal labels Aug 5, 2020
@JackBoosY JackBoosY requested a review from PhoebeHui August 5, 2020 08:48
@JackBoosY
Copy link
Contributor Author

@strega-nil Please merge this PR first.

@PhoebeHui PhoebeHui changed the title [vcpkg baseline] Temporary disable paraview build on OSX [vcpkg baseline] Disable paraview build on OSX Aug 5, 2020
@PhoebeHui PhoebeHui added the info:needs-maintainer-attention Lets the current 'on rotation' vcpkg maintainer know they need to look at this. label Aug 5, 2020
@Neumann-A
Copy link
Contributor

instead of reducing CI coverage I would rather see #12755 being merged instead of this one

@Neumann-A
Copy link
Contributor

@JackBoosY: close since #12755 has been merge instead

@strega-nil strega-nil closed this Aug 5, 2020
@JackBoosY JackBoosY deleted the dev/jack/disable_paraview_osx_temp branch August 5, 2020 21:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

category:vcpkg-bug The issue is with the vcpkg system (including helper scripts in `scripts/cmake/`) 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