Skip to content

First stab at adding basic windows MSVC test #1898

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.

Sign up for GitHub

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

Jump to bottom
Merged
willend merged 29 commits into from
Mar 24, 2025
Merged

First stab at adding basic windows MSVC test #1898

willend merged 29 commits into from
Mar 24, 2025

Conversation

@tkittel
Copy link
Contributor

@tkittel tkittel commented Mar 20, 2025

First stab at adding basic windows MSVC test

@farhi
Copy link
Contributor

farhi commented Mar 21, 2025

You seem to have a lot of fun @willend ;-) ?

@willend
Copy link
Contributor

willend commented Mar 21, 2025

@farhi as usual, Windows sucks a bit indeed. :-D

@willend
Copy link
Contributor

willend commented Mar 24, 2025

@tkittel #1899 should probably superseede this mess that I left here. :-D

willend added 21 commits March 24, 2025 14:14
@willend
Call mcrun with relative path, cat the config file
cd0ee5f
@willend
Add wget dependency
c914f21
@willend
Use ilammy/msvc-dev-cmd@v1
cfc11ec
@willend
Implement @MCPLFLAGS@ and use CMD(ncrystal-config --show buildflags) …
67a7cdd 
…on MCPL2 windows
@willend
Add MCPLFLAGS in parsing DEPENDENCIES
c187c63
@willend
Add msmpi
109404a
@willend
Tar up run_ folders and provide for download
55ad1ee
@willend
REVISIT Remove extra USE_MPI bracing of MPI_MASTER that handles this …
d5cca82 
…internally
@willend
Re-enable MCPL_output and MPI
8cbcab7
@willend
Put MCPL instrs in own test category
434a8fa
@willend
Add McXtrace test, but don't run MPI for MCPL yet
379e81c
@willend
Use ${{ matrix.os }} for output
ea8e1be
@willend
Path correction and test look at %MCXTRACE%
1236984
@willend
MPI_MASTER without USE_MPI for MCPL_input McXtrace
6683cd9
@willend
Try enabling MPI on MCPL_input McXtrace
319d3e2
@willend
MPI_MASTER fixes for McXtrace MCPL_output
1770c72
@willend
Enable MCPL_output MPI McXtrace
6f735be
@willend
mingw / gcc based workflow for mcstas windows
900801b
@willend
Add workflow for mingw McXtrace
af09c1e
@willend
Windows has its own basictest track
9c2da59
@willend
Add MCPLFLAGS in these config snips
00decf0
@willend
Copy link
Contributor

willend commented Mar 24, 2025

@tkittel all checks out - I will now merge here and close #1899

@willend willend merged commit fbd6ada into mccode-dev:main Mar 24, 2025
28 checks passed
@willend willend mentioned this pull request Mar 24, 2025
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@tkittel @farhi @willend