From owner-svn-ports-all@FreeBSD.ORG Thu Aug 21 21:25:40 2014 Return-Path: Delivered-To: svn-ports-all@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [8.8.178.115]) (using TLSv1 with cipher ADH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTPS id 28380C3C; Thu, 21 Aug 2014 21:25:40 +0000 (UTC) Received: from shepard.synsport.net (mail.synsport.com [208.69.230.148]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id D5128391C; Thu, 21 Aug 2014 21:25:39 +0000 (UTC) Received: from [192.168.0.22] (unknown [130.255.19.191]) (using TLSv1 with cipher DHE-RSA-AES128-SHA (128/128 bits)) (No client certificate requested) by shepard.synsport.net (Postfix) with ESMTP id 2DCB043BA8; Thu, 21 Aug 2014 16:25:22 -0500 (CDT) Message-ID: <53F663B2.3000800@marino.st> Date: Thu, 21 Aug 2014 23:25:06 +0200 From: John Marino Reply-To: marino@freebsd.org User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:24.0) Gecko/20100101 Thunderbird/24.5.0 MIME-Version: 1.0 To: Hiroki Sato , ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org Subject: Re: svn commit: r365590 - in head/cad/spice: . files References: <201408211941.s7LJf6de048334@svn.freebsd.org> In-Reply-To: <201408211941.s7LJf6de048334@svn.freebsd.org> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-BeenThere: svn-ports-all@freebsd.org X-Mailman-Version: 2.1.18-1 Precedence: list List-Id: SVN commit messages for the ports tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 21 Aug 2014 21:25:40 -0000 On 8/21/2014 21:41, Hiroki Sato wrote: > Author: hrs > Date: Thu Aug 21 19:41:06 2014 > New Revision: 365590 > URL: http://svnweb.freebsd.org/changeset/ports/365590 > QAT: https://qat.redports.org/buildarchive/r365590/ > > Log: > - Cleanup ${FILESDIR}/Makefile. > - Remove MAKE_JOBS_UNSAFE. > > Modified: > head/cad/spice/Makefile > head/cad/spice/files/Makefile > > Modified: head/cad/spice/Makefile > ============================================================================== > --- head/cad/spice/Makefile Thu Aug 21 19:36:06 2014 (r365589) > +++ head/cad/spice/Makefile Thu Aug 21 19:41:06 2014 (r365590) > @@ -20,7 +20,8 @@ LICENSE_FILE= ${WRKSRC}/COPYRIGHT > > WRKSRC= ${WRKDIR}/${PORTNAME}3f5 > MAKEFILE= ${FILESDIR}/Makefile > -MAKE_JOBS_UNSAFE= yes > +MAKE_ENV= WITH_INSTALL_AS_USER=yes > +ALL_TARGET= build _manpages > > PORTDOCS= spice3f3.ps.gz > PORTEXAMPLES= bjtnoise.cir bsim1tst.cir bsim2tst.cir diffpair.cir \ > > Modified: head/cad/spice/files/Makefile > ============================================================================== > --- head/cad/spice/files/Makefile Thu Aug 21 19:36:06 2014 (r365589) > +++ head/cad/spice/files/Makefile Thu Aug 21 19:41:06 2014 (r365590) > @@ -1,31 +1,34 @@ > # > # $FreeBSD$ > # > -all: > +build: > ${SH} util/build FreeBSD default > + strip ${PROGRAMS:S,^,FreeBSD/obj/bin/,} > > -install: > - mkdir -p ${DESTDIR}${PREFIX}/share/spice/helpdir \ > - ${DESTDIR}${PREFIX}/share/spice/scripts > - cd FreeBSD/obj/bin && \ > - ${BSD_INSTALL_PROGRAM} \ > - spice3 nutmeg sconvert help proc2mod multidec \ > - ${DESTDIR}${PREFIX}/bin > - ${BSD_INSTALL_DATA} lib/helpdir/spice.txt \ > - ${DESTDIR}${PREFIX}/share/spice/helpdir > - ${BSD_INSTALL_SCRIPT} lib/scripts/setplot lib/scripts/spinit \ > - ${DESTDIR}${PREFIX}/share/spice/scripts > - ${BSD_INSTALL_DATA} lib/mfbcap lib/news \ > - ${DESTDIR}${PREFIX}/share/spice > - ${BSD_INSTALL_MAN} man/man5/mfbcap.5 \ > - ${DESTDIR}${PREFIX}/man/man5/mfbcap.5 > - ${BSD_INSTALL_MAN} man/man1/sconvert.1 \ > - ${DESTDIR}${PREFIX}/man/man1/sconvert.1 > - ${BSD_INSTALL_MAN} man/man1/nutmeg.1 \ > - ${DESTDIR}${PREFIX}/man/man1/nutmeg.1 > - ${BSD_INSTALL_MAN} man/man1/spice.1 \ > - ${DESTDIR}${PREFIX}/man/man1/spice.1 > - ${BSD_INSTALL_MAN} man/man1/spice.1 \ > - ${DESTDIR}${PREFIX}/man/man1/spice3.1 > - ${BSD_INSTALL_MAN} man/man3/mfb.3 \ > - ${DESTDIR}${PREFIX}/man/man3/mfb.3 > +PREFIX?= /usr/local > + > +BINDIR= ${PREFIX}/bin > +SHAREFILESDIR= ${PREFIX}/share/spice > +HELPFILESDIR= ${SHAREFILESDIR}/helpdir > +SCRIPTFILESDIR= ${SHAREFILESDIR}/scripts > + > +FILESGROUPS= PROGRAMS HELPFILES SCRIPTFILES SHAREFILES > + > +PROGRAMSMODE= ${BINMODE} > +PROGRAMSOWN= ${BINOWN} > +PROGRAMSGRP= ${BINGRP} > + > +PROGRAMS= spice3 nutmeg sconvert help proc2mod multidec > +HELPFILES= spice.txt > +SCRIPTFILES= setplot spinit > +SHAREFILES= mfbcap news > + > +MAN= spice.1 nutmeg.1 sconvert.1 mfb.3 mfbcap.5 > +MLINKS= spice.1 spice3.1 > +MANDIR= ${PREFIX}/man/man > + > +beforeinstall: > + mkdir -p ${DESTDIR}${HELPFILESDIR} ${DESTDIR}${SCRIPTFILESDIR} > + > +.PATH: FreeBSD/obj/bin lib/helpdir lib/scripts lib man/man1 man/man3 man/man5 > +.include I'm sorry, but using freebsd-specific in a ports vendor makefile is NOT an improvement and frankly puts the build at risk on DragonFly. I wish there was a rule that ports should not use system make fragments. This is not a good practice. This port had a perfectly working and generic makefile before. There's a good chance this just broke spice on DragonFly as the system make file these are different. John