On 1/22/21 4:51 AM, Daniel P. Berrangé wrote:
Even after the previous patch, in order to build without netcf you would need to add "-Dnetcf=disabled" to the meson commandline (or uninstall netcf-devel). This patch makes -Dnetcf=disabled the default. (Without this change, a lot of people would just blindly continue building with netcf enabled.)
Signed-off-by: Laine Stump <laine@redhat.com> --- meson.build | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) I don't much like this idea. I consider it a bug if I have installed the -devel package for a pre-requisite and it isn't
On Fri, Jan 22, 2021 at 03:01:56AM -0500, Laine Stump wrote: then detected.
If a distro no longer wants to support netcf why not just retire the netcf package from that distro version(s), that way users won't have it installed in the first place ?
Yeah, that's what patch 3 does. I added this patch in later as a way of preventing developers (and people who routinely build their own packages from upstream without using the "rpm" target) from naively continuing to use a setup that is deprecated (and then wondering why it behaves differently when the build locally). I'm okay leaving out this patch and just recommending that everyone do their own builds, though.