alibuild icon indicating copy to clipboard operation
alibuild copied to clipboard

alibuild recipe (reference) : requirements

Open maireiphc opened this issue 3 years ago • 4 comments

Adding extra details about the logic I understood for the fields :

  • requires,
  • build_requires,
  • runtime_requires

This comes after some mail exchanges with Adrian Sevcenco, Giulio Eulisse + some local tests on one alidist recipe (lhapdf.sh). To be revised by Giulio to make sure that this is correct.

maireiphc avatar Aug 11 '22 14:08 maireiphc

Codecov Report

Merging #771 (4ab92a1) into master (09a707a) will not change coverage. The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #771   +/-   ##
=======================================
  Coverage   87.23%   87.23%           
=======================================
  Files          29       29           
  Lines        2984     2984           
=======================================
  Hits         2603     2603           
  Misses        381      381           

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

codecov-commenter avatar Aug 11 '22 14:08 codecov-commenter

Thank you for this. We should probably check the the runtime_requires behave correctly. I do no think we ever used them...

ktf avatar Aug 11 '22 15:08 ktf

I confess that I discovered the runtime_requires feature by re-reading your text a few hours ago (in the env variables). I extrapolated the text, I did not test the thing. I have been so far unaware of such a thing to exist. I indeed never stumbled upon runtime_requires in any recipe. At the moment, there is no occurrence of it among our alidist recipes.

maireiphc avatar Aug 11 '22 16:08 maireiphc