Skip to content

Fix: Needs to be removed workaround "Until the fb_assert could be con…#9044

Merged
AlexPeshkoff merged 1 commit into
FirebirdSQL:masterfrom
notesoft:prepare-for-CMake-Clang-install-services.cpp
Jun 3, 2026
Merged

Fix: Needs to be removed workaround "Until the fb_assert could be con…#9044
AlexPeshkoff merged 1 commit into
FirebirdSQL:masterfrom
notesoft:prepare-for-CMake-Clang-install-services.cpp

Conversation

@Khalyutin
Copy link
Copy Markdown
Contributor

…verted to a function/object linked with each module we need this ugly workaround. ... gds__log ..." in services.cpp - lld-link: error: duplicate symbol: gds__log; Defined at "yvalve_common.dir/yvalve/gds.cpp.obj" and src\utilities\install\services.cpp:567

[build] [ 70%] Linking CXX executable ../../firebird/instsvc.exe
[build] lld-link: error: duplicate symbol: gds__log
[build] >>> defined at \src\utilities\install\services.cpp:567
[build] >>>            CMakeFiles/instsvc.dir/install/services.cpp.obj
[build] >>> defined at ../CMakeFiles/yvalve_common.dir/yvalve/gds.cpp.obj
[build] clang++: error: linker command failed with exit code 1 (use -v to see invocation)

…verted to a function/object linked with each module we need this ugly workaround. ... gds__log ..." in services.cpp - lld-link: error: duplicate symbol: gds__log; Defined at "yvalve_common.dir/yvalve/gds.cpp.obj" and src\utilities\install\services.cpp:567

```
[build] [ 70%] Linking CXX executable ../../firebird/instsvc.exe
[build] lld-link: error: duplicate symbol: gds__log
[build] >>> defined at \src\utilities\install\services.cpp:567
[build] >>>            CMakeFiles/instsvc.dir/install/services.cpp.obj
[build] >>> defined at ../CMakeFiles/yvalve_common.dir/yvalve/gds.cpp.obj
[build] clang++: error: linker command failed with exit code 1 (use -v to see invocation)
```
@hvlad hvlad self-assigned this Jun 2, 2026
@AlexPeshkoff AlexPeshkoff merged commit a141ac8 into FirebirdSQL:master Jun 3, 2026
22 of 23 checks passed
@AlexPeshkoff
Copy link
Copy Markdown
Member

Vlad sorry noticed too late that you self assigned it. We discussed this issue...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants