Update VS solution/project files #2964
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- fix missing include dirs in RunRegistry.bat - update dependencies on IfW MD in solution - remove the compiler specification form the vfproj files (old ifx installs wouldn't work)
- InflowWind_driver - remove `registry` dependency (failed at linking, no other drivers have this) - add error handling in batch file to run the devenv builds with VS - drop the `_x64` part of names in test part of `deploy` script
Collaborator
Author
|
To do:
|
deslaughter
approved these changes
Aug 18, 2025
Collaborator
deslaughter
left a comment
deslaughter
left a comment
There was a problem hiding this comment.
It's working in Visual Studio 2022 and Intel oneAPI IFX 2025.2.1
andrew-platt
force-pushed
the
b/dev-tc_VSbuild
branch
from
5161117 to
7e74a3e
Compare
Collaborator
Author
|
Some issues remaining with the
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Ready to merge
Feature or improvement description
A few pieces were missing in the
RunRegistry.batafter mergingdevin.Related issue, if one exists
Reported internally
Impacted areas of the software
Compilation with VS on Windows, and on GH actions.
Additional supporting information
SeaState was coupled to InflowWind, but not updated in the VS project. This resulted in missing dependencies while building MoorDyn.
Test results, if applicable
No test results affected.