<div dir="ltr"><div>By the way, on the NUT CI farm the libwrap is present on some (though not all) systems - covering linux, freebsd, openindiana... and neither complained about `sockdebug` :\</div><div><br></div><div>What version do you have? Maybe it is some alternate implementation?</div><div><br></div><div>Jim<br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Nov 9, 2023 at 3:44 PM Greg Troxel <<a href="mailto:gdt@lexort.com">gdt@lexort.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">I am (belatedly) updating pkgsrc to 2.8.1 (+ bugfix).<br>
<br>
(FWIW, I think a 2.8.1.1 or 2.8.2 immediately with the fix is in order.<br>
>From a packaging viewpoint, the effort to update for a release is about<br>
3 minutes plus time to adapt anythhing that has changed.  So I'd much<br>
rather have releases more often.)<br>
<br>
In the pkgsrc build, nut finds tcp wrappers because they are part of the<br>
base system.  That's generally ok.<br>
<br>
  checking whether to enable libwrap (tcp-wrappers) support... yes<br>
<br>
<br>
There is a program sockdebug.c in server:<br>
<br>
  Making all in server<br>
  `<a href="http://libparseconf.la" rel="noreferrer" target="_blank">libparseconf.la</a>' is up to date.<br>
    CC       sockdebug.o<br>
  `<a href="http://libcommon.la" rel="noreferrer" target="_blank">libcommon.la</a>' is up to date.<br>
    CC       upsd.o<br>
    CC       user.o<br>
    CC       conf.o<br>
    CC       netssl.o<br>
    CC       sstate.o<br>
    CC       desc.o<br>
    CC       netget.o<br>
    CC       netmisc.o<br>
    CC       netlist.o<br>
    CC       netuser.o<br>
    CC       netset.o<br>
    CC       netinstcmd.o<br>
    CCLD     sockdebug<br>
  ld: /usr/lib/libwrap.so: undefined reference to `deny_severity'<br>
  ld: /usr/lib/libwrap.so: undefined reference to `allow_severity'<br>
  *** [sockdebug] Error code 1<br>
<br>
as I understand it, the tcprappers interface requires those variables to<br>
exist.  Nut does define them:<br>
<br>
  work/nut-2.8.1/server/upsd.c:int        allow_severity = LOG_INFO;<br>
  work/nut-2.8.1/server/upsd.c:int        deny_severity = LOG_WARNING;<br>
<br>
but not in sockdebug.<br>
<br>
In server/Makefile.am, tcpwrappers are added for all targets:<br>
<br>
  if WITH_WRAP<br>
     LDADD += $(LIBWRAP_LIBS)<br>
  endif<br>
<br>
<br>
I don't know if the right fix is<br>
<br>
  define these in sockdebug.c<br>
<br>
  change LDADD to be only for upsd<br>
<br>
or something else.<br>
<br>
I think I'm going to just patch out building sockdebug for now.  Or is<br>
that supposed to be installed for user use?  (If so, it really should<br>
have nut in the name.)<br>
<br>
<br>
<br>
_______________________________________________<br>
Nut-upsdev mailing list<br>
<a href="mailto:Nut-upsdev@alioth-lists.debian.net" target="_blank">Nut-upsdev@alioth-lists.debian.net</a><br>
<a href="https://alioth-lists.debian.net/cgi-bin/mailman/listinfo/nut-upsdev" rel="noreferrer" target="_blank">https://alioth-lists.debian.net/cgi-bin/mailman/listinfo/nut-upsdev</a><br>
</blockquote></div>