[PATCH] Make AD DC build require Jannsson JSON libs, fix fileserver without it

[Ralph Böhme]

On Fri, Jun 22, 2018 at 10:49:00AM -0700, Jeremy Allison wrote:
> On Fri, Jun 22, 2018 at 07:35:45PM +0200, Ralph Böhme wrote:
> > hm, I disagree. I strongly feel that it doesn't add too much upstream
> > maintanance having a --without-json-audit switch that allows building on older
> > systems that are lacking libjansson.
> 
> Yeah but it's json today, it'll be something else tomorrow.

Nah, today is today and tomorrow is tomorrow. Today, RHEL 6 is still a supported
release and will be til 2020. Or am I missing something?

> IMHO I think it's OK for the AD-DC to require more packages/resources
> than a fileserver build.

sorry, but I don't get it. Why is json audit logging a *requirement*? It might
be a required features for some downstream consumers, it certainly it not a
requirement for all, is it?

> It's a big complex beast that does a lot of stuff, and it's
> still rapidly under development with more features being added.
> 
> If we try and make it compatible with only the packages available
> on 8+ year old distro's we're going to make development more complex
> than it already is.

Hm, how much work can it be maintaining a build without libjansson? 

-slow

-- 
Ralph Boehme, Samba Team       https://samba.org/
Samba Developer, SerNet GmbH   https://sernet.de/en/samba/
GPG Key Fingerprint:           FAE2 C608 8A24 2520 51C5
                               59E4 AA1E 9B71 2639 9E46