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

Two fixes to win32 cross compilation #32072

Merged
merged 1 commit into from
May 18, 2019
Merged

Two fixes to win32 cross compilation #32072

merged 1 commit into from
May 18, 2019

Conversation

Keno
Copy link
Member

@Keno Keno commented May 18, 2019

  • The default mingw32 gfortran on ubuntu prints the version as GNU Fortran (GCC) 7.3-win32 20180312,
    rather than 7.3.0 most other gfortrans do, so be robust to that in normalize_triplet
  • When shelling out to python use the host path rather than the windows-equivalent path, since python
    is a host tool.

- The default mingw32 gfortran on ubuntu prints the version as `GNU Fortran (GCC) 7.3-win32 20180312`,
  rather than `7.3.0` most other gfortrans do, so be robust to that in normalize_triplet
- When shelling out to python use the host path rather than the windows-equivalent path, since python
  is a host tool.
@Keno Keno requested a review from staticfloat May 18, 2019 17:34
@staticfloat staticfloat merged commit 3c92942 into master May 18, 2019
@staticfloat staticfloat deleted the kf/win32cross branch May 18, 2019 20:06
@ViralBShah
Copy link
Member

backport to 1.2?

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 this pull request may close these issues.

5 participants