conda-forge / openpmd-api-feedstock

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

Windows: Install Prefix #30

Closed ax3l closed 5 years ago

ax3l commented 5 years ago

I just learned that unix-like packages on conda-forge should be installed in a \Library prefix :) https://docs.conda.io/projects/conda-build/en/latest/user-guide/environment-variables.html#environment-variables-set-during-the-build-process

https://github.com/conda-forge/pybind11-feedstock/issues/30 Thanks @sdebionne !

Checklist

conda-forge-linter commented 5 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.

ax3l commented 4 years ago

Oh no, I just learned that we falsely assumed that Library\lib would work and be in %PATH% on user systems. Turns out, it is not and this is a defect :(

Ref.: https://github.com/ContinuumIO/anaconda-issues/issues/233#issuecomment-184455643

No detail on this in the official docs: https://docs.conda.io/projects/conda-build/en/latest/user-guide/environment-variables.html#environment-variables-set-during-the-build-process

We should go to Library\bin instead, which is in %PATH%...

cc @sdebionne

ax3l commented 4 years ago

Ah all good, our .dll is indeed installed in Library\bin :)