Skip to content

improve handling of failed hooks#194

Merged
kdudka merged 3 commits into
csutils:mainfrom
kdudka:host-cppcheck
Nov 8, 2024
Merged

improve handling of failed hooks#194
kdudka merged 3 commits into
csutils:mainfrom
kdudka:host-cppcheck

Conversation

@kdudka

@kdudka kdudka commented Nov 8, 2024

Copy link
Copy Markdown
Member

See the individual commit messages for details.

Resolves: #117

No change in behavior intended with this commit.

Related: csutils#117
... to make it clear that the option requires statically linked cppcheck
installed on the host.

Resolves: csutils#117
Closes: csutils#194
@kdudka kdudka requested review from lzaoral and siteshwar November 8, 2024 08:36
@kdudka kdudka self-assigned this Nov 8, 2024

@siteshwar siteshwar left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM on a quick look.

@kdudka

kdudka commented Nov 8, 2024

Copy link
Copy Markdown
Member Author

@siteshwar Thanks for review!

@kdudka kdudka merged commit 014012a into csutils:main Nov 8, 2024
@kdudka kdudka deleted the host-cppcheck branch November 8, 2024 11:44
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.

cppcheck: post-depinst hook failed with host cppcheck

2 participants