Skip to content

add command line option to select single project configuration of loaded solution#2523

Merged
danmar merged 4 commits into
cppcheck-opensource:masterfrom
lg2de:CommandLineProject
Apr 19, 2020
Merged

add command line option to select single project configuration of loaded solution#2523
danmar merged 4 commits into
cppcheck-opensource:masterfrom
lg2de:CommandLineProject

Conversation

@lg2de

@lg2de lg2de commented Feb 5, 2020

Copy link
Copy Markdown
Contributor

In #1795 a command line option was introduced, but the PR was not completed.
Here is the change again, limited to command line access.
Thanks to @fuzzelhjb providing base implementation.

Maybe missing:

  • check whether --project is active and provided before --project-configuration

Comment thread cli/cmdlineparser.cpp Outdated
@danmar

danmar commented Feb 10, 2020

Copy link
Copy Markdown
Collaborator

Can you please review the manual also. See man/manual.md.

@danmar

danmar commented Feb 11, 2020

Copy link
Copy Markdown
Collaborator

I also have the feeling that you can limit the configurations to check with --platform.. If you use --platform=win64 then no win32 configuration should be analysed. Can you please double check this?

@lg2de

lg2de commented Feb 11, 2020

Copy link
Copy Markdown
Contributor Author

I also have the feeling that you can limit the configurations to check with --platform.. If you use --platform=win64 then no win32 configuration should be analysed. Can you please double check this?

No, I do not think so:
https://github.com/danmar/cppcheck/blob/099dd57eb800e8d21a1feb1107fc846ccfca0fb1/cli/cmdlineparser.cpp#L1103-L1104

This sound like a different option.

@danmar

danmar commented Feb 12, 2020

Copy link
Copy Markdown
Collaborator

ok.. I have the feeling that works in the GUI. Maybe we don't have that in CLI yet.

Comment thread lib/importproject.h Outdated
@danmar danmar merged commit 095fd2b into cppcheck-opensource:master Apr 19, 2020
@danmar

danmar commented Apr 19, 2020

Copy link
Copy Markdown
Collaborator

ok this will be available in next release.

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.

2 participants