Skip to content

Enable BUILD_CORE_DLL#8016

Open
pfultz2 wants to merge 2 commits into
cppcheck-opensource:mainfrom
pfultz2:build-core-dll
Open

Enable BUILD_CORE_DLL#8016
pfultz2 wants to merge 2 commits into
cppcheck-opensource:mainfrom
pfultz2:build-core-dll

Conversation

@pfultz2

@pfultz2 pfultz2 commented Dec 6, 2025

Copy link
Copy Markdown
Contributor

No description provided.

@firewave

firewave commented Dec 7, 2025

Copy link
Copy Markdown
Collaborator

The build is still not tested. The DLL it produces is also unusable because we generate actual libraries for everything (which I stated should not be done). That feature should just be removed and Visual Studio should also not generate a DLL (as also I stated before).

@danmar

danmar commented Dec 8, 2025

Copy link
Copy Markdown
Collaborator

@firewave could you explain the overall problem with generating a DLL that is reused by both the GUI and CLI? It sounds like a good idea to me technically to generate it.. however it must of course be tested in some way..

@pfultz2

pfultz2 commented Dec 9, 2025

Copy link
Copy Markdown
Contributor Author

The build is still not tested.

I did test this on linux/mac and it does work. I dont have access to windows to test it though.

The DLL it produces is also unusable because we generate actual libraries for everything (which I stated should not be done). That feature should just be removed and Visual Studio should also not generate a DLL (as also I stated before).

It only builds a DLL for cppcheck-core.

All the other libraries that are generated are static and are linked into the executable or the DLL, so they can be discarded and do not make the DLL unusable.

@sonarqubecloud

sonarqubecloud Bot commented Dec 9, 2025

Copy link
Copy Markdown

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