Update of devel/libfirm to 1.19.1 lang/cparser to 0.9.12 libfirm: configure patch is required to disable "-fvisilbility=hidden"-feature which removes certain parts in the resulting library which are required by cparser. - libfirm.so was raised to .1 by upstream - I haven't found any depending ports other than cparser. cparser: Makefile patch is no longer required. Fix: Patch attached with submission follows:
Responsible Changed From-To: freebsd-ports-bugs->crees Mine
State Changed From-To: open->analyzed I've replaced your huge patch to configure with a small patch to configure.ac and rerun autoconf, as is the usual policy. http://www.bayofrum.net/~crees/patches/libfirm-ac.diff Taken for Tinderbox run.
State Changed From-To: analyzed->closed cd ../
crees 2011-08-12 17:13:55 UTC FreeBSD ports repository Modified files: devel/libfirm Makefile distinfo pkg-plist lang/cparser Makefile distinfo Added files: devel/libfirm/files patch-configure.ac Removed files: lang/cparser/files patch-Makefile Log: - Update devel/libfirm to 1.19.1 - Update lang/cparser to 0.9.12 and bump LIB_DEPENDS for libfirm PR: ports/159627 Submitted by: Armin Pirkovitsch <armin@frozen-zone.org> (maintainer) Revision Changes Path 1.6 +2 -2 ports/devel/libfirm/Makefile 1.6 +2 -2 ports/devel/libfirm/distinfo 1.1 +12 -0 ports/devel/libfirm/files/patch-configure.ac (new) 1.4 +7 -12 ports/devel/libfirm/pkg-plist 1.6 +2 -2 ports/lang/cparser/Makefile 1.6 +2 -2 ports/lang/cparser/distinfo 1.2 +0 -11 ports/lang/cparser/files/patch-Makefile (dead) _______________________________________________ cvs-all@freebsd.org mailing list http://lists.freebsd.org/mailman/listinfo/cvs-all To unsubscribe, send any mail to "cvs-all-unsubscribe@freebsd.org"
State Changed From-To: closed->open Munged message, sorry...
State Changed From-To: open->closed Committed. Thanks!
Hey, Apparently rdivacky@ found that cparser doesn't work on amd64 without the patch at [1]. May I have your approval for it? Chris [1] http://www.bayofrum.net/~crees/patches/cparser-amd64.diff
crees 2011-08-13 06:41:17 UTC FreeBSD ports repository Modified files: lang/cparser Makefile Added files: lang/cparser/files patch-firm_machine-c Log: Fix on amd64 -- cputype is called x86_64 here. PR: ports/159627 Submitted by: rdivacky Approved by: Armin Pirkovitsch <armin@frozen-zone.org> (maintainer) Revision Changes Path 1.7 +1 -0 ports/lang/cparser/Makefile 1.1 +25 -0 ports/lang/cparser/files/patch-firm_machine-c (new) _______________________________________________ cvs-all@freebsd.org mailing list http://lists.freebsd.org/mailman/listinfo/cvs-all To unsubscribe, send any mail to "cvs-all-unsubscribe@freebsd.org"
On 13 August 2011 06:59, Armin Pirkovitsch <armin@frozen-zone.org> wrote: > Hi! > > Of course you have my approval for that patch. > > Cheers, > Armin > > Excellent, thanks. If you'd be so kind as to pass the patch upstream whenever's convenient, it'd mean we don't have to keep it there, and other OSs would also find porting easier. Chris
crees 2011-08-13 09:03:38 UTC FreeBSD ports repository Modified files: lang/cparser/files patch-firm_machine-c Log: Change +++ path too-- if one has a /tmp/firm_machine.c for whatever reason patching fails. PR: ports/159627 Submitted by: rdivacky Approved by: maintainer (implicit) Revision Changes Path 1.3 +2 -2 ports/lang/cparser/files/patch-firm_machine-c _______________________________________________ cvs-all@freebsd.org mailing list http://lists.freebsd.org/mailman/listinfo/cvs-all To unsubscribe, send any mail to "cvs-all-unsubscribe@freebsd.org"