Re: [PATCH] Broken logic in lang/python27 regarding NIS

[Greg Troxel <gdt%lexort.com@localhost>]

"Dr. Thomas Orgis" <thomas.orgis%uni-hamburg.de@localhost> writes:

> Am Tue, 26 Mar 2019 08:25:33 -0400
> schrieb Greg Troxel <gdt%lexort.com@localhost>:
>
>> So, I would tempted to make this change instead:
>> 
>>   -.  elif ${OPSYS} != "NetBSD" || exists(/usr/bin/ypcat)
>>   +.  elif exists(/usr/bin/ypcat)
>> 
>> but I wonder why it is written the way it is.
>
> The != NetBSD indeed is curious. Perhaps it was == NetBSD ||
> exists(/usr/bin/ypcat)? But anyhow, this is being discussed in multiple
> PRs on GNATS, sadly without final resolution yet. The direction to go
> seems to test for the respective headers instead. I myself need to
> check why the presence of /usr/include/rpcsvc/yp_prot.h is not enough
> to make the python NIS stuff build on my system.

You didn't answer my questions about Linux and NIS, so I'll pause in
thinking about this.  Also, we are about to start freeze, and I don't
want to make a change I don't 100% understand to something with
thousands of dependencies right now.  Perhaps after the freeze we can
drop the netbsd part of the conditional.