From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.io!.POSTED.blaine.gmane.org!not-for-mail From: Eli Zaretskii Newsgroups: gmane.emacs.bugs Subject: bug#64577: 29.0.92; configure on NetBSD forces unstable terminfo library Date: Thu, 13 Jul 2023 08:19:21 +0300 Message-ID: <83jzv45qxy.fsf@gnu.org> References: <87v8eqrxq8.fsf@notcom.org> <837cr572jt.fsf@gnu.org> <83zg415k7c.fsf@gnu.org> <83wmz55d2l.fsf@gnu.org> Injection-Info: ciao.gmane.io; posting-host="blaine.gmane.org:116.202.254.214"; logging-data="8202"; mail-complaints-to="usenet@ciao.gmane.io" Cc: 64577@debbugs.gnu.org To: Valtteri Vuorikoski Original-X-From: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane-mx.org@gnu.org Thu Jul 13 07:20:20 2023 Return-path: Envelope-to: geb-bug-gnu-emacs@m.gmane-mx.org Original-Received: from lists.gnu.org ([209.51.188.17]) by ciao.gmane.io with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1qJokV-0001y2-FB for geb-bug-gnu-emacs@m.gmane-mx.org; Thu, 13 Jul 2023 07:20:19 +0200 Original-Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1qJokJ-0003dh-4W; Thu, 13 Jul 2023 01:20:07 -0400 Original-Received: from eggs.gnu.org ([2001:470:142:3::10]) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1qJokE-0003dQ-Sm for bug-gnu-emacs@gnu.org; Thu, 13 Jul 2023 01:20:03 -0400 Original-Received: from debbugs.gnu.org ([209.51.188.43]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1qJokE-00080M-A7 for bug-gnu-emacs@gnu.org; Thu, 13 Jul 2023 01:20:02 -0400 Original-Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1qJokD-0006Ab-T1 for bug-gnu-emacs@gnu.org; Thu, 13 Jul 2023 01:20:01 -0400 X-Loop: help-debbugs@gnu.org Resent-From: Eli Zaretskii Original-Sender: "Debbugs-submit" Resent-CC: bug-gnu-emacs@gnu.org Resent-Date: Thu, 13 Jul 2023 05:20:01 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 64577 X-GNU-PR-Package: emacs Original-Received: via spool by 64577-submit@debbugs.gnu.org id=B64577.168922555523636 (code B ref 64577); Thu, 13 Jul 2023 05:20:01 +0000 Original-Received: (at 64577) by debbugs.gnu.org; 13 Jul 2023 05:19:15 +0000 Original-Received: from localhost ([127.0.0.1]:53173 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1qJojT-000699-BN for submit@debbugs.gnu.org; Thu, 13 Jul 2023 01:19:15 -0400 Original-Received: from eggs.gnu.org ([209.51.188.92]:51280) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1qJojQ-00068u-Sn for 64577@debbugs.gnu.org; Thu, 13 Jul 2023 01:19:14 -0400 Original-Received: from fencepost.gnu.org ([2001:470:142:3::e]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1qJojK-0007vX-Dm; Thu, 13 Jul 2023 01:19:06 -0400 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=gnu.org; s=fencepost-gnu-org; h=References:Subject:In-Reply-To:To:From:Date: mime-version; bh=70emL9Xnw8Ksq5Bh7TBctlk/mZtj6OV1vAxSWPzaStE=; b=P9SEgKEcjJpG SWaXmew36LwgHATr60cHNLRDJoLzOFarWOTLKTUyMkOIHwFKOgFowfBbZyEpRPXsdIf/wVdnaV+I5 JpFc+2O1MvAaSE+tEccgPNR7KwenEDGIzmHKHd+M+xRGifP2ASXW+e55Zo/+fnaI1k0q+il/NawQS 3W7slDdpSE7nwu/2uiOgHWrSIonUIQ98sRE0b2jq7VcbRD7hvFNyK+vFxRpqBz9Wt9vYLtJaKw4pK DnewWxPQ969q1OSwP7ukNVHfEn0s+oL0cBi+gE2OapI/MmFlVDfYiObiv28T7mgai66PGFXRQj4Sw gfkRrixeTDjcKie6rVG/dQ==; Original-Received: from [87.69.77.57] (helo=home-c4e4a596f7) by fencepost.gnu.org with esmtpsa (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1qJojJ-0006go-SN; Thu, 13 Jul 2023 01:19:06 -0400 In-Reply-To: (message from Valtteri Vuorikoski on Wed, 12 Jul 2023 23:22:16 +0300) X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list X-BeenThere: bug-gnu-emacs@gnu.org List-Id: "Bug reports for GNU Emacs, the Swiss army knife of text editors" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane-mx.org@gnu.org Original-Sender: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane-mx.org@gnu.org Xref: news.gmane.io gmane.emacs.bugs:264997 Archived-At: > Date: Wed, 12 Jul 2023 23:22:16 +0300 > From: Valtteri Vuorikoski > Cc: 64577@debbugs.gnu.org > > On Wed, Jul 12, 2023 at 07:06:42PM +0300, Eli Zaretskii wrote: > > > It fixes the immediate problem, yes. But I don't think this special > > > case should stick around, since it seems relevant only to a very > > > ancient version of NetBSD. > > > > We don't want to drop support of old versions just because we can, and > > we don't really know how "ancient" those ancient versions should > > actually be to require this special case. We know that in May 2010 it > > was TRT, but not when it stopped being TRT. Without knowing that, I'd > > like to avoid removing this, at least until we hear a real problem > > with it. Asking NetBSD users to install ncurses doesn't sound like a > > problem to me. > > Can't say I'm happy about keeping the magical special case, which will > result in a different thing happening than what configure prints out. That happens on other systems as well, look at the code around the netbsd case. The fact that the configure script says "ncurses" is not a promise that ncurses will be used, it's just a status report of that particular test. > The reason not much has been heard about this is probably that most > users likely use packaged emacs, and the pkgsrc build system forcibly sets > TERMINFO=yes if it determines that this is a NetBSD with the relevant > headers available. See this package build makefile: > http://cvsweb.netbsd.org/bsdweb.cgi/pkgsrc/editors/emacs21/Makefile.common?rev=1.21&content-type=text/x-cvsweb-markup > "CPPFLAGS+=-DTERMINFO" (similarly check exists for emacsen up to 28). If you could find out since when did the NetBSD pkgsrc system start using this override, it could be a data point that we could consider in making these decisions. > How about this then, this way should preserve the intent of the > original special casing while honoring the library selection configure > ended up with: > > --- a/configure.ac > +++ b/configure.ac > @@ -5212,9 +5212,8 @@ AC_DEFUN > ;; > > netbsd) > - if test "x$LIBS_TERMCAP" != "x-lterminfo"; then > + if test "x$LIBS_TERMCAP" = "x-ltermcap"; then > TERMINFO=no > - LIBS_TERMCAP="-ltermcap" > fi > ;; > > On ancient NetBSD (without ncurses) LIBS_TERMCAP should be detected as > -ltermcap, and this will ensure that TERMINFO stays undefined > (probably a no-op). On a newer system where someone actually wants to use > termcap and forces LIBS_TERMCAP, this will make sure that TERMINFO is > "no" even though libtermcap is a symlink to libterminfo (= only > the termcap compatibility functions will be used and terminfo stuff > will be ignored even though configure sees tputs() and friends). > > The end result should be the same binary as pkgsrc packaging has > provided for years, except that on a system with ncurses installed > it will be used. I don't mind installing the above instead of my proposal, thanks.