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

Install also the Map++ API headers #2392

Merged
merged 2 commits into from
Aug 27, 2024
Merged

Conversation

sanguinariojoe
Copy link

Feature or improvement description
This PR just adds all the needed files to use the produced MAP++ library by 3rd parties.

On top of that, it renames the library as mappplib (Map++ library) and guard the header files under a mappp folder. Otherwise one might expect collisions with some other packages installed on the system (maplib, mapsys and mapapi seem like names that might happen even as system files on exotic OSs)

Related issue, if one exists
None

Impacted areas of the software
Map++ and the related CMake files

Additional supporting information
Not much else to say

Test results, if applicable
Nothing changed actually

  • Update the documentation version in docs/conf.py
  • Update the versions in docs/source/user/api_change.rst
  • Verify readthedocs builds correctly
  • Create a tag in OpenFAST
  • Create a merge commit in r-test and add a corresponding tag
  • Compile executables for Windows builds
    • FAST_SFunc.mexw64
    • OpenFAST-Simulink_x64.dll
    • openfast_x64.exe
    • DISCON.dll

@andrew-platt andrew-platt self-assigned this Aug 27, 2024
@andrew-platt andrew-platt self-requested a review August 27, 2024 16:27
@deslaughter deslaughter merged commit e204587 into OpenFAST:dev Aug 27, 2024
21 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.

3 participants