Re: net/kea: 1.2.0

On Sun, Aug 27, 2017 at 11:19:44AM +0100, Stuart Henderson wrote: > On 2017/08/27 12:07, Patrik Lundin wrote: > > On Sat, Aug 12, 2017 at 11:39:14PM +0200, Patrik Lundin wrote: > > > > > > Since there have been no further feedback, is there a chance the latest > > > diff can get commited? > > >

Re: net/kea: 1.2.0

On 2017/08/27 12:07, Patrik Lundin wrote: > On Sat, Aug 12, 2017 at 11:39:14PM +0200, Patrik Lundin wrote: > > > > Since there have been no further feedback, is there a chance the latest > > diff can get commited? > > > > Ping. The COMPILER infrastructure changed in the meantime, I'm leaving

Re: net/kea: 1.2.0

On Sat, Aug 12, 2017 at 11:39:14PM +0200, Patrik Lundin wrote: > > Since there have been no further feedback, is there a chance the latest > diff can get commited? > Ping. -- Patrik Lundin

Re: net/kea: 1.2.0

On Thu, Aug 03, 2017 at 10:24:00PM +0200, Patrik Lundin wrote: > > Thanks for spotting that. New diff against fresh ports tree below (and > now portcheck is happy as well). > Since there have been no further feedback, is there a chance the latest diff can get commited? -- Patrik Lundin

Re: net/kea: 1.2.0

Hello Stuart, thanks for the review. On Thu, Aug 03, 2017 at 06:43:11PM +0100, Stuart Henderson wrote: > > clang is already used as the compiler on i386/amd64/aarch64. Just using > "COMPILER=clang" means this won't build on some arch which do have c++11 > support from ports gcc. > > For most

Re: net/kea: 1.2.0

On 2017/08/03 19:12, Patrik Lundin wrote: > * Start using clang as the compiler. Kea 1.2.0 requires C++11 support. > With the clang work taking place in base I noticed espie@ had already > added a patch for a clang build breaking problem. This has been reported and > fixed upstream for later

Re: net/kea: 1.1.0

Patrik Lundin writes: > Hello, > > See below for an update to net/kea from 1.0.0 to 1.1.0. Committed, thanks. [...] > This also brings up another question of mine: where to place logs. > > Currently the default setup creates logs in the following places: > /var/kea/kea.log

Re: net/kea: 1.1.0

On Mon, Oct 10, 2016 at 12:28:58AM +0200, Patrik Lundin wrote: > Hello, > > See below for an update to net/kea from 1.0.0 to 1.1.0. > Ping. -- Patrik Lundin

Re: net/kea: Ensure base awk is used

On 2016-06-27, Stuart Henderson wrote: > ...but there's also this in config.site, which seems incorrect: > > ac_cv_prog_AWK=${ac_cv_prog_AWK=awk} If you mean that it should use the full path, then no, it's correct as is. Putting /usr/bin/awk there causes build errors,

Re: net/kea: Helpfully failing test in ./configure

On Mon, Jun 27, 2016 at 07:44:07PM +0200, Jeremie Courreges-Anglas wrote: > Patrik Lundin writes: > > > > > This means that the failing "test" can actually be thought of as a > > feature. It is of course brittle, and will modify the build parameters > > if someone decides to

Re: net/kea: Helpfully failing test in ./configure

On Mon, Jun 27, 2016 at 07:51:33PM +, Christian Weisgerber wrote: > On 2016-06-27, Patrik Lundin wrote: > > > CXX_DUMP_VERSION=`$CXX -dumpversion | cut -f1-2 -d.` > > if test "$CXX_DUMP_VERSION" \< "4.5"; then > >

Re: net/kea: Helpfully failing test in ./configure

On 2016-06-27, Patrik Lundin wrote: > CXX_DUMP_VERSION=`$CXX -dumpversion | cut -f1-2 -d.` > if test "$CXX_DUMP_VERSION" \< "4.5"; then >WARNING_GCC_44_STRICT_ALIASING_CFLAG="-fno-strict-aliasing" > fi > > The error message is thrown because the builtin test does not

Re: net/kea: Ensure base awk is used

On 2016-06-27, Stuart Henderson wrote: >> That should already be set. Is it not picking up config.site for some reason? > > Oh, it is ac_cv_*prog*_AWK that is getting set, for some reason kea > is also looking at ac_cv_*path*_AWK. > > Perhaps we should add the path variant

Re: net/kea: Ensure base awk is used

On 2016/06/27 21:10, David Coppa wrote: > On Mon, Jun 27, 2016 at 6:26 PM, Stuart Henderson > wrote: > > > Perhaps we should add the path variant to config.site then.. > > It looks like the right approach to me, ok dcoppa@. These are the relevant ports, bacula-7.4.0

Re: net/kea: Ensure base awk is used

On Mon, Jun 27, 2016 at 6:26 PM, Stuart Henderson wrote: > Perhaps we should add the path variant to config.site then.. It looks like the right approach to me, ok dcoppa@. ciao! David

Re: net/kea: Helpfully failing test in ./configure

Patrik Lundin writes: > Hello, > > When looking at the output of ./configure in net/kea I noticed the > following warning which I have previously missed: > === > ./configure[15929]: test: <: unexpected operator/operand > === > > The responsible code in the configure script

Re: net/kea: Ensure base awk is used

On 2016/06/27 17:19, Stuart Henderson wrote: > On 2016/06/27 18:15, Patrik Lundin wrote: > > +CONFIGURE_ENV+= ac_cv_path_AWK=awk > > That should already be set. Is it not picking up config.site for some reason? > Oh, it is ac_cv_*prog*_AWK that is getting set, for some reason kea is also

Re: net/kea: Ensure base awk is used

On 2016/06/27 18:15, Patrik Lundin wrote: > +CONFIGURE_ENV+= ac_cv_path_AWK=awk That should already be set. Is it not picking up config.site for some reason?

Re: net/kea

On Thu, Dec 24, 2015 at 02:53:06AM +, Stuart Henderson wrote: > > I saw it on oss-sec first, then on ISC's security RSS feed (and as if > to emphasize the slightly random nature of that feed it was followed > by release notes for 0.9, 0.9.2-beta and 0.9.2 :-) I read oss-sec anyway, > and

Re: net/kea

Updated tar.gz for the 0.9.2-P1 crash fix ("Improved handling of incoming packets with invalid client-id and DUID.") I also added an XXX comment to remind us to update the license marker for 1.0 because ISC are moving to the Apache license :'( I'm still looking for OKs to import if anyone has

Re: net/kea

On Wed, Dec 23, 2015 at 11:33:30AM +, Stuart Henderson wrote: > Updated tar.gz for the 0.9.2-P1 crash fix ("Improved handling of incoming > packets with invalid client-id and DUID.") I also added an XXX comment > to remind us to update the license marker for 1.0 because ISC are moving > to the

Re: net/kea

On Wed, Dec 23, 2015 at 11:33:30AM +, Stuart Henderson wrote: > Updated tar.gz for the 0.9.2-P1 crash fix ("Improved handling of incoming > packets with invalid client-id and DUID.") > Nice catch! I had not seen any word of this release on the kea mailing lists, how did you notice it? > > I

Re: net/kea

On 23 December 2015 at 21:53, Stuart Henderson wrote: > On 2015/12/23 14:47, Patrik Lundin wrote: >> On Wed, Dec 23, 2015 at 11:33:30AM +, Stuart Henderson wrote: >> > Updated tar.gz for the 0.9.2-P1 crash fix ("Improved handling of incoming >> > packets with invalid

Re: net/kea

On 2015/12/23 14:47, Patrik Lundin wrote: > On Wed, Dec 23, 2015 at 11:33:30AM +, Stuart Henderson wrote: > > Updated tar.gz for the 0.9.2-P1 crash fix ("Improved handling of incoming > > packets with invalid client-id and DUID.") > > > > Nice catch! I had not seen any word of this release on

Re: net/kea

On 2015/12/23 22:03, Kenneth Westerback wrote: > On 23 December 2015 at 21:53, Stuart Henderson wrote: > > On 2015/12/23 14:47, Patrik Lundin wrote: > >> On Wed, Dec 23, 2015 at 11:33:30AM +, Stuart Henderson wrote: > >> > Updated tar.gz for the 0.9.2-P1 crash fix

Re: net/kea

On 2015/12/09 20:33, Patrik Lundin wrote: > On Tue, Dec 08, 2015 at 11:15:04PM +, Stuart Henderson wrote: > > On 2015/12/08 23:59, Patrik Lundin wrote: > > > I have posted a question to kea-dev regarding this: > > > https://lists.isc.org/pipermail/kea-dev/2015-December/000588.html > > > > > >

Re: net/kea

On 12/8/2015 5:41 PM, Patrik Lundin wrote: > On Tue, Dec 08, 2015 at 11:18:53PM +0100, Patrik Lundin wrote: >> >> What is picking up botan? I looked at every @bin and @lib with ldd and I >> can't see anyone referring to that library. What am i missing? >> > >>From

Re: net/kea

On Tue, Dec 08, 2015 at 11:15:04PM +, Stuart Henderson wrote: > On 2015/12/08 23:59, Patrik Lundin wrote: > > I have posted a question to kea-dev regarding this: > > https://lists.isc.org/pipermail/kea-dev/2015-December/000588.html > > > > Let's see how that goes. > > Good idea. > I have

Re: net/kea

On Wed, Dec 09, 2015 at 08:33:10PM +0100, Patrik Lundin wrote: > > Attached is a port with the explicit flag set and a shortened > LIB_DEPENDS/WANTLIB list. I have tested that it ignores the use of Botan > when it is installed. Let me know what you think. > Stuart: It would of course be nice to

Re: net/kea

On Tue, Dec 08, 2015 at 07:39:24PM +0100, Patrik Lundin wrote: > > not a port problem, but "message" is a terrible name for sonething that > > upstream want to place in a system directory! > > I have mentioned this in my upstream kea-dev thread. > FYI: upstream has created a ticket which aims

Re: net/kea

On Mon, Dec 07, 2015 at 09:59:44AM +, Stuart Henderson wrote: > > > > 1. When running "make update-plist" I get the following messages: > > === > > make-plist: Bogus element outside of every prefix: /etc/kea/kea.conf > > make-plist: Bogus element outside of every prefix: /etc/kea/keactrl.conf

Re: net/kea

On Tue, Dec 08, 2015 at 10:41:07PM +, Stuart Henderson wrote: > > I see it in perfdhcp. Actually it looks like there's another option, we can > instead use CONFIGURE_ARGS+= --with-openssl - which would you prefer? > > $ objdump -p ../fake-amd64/usr/local/sbin/perfdhcp > [...] > NEEDED

Re: net/kea

On Tue, Dec 08, 2015 at 11:18:53PM +0100, Patrik Lundin wrote: > > What is picking up botan? I looked at every @bin and @lib with ldd and I > can't see anyone referring to that library. What am i missing? > >From http://kea.isc.org/docs/kea-guide.html#required-software: === Kea supports two

Re: net/kea

On 2015/12/08 23:18, Patrik Lundin wrote: > On Tue, Dec 08, 2015 at 09:46:34PM +, Stuart Henderson wrote: > > Thanks, this is looking good to me, Only two changes I'd make. First is > > in the rc script, what you have isn't wrong, but the mkdir/chown/chmod > > lines can be replaced with a

Re: net/kea

On 2015/12/08 23:59, Patrik Lundin wrote: > On Tue, Dec 08, 2015 at 10:41:07PM +, Stuart Henderson wrote: > > > > I see it in perfdhcp. Actually it looks like there's another option, we can > > instead use CONFIGURE_ARGS+= --with-openssl - which would you prefer? > > > > $ objdump -p

Re: net/kea

On Tue, Dec 08, 2015 at 09:46:34PM +, Stuart Henderson wrote: > Thanks, this is looking good to me, Only two changes I'd make. First is > in the rc script, what you have isn't wrong, but the mkdir/chown/chmod > lines can be replaced with a single command: > > install -d -o root:_kea -m

Re: net/kea

Thanks, this is looking good to me, Only two changes I'd make. First is in the rc script, what you have isn't wrong, but the mkdir/chown/chmod lines can be replaced with a single command: install -d -o root:_kea -m 775 ${LOCALSTATEDIR}/run/kea Second change is WANTLIB/LIB_DEPENDS; it

Re: net/kea

On 2015/12/06 21:19, Patrik Lundin wrote: > Hello, > > I am working on a port for Kea, the new DHCP server from ISC. > > === > # cat pkg/DESC > Kea is a new open source DHCPv4/DHCPv6 server being developed by Internet > Systems Consortium. The objective of this project is to provide a very >