Updated like suggested by Stuart & others. Dropped liburcu flavor. Tested it and pthread is faster.
pthread queries/sec: 24000
liburcu queries/sec: 12000
rc: ${VARBASE} didn't work. Other ports also use fixed path (/var/xxx).
net_gdnsd.tar.gz
Description: GNU Zip compressed data
Thanks,
Regards,
Sam
--
Samuel Streiner
samuel.strei...@gmail.com
Am 07.04.2014 um 18:12 schrieb Stuart Henderson <st...@openbsd.org>:
> On 2014/04/07 17:09, Stuart Henderson wrote:
>> On 2014/04/07 17:44, Joerg Jung wrote:
>>> On Mon, Apr 07, 2014 at 02:43:43PM +0200, Samuel Streiner wrote:
>>>
>>>> Please find attached a new port of gdnsd - geographic dns daemon
>>>>
>>>> Please note that the gdnsd port has a flavor liburcu, witch adds a
>>>> dependency
>>>> (devel/liburcu). This port (devel/liburcu) does not exist jet. Please note
>>>> the second mail
>>>> [NEW] devel/liburcu. You will only need devel/liburcu if you use the
>>>> liburcu flavor.
>>>> Per default gdnsd will be build using pthread.
>>>>
>>>> Feedback & Testing (also on other platforms than amd64&i386) welcome,
>>>
>>> Works fine here on amd64.
>>> IMHO this port is in a good shape and mostly ready.
>>> If another ports developer gives an ok, I'm happy to commit this.
>>>
>>> Thanks,
>>> Regards,
>>> Joerg
>>>
>>
>> Not tested, but from reading :-
>>
>> - should be SHARED_ONLY and regen plists (merge pfrag.shared)
>> - static libraries for plugins not needed; try LIBTOOL_FLAGS=
>> --tag=disable-static
>> - .la files for plugins probably not needed, try rm'ing in post-install, but
>> check it still works without
>> - DESCR is a bit of a mess
>> - patches should be generated with 'make update-patches'
>>
>> CONFIGURE_ARGS+= LIBEV_CFLAGS="-I/usr/local/include" \
>> LIBEV_LIBS="-L/usr/local/lib -lev" \
>> ^^^^^^^^^^^^^^
>> should be ${LOCALBASE}
>> --with-rootdir=/var/gdnsd
>> ^^^^
>> should be ${VARBASE}
>>
>> BUILD_DEPENDS= devel/libtool \
>> devel/ragel \
>> devel/autoconf/2.69 \
>> devel/automake/1.11
>> last 2 should be ${MODGNU_AUTOCONF_DEPENDS} and ${MODGNU_AUTOMAKE_DEPENDS}
>>
>> is gnu libtool just for m4 files for autoconf/automake, or something else?
>>
>> RUN_DEPENDS= net/GeoIP
>>
>> are you sure this is only a run dependency, not a LIB_DEPENDS or
>> BUILD_DEPENDS?
>>
>> COMMENT= Authoritative-only GeoIP-aware DNS server
>>
>> lowercase start of comment
>>
>> gdnsd.rc: daemon="/usr/local/sbin/gdnsd", daemon_flags="-d /var/gdnsd"
>> should use ${TRUEPREFIX} and ${VARBASE}
>>
>
> Oh, another thing, is there really any point adding complexity by
> making liburcu optional? Either it's useful, in which case the port
> should just use it anyway, or it's not, in which case it should be
> dropped.
>
