Add definition for X11_RGBFILE and XMLLINT in config.h #109
Merged
MicheleC
merged 1 commits from feat/add_definitions
into master
4 years ago
Loading…
Reference in new issue
There is no content yet.
Delete Branch 'feat/add_definitions'
Deleting a branch is permanent. It CANNOT be undone. Continue?
Not stricto sensus needed but why not having them?
You might also request xmllint in your Deb packaging.
##### find the rgb.txt file
message( STATUS "Looking for rgb.txt" )
find_file( PATH_RGB rgb.txt PATHS /usr/share/X11 )
Just curious, why we hint to just one path here and we leave the others separately in tdeui/kcolordialog.cppL#632? Wouldn't it make more sense to add those additional paths here too?
Looks good to me, other than the comment I wrote before.
Add definition for X11_RGBFILE and XMLLINT in config.hto WIP:Add definition for X11_RGBFILE and XMLLINT in config.h 4 years agoI'd like to leave it as-is, @MicheleC feel free to adjust to your liking.
WIP:Add definition for X11_RGBFILE and XMLLINT in config.hto Add definition for X11_RGBFILE and XMLLINT in config.h 4 years agoIt looks good – just one small note.
/usr/share/latex2html/styles
/usr/share/nvim/runtime
/usr/X11R6/lib/X11
/usr/openwin/lib/X11
It's great that there is such a large list of paths.
One note: I believe that XOrg/X11 related paths should be listed before other potential paths.
We should avoid specific paths (like vim, neovim, latex and so on), as per IRC discussion.
Looks good to me
Greg, could you rebase on top of master? To keep your GPG signature.
2706c5674e
into master 4 years agoThanks, merge done! 👍
Reviewers
2706c5674e
.