adridg abandoned this revision.
adridg added a comment.
Thanks for looking at this, Harald. For various downstream packaging reasons,
we're going to be stuck with patching, so I'm going to give up on this
particular patch.
REPOSITORY
R244 KCoreAddons
REVISION DETAIL
https://phabricator.
sitter added a comment.
I do wonder if we couldn't just move the freebsd path to the end of the list
and drop the ifdef. As far as linux is concerned we still obey the lookup order
but simply have an additional path where we may look (and where the file should
not ever exist on linux). Conve
adridg edited the test plan for this revision.
REPOSITORY
R244 KCoreAddons
REVISION DETAIL
https://phabricator.kde.org/D21305
To: adridg, sitter
Cc: kde-frameworks-devel, michaelh, ngraham, bruns
adridg edited the test plan for this revision.
REPOSITORY
R244 KCoreAddons
REVISION DETAIL
https://phabricator.kde.org/D21305
To: adridg, sitter
Cc: kde-frameworks-devel, michaelh, ngraham, bruns
adridg created this revision.
adridg added a reviewer: sitter.
Herald added a project: Frameworks.
Herald added a subscriber: kde-frameworks-devel.
adridg requested review of this revision.
REVISION SUMMARY
- After much hemming and hawing we ended up with /usr/local/etc/os-release,
which isn't
