Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fmt.dll ends up in wrong directory on Windows #5

Closed
robquant opened this issue Nov 29, 2019 · 9 comments
Closed

fmt.dll ends up in wrong directory on Windows #5

robquant opened this issue Nov 29, 2019 · 9 comments

Comments

@robquant
Copy link
Contributor

On Windows the shared library fmt.dll is installed in %CONDA_PREFIX%\Library\lib (as is the static lib fmt.lib). However, the canonical path for shared libs in conda environments is %CONDA_PREFIX%\Library\bin. Therefore the shared lib is not found when running a programm that is linked against it.

@tdegeus
Copy link
Member

tdegeus commented Nov 29, 2019

@robquant I'm not a Windows expert. Could you confirm that this fmtlib/fmt#1432 would fix the issue?

@tdegeus
Copy link
Member

tdegeus commented Nov 29, 2019

@robquant Also can you confirm if/that CMake also still knows where to look on Windows?

@robquant
Copy link
Contributor Author

@tdegeus I tried your fix, but needed to make some changes to it: fmtlib/fmt#1437
With this the dll is copied to the right place and cmake is also able to link to it later.

@tdegeus
Copy link
Member

tdegeus commented Nov 29, 2019

Great @robquant !

It would be great @vitaut if you could release a new version if fmtlib/fmt#1437 is merged. The alternative would be to add patches to the feedstock here (e.g. #6 ), but given the number of commits since the last release this becomes a bit clumsy.

@tdegeus
Copy link
Member

tdegeus commented Nov 29, 2019

And thanks @robquant for the report, and the improvement of my PR.

@robquant
Copy link
Contributor Author

Thanks @tdegeus for pointing me to the right place in the CMakeLists.txt!

@vitaut
Copy link

vitaut commented Nov 30, 2019

I'm currently working on putting together a new release and fmtlib/fmt#1437 can be part of it.

@tdegeus
Copy link
Member

tdegeus commented Dec 4, 2019

@robquant Can you confirm that this issue is fixed in version 6.1.0?

@robquant
Copy link
Contributor Author

robquant commented Dec 4, 2019

@tdegeus Yes, fmt.dll is now in the right place!

@robquant robquant closed this as completed Dec 4, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants