From owner-svn-ports-all@FreeBSD.ORG Mon Feb 9 23:14:13 2015 Return-Path: Delivered-To: svn-ports-all@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [8.8.178.115]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTPS id AC741442; Mon, 9 Feb 2015 23:14:13 +0000 (UTC) Received: from mail-yh0-x232.google.com (mail-yh0-x232.google.com [IPv6:2607:f8b0:4002:c01::232]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (Client CN "smtp.gmail.com", Issuer "Google Internet Authority G2" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 679E6EEE; Mon, 9 Feb 2015 23:14:13 +0000 (UTC) Received: by mail-yh0-f50.google.com with SMTP id a41so9083311yho.9; Mon, 09 Feb 2015 15:14:12 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:sender:in-reply-to:references:date:message-id:subject :from:to:cc:content-type; bh=mNTkdVrsBKQugHnaB7iodnEu12e7Y0j4ZR0GhAmNp3A=; b=LP5NGfndUdvmm0aRSTpTPJhlOM/IfFSP5fYZmPwZgmMZsJ8ZoKnptFI4ijy7L5KWYm aYXxiFzwoMOlHokQNUWIxjc+Vj6y0OOHGu3f210+BJSppU3Jl2edcfuaPZdyVfcsQmvv NAhEJboRYhGgYMWGoY1+cWo/a1pc9frBcuEEGwrSxIbQN1bI+rxxYyfdU7QK+muXi6RE L6nuWYSsNhPGveQVj+g6ZB7GSOPXQ+VDf4KtGh8v2CdiFRPXNCaH6/+8u9kaJ40BymHN kWUtVf6bFmjJJFW9vN/ExXT1DWEwgP2ARx7enybZcVDD7WF18pjMW+uHvLN9KURUMSJb 3pMQ== MIME-Version: 1.0 X-Received: by 10.236.231.197 with SMTP id l65mr7363821yhq.33.1423523652458; Mon, 09 Feb 2015 15:14:12 -0800 (PST) Sender: antoine.brodin.freebsd@gmail.com Received: by 10.170.142.198 with HTTP; Mon, 9 Feb 2015 15:14:12 -0800 (PST) In-Reply-To: <201502090850.t198o3sw010099@svn.freebsd.org> References: <201502090850.t198o3sw010099@svn.freebsd.org> Date: Tue, 10 Feb 2015 00:14:12 +0100 X-Google-Sender-Auth: 4bhZDs6_ONOXWYqtZHLKrPQZPdo Message-ID: Subject: Re: svn commit: r378718 - in head/x11-toolkits/fltk: . files From: Antoine Brodin To: Alexey Dokuchaev Content-Type: text/plain; charset=UTF-8 Cc: "svn-ports-head@freebsd.org" , "svn-ports-all@freebsd.org" , "ports-committers@freebsd.org" 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: Mon, 09 Feb 2015 23:14:13 -0000 On Mon, Feb 9, 2015 at 9:50 AM, Alexey Dokuchaev wrote: > Author: danfe > Date: Mon Feb 9 08:50:02 2015 > New Revision: 378718 > URL: https://svnweb.freebsd.org/changeset/ports/378718 > QAT: https://qat.redports.org/buildarchive/r378718/ > > Log: > - Fix the breakage caused by previous commit (r378707): restore sane order of > knobs; resurrect documentation/Makefile patch; bring back recalculation of > ${ETCDIR} length in Fl_Preferences.cxx, trim CONFIGURE_ARGS (again), etc. > - Restore correct LDFLAGS+= -L../src (previously -L${WRKSRC}/src), otherwise > the build fails with e.g. previous version of FLTK libraries installed s/fix/totally break all dependent ports/ ... /usr/local/bin/fltk-config:LDFLAGS=" -L../src -L/usr/local/lib -fstack-protector -L/usr/local/lib" /usr/local/share/examples/fltk/menubar.cxx:#include "../src/flstring.h" /usr/local/include/FL/mac.H:# include "../src/Fl_Font.H" Please test your fixes. Cheers, Antoine > - Take maintainership to prevent Kato's monkeys jeopardizing it all over again > - Install code examples as PORTEXAMPLES (we do not install any documentation > because it requires LaTeX stack to build) > - Since `post-patch' target is a bit too verbose now, make it .SILENT > - Do not compile things with -Os by default (already honors user's CXXFLAGS) > - Do not blindly strip libraries and fluid(1) binary (respect ${STRIP} value) > > Added: > head/x11-toolkits/fltk/files/patch-documentation_Makefile > - copied, changed from r378706, head/x11-toolkits/fltk/files/patch-documentation_Makefile > Modified: > head/x11-toolkits/fltk/Makefile > head/x11-toolkits/fltk/files/patch-configure > > Modified: head/x11-toolkits/fltk/Makefile > ============================================================================== > --- head/x11-toolkits/fltk/Makefile Mon Feb 9 08:23:50 2015 (r378717) > +++ head/x11-toolkits/fltk/Makefile Mon Feb 9 08:50:02 2015 (r378718) > @@ -3,50 +3,43 @@ > > PORTNAME= fltk > PORTVERSION= 1.3.3 > -DISTVERSIONSUFFIX= -source > +PORTREVISION= 1 > CATEGORIES= x11-toolkits > -MASTER_SITES= http://fltk.org/pub/%SUBDIR%/ > -MASTER_SITE_SUBDIR= ${PORTNAME}/${PORTVERSION} > +MASTER_SITES= http://fltk.org/pub/${PORTNAME}/${PORTVERSION}/ > +DISTNAME= ${PORTNAME}-${PORTVERSION}-source > > -MAINTAINER= ports@FreeBSD.org > +MAINTAINER= danfe@FreeBSD.org > COMMENT= Cross-platform C++ graphical user interface toolkit > > LICENSE= LGPL20 > > -LIB_DEPENDS= libjpeg.so:${PORTSDIR}/graphics/jpeg \ > - libpng.so:${PORTSDIR}/graphics/png > - > -WRKSRC= ${WRKDIR}/${PORTNAME}-${PORTVERSION} > +LIB_DEPENDS= libpng.so:${PORTSDIR}/graphics/png \ > + libjpeg.so:${PORTSDIR}/graphics/jpeg > > USES= pkgconfig > USE_XORG= xft > USE_GL= glu > -GNU_CONFIGURE= yes > -CONFIGURE_ARGS= --enable-threads \ > - --enable-shared \ > - --enable-xft > USE_LDCONFIG= yes > +GNU_CONFIGURE= yes > +CONFIGURE_ARGS= --enable-shared > + > +WRKSRC= ${WRKDIR}/${PORTNAME}-${PORTVERSION} > > CPPFLAGS+= -I${LOCALBASE}/include > -LDFLAGS+= -L${LOCALBASE}/lib > +# -L../src is needed to be able to build new version with older one installed > +LDFLAGS+= -L../src -L${LOCALBASE}/lib > > -PORTDOCS= * > +PORTEXAMPLES= * > > -OPTIONS_DEFINE= DOCS > +OPTIONS_DEFINE= EXAMPLES > > -post-patch: > - @${REINPLACE_CMD} -e \ > - '/.SILENT/s|^|#| ; \ > - /echo /s|^|#|' ${WRKSRC}/makeinclude.in > - @${REINPLACE_CMD} -e \ > - '/cat[13]/s|^|#|' ${WRKSRC}/documentation/Makefile > - @${REINPLACE_CMD} -e \ > - 's|"/etc/fltk/"|"${ETCDIR}/"|' ${WRKSRC}/src/Fl_Preferences.cxx > - > -post-install: > - @(cd ${WRKSRC}/test && ${SETENV} ${MAKE_ENV} ${MAKE_CMD} \ > - ${MAKE_FLAGS} ${MAKEFILE} ${MAKE_ARGS} install) > - @${STRIP_CMD} ${STAGEDIR}${PREFIX}/bin/fluid > - @${STRIP_CMD} ${STAGEDIR}${PREFIX}/lib/libfltk*.so.[0-9]* > +post-patch: .SILENT > + prefix="${PREFIX}" ; plength=$$(($${#prefix}+10)) ; \ > + ${REINPLACE_CMD} -e "\|/etc/fltk|{s||${PREFIX}&|;s|10|$${plength}|;}" \ > + ${WRKSRC}/src/Fl_Preferences.cxx > + ${REINPLACE_CMD} -e 's,$$(docdir)/examples,${EXAMPLESDIR},' \ > + ${WRKSRC}/test/Makefile > + ${REINPLACE_CMD} -E '/^INSTALL_(BIN|LIB)/s,-m,${STRIP} &, ; \ > + s,^\.SILENT,#&,' ${WRKSRC}/makeinclude.in > > .include > > Modified: head/x11-toolkits/fltk/files/patch-configure > ============================================================================== > --- head/x11-toolkits/fltk/files/patch-configure Mon Feb 9 08:23:50 2015 (r378717) > +++ head/x11-toolkits/fltk/files/patch-configure Mon Feb 9 08:50:02 2015 (r378718) > @@ -9,3 +9,21 @@ > DSOFLAGS="$X_LIBS $DSOFLAGS" > $as_echo "#define USE_X11 1" >>confdefs.h > > +@@ -8137,17 +8137,10 @@ > + # Don't use -fforce-mem, -fforce-addr, or -fcaller-saves. > + # They all seem to make either no difference or enlarge > + # the code by a few hundred bytes. > +- # > +- # "-Os" seems to be the best compromise between speed and > +- # code size. "-O3" and higher seem to make no effective > +- # difference in the speed of the code, but does bloat the > +- # library 10+%. > + # > + > + if test "x$with_optim" != x; then > + OPTIM="$with_optim $OPTIM" > +- else > +- OPTIM="-Os $OPTIM" > + fi > + fi > + > > Copied and modified: head/x11-toolkits/fltk/files/patch-documentation_Makefile (from r378706, head/x11-toolkits/fltk/files/patch-documentation_Makefile) > ============================================================================== > --- head/x11-toolkits/fltk/files/patch-documentation_Makefile Mon Feb 9 02:26:32 2015 (r378706, copy source) > +++ head/x11-toolkits/fltk/files/patch-documentation_Makefile Mon Feb 9 08:50:02 2015 (r378718) > @@ -1,5 +1,14 @@ > --- documentation/Makefile.orig 2011-08-27 01:33:42.000000000 +0000 > +++ documentation/Makefile 2011-08-27 01:39:59.000000000 +0000 > +@@ -73,7 +73,7 @@ > + > + install: $(MANPAGES) > + echo "Installing documentation files in $(DESTDIR)$(docdir) ..." > +- -$(INSTALL_DIR) $(DESTDIR)$(docdir) > ++# -$(INSTALL_DIR) $(DESTDIR)$(docdir) > + if test -f html/index.html ; then \ > + for file in html/* ; do \ > + $(INSTALL_DATA) $$file $(DESTDIR)$(docdir); \ > @@ -84,11 +84,6 @@ > $(INSTALL_DATA) fltk.pdf $(DESTDIR)$(docdir); \ > fi >