Re: libevent evutil.h

2014-10-31 Thread Landry Breuil
On Thu, Oct 30, 2014 at 02:43:32AM +0100, Alexander Bluhm wrote: > Hi, > > libevent has compatibilty wrappers in evutil. Our tree does not > use them anymore, but they are still part of libevent's interface. > > I don't want to include them automatically, so I suggest to remove > evutil.h from e

Re: libevent evutil.h

2014-10-30 Thread Nicholas Marriott
Thanks Landry On Thu, Oct 30, 2014 at 09:40:14AM +0100, Landry Breuil wrote: > On Thu, Oct 30, 2014 at 08:28:56AM +, Nicholas Marriott wrote: > > No I think we are keeping the evutil.h file for now, the idea is just to > > stop event.h including it. > > OKay, bulk started with just the eve

Re: libevent evutil.h

2014-10-30 Thread Landry Breuil
On Thu, Oct 30, 2014 at 08:28:56AM +, Nicholas Marriott wrote: > No I think we are keeping the evutil.h file for now, the idea is just to > stop event.h including it. OKay, bulk started with just the event.h diff. Landry > On Thu, Oct 30, 2014 at 09:20:49AM +0100, Landry Breuil wrote: > > On

Re: libevent evutil.h

2014-10-30 Thread Nicholas Marriott
No I think we are keeping the evutil.h file for now, the idea is just to stop event.h including it. On Thu, Oct 30, 2014 at 09:20:49AM +0100, Landry Breuil wrote: > On Thu, Oct 30, 2014 at 08:14:35AM +, Nicholas Marriott wrote: > > I'd like to see evutil.h go so I'm happy with this idea but

Re: libevent evutil.h

2014-10-30 Thread Landry Breuil
On Thu, Oct 30, 2014 at 08:14:35AM +, Nicholas Marriott wrote: > I'd like to see evutil.h go so I'm happy with this idea but yes you will > need to make sure it doesn't break ports, there are still quite a few > ports that depend on the base libevent. I'll run a bulk build with that. Apply dif

Re: libevent evutil.h

2014-10-30 Thread Nicholas Marriott
I'd like to see evutil.h go so I'm happy with this idea but yes you will need to make sure it doesn't break ports, there are still quite a few ports that depend on the base libevent. On Thu, Oct 30, 2014 at 02:43:32AM +0100, Alexander Bluhm wrote: > Hi, > > libevent has compatibilty wrappers in

Re: libevent evutil.h

2014-10-29 Thread Theo de Raadt
> libevent has compatibilty wrappers in evutil. Our tree does not > use them anymore, but they are still part of libevent's interface. > > I don't want to include them automatically, so I suggest to remove > evutil.h from event.h. A version bump should not be necessary as > the library itself do

libevent evutil.h

2014-10-29 Thread Alexander Bluhm
Hi, libevent has compatibilty wrappers in evutil. Our tree does not use them anymore, but they are still part of libevent's interface. I don't want to include them automatically, so I suggest to remove evutil.h from event.h. A version bump should not be necessary as the library itself does not