conda-forge / pythran-feedstock

A conda-smithy repository for pythran.
BSD 3-Clause "New" or "Revised" License
1 stars 11 forks source link

Relax C++ compiler pin #65

Closed saraedum closed 2 years ago

saraedum commented 2 years ago

The runtime dependencies of pythran had compiler('cxx'). The intention here was to make sure that some C++ compiler is available. However, this actually forces this to be the compiler that was used when building the package, i.e., GCC 9. This causes trouble when trying to install a package that requires a newer compiler at the same time, e.g., when installing cppyy=2.3.0 and pythran=0.11.0.

saraedum commented 2 years ago

@conda-forge-admin, please rerender.

conda-forge-linter commented 2 years ago

Hi! This is the friendly automated conda-forge-linting service.

I wanted to let you know that I linted all conda-recipes in your PR (recipe) and found some lint.

Here's what I've got...

For recipe:

conda-forge-linter commented 2 years ago

Hi! This is the friendly automated conda-forge-linting service.

I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

saraedum commented 2 years ago

Maybe we don't actually want to fix this and just rerender the feedstock so it builds with the latest compilers instead.

serge-sans-paille commented 2 years ago

I understand the issue... I trust you on that one. Maybe require either clang++ or g++ ?

saraedum commented 2 years ago

Are these the only C++ compilers that are going to work?

serge-sans-paille commented 2 years ago

On Wed, Mar 16, 2022 at 05:38:12PM -0700, Julian Rüth wrote:

Are these the only C++ compilers that are going to work?

That's the only I have tested :-)