NetBSD Problem Report #34591

From www@NetBSD.org  Sat Sep 23 04:33:27 2006
Return-Path: <www@NetBSD.org>
Received: by narn.NetBSD.org (Postfix, from userid 31301)
	id 380FF63B9FB; Sat, 23 Sep 2006 04:33:27 +0000 (UTC)
Message-Id: <20060923043327.380FF63B9FB@narn.NetBSD.org>
Date: Sat, 23 Sep 2006 04:33:27 +0000 (UTC)
From: dhgutteridge@sympatico.ca
Reply-To: dhgutteridge@sympatico.ca
To: gnats-bugs@NetBSD.org
Subject: Patch file that fixed DNS lookup problems in www/dillo was mistakenly removed in CVS
X-Send-Pr-Version: www-1.0

>Number:         34591
>Category:       pkg
>Synopsis:       Patch file that fixed DNS lookup problems in www/dillo was mistakenly removed in CVS
>Confidential:   no
>Severity:       non-critical
>Priority:       medium
>Responsible:    adam
>State:          open
>Class:          sw-bug
>Submitter-Id:   net
>Arrival-Date:   Sat Sep 23 04:35:00 +0000 2006
>Last-Modified:  Mon May 25 03:50:02 +0000 2009
>Originator:     David H. Gutteridge
>Release:        
>Organization:
>Environment:
>Description:
The patch file I submitted in PR31205 and committed by Tim Rightnour (garbled@) was removed in more recent changes to www/dillo.  I'm not sure why, presumably by accident.  It was needed to fix DNS query lockups in older versions of NetBSD that didn't have reentrant resolver functions.

In any case, dillo now provides an option in its configure script that allows the threaded DNS feature to be disabled, so here's my attempt at a new patch that takes advantage of this.  (I'm not able to test it, unfortunately, as everything I have is running > 3.0.  I did have a 1.6.2 box, but it had a hardware failure.)

(This isn't quite as precise with the NetBSD version as my old patch, but I would assume impacts to users running 2.99.x would be minimal.)


>How-To-Repeat:

>Fix:
--- Makefile~	2006-09-22 22:43:11.000000000 -0400
+++ Makefile	2006-09-23 00:17:02.000000000 -0400
@@ -21,6 +21,11 @@

 .include "options.mk"

+# NetBSD < 3.0 does not have reentrant resolver functions
+.if ${OPSYS} == "NetBSD" && !empty(OS_VERSION:M[1-2].*)
+CONFIGURE_ARGS+=	--disable-threaded-dns
+.endif
+
 post-install:
 	${INSTALL_DATA_DIR} ${PREFIX}/share/doc/dillo
 	${INSTALL_DATA} ${WRKSRC}/doc/*.txt ${PREFIX}/share/doc/dillo

>Release-Note:

>Audit-Trail:

Responsible-Changed-From-To: pkg-manager->adam
Responsible-Changed-By: snj@netbsd.org
Responsible-Changed-When: Sat, 23 Sep 2006 07:43:27 +0000
Responsible-Changed-Why:
Adam removed it.


From: Joerg Sonnenberger <joerg@britannica.bec.de>
To: gnats-bugs@NetBSD.org
Cc: 
Subject: Re: pkg/34591: Patch file that fixed DNS lookup problems in www/dillo
    was mistakenly removed in CVS
Date: Sat, 23 Sep 2006 14:23:48 +0200

 On Sat, Sep 23, 2006 at 04:35:00AM +0000, dhgutteridge@sympatico.ca wrote:
 > --- Makefile~	2006-09-22 22:43:11.000000000 -0400
 > +++ Makefile	2006-09-23 00:17:02.000000000 -0400
 > @@ -21,6 +21,11 @@
 >  
 >  .include "options.mk"
 >  
 > +# NetBSD < 3.0 does not have reentrant resolver functions
 > +.if ${OPSYS} == "NetBSD" && !empty(OS_VERSION:M[1-2].*)
 > +CONFIGURE_ARGS+=	--disable-threaded-dns
 > +.endif

 This would also apply to DragonFly, FreeBSD 4 and most likely OpenBSD.
 Not sure about the later though.

 Joerg

From: "David H. GUTTERIDGE" <dhgutteridge@sympatico.ca>
To: gnats-bugs@NetBSD.org
Cc: 
Subject: Re: pkg/34591: Patch file that fixed DNS lookup problems in www/dillo was mistak
Date: Sat, 23 Sep 2006 17:21:30 +0000

 >  This would also apply to DragonFly, FreeBSD 4 and most likely OpenBSD.
 >  Not sure about the later though.
 >
 >  Joerg

 When I created the original patch I submitted, I emailed the DragonFly users 
 list
 to see if it would be relevant for that OS too, but didn't get any response, 
 I
 guess I should have just asked you, Joerg.

 FreeBSD changed this particular family of functions over to be reentrant in 
 their
 5.5 release.  From looking at OpenBSD's ports CVS I can see they've simply
 changed the dns.c file to prevent the threaded code from being compiled in, 
 so
 that suggests we can safely follow their method for now for their OS.

 I've attached a more comprehensive patch below that handles all four major 
 BSDs
 for this package, I hope it's to everyone's liking.  Tim (garbled@) was 
 hesitant about
 the idea of bluntly disabling something for a given OS, but I don't see any 
 other way
 to do it, if DragonFly BSD and OpenBSD don't have the functionality yet.

 Dave

 --- Makefile~	2006-09-22 22:43:11.000000000 -0400
 +++ Makefile	2006-09-23 13:12:01.000000000 -0400
 @@ -21,6 +21,20 @@

 .include "options.mk"

 +# NetBSD < 3.0 does not have reentrant resolver functions
 +.if ${OPSYS} == "NetBSD" && !empty(OS_VERSION:M[1-2].*)
 +CONFIGURE_ARGS+=	--disable-threaded-dns
 +# DragonFly BSD as of 1.6 does not have reentrant resolver functions
 +.elif ${OPSYS} == "DragonFly"
 +CONFIGURE_ARGS+=	--disable-threaded-dns
 +# OpenBSD as of 3.9 does not have reentrant resolver functions
 +.elif ${OPSYS} == "OpenBSD"
 +CONFIGURE_ARGS+=	--disable-threaded-dns
 +# FreeBSD < 5.5 does not have (applicable) reentrant resolver functions
 +.elif ${OPSYS} == "FreeBSD" && (!empty(OS_VERSION:M[1-4].*) || 
 !empty(OS_VERSION:M5.[0-4]*))
 +CONFIGURE_ARGS+=	--disable-threaded-dns
 +.endif
 +
 post-install:
 	${INSTALL_DATA_DIR} ${PREFIX}/share/doc/dillo
 	${INSTALL_DATA} ${WRKSRC}/doc/*.txt ${PREFIX}/share/doc/dillo


From: "David H. Gutteridge" <dhgutteridge@sympatico.ca>
To: gnats-bugs@netbsd.org
Cc: 
Subject: Re: pkg/34591: Patch file that fixed DNS lookup problems in www/dillo was mistakenly removed
Date: Tue, 19 May 2009 20:34:10 -0400

 This was for the older version of dillo, and as far as NetBSD goes, it
 was only relevant for releases that are now EOLed.  I believe there are
 supposed to be configure arguments that can be set now to effect the
 same, should someone care.  In other words, I'm fine to close this.

 Dave

From: David Holland <dholland-pbugs@netbsd.org>
To: gnats-bugs@NetBSD.org
Cc: 
Subject: Re: pkg/34591: Patch file that fixed DNS lookup problems in
	www/dillo was mistakenly removed
Date: Mon, 25 May 2009 03:48:33 +0000

 On Wed, May 20, 2009 at 12:35:02AM +0000, David H. Gutteridge wrote:
  >  This was for the older version of dillo, and as far as NetBSD goes, it
  >  was only relevant for releases that are now EOLed.  I believe there are
  >  supposed to be configure arguments that can be set now to effect the
  >  same, should someone care.  In other words, I'm fine to close this.

 There are configure arguments, but no autodetection, so we ought to
 keep something in place until all of the OS releases involved are
 officially dead.

 Does anyone know the current state of the non-NetBSD versions cited
 earlier in the PR?

 -- 
 David A. Holland
 dholland@netbsd.org

>Unformatted:

NetBSD Home
NetBSD PR Database Search

(Contact us) $NetBSD: query-full-pr,v 1.39 2013/11/01 18:47:49 spz Exp $
$NetBSD: gnats_config.sh,v 1.8 2006/05/07 09:23:38 tsutsui Exp $
Copyright © 1994-2007 The NetBSD Foundation, Inc. ALL RIGHTS RESERVED.