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

[question] automatic PATH prepending with the "bin" folder is disabled in conan 2.0 ? #13353

Closed
1 task done
todorico opened this issue Mar 7, 2023 · 3 comments
Closed
1 task done
Assignees

Comments

@todorico
Copy link

todorico commented Mar 7, 2023

What is your question?

Hi ! Recently I tried using Conan 2.0 on the recipes I maintain to be compatible with both 1.x and 2.x. However, there was a difference that I wasn't expecting. Unlike Conan 1.x, the bin folder wasn't automatically prepended to the PATH variable in Conan 2.x. It forces me to add buildenv_info / runenv_info to a lot of my recipes. Is this the intended behavior ? I couldn't find any reference to this behavior in the "Conan migration guide to 2.0".

Have you read the CONTRIBUTING guide?

  • I've read the CONTRIBUTING guide
@memsharded
Copy link
Member

Hi @todorico

No, this is definitely not expected, we would need more details to reproduce.
The bin folder of tool_requires should be automatically added to PATH in the consumers, automatically injected by the implicit VirtualBuildEnv usage (no need to even declare it).

There was a bug when using MesonToolchain, that has been fixed in #13346. If you were using MesonToolchain, then that could be the cause. If not, please report how to reproduce this. Thanks!

@todorico
Copy link
Author

Hi @memsharded,

After further investigation, I realized that I must have made a mistake in my initial report. I have been unable to reproduce the issue that I reported earlier, and the problem seems to have been resolved on its own. Therefore, I believe that this issue can now be closed.

Thank you for your time and effort in helping me resolve this issue. Your assistance is greatly appreciated!

@memsharded
Copy link
Member

Happy that the issue has been resolved :) closing the ticket.

Glad to help!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants