cppcheck: Enable building the GUI#8561
Conversation
|
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 |
|
The GUI part should probably be split into its own package, like |
JacekJagosz
force-pushed
the
enable-gui-in-cppcheck
branch
from
6a33d42 to
a389626
Compare
|
Could we pull in this patch to fix the localization installation location? It would be nice to not have them dumped straight into |
**Summary** Enable building the GUI
JacekJagosz
force-pushed
the
enable-gui-in-cppcheck
branch
from
a389626 to
6f00dfd
Compare
EbonJaeger
left a comment
EbonJaeger
left a comment
There was a problem hiding this comment.
A couple more things, after thinking about it some more:
- Could a component, summary, and description be added for the new subpackage?
- The
-guisubpackage should have a runtime dependency oncppcheck - 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.
| patterns : | ||
| - gui : | ||
| - /usr/bin/cppcheck-gui | ||
| - /usr/share/Cppcheck/lang/* | ||
| - /usr/share/applications/* | ||
| - /usr/share/icons/* |
There was a problem hiding this comment.
Could this be moved to the bottom of the file? It makes everything easier to find.
|
Hello, do you still have interest in continuing with this Pull Request? |
|
I do want to finish it, just life has been incredibly hard. The GUI is a really useful tool |
|
Understood. No rush, I was just making sure. I hope that everything works itself out! |
2 participants
Summary
Enable building the GUI
Test Plan
Used it for multiple days to analyze my code without issues
Checklist