Date: Sat, 26 Oct 2019 09:18:00 -0600 From: Adam Weinberger <adamw@adamw.org> To: Po-Chuan Hsieh <sunpoet@freebsd.org> Cc: ports-committers <ports-committers@freebsd.org>, svn-ports-all <svn-ports-all@freebsd.org>, svn-ports-head <svn-ports-head@freebsd.org> Subject: Re: svn commit: r515633 - head/print/texinfo Message-ID: <CAP7rwchFU7Hj9K9=03LBk9VV10O4JbDdsFyYq%2ByEsGd-DN_=vQ@mail.gmail.com> In-Reply-To: <CAMHz58SF5SGWydBdbN0cnCrHjgDkKfrXPFhteL=7%2BEtFVrx-zw@mail.gmail.com> References: <201910251909.x9PJ9KPq054081@repo.freebsd.org> <CAP7rwchpefkMOe4y2HvmOdqPyb%2BmXZfw31CMiam95ieRwmTTPQ@mail.gmail.com> <CAMHz58SF5SGWydBdbN0cnCrHjgDkKfrXPFhteL=7%2BEtFVrx-zw@mail.gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
On Sat, Oct 26, 2019 at 7:54 AM Po-Chuan Hsieh <sunpoet@freebsd.org> wrote: > > > > On Sat, Oct 26, 2019 at 3:24 AM Adam Weinberger <adamw@adamw.org> wrote: >> >> On Fri, Oct 25, 2019 at 1:09 PM Sunpoet Po-Chuan Hsieh >> <sunpoet@freebsd.org> wrote: >> > >> > Author: sunpoet >> > Date: Fri Oct 25 19:09:20 2019 >> > New Revision: 515633 >> > URL: https://svnweb.freebsd.org/changeset/ports/515633 >> > >> > Log: >> > Update to 6.7 >> > >> > - Update texinfo.tex >> > >> > Changes: https://git.savannah.gnu.org/cgit/texinfo.git/plain/NEWS >> > https://lists.gnu.org/archive/html/bug-texinfo/2019-09/msg00018.html >> > https://git.savannah.gnu.org/cgit/texinfo.git/log/doc/texinfo.tex >> > PR: 240976 >> > Exp-run by: antoine >> >> Hi Sunpoet, >> >> There are some NLS problems here. If texinfo and p5-Locale-libintl are >> both built WITHOUT_NLS, you get: >> > > Hi Adam, > > Good catch! > Though I've disabled NLS on my box, gettext is still installed by other ports. > Please try https://people.freebsd.org/~sunpoet/patch/print-texinfo-NLS.txt > Thanks! > > Regards, > sunpoet The patch fixes the build failure, thank you for that! Just a head's up, with this patch, in both scenarios (texinfo/p5-L-l OFF/OFF, OFF/ON), QA reports: Error: /usr/local/lib/texinfo/Parsetexi.so is linked to /usr/local/lib/libiconv.so.2 from converters/libiconv but it is not declared as a dependency Warning: you need USES+=iconv, USES+=iconv:wchar_t, or USES+=iconv:translit depending on needs # Adam -- Adam Weinberger adamw@adamw.org https://www.adamw.org
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAP7rwchFU7Hj9K9=03LBk9VV10O4JbDdsFyYq%2ByEsGd-DN_=vQ>