CPP-1006 Initial version of a Windows config for Github Actions #567
Conversation
absurdfarce
changed the title
| OPENSSL_ROOT_DIR: C:/vcpkg/packages/openssl_x64-windows | ||
| steps: | ||
| - uses: actions/checkout@v4 | ||
| - uses: ilammy/msvc-dev-cmd@v1 |
There was a problem hiding this comment.
Appears to be necessary to setup MSVC env properly. More at the Githubs.
| - uses: actions/checkout@v4 | ||
| - uses: ilammy/msvc-dev-cmd@v1 | ||
| - name: Install dependencies | ||
| run: vcpkg install libuv zlib:x64-windows-static krb5 openssl |
There was a problem hiding this comment.
Static zlib required for build
| run: | | ||
| echo "PATH=${{ env.PATH }};${{ env.LIBUV_BIN_DIR }};${{ env.KERBEROS_BIN_DIR }};${{ env.OPENSSL_BIN_DIR }}" >> $env:GITHUB_ENV | ||
| - name: Fix name of static zlib dir | ||
| run: ln -s ${{ env.ZLIB_LIB_DIR }}/zlib.lib ${{ env.ZLIB_LIB_DIR }}/zlibstatic.lib |
There was a problem hiding this comment.
zlibstatic is apparently the expected name but the static build uses the default filename... so we manually fix it here
| @@ -22,7 +22,6 @@ | |||
| #include <string.h> | |||
|
|
|||
| #include <gssapi/gssapi.h> | |||
| #include <gssapi/gssapi_generic.h> | |||
There was a problem hiding this comment.
This header file isn't included in the krb5 vcpkg build. At least for modern versions of the gssapi lib this header isn't especially interesting so it seemed easy enough to remove.
|
Worth noting that this isn't 💯 yet. Outstanding issues at this writing:
I'm not necessarily looking for bullet-proof answers to each of these points before we merge this; I expect this process will iteratively get better over time. |
No description provided.