-
Notifications
You must be signed in to change notification settings - Fork 519
Paraview 6 updates #192
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
base: develop
Are you sure you want to change the base?
Paraview 6 updates #192
Conversation
31aa625 to
f8c53cc
Compare
| for plat in ["linux", "darwin", "freebsd"]: | ||
| # Handle X11 dependencies | ||
| # X is only used on Unix like platforms | ||
| conflicts("glx", when="~x") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is wayland unsupported?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It isn't not supported, but I think the state of the art for gl on wayland is to use glx + xwayland.
vicentebolea
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ParaView bites looks great
f8c53cc to
7f2ba1a
Compare
vicentebolea
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
libxslt is failing in the CI
|
Added remaining missing maintainer. |
|
FYI. |
8f25dff to
3de6adc
Compare
cbc67c7 to
c9d95d4
Compare
7b8cd9a to
a434755
Compare
e173af6 to
bea28fe
Compare
This is needed by some packages to override CMakes default FindPackage behavior.
Co-authored-by: Kenneth Moreland <morelandkd@ornl.gov>
This is a half fix. Projects that use the glvnd targets from CMake will fail due to inconsistent detection of the GL libraries regardless of settings.
libpxbackend is needed by the top-level
3a6711f to
7c6875e
Compare
|
@kwryankrattiger I can't help but note that this PR is quite old and we are still waiting for ParaView 6 to be available in Spack. What is the status of this? |
|
There are a couple of files with conflicts that need to be resolved. |
Yes, it is quite old :( The main blocker on this was that an unrelated package was getting stuck on a concretization error. To address this more systematically the X, GL, and Qt updates were extracted into separate PRs. These two PRs that have since found themselves hung up in review:
These are being ignored, there will be a rebase to only contain the paraview related changes at some point. I am hopeful to remaining two PRs will get merged sooner rather than later. |
* Updates to X related packages spack#192 * fix typo in options Co-authored-by: Wouter Deconinck <wdconinc@gmail.com> --------- Co-authored-by: Wouter Deconinck <wdconinc@gmail.com>
Updates for ParaView 6