From owner-freebsd-gecko@FreeBSD.ORG Sun Oct 23 00:20:12 2011 Return-Path: Delivered-To: gecko@hub.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id BF4FF106566C for ; Sun, 23 Oct 2011 00:20:12 +0000 (UTC) (envelope-from gnats@FreeBSD.org) Received: from freefall.freebsd.org (freefall.freebsd.org [IPv6:2001:4f8:fff6::28]) by mx1.freebsd.org (Postfix) with ESMTP id 966FF8FC08 for ; Sun, 23 Oct 2011 00:20:12 +0000 (UTC) Received: from freefall.freebsd.org (localhost [127.0.0.1]) by freefall.freebsd.org (8.14.4/8.14.4) with ESMTP id p9N0KC0n054029 for ; Sun, 23 Oct 2011 00:20:12 GMT (envelope-from gnats@freefall.freebsd.org) Received: (from gnats@localhost) by freefall.freebsd.org (8.14.4/8.14.4/Submit) id p9N0KC0J054028; Sun, 23 Oct 2011 00:20:12 GMT (envelope-from gnats) Date: Sun, 23 Oct 2011 00:20:12 GMT Message-Id: <201110230020.p9N0KC0J054028@freefall.freebsd.org> To: gecko@FreeBSD.org From: Nali Toja Cc: Subject: Re: ports/161421: [patch] www/firefox: use libevent2 X-BeenThere: freebsd-gecko@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list Reply-To: Nali Toja List-Id: Gecko Rendering Engine issues List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 23 Oct 2011 00:20:12 -0000 The following reply was made to PR ports/161421; it has been noted by GNATS. From: Nali Toja To: FreeBSD-gnats-submit@freebsd.org Cc: Subject: Re: ports/161421: [patch] www/firefox: use libevent2 Date: Sun, 23 Oct 2011 00:18:37 +0000 --=-=-= Content-Type: text/plain Nali Toja writes: > +++ Mk/bsd.gecko.mk 9 Oct 2011 09:23:18 -0000 > @@ -775,6 +782,10 @@ > @${ECHO_CMD} "#pragma GCC visibility push(default)" >> ${MOZSRC}/${subdir}/iconv.h > @${ECHO_CMD} "#include \"${LOCALBASE}/include/iconv.h\"" >> ${MOZSRC}/${subdir}/iconv.h > @${ECHO_CMD} "#pragma GCC visibility pop" >> ${MOZSRC}/${subdir}/iconv.h > + @${ECHO_CMD} "#pragma GCC system_header" >> ${MOZSRC}/${subdir}/event.h > + @${ECHO_CMD} "#pragma GCC visibility push(default)" >> ${MOZSRC}/${subdir}/event.h > + @${ECHO_CMD} "#include " >> ${MOZSRC}/${subdir}/event.h > + @${ECHO_CMD} "#pragma GCC visibility pop" >> ${MOZSRC}/${subdir}/event.h Here is a better version for this hunk, including fix. --=-=-= Content-Type: text/x-patch Content-Disposition: attachment; filename=h.diff Index: Mk/bsd.gecko.mk =================================================================== RCS file: /a/.csup/ports/Mk/bsd.gecko.mk,v retrieving revision 1.42 diff -u -p -r1.42 bsd.gecko.mk --- Mk/bsd.gecko.mk 28 Sep 2011 03:14:22 -0000 1.42 +++ Mk/bsd.gecko.mk 9 Oct 2011 09:23:18 -0000 @@ -764,10 +774,14 @@ gecko-post-patch: ${WRKSRC}/configure .for subdir in config/system_wrappers nsprpub/config/system_wrappers js/src/config/system_wrappers_js @${MKDIR} ${MOZSRC}/${subdir} - @${ECHO_CMD} "#pragma GCC system_header" >> ${MOZSRC}/${subdir}/iconv.h - @${ECHO_CMD} "#pragma GCC visibility push(default)" >> ${MOZSRC}/${subdir}/iconv.h - @${ECHO_CMD} "#include \"${LOCALBASE}/include/iconv.h\"" >> ${MOZSRC}/${subdir}/iconv.h - @${ECHO_CMD} "#pragma GCC visibility pop" >> ${MOZSRC}/${subdir}/iconv.h +.for h in iconv.h:\"${LOCALBASE}/include/iconv.h\" \ + event.h: \ + evutil.h: + @${ECHO_CMD} "#pragma GCC system_header" >> ${MOZSRC}/${subdir}/${h:C/:.*//} + @${ECHO_CMD} "#pragma GCC visibility push(default)" >> ${MOZSRC}/${subdir}/${h:C/:.*//} + @${ECHO_CMD} "#include ${h:C/.*://}" >> ${MOZSRC}/${subdir}/${h:C/:.*//} + @${ECHO_CMD} "#pragma GCC visibility pop" >> ${MOZSRC}/${subdir}/${h:C/:.*//} +.endfor .endfor .for subdir in "" nsprpub js/src @if [ -f ${MOZSRC}/${subdir}/config/system-headers ] ; then \ --=-=-=--