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

vtk 9.0.1 #60548

Closed
wants to merge 7 commits into from
Closed

vtk 9.0.1 #60548

wants to merge 7 commits into from

Conversation

dtrodrigues
Copy link
Member

  • Have you followed the guidelines for contributing?
  • Have you checked that there aren't other open pull requests for the same formula update/change?
  • Have you built your formula locally with brew install --build-from-source <formula>, where <formula> is the name of the formula you're submitting?
  • Is your test running fine brew test <formula>, where <formula> is the name of the formula you're submitting?
  • Does your build pass brew audit --strict <formula> (after doing brew install <formula>)?

Previous work was done here: #58621

@dtrodrigues
Copy link
Member Author

gdcm build failure reported upstream: https://sourceforge.net/p/gdcm/bugs/509/

@dtrodrigues
Copy link
Member Author

pcl does not yet support vtk 9. Current progress here: PointCloudLibrary/pcl#4262

@SMillerDev
Copy link
Member

Maybe we can make an vtk@8 formula for those 2 formula and deprecate it right away?

@dtrodrigues dtrodrigues force-pushed the vtk-9.0.1 branch 2 times, most recently from faff26f to 25c9356 Compare September 7, 2020 16:07
@dtrodrigues
Copy link
Member Author

In summary:

  • opencv needed a small patch for vtk 9 which has been merged: vtkCocoaInteractorFix.mm: explicity include vtkVersion.h opencv/opencv_contrib#2659
  • vtk@8.2 was created in a deprecated state for the following three formulae
    • pcl: actively working on supporting vtk 9
    • gdcm: has a recently open issue about vtk 9 support
    • itk: looks like it builds with vtk 9 without gdcm, but since it's configured to use gdcm, it needs vtk@8.2

pcl uses HomeBrew in their CI, so getting this merged will facilitate support for them to upgrade.

@SMillerDev
Copy link
Member

For itk I'd say we drop gdcm support then.

@dtrodrigues
Copy link
Member Author

Removed -DModule_ITKLevelSetsv4Visualization=ON from itk per InsightSoftwareConsortium/ITK#739 (comment) and trying it without gdcm.

@dtrodrigues
Copy link
Member Author

dtrodrigues commented Sep 9, 2020

Looks like if you don't tell itk to use the system gdcm, itk will build gdcm and openjpeg from its ThirdParty directory and then will try and install the openjpen pkg-config file, which conflicts with the openjpeg Homebrew formula which is also being installed (see https://github.com/Homebrew/homebrew-core/pull/60548/checks?check_run_id=1086307019#step:5:13006). I've put the itk dependency on vtk@8.2 back in and gdcm back in.

Copy link
Member

@SMillerDev SMillerDev left a comment

Choose a reason for hiding this comment

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

Thanks @dtrodrigues ! Without your contributions it'd be impossible to keep homebrew going. You can feel good knowing that you've made the world a lot better for homebrew users around the world! 👍 🎉

@BrewTestBot
Copy link
Member

🤖 A scheduled task has triggered a merge.

@dtrodrigues dtrodrigues deleted the vtk-9.0.1 branch October 19, 2020 03:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants