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

CMake: Search for windows message compiler #3677

Merged
merged 1 commit into from Mar 26, 2019

Conversation

dnakamura
Copy link
Contributor

Don't just assume that its on the path

Signed-off-by: Devin Nakamura devinn@ca.ibm.com

Copy link
Contributor

@DanHeidinga DanHeidinga left a comment

Choose a reason for hiding this comment

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

lgtm

@DanHeidinga
Copy link
Contributor

Actually, looking at the appveyor build, I see it failed due to:

'CMAKE_MC_COMPILER-NOTFOUND' is not recognized as an internal or external command,
1207         operable program or batch file.

@dnakamura Is this a path setup issue?

Don't just assume that its on the path

Signed-off-by: Devin Nakamura <devinn@ca.ibm.com>
@dnakamura
Copy link
Contributor Author

@DanHeidinga The issue was that when cmake was run, mc was not on the path, but it was at build time. Have fixed by adding a hint which points to the windows SDK dir

@charliegracie
Copy link
Contributor

@genie-omr build win xlinux

set(kit_dir "${kit_dir}/bin/x86")
endif()
# find the message compiler
find_program(CMAKE_MC_COMPILER mc.exe HINTS "${kit_dir}")
Copy link
Contributor

Choose a reason for hiding this comment

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

Could the kit_dir be some call that a user could specify if they have it installed in a random location that they do not want to have on their path?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

no matter where they install it, the location should still be stored in the registry (which is where we pull kit_dir from). If for whatever reason its not in the registry you could still specify -DCMAKE_MC_COMPILER=xxxx on the command line.

@charliegracie charliegracie merged commit b6850a5 into eclipse:master Mar 26, 2019
@dnakamura dnakamura deleted the mc branch October 27, 2020 13:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants