From cmt@elch.exwg.net  Tue May 14 19:44:37 2013
Return-Path: <cmt@elch.exwg.net>
Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1])
	by hub.freebsd.org (Postfix) with ESMTP id 52808722
	for <FreeBSD-gnats-submit@freebsd.org>; Tue, 14 May 2013 19:44:37 +0000 (UTC)
	(envelope-from cmt@elch.exwg.net)
Received: from smtp.burggraben.net (base.exwg.net [IPv6:2a01:4f8:140:50a2::2])
	by mx1.freebsd.org (Postfix) with ESMTP id D2D8C790
	for <FreeBSD-gnats-submit@freebsd.org>; Tue, 14 May 2013 19:44:36 +0000 (UTC)
Received: from localhost (localhost [127.0.0.1])
	by smtp.burggraben.net (Postfix) with ESMTP id 7B2ED6003CE
	for <FreeBSD-gnats-submit@freebsd.org>; Tue, 14 May 2013 21:44:35 +0200 (CEST)
Received: from smtp.burggraben.net ([127.0.0.1])
	by localhost (ns.burggraben.net [127.0.0.1]) (amavisd-new, port 10024)
	with ESMTP id q8lbVlL+vdSv for <FreeBSD-gnats-submit@freebsd.org>;
	Tue, 14 May 2013 21:44:34 +0200 (CEST)
Received: from elch.exwg.net (elch.exwg.net [IPv6:2001:470:7b43:1:6a05:caff:fe01:aae9])
	(using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits))
	(Client CN "elch.exwg.net", Issuer "Christoph Moench-Tegeder" (verified OK))
	by smtp.burggraben.net (Postfix) with ESMTPS
	for <FreeBSD-gnats-submit@freebsd.org>; Tue, 14 May 2013 21:44:34 +0200 (CEST)
Received: by elch.exwg.net (Postfix, from userid 1000)
	id EE6C031016; Tue, 14 May 2013 21:44:33 +0200 (CEST)
Message-Id: <20130514194433.EE6C031016@elch.exwg.net>
Date: Tue, 14 May 2013 21:44:33 +0200 (CEST)
From: Christoph Moench-Tegeder <cmt@burggraben.net>
Reply-To: Christoph Moench-Tegeder <cmt@burggraben.net>
To: FreeBSD-gnats-submit@freebsd.org
Cc:
Subject: ftp/wget : fix build with texinfo-5 installed
X-Send-Pr-Version: 3.113
X-GNATS-Notify:

>Number:         178646
>Category:       ports
>Synopsis:       ftp/wget : fix build with texinfo-5 installed
>Confidential:   no
>Severity:       non-critical
>Priority:       medium
>Responsible:    vd
>State:          closed
>Quarter:        
>Keywords:       
>Date-Required:  
>Class:          change-request
>Submitter-Id:   current-users
>Arrival-Date:   Tue May 14 19:50:01 UTC 2013
>Closed-Date:    Fri May 17 08:13:30 UTC 2013
>Last-Modified:  Fri May 17 08:20:00 UTC 2013
>Originator:     Christoph Moench-Tegeder
>Release:        FreeBSD 9.1-RELEASE-p3 amd64
>Organization:
>Environment:
System: FreeBSD elch.exwg.net 9.1-RELEASE-p3 FreeBSD 9.1-RELEASE-p3 #2 r250106: Tue Apr 30 18:06:16 CEST 2013 cmt@elch.exwg.net:/usr/obj/usr/src/sys/ELCH amd64

	ftp/wget      wget-1.14_2
	print/texinfo texinfo-5.1.20130419

>Description:
	ftp/wget fails to build when the texinfo port (print/texinfo) is
	installed, as texinfo 5 is way more strict with it's input than the
	texinfo 4.8 we have in base. In particular, the build fails on wrongly
	nested @itemx/@item constructs in doc/wget.text
>How-To-Repeat:
	
>Fix:
	The problem has already been fixed in upstream git, but not yet
	released. I took the verbatim patch from
	http://git.savannah.gnu.org/cgit/wget.git/patch/doc/wget.texi?id=9e8d899a356d95be34737488bb8fd59d4c6b3dbd
	(thus leaving copyright info on the patch itself intact) and have
	it patched onto the wget source via the post-patch target.
	Ports-Patch via HTTP: http://burggraben.net/hacks/wget_texinfo5.patch.gz

svn status

M       Makefile
A       files/001-fix-texinfo.diff


Index: Makefile
===================================================================
--- Makefile	(revision 318192)
+++ Makefile	(working copy)
@@ -85,5 +85,7 @@
 	${REINPLACE_CMD} -e 's,/usr/local/etc/wgetrc,${PREFIX}/etc/wgetrc,' \
 			 -e 's,@file{},@file{${PREFIX}},' \
 		${WRKSRC}/doc/sample.wgetrc* ${WRKSRC}/doc/wget.texi
+	pwd
+	${PATCH} -d ${WRKSRC} -p1 < ${FILESDIR}/001-fix-texinfo.diff
 
 .include <bsd.port.mk>
Index: files/001-fix-texinfo.diff
===================================================================
--- files/001-fix-texinfo.diff	(revision 0)
+++ files/001-fix-texinfo.diff	(working copy)
@@ -0,0 +1,59 @@
+From 9e8d899a356d95be34737488bb8fd59d4c6b3dbd Mon Sep 17 00:00:00 2001
+From: Stefano Lattarini <stefano.lattarini@gmail.com>
+Date: Thu, 08 Nov 2012 11:15:41 +0000
+Subject: docs: fix errors and warnings with Texinfo 5
+
+Or rather, with the development version 4.13.90, which will eventually
+become Texinfo 5.0.
+
+* doc/wget.texi: Use '@item' instead of '@itemx' in several places,
+as Texinfo 5 refuses to process an '@itemx' that is not preceded by an
+'@item'.
+
+Copyright-paperwork-exempt: yes
+Signed-off-by: Stefano Lattarini <stefano.lattarini@gmail.com>
+---
+(limited to 'doc/wget.texi')
+
+diff --git a/doc/wget.texi b/doc/wget.texi
+index 400debe..8fd6801 100644
+--- a/doc/wget.texi
++++ b/doc/wget.texi
+@@ -875,7 +875,7 @@ recommendation to block many unrelated users from a web site due to the
+ actions of one.
+ 
+ @cindex proxy
+-@itemx --no-proxy
++@item --no-proxy
+ Don't use proxies, even if the appropriate @code{*_proxy} environment
+ variable is defined.
+ 
+@@ -976,7 +976,7 @@ are outside the range of @sc{ascii} characters (that is, greater than
+ whose encoding does not match the one used locally.
+ 
+ @cindex IPv6
+-@itemx -4
++@item -4
+ @itemx --inet4-only
+ @itemx -6
+ @itemx --inet6-only
+@@ -3093,7 +3093,7 @@ display properly---the same as @samp{-p}.
+ Change setting of passive @sc{ftp}, equivalent to the
+ @samp{--passive-ftp} option.
+ 
+-@itemx password = @var{string}
++@item password = @var{string}
+ Specify password @var{string} for both @sc{ftp} and @sc{http} file retrieval. 
+ This command can be overridden using the @samp{ftp_password} and 
+ @samp{http_password} command for @sc{ftp} and @sc{http} respectively.
+@@ -3604,7 +3604,7 @@ In addition to the environment variables, proxy location and settings
+ may be specified from within Wget itself.
+ 
+ @table @samp
+-@itemx --no-proxy
++@item --no-proxy
+ @itemx proxy = on/off
+ This option and the corresponding command may be used to suppress the
+ use of proxy, even if the appropriate environment variables are set.
+--
+cgit v0.9.0.2


>Release-Note:
>Audit-Trail:
Responsible-Changed-From-To: freebsd-ports-bugs->vd 
Responsible-Changed-By: edwin 
Responsible-Changed-When: Tue May 14 19:50:10 UTC 2013 
Responsible-Changed-Why:  
Over to maintainer (via the GNATS Auto Assign Tool) 

http://www.freebsd.org/cgi/query-pr.cgi?pr=178646 
State-Changed-From-To: open->closed 
State-Changed-By: vd 
State-Changed-When: Fri May 17 08:11:30 UTC 2013 
State-Changed-Why:  
Committed, thanks! 

Note: use /usr/ports/Tools/scripts/splitpatch.pl to generate the name 
of the patch file to be placed in files/ 

http://www.freebsd.org/cgi/query-pr.cgi?pr=178646 

From: dfilter@FreeBSD.ORG (dfilter service)
To: bug-followup@FreeBSD.org
Cc:  
Subject: Re: ports/178646: commit references a PR
Date: Fri, 17 May 2013 08:11:16 +0000 (UTC)

 Author: vd
 Date: Fri May 17 08:11:03 2013
 New Revision: 318361
 URL: http://svnweb.freebsd.org/changeset/ports/318361
 
 Log:
   ftp/wget: fix build with texinfo-5 installed
   
   PR:		ports/178646
   Submitted by:	Christoph Moench-Tegeder <cmt@burggraben.net>
 
 Added:
   head/ftp/wget/files/patch-doc__wget.texi   (contents, props changed)
 
 Added: head/ftp/wget/files/patch-doc__wget.texi
 ==============================================================================
 --- /dev/null	00:00:00 1970	(empty, because file is newly added)
 +++ head/ftp/wget/files/patch-doc__wget.texi	Fri May 17 08:11:03 2013	(r318361)
 @@ -0,0 +1,40 @@
 +--- doc/wget.texi
 ++++ doc/wget.texi
 +@@ -875,7 +875,7 @@ recommendation to block many unrelated users from a web site due to the
 + actions of one.
 + 
 + @cindex proxy
 +-@itemx --no-proxy
 ++@item --no-proxy
 + Don't use proxies, even if the appropriate @code{*_proxy} environment
 + variable is defined.
 + 
 +@@ -976,7 +976,7 @@ are outside the range of @sc{ascii} characters (that is, greater than
 + whose encoding does not match the one used locally.
 + 
 + @cindex IPv6
 +-@itemx -4
 ++@item -4
 + @itemx --inet4-only
 + @itemx -6
 + @itemx --inet6-only
 +@@ -3093,7 +3093,7 @@ display properly---the same as @samp{-p}.
 + Change setting of passive @sc{ftp}, equivalent to the
 + @samp{--passive-ftp} option.
 + 
 +-@itemx password = @var{string}
 ++@item password = @var{string}
 + Specify password @var{string} for both @sc{ftp} and @sc{http} file retrieval. 
 + This command can be overridden using the @samp{ftp_password} and 
 + @samp{http_password} command for @sc{ftp} and @sc{http} respectively.
 +@@ -3604,7 +3604,7 @@ In addition to the environment variables, proxy location and settings
 + may be specified from within Wget itself.
 + 
 + @table @samp
 +-@itemx --no-proxy
 ++@item --no-proxy
 + @itemx proxy = on/off
 + This option and the corresponding command may be used to suppress the
 + use of proxy, even if the appropriate environment variables are set.
 +--
 +cgit v0.9.0.2
 _______________________________________________
 svn-ports-all@freebsd.org mailing list
 http://lists.freebsd.org/mailman/listinfo/svn-ports-all
 To unsubscribe, send any mail to "svn-ports-all-unsubscribe@freebsd.org"
 
>Unformatted:
