Skip to content

cppcheck: Enable building the GUI#8561

Open
JacekJagosz wants to merge 1 commit into
getsolus:mainfrom
JacekJagosz:enable-gui-in-cppcheck
Open

cppcheck: Enable building the GUI#8561
JacekJagosz wants to merge 1 commit into
getsolus:mainfrom
JacekJagosz:enable-gui-in-cppcheck

Conversation

@JacekJagosz
Copy link
Copy Markdown
Contributor

@JacekJagosz JacekJagosz commented Apr 15, 2026

Summary

Enable building the GUI

Test Plan
Used it for multiple days to analyze my code without issues

Checklist

  • Package was built and tested against unstable
  • This change could gainfully be listed in the weekly sync notes once merged

@JacekJagosz
Copy link
Copy Markdown
Contributor Author

Only problem I can see with including it is that now someone that installs it as a dependency, will now also get the GUI. For example kdevelop is a reverse dependency

@EbonJaeger
Copy link
Copy Markdown
Member

The GUI part should probably be split into its own package, like cppcheck-gui.

@JacekJagosz JacekJagosz force-pushed the enable-gui-in-cppcheck branch from 6a33d42 to a389626 Compare April 15, 2026 22:35
@EbonJaeger
Copy link
Copy Markdown
Member

Could we pull in this patch to fix the localization installation location? It would be nice to not have them dumped straight into /usr/bin/.

**Summary**

Enable building the GUI
@JacekJagosz JacekJagosz force-pushed the enable-gui-in-cppcheck branch from a389626 to 6f00dfd Compare April 19, 2026 13:51
Copy link
Copy Markdown
Member

@EbonJaeger EbonJaeger left a comment

Choose a reason for hiding this comment

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

A couple more things, after thinking about it some more:

  1. Could a component, summary, and description be added for the new subpackage?
  2. The -gui subpackage should have a runtime dependency on cppcheck
  3. Stretch goal: It would be great if an AppStream metainfo file could be added for the GUI package so it can be found in GNOME Software and Discover.

Comment on lines +17 to +22
patterns :
- gui :
- /usr/bin/cppcheck-gui
- /usr/share/Cppcheck/lang/*
- /usr/share/applications/*
- /usr/share/icons/*
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.

Could this be moved to the bottom of the file? It makes everything easier to find.

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

Labels

None yet

Projects

Status: Triage

Development

Successfully merging this pull request may close these issues.

2 participants