-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Remove vtk from CMake build #6476
Remove vtk from CMake build #6476
Conversation
+@jwnimmer-tri for feature review. |
+@soonho-tri for platform review, please. Reviewed 8 of 8 files at r1. Comments from Reviewable |
BTW, a related issue: When this PR gets merged, can we update http://drake.mit.edu/mac.html to remove Reviewed 8 of 8 files at r1. Comments from Reviewable |
OK. We don't have this one anymore at least. Review status: all files reviewed at latest revision, all discussions resolved, some commit checks failed. Comments from Reviewable |
Still needed for Director for the moment, though that will change in the next few days. Review status: all files reviewed at latest revision, all discussions resolved, some commit checks failed. Comments from Reviewable |
I see. Thanks! Review status: all files reviewed at latest revision, all discussions resolved, some commit checks failed. Comments from Reviewable |
Reviewed 4 of 4 files at r2. Comments from Reviewable |
1 similar comment
Reviewed 4 of 4 files at r2. Comments from Reviewable |
Reviewed 4 of 4 files at r3. Comments from Reviewable |
Toward #3129.
This change is