spack icon indicating copy to clipboard operation
spack copied to clipboard

hepmc3: add v3.3.1

Open paulgessinger opened this issue 9 months ago • 10 comments

Adds version 3.3.1 of HepMC3.

paulgessinger avatar Mar 28 '25 12:03 paulgessinger

@luketpickering can you review this PR?

This PR modifies the following package(s), for which you are listed as a maintainer:

  • hepmc3

spackbot-app[bot] avatar Mar 28 '25 12:03 spackbot-app[bot]

@wdconinc added CMake constraint now. Can I do it like this with depends_on or do I need to introduce separate conflicts?

paulgessinger avatar Mar 29 '25 07:03 paulgessinger

@wdconinc added CMake constraint now. Can I do it like this with depends_on or do I need to introduce separate conflicts?

Yes, perfectly fine. You didn't even need to modify the older depends_on, and could have used:

    depends_on("[email protected]:", type="build")
    depends_on("[email protected]:", when="@3.3.1:", type="build")

since the second one will just override the first one for newer versions.

wdconinc avatar Mar 29 '25 14:03 wdconinc

There's an issue with ci that's under investigation. Might be Monday before we can merge this.

wdconinc avatar Mar 29 '25 14:03 wdconinc

@wdconinc CI failure seems unrelated

paulgessinger avatar Apr 29 '25 11:04 paulgessinger

The CI error seems to be due to whizard compiling with -std=c++11 even though it (through hepmc3?) depends on root compiled with C++20. @vvolkl You're a whizard user, right?

wdconinc avatar Apr 29 '25 13:04 wdconinc

Whizard issue hopefully resolved by https://github.com/spack/spack/pull/50255.

wdconinc avatar Apr 30 '25 18:04 wdconinc

The failure looks like ROOT's classic C++17 polyfill inconsistency, where ROOT itself is built with C++17 but a depending project is not.

I'm struggling to see how this can be related.

paulgessinger avatar May 09 '25 16:05 paulgessinger

Hi @paulgessinger, this package-related pull request can be migrated from spack/spack to spack/spack-packages using the migrate-pkg-prs utility.

We encourage authors to run the migration script themselves to preserve author attribution.

You can migrate all your open pull requests at once, following the steps in the documentation.


This comment was posted automatically.

haampie avatar Jun 06 '25 12:06 haampie

This pull request has been automatically marked as stale because it has not had any activity in the last 6 months. It will be closed in 30 days if there is no further activity.

If the pull request is waiting for a reply from reviewers, feel free to ping them as a reminder. If it is waiting and has no assigned reviewer, feel free to ping @spack/spack-releasers or simply leave a comment saying this should not be marked stale. This will reset the pull request's stale state.

To get more eyes on your pull request, you can post a link in the #pull-requests channel of the Spack Slack. Thank you for your contributions!

github-actions[bot] avatar Dec 04 '25 01:12 github-actions[bot]