Update to 4.04. It is important to make devel/hs-hlibev work correctly as it
does not link properly with libev 3.x.
Fix: Patch attached with submission follows:
Over to maintainer (via the GNATS Auto Assign Tool)
There is a problem in your patch. It install event.h header that would
conflict with devel/libevent port. So I have to add CONFLICTS with
libevent. The other option is to exclude event.h from install (that was
the option that I've used before). So what's your opinion on that?
On 08/26/11 14:58, Vsevolod Stakhov wrote:
> what's your opinion [...]?
I would prefer installing the event.h header too as it may be surprising
for the user to have it discarded. And with devel/libevent installed,
it may be misleading since she may think that header is installed by
In my humble opinion, the ultimate solution would be to ask both libev
and libevent upstream to move their headers under something like
$includedir/libev and $includedir/libevent, respectively.
I have not really tested whether devel/hs-hlibev requires event.h to be
present at all so not installing that file may cause breakage for it.
On 08/29/2011 04:10 AM, Gabor PALI wrote:
> The following reply was made to PR ports/159932; it has been noted by GNATS.
> From: Gabor PALI<pgj@FreeBSD.org>
> To: Vsevolod Stakhov<vsevolod@FreeBSD.org>
> Cc: bug-followup@FreeBSD.org
> Subject: Re: ports/159932: [update] devel/libev: Update to 4.04
> Date: Mon, 29 Aug 2011 01:28:15 +0200
> On 08/26/11 14:58, Vsevolod Stakhov wrote:
> > what's your opinion [...]?
> I would prefer installing the event.h header too as it may be surprising
> for the user to have it discarded. And with devel/libevent installed,
> it may be misleading since she may think that header is installed by
Well, here are my points when I've decided NOT to install event.h in
First of all libev install event.h for those applications that are using
libevent interface but via emulating libevent API by libev. So ports
that depends on libev MUST NOT use event.h header in any way (they
should definitely use ev.h header). And ports that are depending on
libevent MUST use event.h header and MUST depend on devel/libevent and
on devel/libev. So installing event.h header by libev is a confusing
action. If devel/libevent would be made as a conflict for devel/libev
ports depending libevent still would be broken without it. And ports
depending on libev are not using event.h header. So from my point of
view this file is useless.
> In my humble opinion, the ultimate solution would be to ask both libev
> and libevent upstream to move their headers under something like
> $includedir/libev and $includedir/libevent, respectively.
Yes, but this requires review of all ports depending on libev or
libevent and maybe adding some knobs to portmk like
> I have not really tested whether devel/hs-hlibev requires event.h to be
> present at all so not installing that file may cause breakage for it.
Well, libev community grows rather fast, so it is definitely required to
solve conflicts between these two libraries.
On 08/29/11 12:30, Vsevolod Stakhov wrote:
> First of all libev install event.h for those applications that are
> using libevent interface but via emulating libevent API by libev.
Right. Now I understand your point. I will test devel/hs-hlibev
without event.h installed (by devel/libev) and get back to you with the
results. I expect it to work but better safe than sorry.
> Well, libev community grows rather fast, so it is definitely required
> to solve conflicts between these two libraries.
I do not really want to make a problem out of this issue, but it is
indeed need to be taken care of in the long run.
vsevolod 2011-09-16 16:17:40 UTC
FreeBSD ports repository
devel/libev Makefile distinfo pkg-plist
devel/libev/files patch-configure patch-ev.c
- Update to 4.04 
- Add libev.pc 
PR: 159932 , 160030 
Submitted by: pgj , Aldis Berjoza <aldis at bsdroot.lv> 
Revision Changes Path
1.11 +9 -6 ports/devel/libev/Makefile
1.10 +2 -2 ports/devel/libev/distinfo
1.1 +10 -0 ports/devel/libev/files/libev.pc.in (new)
1.2 +12 -46 ports/devel/libev/files/patch-configure
1.2 +0 -32 ports/devel/libev/files/patch-ev++.h (dead)
1.2 +4 -4 ports/devel/libev/files/patch-ev.c
1.4 +2 -1 ports/devel/libev/pkg-plist
firstname.lastname@example.org mailing list
To unsubscribe, send any mail to "email@example.com"
Committed with some modifications, thanks!