Skip to content

Conversation

@deslaughter
Copy link
Collaborator

This PR is ready to be merged.

Feature or improvement description

This PR fixes the issue with building Windows executables in Github Actions. It also updates the action to use the latest Intel oneAPI Fortran Essentials toolkit with reduces the runtime to 1 hour and upgrades to the IFX compiler instead of IFORT. These changes worked when manually running the deploy workflow for this branch.

I think the main issue was leftover files from previous builds in the build/lib directory, so a command was used to remove it before building the Release_MATLAB configuration.

Impacted areas of the software

deploy.yml and build_windows_executables.bat

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR fixes Windows executable build failures in GitHub Actions by updating the Intel oneAPI installation process and cleaning up leftover build files. The changes upgrade from the older IFORT compiler to IFX and reduce workflow runtime to approximately 1 hour.

Changes:

  • Consolidated two separate Intel oneAPI installations (BaseKit + HPCKit) into a single Fortran Essentials 2025.3.0 installation
  • Added cleanup of build/lib directory before building the MATLAB configuration to prevent issues from leftover files

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
.github/workflows/deploy.yml Simplified Intel oneAPI installation by replacing BaseKit and HPCKit with Fortran Essentials toolkit
.github/scripts/build_windows_executables.bat Added build/lib directory cleanup before MATLAB solution build to prevent leftover file conflicts

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@andrew-platt andrew-platt merged commit 69cab72 into OpenFAST:dev Jan 21, 2026
29 of 30 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants