From mboxrd@z Thu Jan 1 00:00:00 1970 From: Leo Famulari Subject: Re: [PATCH] gnu: lynx: Support HTTPS (SSL) connections Date: Thu, 3 Mar 2016 22:01:40 -0500 Message-ID: <20160304030140.GA30676@jasmine> References: <1457059066-8060-1-git-send-email-tobias.geerinckx.rice@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from eggs.gnu.org ([2001:4830:134:3::10]:55411) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1abfzu-0004w0-VO for guix-devel@gnu.org; Thu, 03 Mar 2016 22:01:47 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1abfzq-0007CT-VN for guix-devel@gnu.org; Thu, 03 Mar 2016 22:01:46 -0500 Received: from out1-smtp.messagingengine.com ([66.111.4.25]:36137) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1abfzq-0007C8-Pt for guix-devel@gnu.org; Thu, 03 Mar 2016 22:01:42 -0500 Content-Disposition: inline In-Reply-To: <1457059066-8060-1-git-send-email-tobias.geerinckx.rice@gmail.com> List-Id: "Development of GNU Guix and the GNU System distribution." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: guix-devel-bounces+gcggd-guix-devel=m.gmane.org@gnu.org Sender: guix-devel-bounces+gcggd-guix-devel=m.gmane.org@gnu.org To: tobias.geerinckx.rice@gmail.com Cc: guix-devel@gnu.org On Fri, Mar 04, 2016 at 03:37:46AM +0100, tobias.geerinckx.rice@gmail.com wrote: > From: Tobias Geerinckx-Rice > > * gnu/packages/lynx.scm (lynx)[inputs]: Add 'openssl'. > [arguments]: Convert to list; add configure flag for SSL support. Wow, thanks for this patch! Can you say if you learned anything else... "interesting" about lynx and https support? For example, a couple months ago I was reading our bug reports and saw an old one about https support in w3m (another console browser). I dug a little deeper and realized that https support was completely broken by default. You can see the result in commit 62339e2d493bf87. So, do you know if lynx is still supporting broken ciphers and protocols, or if there are other problems of that nature? Also, what is role of gnutls once this patch is applied? Does lynx need to refer to both gnutls and openssl? > --- > gnu/packages/lynx.scm | 30 +++++++++++++++++------------- > 1 file changed, 17 insertions(+), 13 deletions(-) > > diff --git a/gnu/packages/lynx.scm b/gnu/packages/lynx.scm > index 3182b3e..7e1ef1d 100644 > --- a/gnu/packages/lynx.scm > +++ b/gnu/packages/lynx.scm > @@ -48,24 +48,28 @@ > ("libidn" ,libidn) > ("gnutls" ,gnutls) > ("libgcrypt" ,libgcrypt) > + ("openssl" ,openssl) > ("unzip" ,unzip) > ("zlib" ,zlib) > ("gzip" ,gzip) > ("bzip2" ,bzip2))) > (arguments > - `(#:configure-flags '("--with-pkg-config" > - "--with-screen=ncurses" > - "--with-zlib" > - "--with-bzlib" > - "--with-gnutls" > - ;; "--with-socks5" ; XXX TODO > - "--enable-widec" > - "--enable-ascii-ctypes" > - "--enable-local-docs" > - "--enable-htmlized-cfg" > - "--enable-gzip-help" > - "--enable-nls" > - "--enable-ipv6") > + `(#:configure-flags > + (list "--with-pkg-config" > + "--with-screen=ncurses" > + "--with-zlib" > + "--with-bzlib" > + "--with-gnutls" > + (string-append "--with-ssl=" > + (assoc-ref %build-inputs "openssl")) > + ;; "--with-socks5" ; XXX TODO > + "--enable-widec" > + "--enable-ascii-ctypes" > + "--enable-local-docs" > + "--enable-htmlized-cfg" > + "--enable-gzip-help" > + "--enable-nls" > + "--enable-ipv6") > #:tests? #f ; no check target > #:phases (alist-replace > 'install > -- > 2.6.3 > >