<div dir="ltr">Thanks, I think it would not hurt to add the variables into the source if that helps?<br> <br><div>A bit puzzled why it wants TCP wrappers though, the program is primarily about the Unix socket access.</div><div><br></div><div>It can be used by end-users or more likely by developers for troubleshooting; potentially for some automations that act like a NUT driver. Not intended as a prime-time mechanism, but could have its uses...</div><div><br></div><div>As for the name, it is supposed to be installed into `--libexecdir` by default, where it might indeed confuse or collide (if using the common system path by silent default). Package recipes I've seen recently had used customized paths like `/usr/libexec/nut` similar to how `sysconfdir` is `/etc/nut` or `/etc/ups` (not using silent default `/etc`), so did not think much of it.</div><div><br></div><div>Regarding the release rituals - it is a bit more than 3 minutes, poking here and there (nut-website snapshots, github release drafting, and stuff), but after a few iterations it at least got decently documented in maintainer notes. Also helps to not remember it from scratch after a decade, but actually 
*remember*from a more recent experience :)</div><div><br></div><div>The annoying thing with "urgent" releases is moving GitHub milestone tasks (what we expected to implement here in a leisurely cadence), which suddenly are not getting completed in the predicted release number, to another milestone. But this is also not a prohibitive show-stopper...<br></div><div><br></div><div>Jim</div><div><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>