NetBSD Problem Report #50105

From simon@ducks2.heysi.co.uk  Thu Jul 30 18:24:15 2015
Return-Path: <simon@ducks2.heysi.co.uk>
Received: from mail.netbsd.org (mail.netbsd.org [149.20.53.66])
	(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits))
	(Client CN "mail.netbsd.org", Issuer "Postmaster NetBSD.org" (verified OK))
	by mollari.NetBSD.org (Postfix) with ESMTPS id 9868FA654F
	for <gnats-bugs@gnats.NetBSD.org>; Thu, 30 Jul 2015 18:24:15 +0000 (UTC)
Message-Id: <20150730182412.E31E14F415@ducks2.heysi.co.uk>
Date: Thu, 30 Jul 2015 19:24:12 +0100 (BST)
From: base@atomicules.co.uk
Reply-To: base@atomicules.co.uk
To: gnats-bugs@gnats.NetBSD.org
Subject: Update pkgsrc/www/elinks from 0.12pre6 to 0.13 to fix SSL Error
X-Send-Pr-Version: 3.95

>Number:         50105
>Category:       pkg
>Synopsis:       Update pkgsrc/www/elinks from 0.12pre6 to 0.13 to fix SSL Error
>Confidential:   no
>Severity:       serious
>Priority:       low
>Responsible:    pkg-manager
>State:          closed
>Class:          change-request
>Submitter-Id:   net
>Arrival-Date:   Thu Jul 30 18:25:00 +0000 2015
>Closed-Date:    Sun Aug 09 15:13:39 +0000 2015
>Last-Modified:  Sun Aug 09 21:05:01 +0000 2015
>Originator:     base@atomicules.co.uk
>Release:        NetBSD 6.1.5
>Organization:
>Environment:
System: NetBSD 6.1.5 (XEN3_DOMU) amd64
Architecture: x86_64
Machine: amd64
>Description:
	Elinks 0.12pre6 is unable to access certain sites via https. This is one
	such report:
	http://thread.gmane.org/gmane.comp.web.elinks.user/2191/focus=2193 I do not
	know the reason for this, but I do know (and have verified) it is fixed in
	version 0.13. I was going to create a wip package for 0.13, but Thomas
	Klausner suggested creating a PR with a diff instead for the www package.
>How-To-Repeat:
	Try to access https://www.cenapaliw.pl in elinks 0.12pre6. You will receive
	SSL Error.
>Fix:
	Update to elinks-current-0.13. I have a diff available to update
	pkgsrc/www/elinks and hopefully I'll be able to attach that somewhere once
	this PR is created. I used the same set of patches against the 0.12pre6
	version, but adjusted them to suit the 0.13 codebase. Patches patch-ab and
	patch-ac were no longer required.

>Release-Note:

>Audit-Trail:
From: atomicules <base@atomicules.co.uk>
To: gnats-bugs@NetBSD.org
Cc: 
Subject: Re: pkg/50105: Update pkgsrc/www/elinks from 0.12pre6 to 0.13 to fix
 SSL Error
