https://bugs.kde.org/show_bug.cgi?id=499828
Gleb Popov <6year...@gmail.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |6year...@gmail.com
--- Comment #3 from Gleb Popov <6year...@gmail.com> ---
(In reply to Noah Davis from comment #2)
> Thanks for providing a patch. I'm not sure if I want to use it though.
>
> I try to avoid optional dependencies in general for a few reasons:
>
> 1. To prevent users from installing it in a way that makes them think the
> software is buggy or missing features.
The dependency can be made required by default but optional if a CMake option
is passed. This protects users from accidentally coming up with a broken build
configuration, but still allows "advanced" users to get a custom build.
> 4. To prevent distros from packaging it in a way that is incorrect that leads
> to the scenario in reason 1.
This is exactly what defaults are for.
But I agree that the patch is somewhat invasive.
--
You are receiving this mail because:
You are watching all bug changes.
