Skip to content

Disallow Geant4_julia_jll v0.2.3, it's broken#503

Closed
oschulz wants to merge 2 commits intomainfrom
pin-Geant4_julia_jll
Closed

Disallow Geant4_julia_jll v0.2.3, it's broken#503
oschulz wants to merge 2 commits intomainfrom
pin-Geant4_julia_jll

Conversation

@oschulz
Copy link
Member

@oschulz oschulz commented Jun 19, 2025

We can't limit Geant4_jll directly to avoid the length issue in v11.3.2+0, since

Geant4_jll = "11.2.0 - 11.3.2+0"

is not allowed, but limiting Geant4_julia_jll seems to work as well.

@fhagemann
Copy link
Collaborator

fhagemann commented Jun 19, 2025

Can we not wait until Geant4 v0.2.2 is released? And then let the General registry figure out, which versions are compatible with each other?

@fhagemann fhagemann marked this pull request as draft June 19, 2025 11:30
@oschulz
Copy link
Member Author

oschulz commented Jun 19, 2025

Can we not wait until Geant4 v0.2.2 is released? And then let the General registry figure out, which versions are compatible with each other?

People will have lot's of trouble at the pulse-shape simulation workshop next week, if SSD+Geant4 doesn't work out of the box ...

We can make a new release as soon as the new build is out and works.

@oschulz
Copy link
Member Author

oschulz commented Jun 19, 2025

There's really no downside to having an SSD version in between that restricts Geant4_jll.

@oschulz oschulz requested a review from fhagemann June 19, 2025 18:34
@oschulz
Copy link
Member Author

oschulz commented Jun 21, 2025

Ping @fhagemann

@oschulz oschulz marked this pull request as ready for review June 21, 2025 10:42
@fhagemann
Copy link
Collaborator

Can we wait until tomorrow evening and see if a new version of Geant4 gets released. The issue is, that within Geant4 length is used instead of Base.length. But as said, this is already fixed on main.

@fhagemann
Copy link
Collaborator

@fhagemann
Copy link
Collaborator

fhagemann commented Jun 21, 2025

Geant4 v0.2.2 was released.

Can we instead try restricting Geant4 to minimum 0.2.2, now that’s it’s released? This should fix the failing tests and the error reported here.

@fhagemann fhagemann closed this Jun 25, 2025
@fhagemann fhagemann deleted the pin-Geant4_julia_jll branch July 7, 2025 16:04
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