Date: Thu, 30 Jul 2015 19:28:50 +0100

 --FL5UXtIhxfXey3p5
 Content-Type: text/plain; charset=utf-8; format=flowed
 Content-Disposition: inline

 Attempting to attach patch/diff for this PR

 --FL5UXtIhxfXey3p5
 Content-Type: text/plain; charset=utf-8
 Content-Disposition: attachment; filename="0.13patch"

 ? 0.13patch
 Index: Makefile
 ===================================================================
 RCS file: /cvsroot/pkgsrc/www/elinks/Makefile,v
 retrieving revision 1.58
 diff -u -r1.58 Makefile
 --- Makefile	19 Oct 2014 22:27:48 -0000	1.58
 +++ Makefile	30 Jul 2015 17:17:09 -0000
 @@ -1,8 +1,8 @@
  # $NetBSD: Makefile,v 1.58 2014/10/19 22:27:48 alnsn Exp $

 -DISTNAME=		elinks-0.12pre6
 -PKGNAME=		${DISTNAME:S/pre/rc/}
 -PKGREVISION=		8
 +DISTNAME=		elinks-current-0.13
 +PKGNAME=		elinks-0.13
 +PKGREVISION=		1
  CATEGORIES=		www
  MASTER_SITES=		http://elinks.or.cz/download/ \
  			http://ftp.icm.edu.pl/packages/elinks/
 @@ -11,6 +11,11 @@
  MAINTAINER=		pkgsrc-users@NetBSD.org
  HOMEPAGE=		http://elinks.or.cz/
  COMMENT=		Extended/Enhanced Links
 +LICENSE=		gnu-gpl-v2
 +
 +#This from wip/elinks-snapshot
 +post-extract:
 +	cd ${WRKDIR} && ${MV} elinks-* ${DISTNAME}

  GNU_CONFIGURE=		yes
  USE_PKGLOCALEDIR=	yes
 Index: distinfo
 ===================================================================
 RCS file: /cvsroot/pkgsrc/www/elinks/distinfo,v
 retrieving revision 1.28
 diff -u -r1.28 distinfo
 --- distinfo	3 Nov 2012 14:43:25 -0000	1.28
 +++ distinfo	30 Jul 2015 17:17:09 -0000
 @@ -1,12 +1,10 @@
  $NetBSD: distinfo,v 1.28 2012/11/03 14:43:25 wiz Exp $

 -SHA1 (elinks-0.12pre6.tar.bz2) = 3517795e8a390cb36ca249a5be6514b9784520a5
 -RMD160 (elinks-0.12pre6.tar.bz2) = 99d92dde61ed9c626f10d3d431ed4d325b2e33f6
 -Size (elinks-0.12pre6.tar.bz2) = 2854500 bytes
 -SHA1 (patch-aa) = b188ab3975bcfd6a491c2e41a17b1f31703d4667
 -SHA1 (patch-ab) = a1d9b4246869c1999043c76dbcb35996a6b294fd
 -SHA1 (patch-ac) = a7f4bacbdf4073a5a733fe565b1f51dc755c7e4d
 -SHA1 (patch-ad) = 089e2455c64cbfb48907693a653e76cc97961622
 -SHA1 (patch-ae) = a46c419ec3d96825958903459708b8b131862053
 -SHA1 (patch-ag) = c31bede62a6baa1d43ab1dff639f9d7265d2bece
 -SHA1 (patch-ah) = 25357fd0b0aa08f7ca55f3f8c262d2d1a4874710
 +SHA1 (elinks-current-0.13.tar.bz2) = 220dc78cbf03f10bafc8cb92d9a5f93a4d313e7d
 +RMD160 (elinks-current-0.13.tar.bz2) = 72ac8d33dac4608377f14a8ea15c89e780ffedce
 +Size (elinks-current-0.13.tar.bz2) = 2924345 bytes
 +SHA1 (patch-aa) = 77535c2d6c908aa61311789ced93cf1da1daa071
 +SHA1 (patch-ad) = f4cfb51613892c551dc0a956ec1cca4d69240966
 +SHA1 (patch-ae) = ce6a081cb0d5b05065c9851e62e6e727302b4f7d
 +SHA1 (patch-ag) = f0d9fa00e9464f401165bd25ba53cd47ecfb3c74
 +SHA1 (patch-ah) = 6c59e77b9fc9604e1a950317f0391c91d9830a73
 Index: patches/patch-aa
 ===================================================================
 RCS file: /cvsroot/pkgsrc/www/elinks/patches/patch-aa,v
 retrieving revision 1.11
 diff -u -r1.11 patch-aa
 --- patches/patch-aa	14 Jun 2011 13:36:34 -0000	1.11
 +++ patches/patch-aa	30 Jul 2015 17:17:09 -0000
 @@ -1,17 +1,9 @@
 -$NetBSD: patch-aa,v 1.11 2011/06/14 13:36:34 wiz Exp $
 +$NetBSD$

 ---- configure.orig	2009-07-08 09:57:36.000000000 +0000
 +$Don't know why this is required, but as per pkgsrc elinks-0.12pre6
 +--- configure.orig	2015-07-26 22:45:08.000000000 +0000
  +++ configure
 -@@ -18993,7 +18993,7 @@ if test -z "$disable_lua"; then
 - 		for suffix in "" 50; do
 - 			if test "$cf_result" = no && ( test -f "$luadir/include/lua.h" || \
 - 			   test -f "$luadir/include/lua$suffix/lua.h" ) ; then
 --				LUA_LIBS="-L$luadir/lib -llua$suffix -llualib$suffix -lm"
 -+				LUA_LIBS="-L$luadir/lib -llua$suffix -lm"
 - 				LUA_CFLAGS="-I$luadir/include -I$luadir/include/lua$suffix"
 - 
 - 				LIBS="$LUA_LIBS $LIBS_X"
 -@@ -25064,7 +25064,7 @@ _ACEOF
 +@@ -14143,7 +14143,7 @@ _ACEOF


   # Create LIBDIR #define for config.h
 Index: patches/patch-ab
 ===================================================================
 RCS file: patches/patch-ab
 diff -N patches/patch-ab
 --- patches/patch-ab	6 Nov 2008 08:33:31 -0000	1.7
 +++ /dev/null	1 Jan 1970 00:00:00 -0000
 @@ -1,54 +0,0 @@
 -$NetBSD: patch-ab,v 1.7 2008/11/06 08:33:31 bjs Exp $
 -
 ---- src/scripting/lua/core.c.orig	2008-09-21 04:45:22.000000000 -0400
 -+++ src/scripting/lua/core.c
 -@@ -658,7 +658,7 @@ do_hooks_file(LS, unsigned char *prefix,
 - 	if (file_can_read(file)) {
 - 		int oldtop = lua_gettop(S);
 - 
 --		if (lua_dofile(S, file) != 0)
 -+		if (luaL_dofile(S, file) != 0)
 - 			sleep(3); /* Let some time to see error messages. */
 - 		lua_settop(S, oldtop);
 - 	}
 -@@ -669,13 +669,24 @@ do_hooks_file(LS, unsigned char *prefix,
 - void
 - init_lua(struct module *module)
 - {
 --	L = lua_open();
 --
 --	luaopen_base(L);
 --	luaopen_table(L);
 --	luaopen_io(L);
 --	luaopen_string(L);
 --	luaopen_math(L);
 -+	static const luaL_Reg lualibs[] = {
 -+		{"", luaopen_base},
 -+		{LUA_TABLIBNAME, luaopen_table},
 -+		{LUA_IOLIBNAME, luaopen_io},
 -+		{LUA_OSLIBNAME, luaopen_os},
 -+		{LUA_STRLIBNAME, luaopen_string},
 -+		{LUA_MATHLIBNAME, luaopen_math},
 -+		{NULL, NULL}
 -+	};
 -+	const luaL_Reg *lib = lualibs;
 -+  
 -+  	L = lua_open();
 -+  
 -+	for (; lib->func; lib++) {
 -+		lua_pushcfunction(L, lib->func);
 -+		lua_pushstring(L, lib->name);
 -+		lua_call(L, 1, 0);
 -+	}
 - 
 - 	lua_register(L, LUA_ALERT, l_alert);
 - 	lua_register(L, "current_url", l_current_url);
 -@@ -780,7 +791,7 @@ handle_ret_eval(struct session *ses)
 - 		int oldtop = lua_gettop(L);
 - 
 - 		if (prepare_lua(ses) == 0) {
 --			lua_dostring(L, expr);
 -+			luaL_dostring(L, expr);
 - 			lua_settop(L, oldtop);
 - 			finish_lua();
 - 		}
 Index: patches/patch-ac
 ===================================================================
 RCS file: patches/patch-ac
 diff -N patches/patch-ac
 --- patches/patch-ac	6 Nov 2008 08:33:31 -0000	1.4
 +++ /dev/null	1 Jan 1970 00:00:00 -0000
 @@ -1,13 +0,0 @@
 -$NetBSD: patch-ac,v 1.4 2008/11/06 08:33:31 bjs Exp $
 -
 ---- src/scripting/lua/hooks.c.orig	2008-09-21 04:45:22.000000000 -0400
 -+++ src/scripting/lua/hooks.c
 -@@ -200,7 +200,7 @@ static enum evhook_status
 - script_hook_quit(va_list ap, void *data)
 - {
 - 	if (!prepare_lua(NULL)) {
 --		lua_dostring(lua_state, "if quit_hook then quit_hook() end");
 -+		luaL_dostring(lua_state, "if quit_hook then quit_hook() end");
 - 		finish_lua();
 - 	}
 - 
 Index: patches/patch-ad
 ===================================================================
 RCS file: /cvsroot/pkgsrc/www/elinks/patches/patch-ad,v
 retrieving revision 1.2
 diff -u -r1.2 patch-ad
 --- patches/patch-ad	6 Nov 2008 08:33:31 -0000	1.2
 +++ patches/patch-ad	30 Jul 2015 17:17:09 -0000
 @@ -1,6 +1,7 @@
 -$NetBSD: patch-ad,v 1.2 2008/11/06 08:33:31 bjs Exp $
 +$NetBSD$

 ---- src/intl/gettext/Makefile.orig	2008-09-21 04:45:22.000000000 -0400
 +$Don't know why this is required, but as per pkgsrc elinks-0.12pre6
 +--- src/intl/gettext/Makefile.orig	2015-07-01 14:23:19.000000000 +0000
  +++ src/intl/gettext/Makefile
  @@ -3,6 +3,7 @@ include $(top_builddir)/Makefile.config

 @@ -8,5 +9,5 @@
   builddir = $(top_builddir)/src/intl/gettext
  +libdir = $(localedir)

 - OBJS = \
 -  bindtextdom.o \
 + CFLAGS += $(GETTEXT_CFLAGS)
 + 
 Index: patches/patch-ae
 ===================================================================
 RCS file: /cvsroot/pkgsrc/www/elinks/patches/patch-ae,v
 retrieving revision 1.2
 diff -u -r1.2 patch-ae
 --- patches/patch-ae	6 Nov 2008 08:33:31 -0000	1.2
 +++ patches/patch-ae	30 Jul 2015 17:17:09 -0000
 @@ -1,6 +1,7 @@
 -$NetBSD: patch-ae,v 1.2 2008/11/06 08:33:31 bjs Exp $
 +$NetBSD$

 ---- src/util/memory.c.orig	2008-09-21 04:45:22.000000000 -0400
 +$Don't know why this is required, but as per pkgsrc elinks-0.12pre6
 +--- src/util/memory.c.orig	2015-07-01 14:23:19.000000000 +0000
  +++ src/util/memory.c
  @@ -21,6 +21,12 @@
   #include "util/error.h"
 @@ -15,7 +16,7 @@

   #if !defined(DEBUG_MEMLEAK) && !defined(CONFIG_FASTMEM)

 -@@ -181,7 +187,7 @@ mem_mmap_realloc(void *p, size_t old_siz
 +@@ -165,7 +171,7 @@ mem_mmap_realloc(void *p, size_t old_siz

   	if (new_size) {
   #ifdef HAVE_MREMAP
 Index: patches/patch-ag
 ===================================================================
 RCS file: /cvsroot/pkgsrc/www/elinks/patches/patch-ag,v
 retrieving revision 1.2
 diff -u -r1.2 patch-ag
 --- patches/patch-ag	6 Nov 2008 08:33:31 -0000	1.2
 +++ patches/patch-ag	30 Jul 2015 17:17:09 -0000
 @@ -1,8 +1,7 @@
 -$NetBSD: patch-ag,v 1.2 2008/11/06 08:33:31 bjs Exp $
 +$NetBSD$

 -Don't look for message catalogs in ../po/.
 -
 ---- src/intl/gettext/loadmsgcat.c.orig	2008-09-21 04:45:22.000000000 -0400
 +$Don't know why this is required, but as per pkgsrc elinks-0.12pre6
 +--- src/intl/gettext/loadmsgcat.c.orig	2015-07-01 14:23:19.000000000 +0000
  +++ src/intl/gettext/loadmsgcat.c
  @@ -259,7 +259,7 @@ _nl_load_domain(struct loaded_l10nfile *
   	{
 Index: patches/patch-ah
 ===================================================================
 RCS file: /cvsroot/pkgsrc/www/elinks/patches/patch-ah,v
 retrieving revision 1.1
 diff -u -r1.1 patch-ah
 --- patches/patch-ah	2 Jul 2009 20:04:12 -0000	1.1
 +++ patches/patch-ah	30 Jul 2015 17:17:09 -0000
 @@ -1,8 +1,9 @@
 -$NetBSD: patch-ah,v 1.1 2009/07/02 20:04:12 drochner Exp $
 +$NetBSD$

 ---- src/main/select.c.orig	2008-09-21 10:45:22.000000000 +0200
 +$Don't know why this is required, but as per pkgsrc elinks-0.12pre6
 +--- src/main/select.c.orig	2015-07-01 14:23:19.000000000 +0000
  +++ src/main/select.c
 -@@ -223,7 +223,7 @@ select_loop(void (*init)(void))
 +@@ -229,7 +229,7 @@ select_loop(void (*init)(void))
   	check_bottom_halves();

   	while (!program.terminate) {
 @@ -11,7 +12,7 @@
   		int n, i, has_timer;
   		timeval_T t;

 -@@ -264,7 +264,9 @@ select_loop(void (*init)(void))
 +@@ -270,7 +270,9 @@ select_loop(void (*init)(void))
   		if (has_timer) {
   			/* Be sure timeout is not negative. */
   			timeval_limit_to_zero_or_one(&t);

 --FL5UXtIhxfXey3p5--

From: Thomas Klausner <wiz@NetBSD.org>
To: NetBSD bugtracking <gnats-bugs@NetBSD.org>
Cc: 
Subject: Re: pkg/50105: Update pkgsrc/www/elinks from 0.12pre6 to 0.13 to fix
 SSL Error
Date: Fri, 31 Jul 2015 15:50:28 +0200

 elinks-0.13 is currently named an "Experimental GIT snapshot" and we
 usually don't even update to beta versions (though we did here, we
 currently have the "Unstable prerelease").

 Can you please ask upstream about making an official release?

 >  RCS file: /cvsroot/pkgsrc/www/elinks/patches/patch-aa,v
 >  retrieving revision 1.11
 >  diff -u -r1.11 patch-aa
 >  --- patches/patch-aa	14 Jun 2011 13:36:34 -0000	1.11
 >  +++ patches/patch-aa	30 Jul 2015 17:17:09 -0000
 >  @@ -1,17 +1,9 @@
 >  -$NetBSD: patch-aa,v 1.11 2011/06/14 13:36:34 wiz Exp $
 >  +$NetBSD$
 >   
 >  ---- configure.orig	2009-07-08 09:57:36.000000000 +0000
 >  +$Don't know why this is required, but as per pkgsrc elinks-0.12pre6

 While it's good to add patches to comments and pkglint warns about it,
 please don't add ones that do not provide information like this one.

 Thanks,
  Thomas

From: atomicules <base@atomicules.co.uk>
To: gnats-bugs@NetBSD.org
Cc: pkg-manager@netbsd.org, gnats-admin@netbsd.org, pkgsrc-bugs@netbsd.org
Subject: Re: pkg/50105: Update pkgsrc/www/elinks from 0.12pre6 to 0.13 to fix
 SSL Error
Date: Sat, 1 Aug 2015 11:29:14 +0100

 On 31-Jul-2015 13:55:00, Thomas Klausner wrote:
 >
 > elinks-0.13 is currently named an "Experimental GIT snapshot" and we
 > usually don't even update to beta versions (though we did here, we
 > currently have the "Unstable prerelease").

 I was going by the labels/names used on the main download 
 http://elinks.or.cz/download/ page. I think it is just that the rest 
 of the site hasn't been updated to reflect this. I.e. this page 
 http://elinks.or.cz/download.html still lists 0.11 as the stable 
 release. The main page http://elinks.or.cz/ still refers to 0.11 as 
 current stable and 0.12pre6 as current unstable. However, the git tree 
 also hasn't been tagged with a 0.13 release either, so...

 > Can you please ask upstream about making an official release?

 ...I'll ask upstream about them at least tagging things in git, even 
 if, for whatever reason, they can't update the rest of the site.

 > While it's good to add patches to comments and pkglint warns about it,
 > please don't add ones that do not provide information like this one.

 Yeah, I can't say I thought that was a great idea either. Thanks for 
 the review and the advice.

From: atomicules <base@atomicules.co.uk>
To: gnats-bugs@NetBSD.org
Cc: pkg-manager@netbsd.org, gnats-admin@netbsd.org, pkgsrc-bugs@netbsd.org
Subject: Re: pkg/50105: Update pkgsrc/www/elinks from 0.12pre6 to 0.13 to fix
 SSL Error
Date: Sun, 9 Aug 2015 15:50:30 +0100

 Hi Thomas, 

 On 31-Jul-2015 13:55:00, Thomas Klausner wrote:
 > Can you please ask upstream about making an official release?

 I asked upstream on the Elinks user list since this list has more 
 activity than the dev list (neither are very active lists). I also 
 asked on the Elinks IRC channel. I've not had any response.

 However, I've looked into the releases a bit more and I think I may 
 have been wrong in my initial assumptions. I now think 0.12pre6 is 
 actually the latest unstable release and the reason why there hasn't 
 been a tagged 0.13 release is because it is still being developed 
 against this version. And therefore the elinks-current-0.13 releases 
 really are snapshots of the Git tree.

 In which case this PR is void.

 But then I have another question: Could I then take over maintenance 
 of the pkgsrc/wip/elinks-snapshot package, rather than create a new 
 package, and update that to the 0.13 version? By the looks of it, 
 Blair Sadewitz created it in 2007 to use the 0.12pre6 version because 
 pkgsrc/www/elinks was on 0.11.3. As it stands now this package is 
 redundant.

State-Changed-From-To: open->closed
State-Changed-By: wiz@NetBSD.org
State-Changed-When: Sun, 09 Aug 2015 15:13:39 +0000
State-Changed-Why:
No stable release available, let's wait for it.


From: Thomas Klausner <wiz@NetBSD.org>
To: NetBSD bugtracking <gnats-bugs@NetBSD.org>
Cc: 
Subject: Re: pkg/50105: Update pkgsrc/www/elinks from 0.12pre6 to 0.13 to fix
 SSL Error
Date: Sun, 9 Aug 2015 17:12:57 +0200

 On Sun, Aug 09, 2015 at 03:50:30PM +0100, atomicules wrote:
 > However, I've looked into the releases a bit more and I think I may have
 > been wrong in my initial assumptions. I now think 0.12pre6 is actually the
 > latest unstable release and the reason why there hasn't been a tagged 0.13
 > release is because it is still being developed against this version. And
 > therefore the elinks-current-0.13 releases really are snapshots of the Git
 > tree.
 > 
 > In which case this PR is void.

 Ok.

 > But then I have another question: Could I then take over maintenance of the
 > pkgsrc/wip/elinks-snapshot package, rather than create a new package, and
 > update that to the 0.13 version? By the looks of it, Blair Sadewitz created
 > it in 2007 to use the 0.12pre6 version because pkgsrc/www/elinks was on
 > 0.11.3. As it stands now this package is redundant.

 I'm pretty sure you can.
  Thomas

From: David Holland <dholland-pbugs@netbsd.org>
To: gnats-bugs@NetBSD.org
Cc: 
Subject: Re: pkg/50105: Update pkgsrc/www/elinks from 0.12pre6 to 0.13 to fix
 SSL Error
Date: Sun, 9 Aug 2015 21:02:10 +0000

 On Sun, Aug 09, 2015 at 02:55:00PM +0000, atomicules wrote:
  >  But then I have another question: Could I then take over maintenance 
  >  of the pkgsrc/wip/elinks-snapshot package, rather than create a new 
  >  package, and update that to the 0.13 version? By the looks of it, 
  >  Blair Sadewitz created it in 2007 to use the 0.12pre6 version because 
  >  pkgsrc/www/elinks was on 0.11.3. As it stands now this package is 
  >  redundant.

 Just do it - that's the point of wip. (and bjs@ hasn't been around in
 a long time, alas)

 -- 
 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-2014 The NetBSD Foundation, Inc. ALL RIGHTS RESERVED.