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

mingw tool not using the same search paths between exists and generate methods #4134

Closed
jcbrill opened this issue Apr 13, 2022 · 0 comments · Fixed by #4135
Closed

mingw tool not using the same search paths between exists and generate methods #4134

jcbrill opened this issue Apr 13, 2022 · 0 comments · Fixed by #4135

Comments

@jcbrill
Copy link
Contributor

jcbrill commented Apr 13, 2022

The current implementation of the mingw tool augments the mingw search paths for version specific mingw-w64 folders in the generate function but fails to use the same search paths in the exists function.

mingw-w64 folders installed as C:\mingw-w64\<SPECIFICVERSION>\mingw64 are detected in the generate function. Therefore, the following environment works as expected:

env = Environment(tools=['mingw'])

However, if there are no versions of msvc installed, the following fails due to attempting to build with the msvc tools:

env = Environment()

Because the mingw-w64 specific version folders are not considered in the exists function, the mingw tool is filtered from the win32 compiler candidate list. When the only compiler installed is mingw, this causes the installed list to be empty in which case the default tool is msvc. This causes the default environment to fail without msvc installed.

Additional Information:

  • SCons master branch
  • Python 3.6.8 64-bit (embedded and winpython)
  • Neither embedded or winpython requires installation
  • Windows 10 VMWare Virtual Machine
  • Mingw-w64 installed to C:\mingw-w64\mingw-1110-v9-r0-64-win32-sjlj\mingw64
  • Visual Studio not installed.
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.

1 participant