Drop NIS support

[Rowland penny]

On 24/03/2021 20:23, Andrew Bartlett wrote:
> On Wed, 2021-03-24 at 20:16 +0000, Rowland penny via samba-technical
> wrote:
>> On 24/03/2021 20:11, Ralph Boehme wrote:
>>> Am 3/24/21 um 6:57 PM schrieb Rowland penny via samba-technical:
>>>> On 24/03/2021 17:52, Andrew Bartlett via samba-technical wrote:
>>>>> On Wed, 2021-03-24 at 09:12 -0700, Jeremy Allison via samba-
>>>>> technical
>>>>> wrote:
>>>>>> On Wed, Mar 24, 2021 at 05:10:15PM +0100, Andreas Schneider
>>>>>> via
>>>>>> samba-technical wrote:
>>>>>>> Hi,
>>>>>>>
>>>>>>> could we drop NIS support for Samba 4.15?
>>>>>>>
>>>>>>> This means getting rid of lib/replace/system/nis.h and
>>>>>>> yp_*() ...
>>>>>> +1 from me. Let's see if anyone else is still
>>>>>> using it.
>>>>>>
>>>>>> So the current list for 4.15 will be Tru64, HPUX, NIS.
>>>>>>
>>>>>> Any more for any more ?
>>>>> Just to be clear to others, this is not the so-called NIS
>>>>> support in
>>>>> the AD DC which at one point helped Active Directory Users and
>>>>> Computers managed the rfc2307 user attributes.
>>>>>
>>>>> I'm happy for the real NIS support to go.
>>>>>
>>>>> In terms of other things to trim, what about some of the
>>>>> printing
>>>>> modes?
>>>>>
>>>>> Andrew Bartlett
>>>>>
>>>> How about idmap_hash ? The one that says 'DO NOT USE THIS
>>>> BACKEND' at
>>>> the top of its manpage.
>>> ...unless you know what you're doing.
>>>
>>> There are folks out there using this at scale so removing this is
>>> not
>>> really something we should consider.
>>>
>>> -slow
>>>
>> Well according to the manpage 'DO NOT USE THIS BACKEND', they
>> shouldn't
>> be using it, you cannot have it both ways.
> Rowland, the 'on new installations' is silent, but implied.


It might be to you, but to a normal user it isn't readily known.

>
>> Either it shouldn't be used,
>> or those warnings need to be removed from the manpage.
> The world is not always so black and white (and would be far more
> boring if it was).


You are entering into dangerous waters there, you are not supposed to 
mention what you mentioned 😂

>
> Andrew Bartlett
>

OK, how about this, I rewrite the idmap_hash manpage to say that it is 
only retained for existing users but must not be used for new installations.

Rowland