Skip to content

Conversation

@michaelmckinsey1
Copy link

@michaelmckinsey1 michaelmckinsey1 commented Nov 6, 2025

These are package updates related to the v0.5.0 release of Adiak (https://github.com/LLNL/Adiak/releases/tag/v0.5.0)

  • Add 0.5.0 as a new version
  • Adds Python bindings to Adiak llnl/Adiak#38 Added python bindings to Adiak. This PR enables building with +python to enable the bindings on master. I have tested these changes manually in conjunction with the caliper python bindings, which already exist.

@michaelmckinsey1
Copy link
Author

@spackbot maintainers

@michaelmckinsey1
Copy link
Author

@daboehme @mplegendre

@michaelmckinsey1 michaelmckinsey1 changed the title adiak: Add Python Bindings to Adiak Package adiak: Update Package for v0.5.0 Release Dec 4, 2025
@daboehme
Copy link
Contributor

Looks good to me

depends_on("mpi", when="+mpi")

depends_on("python@3", when="+python", type=("build", "link", "run"))
depends_on("[email protected]:", when="+python", type=("build", "link", "run"))
Copy link
Member

Choose a reason for hiding this comment

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

If you add extends(python, when=+python), then it should not be required to define setup_run_environment below; you could bundle that with the 2 above statements like:

    with when("+python"):
        extends("python")
        depends_on("python@3:", type=("build", "link", "run"))
        depends_on("[email protected]:", type=("build", "link", "run"))

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants