Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Remove some includes and fix a couple of URLs #696

Merged
merged 1 commit into from
Oct 17, 2024
Merged

Remove some includes and fix a couple of URLs #696

merged 1 commit into from
Oct 17, 2024

Conversation

jmcarcell
Copy link
Member

BEGINRELEASENOTES

  • Remove some includes and fix a couple of URLs

ENDRELEASENOTES

@tmadlener
Copy link
Collaborator

Something broke for Julia, that was working before.

@peremato do you have any idea whether a recent Julia update leads to changed behavior that would require writing something back to the Julia installation (which is on CVMFS) rather than some other directory? https://github.com/AIDASoft/podio/actions/runs/11380288908/job/31659310951?pr=696#step:4:783

@andresailer
Copy link
Member

Maybe setting JULIA_DEPOT_PATH to some /tmp folder? (It is the only thing I know about julia...)

@tmadlener
Copy link
Collaborator

See details in https://its.cern.ch/jira/browse/SPI-2666

@tmadlener tmadlener merged commit 87e0fee into AIDASoft:master Oct 17, 2024
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tmadlener tmadlener tmadlener approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@jmcarcell @tmadlener @andresailer