From owner-svn-ports-head@FreeBSD.ORG Tue Dec 4 11:46:03 2012 Return-Path: Delivered-To: svn-ports-head@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [69.147.83.52]) by hub.freebsd.org (Postfix) with ESMTP id 5F35359E; Tue, 4 Dec 2012 11:46:03 +0000 (UTC) (envelope-from bsdkaffee@gmail.com) Received: from mail-qc0-f182.google.com (mail-qc0-f182.google.com [209.85.216.182]) by mx1.freebsd.org (Postfix) with ESMTP id CC4588FC16; Tue, 4 Dec 2012 11:46:02 +0000 (UTC) Received: by mail-qc0-f182.google.com with SMTP id k19so2623805qcs.13 for ; Tue, 04 Dec 2012 03:46:02 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=sender:from:to:cc:subject:date:message-id:user-agent:in-reply-to :references:mime-version:content-transfer-encoding:content-type; bh=v2luzPfegkGG5mxfsqEtlYhnwZWoIjiW8Jb2NIlViW0=; b=flZ5Ap6ZpbgElCr5kjAqWgaEAFFiqQ2uiq8PqgmjKRDlB++TsUaf9cQiDIC14eszpX FjgfPRHuN3ovh00vGupMi/Y6qxudqGroJhmrEsTEiCRzXIf75s235sK8RPvY0etcO9iQ HqjxiVx3zFhtq6W516pRXiluYaG91Q/+um/S8/8vALCUl+qbC5I5Dr+LTRoJce9bzaOY pA0q7K0jEa0FTFKW6iakCbDhvcFrkhTtW0N0/xOUOZK4W3LMKycay6hJAqOXPLUsXvjn f6T/mZIJymZG+Iv4LG48bnYmF+5Ql/MOxhUFzoamN/92HmVbV6hDPRFAEo8x91wds4v3 MxnQ== Received: by 10.49.133.195 with SMTP id pe3mr24257800qeb.58.1354621562016; Tue, 04 Dec 2012 03:46:02 -0800 (PST) Received: from mocha.verizon.net (c-71-61-40-68.hsd1.oh.comcast.net. [71.61.40.68]) by mx.google.com with ESMTPS id z5sm451132qer.8.2012.12.04.03.45.59 (version=TLSv1/SSLv3 cipher=OTHER); Tue, 04 Dec 2012 03:46:00 -0800 (PST) Sender: "Jason E. Hale" From: "Jason E. Hale" To: Gabor Pali Subject: Re: svn commit: r308217 - in head/devel/icmake: . files Date: Tue, 04 Dec 2012 06:45:58 -0500 Message-ID: <1837268.Hp4WNQ0tny@mocha.verizon.net> User-Agent: KMail/4.8.4 (FreeBSD/9.0-RELEASE-p3; KDE/4.8.4; i386; ; ) In-Reply-To: <201212040913.qB49DUfk028183@svn.freebsd.org> References: <201212040913.qB49DUfk028183@svn.freebsd.org> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" Cc: svn-ports-head@freebsd.org, svn-ports-all@freebsd.org, ports-committers@freebsd.org X-BeenThere: svn-ports-head@freebsd.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: SVN commit messages for the ports tree for head List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 04 Dec 2012 11:46:03 -0000 On Tuesday, December 04, 2012 09:13:30 Gabor Pali wrote: > Author: pgj > Date: Tue Dec 4 09:13:30 2012 > New Revision: 308217 > URL: http://svnweb.freebsd.org/changeset/ports/308217 > > Log: > r307972@csupor: pgj | 2012-12-04 09:41:39 +0100 > - Update to 7.19.00 > - Trim Makefile header > - Set license > - Teach more about how to respect CC and CFLAGS > > Feature safe: yes > > Added: > head/devel/icmake/files/patch-exec-bootstrap (contents, props changed) > Modified: > head/devel/icmake/Makefile > head/devel/icmake/distinfo > head/devel/icmake/pkg-descr > head/devel/icmake/pkg-plist > Directory Properties: > head/ (props changed) This commit added a property change to head that I don't think should be there: svk:merge 7042ac7c-f72f-e211-9f8d-6cf0491430cb:/svk/freebsd-ports-local:307972 > > Modified: head/devel/icmake/Makefile > ============================================================================ > == --- head/devel/icmake/Makefile Tue Dec 4 09:10:47 2012 (r308216) > +++ head/devel/icmake/Makefile Tue Dec 4 09:13:30 2012 (r308217) > @@ -1,12 +1,7 @@ > -# New ports collection makefile for: icmake > -# Date created: March 4, 2009 > -# Whom: Gabor Pali > -# > # $FreeBSD$ > -# > > PORTNAME= icmake > -PORTVERSION= 7.12.5 > +PORTVERSION= 7.19.00 > CATEGORIES= devel > MASTER_SITES= SF/${PORTNAME}/${PORTNAME}/${PORTVERSION} > DISTNAME= ${PORTNAME}_${PORTVERSION} > @@ -15,12 +10,14 @@ EXTRACT_SUFX= .orig.tar.gz > MAINTAINER= pgj@FreeBSD.org > COMMENT= An Intelligent C-like Maker > COMMENT should drop leading indefinite article "An" > +LICENSE= GPLv3 > + > BUILD_DEPENDS= bash:${PORTSDIR}/shells/bash \ > gsed:${PORTSDIR}/textproc/gsed > > .if !defined(NO_INSTALL_MANPAGES) > MAN1= icmake.1 icmbuild.1 icmstart.1 > -MAN7= icmstart.rc.7 > +MAN7= icmstart.rc.7 icmconf.7 > .endif > > DOCSDIR= ${PREFIX}/share/doc/${PORTNAME}-${PORTVERSION} > @@ -30,53 +27,85 @@ PLIST_SUB+= PORTVERSION="${PORTVERSION}" > > WRKSRC= ${WRKDIR}/${PORTNAME}-${PORTVERSION} > > +MAKE_ENV= CC="${CC}" CFLAGS="${CFLAGS}" > + ^ These are already set in b.p.m. > post-patch: > - @${REINPLACE_CMD} -e 's|#!/bin/bash|#!${LOCALBASE}/bin/bash|' \ > + @${REINPLACE_CMD} -I "" -e 's|#!/bin/bash|#!${LOCALBASE}/bin/bash|' \ Wouldn't it be easier to just set REINPLACE_ARGS= -I "" ? > ${WRKSRC}/icm_bootstrap \ > ${WRKSRC}/icm_install \ > + ${WRKSRC}/icm_prepare \ > ${WRKSRC}/scripts/convert > - @${REINPLACE_CMD} -e 's|sed|gsed|g' \ > + @${REINPLACE_CMD} -I "" -e 's|sed|gsed|g' \ > ${WRKSRC}/scripts/conversions \ > ${WRKSRC}/scripts/convert > - @${REINPLACE_CMD} -e 's|"usr/bin"|"bin"| ; \ > + @${REINPLACE_CMD} -I "" -e 's|"usr/bin"|"bin"| ; \ > s|"usr/share/icmake"|"share/icmake-${PORTVERSION}"| ; \ > s|"usr/share/man"|"man"| ; \ > s|"usr/lib/icmake"|"lib/icmake-${PORTVERSION}"| ; \ > s|"usr/share/doc/icmake"|"share/doc/icmake-${PORTVERSION}"| ; \ > s|"usr/share/doc/icmake- doc"|"share/doc/icmake-${PORTVERSION}/doc"|' \ > ${WRKSRC}/INSTALL.im > - @${REINPLACE_CMD} -e '\ > + @${REINPLACE_CMD} -I "" -e '\ > s|"ar"|"${AR}"| ; \ > s|"rsv"|"${ARFLAGS}"| ; \ > s|"gcc"|"${CC}"| ; \ > - s|"-c -Wall -O2|"-c ${CFLAGS}| ; \ > s|"gzip"|"${GZIP_CMD}"| ; \ > s|"cp"|"${CP}"| ; \ > s|"mkdir"|"${MKDIR}"| ; \ > s|"rm"|"${RM}"| ; \ > s|"strip"|"${STRIP_CMD}"|' \ > ${WRKSRC}/def/programs > - @${REINPLACE_CMD} -e '\ > - s|gcc|${CC}| ; \ > - s|-O2 -g -Wall|${CFLAGS}| ; \ > - s|cp -r|cp -R|' \ > + @${REINPLACE_CMD} -I "" -e 's|cp -r|cp -R|' \ > ${WRKSRC}/icm_bootstrap \ > ${WRKSRC}/icm_install > + @${REINPLACE_CMD} -I "" -e > 's|#!.*/usr/bin/icmake|#!${LOCALBASE}/bin/icmake|' \ > + ${WRKSRC}/comp/build \ > + ${WRKSRC}/doc/icmake.1 \ > + ${WRKSRC}/doc/manpage/icmake.yo \ > + ${WRKSRC}/doc/manpage/build \ > + ${WRKSRC}/examples/am \ > + ${WRKSRC}/examples/defines \ > + ${WRKSRC}/examples/ds \ > + ${WRKSRC}/examples/ftpxfer \ > + ${WRKSRC}/examples/initialization \ > + ${WRKSRC}/examples/nesteddirectives \ > + ${WRKSRC}/examples/r \ > + ${WRKSRC}/exec/build \ > + ${WRKSRC}/make/build \ > + ${WRKSRC}/pp/build \ > + ${WRKSRC}/rss/build \ > + ${WRKSRC}/un/build > + @${REINPLACE_CMD} -I "" -e 's|"gcc"|"${CC}"|g; s|-Wall -O2|${CFLAGS}|g' \ > + ${WRKSRC}/comp/build \ > + ${WRKSRC}/def/programs \ > + ${WRKSRC}/doc/icmake.doc \ > + ${WRKSRC}/exec/icmconf \ > + ${WRKSRC}/usr/share/icmake/icmconf \ > + ${WRKSRC}/exec/build \ > + ${WRKSRC}/make/build \ > + ${WRKSRC}/pp/build \ > + ${WRKSRC}/rss/build \ > + ${WRKSRC}/un/build > + @${REINPLACE_CMD} -I "" -e '\ > + s|/usr/lib/icmake|/usr/local/lib/icmake-${PORTVERSION}|g; \ > + s|/usr/share/icmake|/usr/local/share/icmake-${PORTVERSION}|g; \ > + s|/etc/icmake/icmake.cf|/usr/local/etc/icmake/icmake.rc|g' \ > + ${WRKSRC}/etc/icmake/icmake.rc > > do-build: > - cd ${WRKSRC} && ./icm_bootstrap ${PREFIX} > + cd ${WRKSRC} && ${SETENV} ${MAKE_ENV} ./icm_bootstrap ${PREFIX} > > do-install: > - cd ${WRKSRC} && ./icm_install strip progs > - cd ${WRKSRC} && ./icm_install scripts > - cd ${WRKSRC} && ./icm_install skel > - cd ${WRKSRC} && ./icm_install etc > + cd ${WRKSRC} && ${SETENV} ${MAKE_ENV} ./icm_install strip progs > + cd ${WRKSRC} && ${SETENV} ${MAKE_ENV} ./icm_install scripts > + cd ${WRKSRC} && ${SETENV} ${MAKE_ENV} ./icm_install skel > + cd ${WRKSRC} && ${SETENV} ${MAKE_ENV} ./icm_install etc > .if !defined(NO_INSTALL_MANPAGES) > - cd ${WRKSRC} && ./icm_install man > + cd ${WRKSRC} && ${SETENV} ${MAKE_ENV} ./icm_install man > .endif > .if !defined(NOPORTDOCS) Should convert to ${PORT_OPTIONS:MDOCS} > - cd ${WRKSRC} && ./icm_install doc > - cd ${WRKSRC} && ./icm_install docdoc > + cd ${WRKSRC} && ${SETENV} ${MAKE_ENV} ./icm_install doc > + cd ${WRKSRC} && ${SETENV} ${MAKE_ENV} ./icm_install docdoc > .endif > > .include > > Modified: head/devel/icmake/distinfo > ============================================================================ > == --- head/devel/icmake/distinfo Tue Dec 4 09:10:47 2012 (r308216) > +++ head/devel/icmake/distinfo Tue Dec 4 09:13:30 2012 (r308217) > @@ -1,2 +1,2 @@ > -SHA256 (icmake_7.12.5.orig.tar.gz) = > e1ce6fac859e91e4ef227860462ff2171d302be9fce7eb9a9c91dcbdb6eb577f -SIZE > (icmake_7.12.5.orig.tar.gz) = 316629 > +SHA256 (icmake_7.19.00.orig.tar.gz) = > 84a463398775b7ef1b76a3e3f72c868f7335c1820b78c28b154fb3be6e8ca819 +SIZE > (icmake_7.19.00.orig.tar.gz) = 329711 > > Added: head/devel/icmake/files/patch-exec-bootstrap > ============================================================================ > == --- /dev/null 00:00:00 1970 (empty, because file is newly added) > +++ head/devel/icmake/files/patch-exec-bootstrap Tue Dec 4 09:13:30 > 2012 (r308217) @@ -0,0 +1,23 @@ > +--- exec/bootstrap.orig 2012-09-07 09:43:56.000000000 +0200 > ++++ exec/bootstrap 2012-12-01 23:45:56.000000000 +0100 > +@@ -1,6 +1,6 @@ > + echo Creating icm-exec > + > +-gcc -O2 -g -Wall -DHAVE_GLOB -c *.c > ++${CC} ${CFLAGS} -g -DHAVE_GLOB -c *.c > + > + mkdir -p tmp/bin > + > +@@ -8,10 +8,10 @@ > + for x in auks var virtual int list string stack opcodefun builtin > + do > + cd $x > +- gcc -O2 -g -Wall -DHAVE_GLOB -c *.c || exit 1 > ++ ${CC} ${CFLAGS} -g -DHAVE_GLOB -c *.c || exit 1 > + cd .. > + done > + > +-gcc -o tmp/bin/icm-exec$1 *.o */*.o ../rss/libicrss.a > ++${CC} -o tmp/bin/icm-exec$1 *.o */*.o ../rss/libicrss.a > + > + rm *.o */*.o > > Modified: head/devel/icmake/pkg-descr > ============================================================================ > == --- head/devel/icmake/pkg-descr Tue Dec 4 09:10:47 2012 (r308216) +++ > head/devel/icmake/pkg-descr Tue Dec 4 09:13:30 2012 (r308217) @@ -4,4 +4,4 > @@ automatic program maintenance and system > platforms. Later on, Icmake was eventually ported to Unix platforms > (SCO and Linux). By now Icmake also runs on a HP-Unix platform. > > -WWW: http://icmake.sourceforge.net/ > +WWW: http://icmake.sourceforge.net/ > Why the change to tab? Single space is the standard. > Modified: head/devel/icmake/pkg-plist > ============================================================================ > == --- head/devel/icmake/pkg-plist Tue Dec 4 09:10:47 2012 (r308216) +++ > head/devel/icmake/pkg-plist Tue Dec 4 09:13:30 2012 (r308217) @@ -1,21 > +1,15 @@ > @comment $FreeBSD$ ^ This isn't neccessary. Tools/scripts/psvn should be used to set the props correctly. > bin/icmake > -bin/icmun > bin/icmbuild > bin/icmstart > -etc/icmake/AUTHOR > -etc/icmake/VERSION > -etc/icmake/YEARS > +bin/icmun > +etc/icmake/icmake.rc > etc/icmake/icmstart.rc > lib/icmake-%%PORTVERSION%%/icm-comp > lib/icmake-%%PORTVERSION%%/icm-exec > lib/icmake-%%PORTVERSION%%/icm-pp > -share/icmake-%%PORTVERSION%%/parser/gramspec/grambuild > -share/icmake-%%PORTVERSION%%/parser/gramspec/header.gr0 > -share/icmake-%%PORTVERSION%%/parser/gramspec/startrule.gr0 > share/icmake-%%PORTVERSION%%/parser/grammar > share/icmake-%%PORTVERSION%%/scanner/lexer > -share/icmake-%%PORTVERSION%%/scanner/scanner.h > share/icmake-%%PORTVERSION%%/scanner/scanner.ih > share/icmake-%%PORTVERSION%%/CLASSES > share/icmake-%%PORTVERSION%%/icmconf > @@ -25,7 +19,6 @@ share/icmake-%%PORTVERSION%%/usage.cc > share/icmake-%%PORTVERSION%%/version.cc > @dirrm etc/icmake > @dirrm lib/icmake-%%PORTVERSION%% > -@dirrm share/icmake-%%PORTVERSION%%/parser/gramspec > @dirrm share/icmake-%%PORTVERSION%%/parser > @dirrm share/icmake-%%PORTVERSION%%/scanner > @dirrm share/icmake-%%PORTVERSION%% -- Jason E. Hale - jhale@ FreeBSD Ports Committer KDE/FreeBSD Team