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.

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

Cyclical link dependencies FastJet addon <-> FccAnalyses? #417

Open
vvolkl opened this issue Oct 28, 2024 · 1 comment
Open

Cyclical link dependencies FastJet addon <-> FccAnalyses? #417

vvolkl opened this issue Oct 28, 2024 · 1 comment

Comments

@vvolkl
Copy link
Member

vvolkl commented Oct 28, 2024

The FastJet addon uses symbols from JetClusteringUtils.cc (here for example) and should actually link to the FccAnalyses target where JetClusteringUtils.cc is built.
But the FccAnalyses target links against the FastJet addon here.

Not sure why the linker does not complain ( it does with apple-clang 16 on macos, where I noticed this). Maybe JetClustering.cc should be moved back to analyzers/dataframe?

@jmcarcell
Copy link
Contributor

I have also found this issue. This should indeed by fixed if we want to compile on MacOS. The options are either to move back JetClusteringUtils.cc to analyzers/dataframe or create an additional library with jet clustering utilities that could be linked in the FastJet plugin.

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

No branches or pull requests

2 participants