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 - Run HEMTT on Windows for binarization #9590

Merged
merged 2 commits into from
Nov 1, 2023

Conversation

Dahlgren
Copy link
Member

@Dahlgren Dahlgren commented Oct 26, 2023

When merged this pull request will:

  • Run a HEMTT Windows build on pull requests and after merge to main
  • Make the Arma 3 Tools made available via arma-actions/arma3-tools for access to binarize_x64.exe, requires pull_request_target instead of pull_request
  • Using pull_request_target means it will use files from base branch by default. To get around this it will copy addons, optionals and include from the PR files into build directory.

Example Windows build: https://github.com/Dahlgren/ACE3/actions/runs/6658953635/job/18096958599, I accidentally flipped the names during test build so -nobin is the binarized one from Windows

IMPORTANT

  • If the contribution affects the documentation, please include your changes in this pull request so the documentation will appear on the website.
  • Development Guidelines are read, understood and applied.
  • Title of this PR uses our standard template Component - Add|Fix|Improve|Change|Make|Remove {changes}.

@jonpas jonpas added area/tools ignore-changelog Release Notes: Excluded labels Oct 26, 2023
@jonpas jonpas added this to the Ongoing milestone Oct 26, 2023
@jonpas jonpas changed the title ci: Run HEMTT on Windows CI - Run HEMTT on Windows for binarization Oct 26, 2023
@jonpas jonpas merged commit dc944b1 into acemod:master Nov 1, 2023
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/tools ignore-changelog Release Notes: Excluded
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants