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: fix empty string warnings #7288
Conversation
@babsingh Could you please have a look at this? |
jenkins build all |
Something is still wrong: In draft mode while I sort that out. |
Prior to this change, numerous warnings are issued by cmake: CMake Warning: Ignoring empty string ("") provided on the command line. Rework how the decision to use 'cygpath -w' is made: Provide USE_PATH_TOOL to GenerateStub.cmake so it can behave as desired. Signed-off-by: Keith W. Campbell <keithc@ca.ibm.com>
I think the problem here was that the test in
was failing because |
0bd1949
to
1691da5
Compare
Tested internally. |
jenkins build all |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Only a known and unrelated failure is seen #7181.
Prior to this change, numerous warnings are issued by cmake:
Rework how the decision to use
cygpath -w
is made: ProvideUSE_PATH_TOOL
toGenerateStub.cmake
so it can behave as desired.This has the same goal as #7283, but without the problem (eclipse-openj9/openj9#19210) it introduced.