Skip to content

added CMake option EXTERNALS_AS_SYSTEM to treat external includes as SYSTEM ones#5386

Merged
danmar merged 2 commits intocppcheck-opensource:mainfrom
firewave:ext-as-sys
Jan 3, 2024
Merged

added CMake option EXTERNALS_AS_SYSTEM to treat external includes as SYSTEM ones#5386
danmar merged 2 commits intocppcheck-opensource:mainfrom
firewave:ext-as-sys

Conversation

@firewave
Copy link
Copy Markdown
Collaborator

@firewave firewave commented Sep 1, 2023

Although these files are part of our repo changes are being done via their original projects so it might make sense to treat these as system includes for some people instead of local ones.

@danmar
Copy link
Copy Markdown
Collaborator

danmar commented Sep 1, 2023

This is probably something that it's best to write a separate trac ticket about. It's intended as an improvement for package maintainers right?

@firewave
Copy link
Copy Markdown
Collaborator Author

firewave commented Sep 1, 2023

This is probably something that it's best to write a separate trac ticket about. It's intended as an improvement for package maintainers right?

Maybe. I assume there might be platforms at some point which do want to suppress this.
Nobody requested it, but in the past I have been flip-flopping about this and so I made it an option.

@danmar
Copy link
Copy Markdown
Collaborator

danmar commented Sep 2, 2023

This is probably something that it's best to write a separate trac ticket about. It's intended as an improvement for package maintainers right?

Maybe. I assume there might be platforms at some point which do want to suppress this. Nobody requested it, but in the past I have been flip-flopping about this and so I made it an option.

Okay.. I believe a ticket would be good.. I think we (including myself) should be better on creating tickets.

It's not required to create tickets for refactorings or pure-CI changes.

Do you have a trac account?

@firewave
Copy link
Copy Markdown
Collaborator Author

firewave commented Sep 2, 2023

Okay.. I believe a ticket would be good.. I think we (including myself) should be better on creating tickets.

It's not required to create tickets for refactorings or pure-CI changes.

Love that. We should include links to the Trac milestones in the GitHub tag descriptions.

@danmar
Copy link
Copy Markdown
Collaborator

danmar commented Sep 2, 2023

Okay.. I believe a ticket would be good.. I think we (including myself) should be better on creating tickets.
It's not required to create tickets for refactorings or pure-CI changes.

Love that. We should include links to the Trac milestones in the GitHub tag descriptions.

👍

I am not sure what you mean with "Github tag descriptions".

@firewave
Copy link
Copy Markdown
Collaborator Author

firewave commented Sep 2, 2023

I am not sure what you mean with "Github tag descriptions".

https://github.com/danmar/cppcheck/releases/tag/2.11 should link to https://trac.cppcheck.net/milestone/2.11 and so on

@danmar
Copy link
Copy Markdown
Collaborator

danmar commented Sep 4, 2023

I am not sure what you mean with "Github tag descriptions".

https://github.com/danmar/cppcheck/releases/tag/2.11 should link to https://trac.cppcheck.net/milestone/2.11 and so on

you mean that the text should contain a link to the corresponding trac milestone? or is there some other way to link it? yes that link sounds reasonable anyway. 👍

@danmar danmar added the merge-after-next-release Wait with merging this PR until after the next Release label Sep 4, 2023
@danmar
Copy link
Copy Markdown
Collaborator

danmar commented Sep 4, 2023

I assumed this can wait until after the release.

@firewave firewave removed the merge-after-next-release Wait with merging this PR until after the next Release label Sep 10, 2023
@danmar danmar merged commit dd869cf into cppcheck-opensource:main Jan 3, 2024
@firewave firewave deleted the ext-as-sys branch January 4, 2024 08:35
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