It appears that delve/premake4 has a set of unit tests, and two of them do not currently pass, as premake tries to read from /etc/ld.so.conf, which we don't have. The attached patch fixes that problem, and implements regression-test target. You can see tinderbox logs at [1]. [1] https://redports.org/buildarchive/20120131150907-284/ Fix: Patch attached with submission follows:
Responsible Changed From-To: freebsd-ports-bugs->miwi I'll take it.
State Changed From-To: open->feedback Hi, this patch is not clean: |--- premake4.orig/files/patch-src-host-scripts.c 1970-01-01 03:00:00.000000000 +0300 |+++ premake4/files/patch-src-host-scripts.c 2012-01-31 16:05:38.000000000 +0200 -------------------------- (Creating file patch-src-host-scripts.c...) Patching file patch-src-host-scripts.c using Plan A... patch: **** malformed patch at line 55: omatch(result, mask, wantfiles)nif mas"
miwi@freebsd.org wrote: > Hi, > > this patch is not clean: > > [...] > > http://www.freebsd.org/cgi/query-pr.cgi?pr=164665 It appears that the patch has one of it's lines wrapped around at 990 characters. (Does web interface do that automatically or was it my mistake?) Anyway, you can get the original patch at [1]. [1] http://tx97.net/~magv/premake4-fix.diff
State Changed From-To: feedback->closed Committed. Thanks!
miwi 2012-03-04 12:27:14 UTC FreeBSD ports repository Modified files: devel/premake4 Makefile Added files: devel/premake4/files patch-src-base-os.lua patch-src-host-scripts.c Log: - Fix broken regression test PR: 164665 Submitted by: maintainer Revision Changes Path 1.2 +5 -1 ports/devel/premake4/Makefile 1.1 +17 -0 ports/devel/premake4/files/patch-src-base-os.lua (new) 1.1 +5 -0 ports/devel/premake4/files/patch-src-host-scripts.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"