-
Notifications
You must be signed in to change notification settings - Fork 354
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
Adding enclave tests to Windows CI #1011
Conversation
bors try |
tryBuild succeeded |
steps { | ||
unstash 'linuxdebug' | ||
bat 'move build linuxbin' | ||
bat 'mkdir build && cd build && cmake -G "Visual Studio 15 2017 Win64" -DADD_WINDOWS_ENCLAVE_TESTS=1 -DLINUX_BIN_DIR=%cd%\\linuxbin\\tests .. && pushd . && "C:\\Program Files (x86)\\Microsoft Visual Studio\\2017\\BuildTools\\Common7\\Tools\\LaunchDevCmd.bat" && popd && cmake --build . --config Debug && ctest -V -C Debug' |
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.
Again, consider breaking this across lines for readability, or pushing this into a separate script file.
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.
@CodeMonkeyLeet Thanks Simon, I'll add a separate script file in my next PR
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.
LGTM
bors r+ |
1011: Adding enclave tests to Windows CI r=johnkord a=johnkord This enables the building of the OE tests on Linux, copying them to a Windows system, and then running the Windows build with the tests built on Linux. NOTE: I had to lower the enclave memory requirements to the props test in order for the test to pass on Windows. Please review this change carefully, as I'm not sure how low it should be. Co-authored-by: John Kordich <johnkord@microsoft.com>
Build succeeded |
This enables the building of the OE tests on Linux, copying them to a Windows system, and then running the Windows build with the tests built on Linux.
NOTE: I had to lower the enclave memory requirements to the props test in order for the test to pass on Windows. Please review this change carefully, as I'm not sure how low it should be.