Skip to content

Revive this package#43

Merged
Moelf merged 5 commits intomainfrom
update
Aug 3, 2025
Merged

Revive this package#43
Moelf merged 5 commits intomainfrom
update

Conversation

@Moelf
Copy link
Member

@Moelf Moelf commented Aug 2, 2025

@jstrube
Copy link
Member

jstrube commented Aug 3, 2025

lciowrap is defined here: https://github.com/JuliaHEP/LCIO_Julia_Wrapper/blob/main/src/lciowrap.cc. Strange that it's not found.

@jstrube
Copy link
Member

jstrube commented Aug 3, 2025

lciowrap is defined here: https://github.com/JuliaHEP/LCIO_Julia_Wrapper/blob/main/src/lciowrap.cc. Strange that it's not found.

nvm. The latest commit tries to import that from LCIO_jll, that's why it's not found.

Copy link
Member

@jstrube jstrube left a comment

Choose a reason for hiding this comment

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

try:
@wrapmodule(() -> "lciowrap.so", :define_julia_module)

@Moelf
Copy link
Member Author

Moelf commented Aug 3, 2025

i had a copy working somewhere, I will remind myself what I did lol

@Moelf
Copy link
Member Author

Moelf commented Aug 3, 2025

@Moelf
Copy link
Member Author

Moelf commented Aug 3, 2025

ok this works locally with the JuliaPackaging/Yggdrasil#11769

@Moelf Moelf merged commit 5cad523 into main Aug 3, 2025
2 of 3 checks passed
@Moelf Moelf deleted the update branch August 3, 2025 23:33
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