RE: nnrpd: adding blacklistd support

Julien ÉLIE <[email protected]> Mon, 28 Mar 2022 16:21:31 +0000
Newsgroups gmane.network.inn
Message-ID <[email protected]>
Hi Andreas,

> Adding it as --with-blacklistd sounds like the right thing to do, yes. 
>
>> I'll have a look.  As I do not run FreeBSD myself, it would 
>> be great if you could somehow test it.
> 
> When you say that you will have a look, does that mean that you want
> to rework the patch yourself?
>
> I'm willing (and was expecting) to open a pull request on GitHub and
> do testing as well as any required rewriting per your review.

Oh, that's kind of you.  It will save me time to do other work for INN
for the next major release.
Of course I don't mind your working on a pull request.
As we did not have GitHub until last year, I was accustomed to
reworking myself patches sent to our mailing-lists.

The idea would be to create a new m4/blacklistd.m4 file (based for
instance on m4/canlock.m4), add related glue in configure.ac and
Makefile.global.in, and surround the new code in nnrpd with
#if defined(HAVE_BLACKLISTD) instructions.

There is an entry in MANIFEST to add too.

Maybe a note about blacklistd in doc/pod/nnrpd.pod? (or any other
file you think more appropriate)
And mention --with-blacklistd in doc/pod/install.pod.

As GitHub continuous integration is run on Ubuntu, it cannot be
tested (so there is nothing to change in the ci directory I think).


Many thanks to you.  It is greatly appreciated!

-- 
Julien ÉLIE
-- 
inn-workers mailing list
[email protected]
https://lists.isc.org/mailman/listinfo/inn-workers