STARlight introduction in AliGenerators

[jackal1-66]

Added STARlight dependency to AliGenerators. STARlight recipe has been edited to pull directly the main branch from the official STARlight repository. In addition the HepMC3 package has been set as a dependency and STARlight is now built with it, so that it could also be useful for RIVET analyses. Tests performed locally are encouraging.

[jackal1-66]

@ktf the error on alidist seems unrelated to the introduction of STARlight. Could you please check?

[ktf]

Fine with me. Just fix up the quotes and then I will merge.

[jackal1-66]

Fine with me. Just fix up the quotes and then I will merge.

I added the double quotes and noticed there was HepMC2 instead of 3. It was working anyway, but better upgrade that little part as well. Tested locally and it works.

[ktf]

Thanks! Could you please use a tagged version as well?

I also noticed the recipe generates its own modulefile. It's probably better to move to use alibuild-generate-module so that eventual runtime dependencies are taken into account. Grep for it in alidist to see how it's used elsewhere.

[ktf]

(the last point can be done in a separate PR, if you prefer)

[jackal1-66]

Thanks! Could you please use a tagged version as well?

I also noticed the recipe generates its own modulefile. It's probably better to move to use alibuild-generate-module so that eventual runtime dependencies are taken into account. Grep for it in alidist to see how it's used elsewhere.

I will move to alibuild-generate-module on a different PR. I prefere being sure that everything works as expected before moving to that (minimum changes at a time).

[ktf]

Thank you for the fixes. Let's have a last spin on the AliGenerators test and then I will merge.

[jackal1-66]

build/O2/alidist-dataflow-cs8 failing is unrelated to this PR

[jackal1-66]

The current alidist failures are not related to this PR.

[chiarazampolli]

Ported to async-v1-01-03, label removed.