Date: Sat, 15 Mar 2014 14:09:40 +0400 From: Andrej Zverev <az@freebsd.org> To: Thierry Thomas <thierry@freebsd.org> Cc: "svn-ports-head@freebsd.org" <svn-ports-head@freebsd.org>, "svn-ports-all@freebsd.org" <svn-ports-all@freebsd.org>, "ports-committers@freebsd.org" <ports-committers@freebsd.org> Subject: Re: svn commit: r348304 - in head/lang/perl5.16: . files Message-ID: <CAD5bB%2BhHjiN7_ET%2BdrfsFj-F63%2B825Oh%2BRnikPnBzqWkY5QFjw@mail.gmail.com> In-Reply-To: <201403150957.s2F9veIk002825@svn.freebsd.org> References: <201403150957.s2F9veIk002825@svn.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
i did think what you will apply patch against x11/rxvt-unicode, but not lang/perl.5.16, but all i can now say. I'm shocked :-) Let see on result. On Sat, Mar 15, 2014 at 1:57 PM, Thierry Thomas <thierry@freebsd.org> wrote: > Author: thierry > Date: Sat Mar 15 09:57:40 2014 > New Revision: 348304 > URL: http://svnweb.freebsd.org/changeset/ports/348304 > QAT: https://qat.redports.org/buildarchive/r348304/ > > Log: > Fix the build of x11/rxvt-unicode with clang34, by defining a more > C++-friendly dNOOP. > > Note: don't apply the patch proposed in the PR, but a patch proposed > by Craig A. Berry at > http://perl5.git.perl.org/perl.git/commit/7db66e12883f0832ca80164b723768b848187bda > as suggested by az@. > > PR: ports/187571 > Submitted by: /me > Reported by: Thomas Hoffmann <trh411 (at) gmail.com> > Approved by: az > > Added: > head/lang/perl5.16/files/patch-perl.h (contents, props changed) > Modified: > head/lang/perl5.16/Makefile > > Modified: head/lang/perl5.16/Makefile > ============================================================================== > --- head/lang/perl5.16/Makefile Sat Mar 15 09:50:04 2014 (r348303) > +++ head/lang/perl5.16/Makefile Sat Mar 15 09:57:40 2014 (r348304) > @@ -3,7 +3,7 @@ > > PORTNAME= perl > PORTVERSION= ${PERL_VERSION} > -PORTREVISION= 7 > +PORTREVISION= 8 > CATEGORIES= lang devel perl5 > MASTER_SITES= CPAN \ > ${MASTER_SITE_LOCAL:S/$/:local/} \ > @@ -77,7 +77,7 @@ PKGDEINSTALL= ${WRKDIR}/use.perl > .include "${PORTSDIR}/Mk/bsd.default-versions.mk" > > # if this port is default due PERL5_DEFAULT > -# change PKGNAME to reflect this > +# change PKGNAME to reflect this > .if ${PERL_VER} == ${PERL5_DEFAULT} > PKGNAMESUFFIX= ${PERL_VERSION:C/\.[0-9]+\.[0-9]+$//} > .else > > Added: head/lang/perl5.16/files/patch-perl.h > ============================================================================== > --- /dev/null 00:00:00 1970 (empty, because file is newly added) > +++ head/lang/perl5.16/files/patch-perl.h Sat Mar 15 09:57:40 2014 (r348304) > @@ -0,0 +1,14 @@ > +--- perl.h.orig 2013-03-04 16:16:22.000000000 +0100 > ++++ perl.h 2014-03-15 10:42:19.000000000 +0100 > +@@ -359,7 +359,11 @@ > + /* Rats: if dTHR is just blank then the subsequent ";" throws an error */ > + /* Declaring a *function*, instead of a variable, ensures that we don't rely > + on being able to suppress "unused" warnings. */ > ++#ifdef __cplusplus > ++#define dNOOP (void)0 > ++#else > + #define dNOOP extern int Perl___notused(void) > ++#endif > + > + #ifndef pTHX > + /* Don't bother defining tTHX and sTHX; using them outside >
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAD5bB%2BhHjiN7_ET%2BdrfsFj-F63%2B825Oh%2BRnikPnBzqWkY5QFjw>