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
Windows build in github action #368
base: main
Are you sure you want to change the base?
Conversation
This script generates bash compatible environment variables from Windows environment variables.
fixed name of logs in github action "upload logs on failure"
Improvements of Windows Build: - run only one test for Win 10 in PRs - disable tests (takes to long currently and a lot of them are failing) - make sure that VS link ist used and not link from msys
|
||
OPTS="${EXTRA_OPTS} ${OPTS} $*" | ||
configure_ovs $OPTS | ||
make -j4 || { cat config.log; exit 1; } |
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.
Please change to make -j
so it will use as many threads as the container / VM has
make -j4 || { cat config.log; exit 1; } | ||
|
||
if [ "$TESTSUITE" ]; then | ||
if ! make check TESTSUITEFLAGS=-j8 RECHECK=yes; then |
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.
In this case we need to drop TESTSUITEFLAGS=-j8
OPTS: --prefix=C:/openvswitch/usr --localstatedir=C:/openvswitch/var --sysconfdir=C:/openvswitch/etc ${{ matrix.opts }} | ||
TESTSUITE: ${{ matrix.testsuite }} | ||
|
||
# linking on latest VS 2019 don't work, most propably due to a VS 2019 issue since VS 16.10 |
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.
This can be dropped.
Thank you for the PR @fw2568. Can you please squash all the commits and add a |
This PR adds a github action to build ovs for windows.
As discussed in openvswitch/ovs-issues#209 use is currently limited as tests cannot be run for windows for a PR and PR requests are not the primary workflow to submit changes to ovs.