Skip site navigation (1)Skip section navigation (2)
Date:      Sun, 13 Sep 2015 14:30:14 +0000
From:      Alexey Dokuchaev <danfe@FreeBSD.org>
To:        Baptiste Daroussin <bapt@FreeBSD.org>
Cc:        ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org
Subject:   Re: svn commit: r396826 - head/x11/metisse
Message-ID:  <20150913143014.GA59597@FreeBSD.org>
In-Reply-To: <201509131424.t8DEOxgd093250@repo.freebsd.org>
References:  <201509131424.t8DEOxgd093250@repo.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Sun, Sep 13, 2015 at 02:24:59PM +0000, Baptiste Daroussin wrote:
> New Revision: 396826
> URL: https://svnweb.freebsd.org/changeset/ports/396826
> 
> Log:
>   Clean options by using options helpers
>   Remove gtk1 option
>   Nuke usage of HAVE_GOME.
>   Please notice that the port is stil wrong as it still relies on the
>   configure script to autodetect things instead of control[l]ing it
>   depending on options.  At least it is now a bit cleaner[.]

(Fixed some typos for you)

Thanks, but I don't see the reason for this part:

> -post-patch: .SILENT
> +post-patch:
>  # Catch up with new <fribidi-types.h> header name
> -	${REINPLACE_CMD} -e 's,fribidi_types\.h,fribidi-types.h,' \
> +	@${REINPLACE_CMD} -e 's,fribidi_types\.h,fribidi-types.h,' \
>  		${WRKSRC}/fvwm-insitu/libs/FBidi.c
>  # Fix desktop entries path
> -	${REINPLACE_CMD} -E 's,^(configdir =).*,\1 ${DESKTOPDIR},' \
> +	@${REINPLACE_CMD} -E 's,^(configdir =).*,\1 ${DESKTOPDIR},' \
>  		${WRKSRC}/fvwm-insitu/opale/session/Makefile.in
> -.if defined(_NOGLADEDIR)
> -	${REINPLACE_CMD} -e 's, install-gladeDATA,,' \
> -		${WRKSRC}/fvwm-insitu/modules/FvwmConfig/Makefile.in
> -.endif
> -	${REINPLACE_CMD} -e 's,return;,return 0;,' \
> +	@${REINPLACE_CMD} -e 's,return;,return 0;,' \

What was wrong with .SILENT?  (I ask because it was originally my port.)

./danfe



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20150913143014.GA59597>