[Nut-upsdev] 2.8.1 build buglet: sockdebug.c

[Jim Klimov]

Thanks, I think it would not hurt to add the variables into the source if
that helps?

A bit puzzled why it wants TCP wrappers though, the program is primarily
about the Unix socket access.

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...

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.

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 :)

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...

Jim


On Thu, Nov 9, 2023 at 3:44 PM Greg Troxel <gdt at lexort.com> wrote:

> I am (belatedly) updating pkgsrc to 2.8.1 (+ bugfix).
>
> (FWIW, I think a 2.8.1.1 or 2.8.2 immediately with the fix is in order.
> From a packaging viewpoint, the effort to update for a release is about
> 3 minutes plus time to adapt anythhing that has changed.  So I'd much
> rather have releases more often.)
>
> In the pkgsrc build, nut finds tcp wrappers because they are part of the
> base system.  That's generally ok.
>
>   checking whether to enable libwrap (tcp-wrappers) support... yes
>
>
> There is a program sockdebug.c in server:
>
>   Making all in server
>   `libparseconf.la' is up to date.
>     CC       sockdebug.o
>   `libcommon.la' is up to date.
>     CC       upsd.o
>     CC       user.o
>     CC       conf.o
>     CC       netssl.o
>     CC       sstate.o
>     CC       desc.o
>     CC       netget.o
>     CC       netmisc.o
>     CC       netlist.o
>     CC       netuser.o
>     CC       netset.o
>     CC       netinstcmd.o
>     CCLD     sockdebug
>   ld: /usr/lib/libwrap.so: undefined reference to `deny_severity'
>   ld: /usr/lib/libwrap.so: undefined reference to `allow_severity'
>   *** [sockdebug] Error code 1
>
> as I understand it, the tcprappers interface requires those variables to
> exist.  Nut does define them:
>
>   work/nut-2.8.1/server/upsd.c:int        allow_severity = LOG_INFO;
>   work/nut-2.8.1/server/upsd.c:int        deny_severity = LOG_WARNING;
>
> but not in sockdebug.
>
> In server/Makefile.am, tcpwrappers are added for all targets:
>
>   if WITH_WRAP
>      LDADD += $(LIBWRAP_LIBS)
>   endif
>
>
> I don't know if the right fix is
>
>   define these in sockdebug.c
>
>   change LDADD to be only for upsd
>
> or something else.
>
> I think I'm going to just patch out building sockdebug for now.  Or is
> that supposed to be installed for user use?  (If so, it really should
> have nut in the name.)
>
>
>
> _______________________________________________
> Nut-upsdev mailing list
> Nut-upsdev at alioth-lists.debian.net
> https://alioth-lists.debian.net/cgi-bin/mailman/listinfo/nut-upsdev
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://alioth-lists.debian.net/pipermail/nut-upsdev/attachments/20231109/658ed361/attachment.htm>