Skip to content
This repository was archived by the owner on Nov 10, 2025. It is now read-only.

specify absolute path of compiler envs#20

Open
minrk wants to merge 1 commit intoconda-forge:mainfrom
minrk:abspath-compilers
Open

specify absolute path of compiler envs#20
minrk wants to merge 1 commit intoconda-forge:mainfrom
minrk:abspath-compilers

Conversation

@minrk
Copy link
Member

@minrk minrk commented Mar 11, 2017

ensures that enabling toolchain specifies a specific compiler on the system, and not the one that may be installed in the env with the same name, unless explicitly requested.

This is the right thing to do for halide (conda-forge/staged-recipes#2559 (comment)), but may not be what people want in general. When people install gcc as a build dep (and toolchain), do they expect the one in the env to be used by default without being explicitly selected? Should they?

ensures that enabling toolchain specifies a specific compiler on the system, and not the one that may be installed in the env with the same name, unless explicitly requested.

This is the right thing to do for halide (conda-forge/staged-recipes#2559 (comment)), but may not be what people want in general. When people install gcc as a build dep (and toolchain), do they expect the one in the env to be used by default without being explicitly selected? Should they?
@conda-forge-linter
Copy link

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.

@isuruf
Copy link
Member

isuruf commented Apr 27, 2017

Another problem with not using absolute path is that if the gcc conda package is used, there is a mismatch between the standard library used. gcc conda package uses 4.8.5 while the docker container has 4.8.2. See conda-forge/ipopt-feedstock#5 for a mismatch in GLIBCXX.

cc @conda-forge/core

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants