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

Change OMCppOMCFactory to static library. #10515

Merged
merged 1 commit into from
Apr 7, 2023

Conversation

mahge
Copy link
Contributor

@mahge mahge commented Apr 7, 2023

  • This was overlooked and made different from the other build system (the original CMake build for CPP runtime used by the OpenModelica autoconf build.)

  • Normally for Linux, macOS and MinGW this was insignificant. However, for MSVC it matters because the library does not export any symbols. For MSVC, when built as DLL, this implies that the library is not supposed to be linked directly (it will only be loaded). So it will not generate the import lib for it.

  - This was overlooked and made different from the other build system
    (the original CMake build for CPP runtime used by the OpenModelica
    autoconf build.)

  - Normally for Linux, macOS and MinGW this was insignificant. However,
    for MSVC it matters because the library does not export any symbols.
    For MSVC, when built as DLL, this implies that the library is not supposed
    to be linked directly (it will only be loaded). So it will not generate
    the import lib for it.
@mahge mahge added COMP/SimRT/C++ Issues and pull requests related to the C++ runtime COMP/Build System Issue and pull request related to the build systems labels Apr 7, 2023
@mahge mahge self-assigned this Apr 7, 2023
@mahge mahge enabled auto-merge (squash) April 7, 2023 09:24
@mahge mahge merged commit 2414d51 into OpenModelica:master Apr 7, 2023
@mahge mahge deleted the minor_fixes_w branch April 7, 2023 09:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
COMP/Build System Issue and pull request related to the build systems COMP/SimRT/C++ Issues and pull requests related to the C++ runtime
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant