[NEW] mail/libopensmtpd

classic Classic list List threaded Threaded
6 messages Options
Reply | Threaded
Open this post in threaded view
|

[NEW] mail/libopensmtpd

Martijn van Duren-7
$ cat libopensmtpd/pkg/DESCR
libopensmtpd is an libevent based library which allows for easy writing
of opensmtpd filter applications in C.
$

This is required for filter-dnsbl and filter-dkim.

Since I'm not too familiar with ports I would like to pay special
attention to the Makefile of both the port as well as the source.

Also, I currently host the release tarballs at my personal server, which
I also use for generic other stuff and might not always be available.
If someone from the ports team has a more stable location to host the
release tarballs let me know.

martijn@

libopensmtpd.tar.gz (1K) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: [NEW] mail/libopensmtpd

Martijn van Duren-7
On 8/24/19 6:29 AM, Martijn van Duren wrote:

> $ cat libopensmtpd/pkg/DESCR
> libopensmtpd is an libevent based library which allows for easy writing
> of opensmtpd filter applications in C.
> $
>
> This is required for filter-dnsbl and filter-dkim.
>
> Since I'm not too familiar with ports I would like to pay special
> attention to the Makefile of both the port as well as the source.
>
> Also, I currently host the release tarballs at my personal server, which
> I also use for generic other stuff and might not always be available.
> If someone from the ports team has a more stable location to host the
> release tarballs let me know.
>
> martijn@
>
Updated diff based on recent smtpd changes.
I haven't changed the version number since it isn't imported in ports
yet.

Questions above still remain.

Now to be placed in mail/openstmpd-filters. Requires additional diff
to prevent PKGNAME to be overwritten.

Index: Makefile.inc
===================================================================
RCS file: /cvs/ports/mail/opensmtpd-filters/Makefile.inc,v
retrieving revision 1.1
diff -u -p -r1.1 Makefile.inc
--- Makefile.inc 4 Sep 2019 12:27:42 -0000 1.1
+++ Makefile.inc 5 Sep 2019 13:12:09 -0000
@@ -1,6 +1,6 @@
 # $OpenBSD: Makefile.inc,v 1.1 2019/09/04 12:27:42 martijn Exp $
 
-PKGNAME = opensmtpd-filter-${FILTER_NAME}-${V}
+PKGNAME ?= opensmtpd-filter-${FILTER_NAME}-${V}
 
 MODGO_TYPE = bin
 MODGO_BINDIR = libexec/smtpd

libopensmtpd.tar.gz (1K) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: [NEW] mail/libopensmtpd

Martijn van Duren-7
ping

On 9/5/19 3:13 PM, Martijn van Duren wrote:

> On 8/24/19 6:29 AM, Martijn van Duren wrote:
>> $ cat libopensmtpd/pkg/DESCR
>> libopensmtpd is an libevent based library which allows for easy writing
>> of opensmtpd filter applications in C.
>> $
>>
>> This is required for filter-dnsbl and filter-dkim.
>>
>> Since I'm not too familiar with ports I would like to pay special
>> attention to the Makefile of both the port as well as the source.
>>
>> Also, I currently host the release tarballs at my personal server, which
>> I also use for generic other stuff and might not always be available.
>> If someone from the ports team has a more stable location to host the
>> release tarballs let me know.
>>
>> martijn@
>>
> Updated diff based on recent smtpd changes.
> I haven't changed the version number since it isn't imported in ports
> yet.
>
> Questions above still remain.
>
> Now to be placed in mail/openstmpd-filters. Requires additional diff
> to prevent PKGNAME to be overwritten.
>
> Index: Makefile.inc
> ===================================================================
> RCS file: /cvs/ports/mail/opensmtpd-filters/Makefile.inc,v
> retrieving revision 1.1
> diff -u -p -r1.1 Makefile.inc
> --- Makefile.inc 4 Sep 2019 12:27:42 -0000 1.1
> +++ Makefile.inc 5 Sep 2019 13:12:09 -0000
> @@ -1,6 +1,6 @@
>  # $OpenBSD: Makefile.inc,v 1.1 2019/09/04 12:27:42 martijn Exp $
>  
> -PKGNAME = opensmtpd-filter-${FILTER_NAME}-${V}
> +PKGNAME ?= opensmtpd-filter-${FILTER_NAME}-${V}
>  
>  MODGO_TYPE = bin
>  MODGO_BINDIR = libexec/smtpd
>

Reply | Threaded
Open this post in threaded view
|

Re: [NEW] mail/libopensmtpd

Mikolaj Kucharski-3
Hi Martijn,

I've used below patch and earlier attached libopensmtpd port and
it worked fine for me with filter-dkimsign.

On Thu, Sep 12, 2019 at 09:13:25AM +0200, Martijn van Duren wrote:

> ping
>
> On 9/5/19 3:13 PM, Martijn van Duren wrote:
> > On 8/24/19 6:29 AM, Martijn van Duren wrote:
> >> $ cat libopensmtpd/pkg/DESCR
> >> libopensmtpd is an libevent based library which allows for easy writing
> >> of opensmtpd filter applications in C.
> >> $
> >>
> >> This is required for filter-dnsbl and filter-dkim.
> >>
> >> Since I'm not too familiar with ports I would like to pay special
> >> attention to the Makefile of both the port as well as the source.
> >>
> >> Also, I currently host the release tarballs at my personal server, which
> >> I also use for generic other stuff and might not always be available.
> >> If someone from the ports team has a more stable location to host the
> >> release tarballs let me know.
> >>
> >> martijn@
> >>
> > Updated diff based on recent smtpd changes.
> > I haven't changed the version number since it isn't imported in ports
> > yet.
> >
> > Questions above still remain.
> >
> > Now to be placed in mail/openstmpd-filters. Requires additional diff
> > to prevent PKGNAME to be overwritten.
> >
> > Index: Makefile.inc
> > ===================================================================
> > RCS file: /cvs/ports/mail/opensmtpd-filters/Makefile.inc,v
> > retrieving revision 1.1
> > diff -u -p -r1.1 Makefile.inc
> > --- Makefile.inc 4 Sep 2019 12:27:42 -0000 1.1
> > +++ Makefile.inc 5 Sep 2019 13:12:09 -0000
> > @@ -1,6 +1,6 @@
> >  # $OpenBSD: Makefile.inc,v 1.1 2019/09/04 12:27:42 martijn Exp $
> >  
> > -PKGNAME = opensmtpd-filter-${FILTER_NAME}-${V}
> > +PKGNAME ?= opensmtpd-filter-${FILTER_NAME}-${V}
> >  
> >  MODGO_TYPE = bin
> >  MODGO_BINDIR = libexec/smtpd
> >

--
Regards,
 Mikolaj

Reply | Threaded
Open this post in threaded view
|

Re: [NEW] mail/libopensmtpd

Stuart Henderson
In reply to this post by Martijn van Duren-7
On 2019/09/12 09:13, Martijn van Duren wrote:

> ping
>
> On 9/5/19 3:13 PM, Martijn van Duren wrote:
> > On 8/24/19 6:29 AM, Martijn van Duren wrote:
> >> $ cat libopensmtpd/pkg/DESCR
> >> libopensmtpd is an libevent based library which allows for easy writing
> >> of opensmtpd filter applications in C.
> >> $
> >>
> >> This is required for filter-dnsbl and filter-dkim.
> >>
> >> Since I'm not too familiar with ports I would like to pay special
> >> attention to the Makefile of both the port as well as the source.
> >>
> >> Also, I currently host the release tarballs at my personal server, which
> >> I also use for generic other stuff and might not always be available.
> >> If someone from the ports team has a more stable location to host the
> >> release tarballs let me know.
> >>
> >> martijn@
> >>
> > Updated diff based on recent smtpd changes.
> > I haven't changed the version number since it isn't imported in ports
> > yet.
> >
> > Questions above still remain.
> >
> > Now to be placed in mail/openstmpd-filters. Requires additional diff
> > to prevent PKGNAME to be overwritten.
> >
> > Index: Makefile.inc
> > ===================================================================
> > RCS file: /cvs/ports/mail/opensmtpd-filters/Makefile.inc,v
> > retrieving revision 1.1
> > diff -u -p -r1.1 Makefile.inc
> > --- Makefile.inc 4 Sep 2019 12:27:42 -0000 1.1
> > +++ Makefile.inc 5 Sep 2019 13:12:09 -0000
> > @@ -1,6 +1,6 @@
> >  # $OpenBSD: Makefile.inc,v 1.1 2019/09/04 12:27:42 martijn Exp $
> >  
> > -PKGNAME = opensmtpd-filter-${FILTER_NAME}-${V}
> > +PKGNAME ?= opensmtpd-filter-${FILTER_NAME}-${V}
> >  
> >  MODGO_TYPE = bin
> >  MODGO_BINDIR = libexec/smtpd
> >
>

port doesn't honour CC - set this: MAKE_FLAGS= CC="${CC}"

trim COMMENT a bit, we don't usually start with a/the or start with
caps. I'd just go for e.g. "library for writing opensmtpd filters"

gcc objects to this:

opensmtpd.c:1967:13: warning: 'port' may be used uninitialized in this function [-Wmaybe-uninitialized]
    port[-1] = ':';
    ~~~~~~~~~^~~~~

start with "SHARED_LIBS= opensmtpd 0.0" and the version number needs to
be under ports control; changing that var in the Makefile must cause the
built library version to change. the version number is available in
LIBopensmtpd_VERSION in the ports Makefile which can then be passed into
the build via MAKE_FLAGS / MAKE_ENV.


Reply | Threaded
Open this post in threaded view
|

Re: [NEW] mail/libopensmtpd

Martijn van Duren-7
On 9/17/19 1:00 AM, Stuart Henderson wrote:

> On 2019/09/12 09:13, Martijn van Duren wrote:
>> ping
>>
>> On 9/5/19 3:13 PM, Martijn van Duren wrote:
>>> On 8/24/19 6:29 AM, Martijn van Duren wrote:
>>>> $ cat libopensmtpd/pkg/DESCR
>>>> libopensmtpd is an libevent based library which allows for easy writing
>>>> of opensmtpd filter applications in C.
>>>> $
>>>>
>>>> This is required for filter-dnsbl and filter-dkim.
>>>>
>>>> Since I'm not too familiar with ports I would like to pay special
>>>> attention to the Makefile of both the port as well as the source.
>>>>
>>>> Also, I currently host the release tarballs at my personal server, which
>>>> I also use for generic other stuff and might not always be available.
>>>> If someone from the ports team has a more stable location to host the
>>>> release tarballs let me know.
>>>>
>>>> martijn@
>>>>
>>> Updated diff based on recent smtpd changes.
>>> I haven't changed the version number since it isn't imported in ports
>>> yet.
>>>
>>> Questions above still remain.
>>>
>>> Now to be placed in mail/openstmpd-filters. Requires additional diff
>>> to prevent PKGNAME to be overwritten.
>>>
>>> Index: Makefile.inc
>>> ===================================================================
>>> RCS file: /cvs/ports/mail/opensmtpd-filters/Makefile.inc,v
>>> retrieving revision 1.1
>>> diff -u -p -r1.1 Makefile.inc
>>> --- Makefile.inc 4 Sep 2019 12:27:42 -0000 1.1
>>> +++ Makefile.inc 5 Sep 2019 13:12:09 -0000
>>> @@ -1,6 +1,6 @@
>>>  # $OpenBSD: Makefile.inc,v 1.1 2019/09/04 12:27:42 martijn Exp $
>>>  
>>> -PKGNAME = opensmtpd-filter-${FILTER_NAME}-${V}
>>> +PKGNAME ?= opensmtpd-filter-${FILTER_NAME}-${V}
>>>  
>>>  MODGO_TYPE = bin
>>>  MODGO_BINDIR = libexec/smtpd
>>>
>>
>
> port doesn't honour CC - set this: MAKE_FLAGS= CC="${CC}"
>
> trim COMMENT a bit, we don't usually start with a/the or start with
> caps. I'd just go for e.g. "library for writing opensmtpd filters"
>
> gcc objects to this:
>
> opensmtpd.c:1967:13: warning: 'port' may be used uninitialized in this function [-Wmaybe-uninitialized]
>     port[-1] = ':';
>     ~~~~~~~~~^~~~~
>
> start with "SHARED_LIBS= opensmtpd 0.0" and the version number needs to
> be under ports control; changing that var in the Makefile must cause the
> built library version to change. the version number is available in
> LIBopensmtpd_VERSION in the ports Makefile which can then be passed into
> the build via MAKE_FLAGS / MAKE_ENV.
>
>
This one should address all the issues.

libopensmtpd.tar.gz (1K) Download Attachment