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

CI: Windows on GH Actions (MSVC, Clang) #840

Open
wants to merge 5 commits into
base: dev
Choose a base branch
from
Open

Conversation

ax3l
Copy link
Member

@ax3l ax3l commented Dec 9, 2020

Add two new builds on GH Actions: MSVC and Clang on Windows.

Clang on Windows support depends on #839

@ax3l ax3l added machine/system machine & HPC system specific issues continuous integration labels Dec 9, 2020
@ax3l ax3l force-pushed the ci-winClang branch 13 times, most recently from 95da488 to 32a2047 Compare December 9, 2020 04:02
@ax3l ax3l requested a review from franzpoeschel December 14, 2020 17:08
@ax3l ax3l force-pushed the ci-winClang branch 3 times, most recently from 0458a9a to bc6cd96 Compare December 23, 2020 10:17
@ax3l ax3l force-pushed the ci-winClang branch 10 times, most recently from 6512f0d to 8d0a974 Compare January 4, 2021 07:38
.github/workflows/windows.yml Outdated Show resolved Hide resolved
.github/workflows/windows.yml Outdated Show resolved Hide resolved
-DCMAKE_C_COMPILER=clang-cl ^
-DCMAKE_CXX_COMPILER=clang-cl ^
-DCMAKE_BUILD_TYPE=Release ^
-DCMAKE_VERBOSE_MAKEFILE=ON
Copy link
Member Author

@ax3l ax3l Jan 5, 2021

Choose a reason for hiding this comment

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

add Python_EXECUTABLE hint?

@ax3l ax3l force-pushed the ci-winClang branch 8 times, most recently from 4202632 to eb4bc51 Compare January 27, 2021 08:25
cmake --build . --config Release --parallel 2
if errorlevel 1 exit 1

dumpbin /dependents D:\a\openPMD-api\openPMD-api\build\lib\site-packages\openpmd_api\openpmd_api_cxx.cp39-win_amd64.pyd
Copy link
Member Author

@ax3l ax3l Jan 27, 2021

Choose a reason for hiding this comment

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

clang-cl (broken for Python modules only):

Dump of file D:\a\openPMD-api\openPMD-api\build\lib\site-packages\openpmd_api\openpmd_api_cxx.cp39-win_amd64.pyd

File Type: DLL

  Image has the following dependencies:

    openPMD.dll
    python39.dll
    MSVCP140.dll
    KERNEL32.dll
    VCRUNTIME140.dll
    api-ms-win-crt-stdio-l1-1-0.dll
    api-ms-win-crt-math-l1-1-0.dll
    api-ms-win-crt-runtime-l1-1-0.dll
    api-ms-win-crt-string-l1-1-0.dll
    api-ms-win-crt-heap-l1-1-0.dll

  Summary

        1000 .00cfg
        6000 .data
        1000 .gehcont
        E000 .pdata
       49000 .rdata
        1000 .reloc
        1000 .rsrc
      146000 .text
        1000 .tls

@ax3l ax3l force-pushed the ci-winClang branch 3 times, most recently from c9dda44 to e1c9dc9 Compare January 27, 2021 18:55
ax3l added 3 commits February 10, 2021 18:03
Add two new builds on GH Actions: MSVC and Clang on Windows.
Fails otherwise (now)
Fix test issue on conda-forge by adding the exact same hints as for
other python tests. Avoid appending the multi-config directory so
we can use the same path hints during test runs from the build dir.
@ax3l ax3l force-pushed the ci-winClang branch 4 times, most recently from c0fe0ec to be51b6f Compare February 11, 2021 04:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
continuous integration machine/system machine & HPC system specific issues
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants