X-Original-To: alpine-devel@lists.alpinelinux.org Delivered-To: alpine-devel@mail.alpinelinux.org Received: from mailfilter.prov.us (heimdall.prov.us [69.209.46.141]) by mail.alpinelinux.org (Postfix) with ESMTP id 3E3E51A15BF6 for ; Tue, 12 Jul 2011 16:54:49 +0000 (UTC) X-ASG-Debug-ID: 1310489688-01f2cd68099e110001-6PaJeb Received: from smtp.prov.us (smtp.prov.us [69.209.46.178]) by mailfilter.prov.us with ESMTP id y3fPb6qli30yUhaF; Tue, 12 Jul 2011 11:54:48 -0500 (CDT) X-Barracuda-Envelope-From: john@hohm.net X-Barracuda-RBL-Trusted-Forwarder: 69.209.46.178 Received: from [172.17.1.3] (unknown [172.17.1.3]) (Authenticated sender: john_hohm_net) by smtp.prov.us (Postfix) with ESMTPA id 5B4AF15D; Tue, 12 Jul 2011 11:54:48 -0500 (CDT) Message-ID: <4E1C7C58.6040405@hohm.net> X-Barracuda-BBL-Trusted-Forwarder: 172.17.1.3 X-Barracuda-RBL-Trusted-Forwarder: 172.17.1.3 Date: Tue, 12 Jul 2011 11:54:48 -0500 From: John Keith Hohm User-Agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.2.17) Gecko/20110424 Thunderbird/3.1.10 X-Mailinglist: alpine-devel Precedence: list List-Id: Alpine Development List-Unsubscribe: List-Post: List-Help: List-Subscribe: MIME-Version: 1.0 To: Natanael Copa CC: alpine-devel@lists.alpinelinux.org, Carlo Landmeter Subject: Re: [alpine-devel] [PATCH] testing/mtr: new aport References: <1310419297-15518-1-git-send-email-john@hohm.net> <20110712102023.10ce93fe@ncopa-desktop.nor.wtbts.net> X-ASG-Orig-Subj: Re: [alpine-devel] [PATCH] testing/mtr: new aport In-Reply-To: <20110712102023.10ce93fe@ncopa-desktop.nor.wtbts.net> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit X-Barracuda-Connect: smtp.prov.us[69.209.46.178] X-Barracuda-Start-Time: 1310489688 X-Barracuda-URL: http://mailfilter.prov.us:80/cgi-mod/mark.cgi X-Virus-Scanned: by bsmtpd at prov.us X-Barracuda-Bayes: INNOCENT GLOBAL 0.5003 1.0000 0.7500 X-Barracuda-Spam-Score: 0.75 X-Barracuda-Spam-Status: No, SCORE=0.75 using global scores of TAG_LEVEL=1000.0 QUARANTINE_LEVEL=1000.0 KILL_LEVEL=9.0 tests= X-Barracuda-Spam-Report: Code version 3.2, rules version 3.2.2.68721 Rule breakdown below pts rule name description ---- ---------------------- -------------------------------------------------- On 07/12/2011 03:20 AM, Natanael Copa wrote: > I had a look at this patch and, sure it works. It would have been > preferable that it could have changed the configure.in file and then > run aclocal&& autoreconf or similar. Instead of error out on missing > res_mkquery it could have set a config.h variable and then do > something like: > > #ifndef HAVE_MK_QUERY > /* res_mkquery from dietlibc libcruft http://www.fefe.de/dietlibc/ */ > ... > #endif Yes, I actually did something like that first locally but the patch from re-running those tools was ridiculously big, since I did not use the exact same automake and autoconf versions as upstream. > and send that patch upstream. > > (even better would probably to remove the entire dns.c and use c-ares > library) Hey, thanks for the pointer to c-ares. Maybe I should check with upstream whether they are likely to accept such a patch. I had first considered replacing dns.c with the public domain djbdns client library files but thought they might not be keen on losing IPv6 support. > That said, I think its good enough for Alpine anyways. > Thanks! Thanks guys for the inclusion, and for working on Alpine. I am quite pleased to be able to stay with Alpine for an upgrade to a new x86_64 incarnation of an important firewall. -- John Keith Hohm --- Unsubscribe: alpine-devel+unsubscribe@lists.alpinelinux.org Help: alpine-devel+help@lists.alpinelinux.org ---