Date: Mon, 16 Sep 2013 09:30:54 -0700 (PDT) From: Jeremy Chadwick <jdc@koitsu.org> To: FreeBSD-gnats-submit@freebsd.org Subject: ports/182159: Fix ftp/wget to work with perl 5.18 Message-ID: <20130916163054.C210573A1B@icarus.home.lan> Resent-Message-ID: <201309161640.r8GGe294019931@freefall.freebsd.org>
next in thread | raw e-mail | index | archive | help
>Number: 182159 >Category: ports >Synopsis: Fix ftp/wget to work with perl 5.18 >Confidential: no >Severity: non-critical >Priority: medium >Responsible: freebsd-ports-bugs >State: open >Quarter: >Keywords: >Date-Required: >Class: update >Submitter-Id: current-users >Arrival-Date: Mon Sep 16 16:40:01 UTC 2013 >Closed-Date: >Last-Modified: >Originator: Jeremy Chadwick >Release: FreeBSD 9.2-PRERELEASE amd64 >Organization: >Environment: System: FreeBSD icarus.home.lan 9.2-PRERELEASE FreeBSD 9.2-PRERELEASE #0 r255608: Mon Sep 16 00:47:48 PDT 2013 root@icarus.home.lan:/usr/obj/usr/src/sys/X7SBA_RELENG_9_amd64 amd64 >Description: ftp/wget does not build when perl 5.18 is used on the system. The failure happens during the pod2man phase during build, and causes the build to bail out: ./texi2pod.pl -D VERSION="1.14" ./wget.texi wget.pod /usr/local/bin/pod2man --center="GNU Wget" --release="GNU Wget 1.14" wget.pod > wget.1 wget.pod around line 2105: Expected text after =item, not a number wget.pod around line 2110: Expected text after =item, not a number wget.pod around line 2116: Expected text after =item, not a number wget.pod around line 2121: Expected text after =item, not a number wget.pod around line 2126: Expected text after =item, not a number wget.pod around line 2131: Expected text after =item, not a number wget.pod around line 2136: Expected text after =item, not a number wget.pod around line 2141: Expected text after =item, not a number POD document had syntax errors at /usr/local/bin/pod2man line 71. The official wget repo lacks a fix for this problem: http://git.savannah.gnu.org/cgit/wget.git/tree/doc ...and I will be contacting them to have it fixed upstream and will follow up to this PR with a official bug number: http://savannah.gnu.org/bugs/?group=wget In the meantime, roughly 4 months ago, some other folks came up with a workaround: http://www.sourcemage.org/issues/546 However they seemed to miss fixing the IPv6 option (-6), so I've included that in the below. >How-To-Repeat: cd /usr/ports/ftp/wget && make, on a system using perl 5.18. >Fix: Apply below patch (generated with "svn diff"). This will hold us over until I can get a proper fix from the official wget folks. A full replacement files/patch-doc__wget.texi will also be available at the below URL once I get a PR number from GNATS: http://jdc.koitsu.org/freebsd/{prnum}/patch-doc__wget.texi Index: files/patch-doc__wget.texi =================================================================== --- files/patch-doc__wget.texi (revision 327410) +++ files/patch-doc__wget.texi (working copy) @@ -1,6 +1,6 @@ ---- doc/wget.texi -+++ doc/wget.texi -@@ -875,7 +875,7 @@ recommendation to block many unrelated users from a web site due to the +--- doc/wget.texi.orig 2012-08-04 01:41:52.000000000 -0700 ++++ doc/wget.texi 2013-09-16 09:14:28.008869641 -0700 +@@ -876,7 +876,7 @@ actions of one. @cindex proxy @@ -9,7 +9,7 @@ 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 +@@ -977,9 +977,9 @@ whose encoding does not match the one used locally. @cindex IPv6 @@ -16,9 +16,54 @@ -@itemx -4 +@item -4 @itemx --inet4-only - @itemx -6 +-@itemx -6 ++@item -6 @itemx --inet6-only -@@ -3093,7 +3093,7 @@ display properly---the same as @samp{-p}. + Force connecting to IPv4 or IPv6 addresses. With @samp{--inet4-only} + or @samp{-4}, Wget will only connect to IPv4 hosts, ignoring AAAA +@@ -2106,32 +2106,32 @@ + + + @table @asis +-@item 0 ++@item .0 + No problems occurred. + +-@item 1 ++@item .1 + Generic error code. + +-@item 2 ++@item .2 + Parse error---for instance, when parsing command-line options, the + @samp{.wgetrc} or @samp{.netrc}... + +-@item 3 ++@item .3 + File I/O error. + +-@item 4 ++@item .4 + Network failure. + +-@item 5 ++@item .5 + SSL verification failure. + +-@item 6 ++@item .6 + Username/password authentication failure. + +-@item 7 ++@item .7 + Protocol errors. + +-@item 8 ++@item .8 + Server issued an error response. + @end table + +@@ -3094,7 +3094,7 @@ Change setting of passive @sc{ftp}, equivalent to the @samp{--passive-ftp} option. @@ -27,7 +72,7 @@ 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 +@@ -3605,7 +3605,7 @@ may be specified from within Wget itself. @table @samp @@ -36,5 +81,3 @@ @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: >Unformatted:
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20130916163054.C210573A1B>