conda-forge / cuda-nvdisasm-feedstock

A conda-smithy repository for cuda-nvdisasm.
BSD 3-Clause "New" or "Revised" License
0 stars 8 forks source link

Why are compilers in the build requirements when nothing's being compiled? #9

Open danpetry opened 6 months ago

danpetry commented 6 months ago

Comment:

This also pulls in stuff to the run environment through strong run exports, maybe that's the intention?

jakirkham commented 6 months ago

To make sure the compiler runtime libraries are getting added

Is there a particular issue you are encountering? If so, could you please share some more details?

danpetry commented 6 months ago

Well, mostly we're just making sure we understand things during review. I guess you found it cleaner to add the compilers to the build requirements rather than list the runtime libs explicitly in run? I.e. the following libs are run dependencies:

libgcc-ng:     11.2.0-h1234567_1   defaults
libgomp:       11.2.0-h1234567_1   defaults
libstdcxx-ng:  11.2.0-h1234567_1   defaults

We are getting a warning saying that libstdcxx-ng isn't being used during the linking checks.

jakirkham commented 6 months ago

Yeah that's what we settled on

We can drop the {{ compiler("cxx") }} dependency if it is causing issues. Would you like to submit a PR removing it?

https://github.com/conda-forge/cuda-nvdisasm-feedstock/blob/0e7e587a10e444633e543bcf806cd6ff08f10876/recipe/meta.yaml#L32