Skip to content

Add ze-validator and dependency updates - #122

Open
rpereira-dev wants to merge 2 commits into
argonne-lcf:mainfrom
anlsys:rpereira-dev/ze-validator-deps
Open

rpereira-dev wants to merge 2 commits into
argonne-lcf:mainfrom
anlsys:rpereira-dev/ze-validator-deps

Conversation

@rpereira-dev

@rpereira-dev rpereira-dev commented Sep 22, 2026 •

Copy link
Copy Markdown

This PR mostly adds dependencies required for ze_validator (the Level Zero validator tool):

  • rgl, a graph library used for cycle detections, that we will likely re-use for other validators in the future
  • rtree3, a red-black tree data-structure: ruby has none standard, and we needed one for tracking allocated memory

Additionally the PR:

  • Alike b24b8cc it adds a few depends_on("gmake", type="build") that were causing build errors on my system
  • Add a variant +ze-validator that:

@TApplencourt

TApplencourt commented Sep 23, 2026 •

Copy link
Copy Markdown
Collaborator

Alike b24b8cc it adds a few depends_on("gmake", type="build") that were causing build errors on my system

yeah know issue inside the spack. I opened it there. Look like every ruby gemm who build some natice c code will need that.

Suprised that old the ruby packaged need it...

See spack/spack-packages#5968

@TApplencourt

Copy link
Copy Markdown
Collaborator

LGTM. Can merge when the validator PR have been merged

@rpereira-dev

rpereira-dev commented Sep 23, 2026 •

Copy link
Copy Markdown
Author

I removed the dependency to rgl.
The lib was not really helpful for what we needed (detecting cycles, and returning the first one if any).
Reflected in THAPI with argonne-lcf/THAPI@e402379

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.

2 participants