Skip to content

test Unix Makefiles #1279

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

Draft
wants to merge 55 commits into
base: main
Choose a base branch
from

Conversation

opensource-krzysztof
Copy link
Collaborator

@opensource-krzysztof opensource-krzysztof commented Apr 18, 2025

Unix Makefiles

@opensource-krzysztof opensource-krzysztof marked this pull request as ready for review April 18, 2025 09:10
@opensource-krzysztof opensource-krzysztof requested a review from a team as a code owner April 18, 2025 09:10
@lukaszstolarczuk lukaszstolarczuk changed the title test NMake Makefiles test Unix Makefiles Apr 18, 2025
@lukaszstolarczuk lukaszstolarczuk marked this pull request as draft May 6, 2025 12:35
@@ -233,6 +46,24 @@ jobs:

- name: Install dependencies
run: vcpkg install --triplet x64-windows
- name: Install llvm-mingw
run: |
curl -LO https://github.com/mstorsjo/llvm-mingw/releases/download/20250430/llvm-mingw-20250430-ucrt-x86_64.zip
Copy link
Contributor

Choose a reason for hiding this comment

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

mingw is probably not needed

@@ -256,12 +87,13 @@ jobs:
run: |
call "C:\Program Files (x86)\Intel\oneAPI\setvars.bat"
call "C:\Program Files (x86)\Intel\oneAPI\setvars-vcvarsall.bat"
bash
Copy link
Contributor

Choose a reason for hiding this comment

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

This is meant to be run with a powershell

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants