On 2/20/2014 4:29 PM, Juan Francisco Cantero Hurtado wrote:
Brian asked me to compile and to test the last version of seed7 on hppa.
The result is a little weird, so let me know if you have a better fix or
just give me the OKs.


So I'm assuming it builds correctly with this fix? But does it work?

Apparently gmake doesn't honor ALL_TARGET on hppa. I tried also running
directly "gmake -f makefile depend s7 s7c" within WRKSRC or removing
ALL_TARGET from the port makefile and adding "all: depend s7 s7c" to
WRKSRC/makefile". Nothing worked. gmake doesn't run "depend".


Blah. That is pretty crappy. I really hope there's a better fix.

~Brian


Index: Makefile
===================================================================
RCS file: /cvs/ports/lang/seed7/Makefile,v
retrieving revision 1.14
diff -u -p -r1.14 Makefile
--- Makefile    19 Jan 2014 20:21:01 -0000      1.14
+++ Makefile    20 Feb 2014 21:08:32 -0000
@@ -1,8 +1,7 @@
  # $OpenBSD: Makefile,v 1.14 2014/01/19 20:21:01 bcallah Exp $

-BROKEN-hppa =  SIGILL compiling prg/s7c
-
  V =           20140119
+REVISION =     0
  COMMENT =     high-level, extensible programming language
  DISTNAME =    seed7_05_${V}
  PKGNAME =     seed7-${V}
@@ -26,7 +25,8 @@ MAKE_FLAGS =  CC="${CC}" LDFLAGS="-Wl,--g
  MAKE_ENV +=   S7_LIB_DIR=${TRUEPREFIX}/lib/seed7/bin \
                SEED7_LIBRARY=${TRUEPREFIX}/lib/seed7/lib
  MAKE_FILE =   makefile
-ALL_TARGET =   depend s7 s7c
+# Surprisingly, ALL_TARGET doesn't work on HPPA.
+#ALL_TARGET =  depend s7 s7c

  CFLAGS +=     -I${X11BASE}/include

@@ -36,6 +36,12 @@ WRKSRC =     ${WRKDIST}/src
  post-patch:
        perl -pi -e "s,-O2,${CFLAGS},g" ${WRKSRC}/makefile
        perl -pi -e "s,/usr,${TRUEPREFIX},g" ${WRKDIST}/doc/s7{,c}.1
+
+do-build:
+       cd ${WRKSRC} && \
+       ${SETENV} ${MAKE_ENV} ${MAKE_PROGRAM} ${MAKE_FLAGS} -f ${MAKE_FILE} depend 
&& \
+       ${SETENV} ${MAKE_ENV} ${MAKE_PROGRAM} ${MAKE_FLAGS} -f ${MAKE_FILE} s7 
&& \
+       ${SETENV} ${MAKE_ENV} ${MAKE_PROGRAM} ${MAKE_FLAGS} -f ${MAKE_FILE} s7c

  do-install:
        ${INSTALL_PROGRAM} ${WRKDIST}/bin/s7{,c} ${PREFIX}/bin


Reply via email to