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

[bug] tools.microsoft.bash:path not set in test() if injected through a tool_requires #12335

Closed
SpaceIm opened this issue Oct 18, 2022 · 1 comment

Comments

@SpaceIm
Copy link
Contributor

SpaceIm commented Oct 18, 2022

Environment Details (include every applicable attribute)

  • Operating System+version: Windows 10
  • Compiler+version: Visual Studio 2017
  • Conan version: 1.52.0
  • Python version: the one of Windows agent in CCI

Steps to reproduce (Include if Applicable)

It's worth noting that in conan-io/conan-center-index#13584, msys2 is in tool_requires of gnu-config test package if build machine is windows, and msys2 properly set tools.microsoft.bash:subsystem and tools.microsoft.bash:path in its package_info() method.

I've tested locally on Windows with conan 1.53.0 and 2 profiles, and I did not noticed any issue, so I guess it's an issue either with conan 1.52.0 or 1 profile, since c3i builds msvc binaries with 1 profile and conan 1.52.0 currently.

Logs (Executed commands with output) (Include/Attach if Applicable)

ERROR: gnu-config/cci.20210814 (test package): Error in test() method, line 23
	self.run("config.guess")
	ConanException: The config 'tools.microsoft.bash:subsystem' and 'tools.microsoft.bash:path' are needed to run commands in a Windows subsystem
********************************************************************************
@SpaceIm
Copy link
Contributor Author

SpaceIm commented Oct 18, 2022

Well, it's the same issue than #11975, and fixed by #12095 in conan 1.53.0

@SpaceIm SpaceIm changed the title [bug] tools.microsoft.bash:path not set in test() if injected through a tool_requires with 1 profile [bug] tools.microsoft.bash:path not set in test() if injected through a tool_requires Oct 18, 2022
@SpaceIm SpaceIm closed this as completed Oct 18, 2022
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

No branches or pull requests

1 participant