iLCSoft / MarlinFastJet

Marlin processor to interface FastJet
GNU General Public License v3.0
2 stars 8 forks source link

Remove support for FastJet 2 #12

Closed andresailer closed 5 years ago

andresailer commented 5 years ago

Any objections to removing the support for FastJet 2 and requiring FastJet 3?

https://github.com/iLCSoft/MarlinFastJet/blob/7a71692e1de71d6a88408db90211649e3b883e13/CMakeLists.txt#L38

I want to use a function that only exists in FastJet3 and would like to avoid having to code around version dependencies.

rete commented 5 years ago

We use FastJet version 3.2.* since a long time now. ILCSoft v01-19-01 already uses FastJet 3.2.0. I guess we can remove the support for version 2.

andresailer commented 5 years ago

Thanks for the quick response, dropped the support in #13