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

[openssl] Enforce cmd for nmake #37716

Merged
merged 1 commit into from
Mar 27, 2024
Merged

Conversation

dg0yt
Copy link
Contributor

@dg0yt dg0yt commented Mar 26, 2024

Fixes #37134.
Fixes #35949.

Comment on lines 1 to 5
# Need cmd to pass quoted CC from nmake to mkbuildinf.pl, GH-37134
find_program(CMD_EXECUTABLE cmd HINTS ENV PATH NO_DEFAULT_PATH REQUIRED)
set(ENV{COMSPEC} "${CMD_EXECUTABLE}")

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this a general problem and should be moved into vcpkg_build_nmake ?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have the same question ;-) It could be moved one step further:
Is this a general problem for windows host and should be moved to vcpkg tool?

@FrankXie05 FrankXie05 self-assigned this Mar 26, 2024
@FrankXie05 FrankXie05 added the category:port-bug The issue is with a library, which is something the port should already support label Mar 26, 2024
@BillyONeal
Copy link
Member

Is this a general problem for windows host and should be moved to vcpkg tool?

I think the answer is yes. See also microsoft/vcpkg-tool#1201

@BillyONeal
Copy link
Member

BillyONeal commented Mar 26, 2024

@dg0yt dg0yt marked this pull request as ready for review March 27, 2024 06:08
@dg0yt
Copy link
Contributor Author

dg0yt commented Mar 27, 2024

I probaly won't touch these parts of the tools soon. I'm not operating under MSVS.

@BillyONeal BillyONeal merged commit 84e0086 into microsoft:master Mar 27, 2024
16 checks passed
@BillyONeal
Copy link
Member

BillyONeal commented Mar 27, 2024

I think the tool is the right place but this is better than nothing and already done :). Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category:port-bug The issue is with a library, which is something the port should already support
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[openssl] Build error on x64-windows [openssl] Build error on x64-windows
4 participants