Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 2 Nov 2018 10:07:44 +0100
From:      Mathieu Arnold <mat@FreeBSD.org>
To:        Mark Linimon <linimon@FreeBSD.org>
Cc:        ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org
Subject:   Re: svn commit: r483758 - head/devel/libpru
Message-ID:  <20181102090744.4f7mkmgjuxvrd776@atuin.in.mat.cc>
In-Reply-To: <201811020135.wA21ZqbL037253@repo.freebsd.org>
References:  <201811020135.wA21ZqbL037253@repo.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help

--ykvbjypibdhyxvj7
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Fri, Nov 02, 2018 at 01:35:52AM +0000, Mark Linimon wrote:
> Author: linimon
> Date: Fri Nov  2 01:35:51 2018
> New Revision: 483758
> URL: https://svnweb.freebsd.org/changeset/ports/483758
>=20
> Log:
>   Fix build with GCC-based architectures.
>  =20
>   PR:		232851
>   Submitted by:	Piotr Kubaj
>=20
> Modified:
>   head/devel/libpru/Makefile
>=20
> Modified: head/devel/libpru/Makefile
> =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D
> --- head/devel/libpru/Makefile	Fri Nov  2 01:34:15 2018	(r483757)
> +++ head/devel/libpru/Makefile	Fri Nov  2 01:35:51 2018	(r483758)
> @@ -11,13 +11,15 @@ COMMENT=3D	Library to interface with PRUs
>  LICENSE=3D	BSD2CLAUSE
> =20
>  IGNORE_DragonFly=3D	only supported on FreeBSD
> -BROKEN_mips=3D	Does not build: unrecognized command line option -Weveryt=
hing
> -BROKEN_mips64=3D	Does not build: unrecognized command line option -Wever=
ything
> -BROKEN_powerpc64=3D	Does not build: unrecognized command line option -We=
verything
> -BROKEN_sparc64=3D	Does not build: unrecognized command line option -Weve=
rything
> -
>  USES=3D		cmake
> =20
>  WRKSRC=3D		${WRKDIR}/rpaulo-libpru-5a74157b82b8
> =20
> -.include <bsd.port.mk>
> +.include <bsd.port.pre.mk>
> +
> +post-patch:
> +.if ${ARCH:Mmips*} || ${ARCH:Mpowerpc*} || ${ARCH} =3D=3D sparc64
> +	${REINPLACE_CMD} -e 's/ -Weverything//'  ${WRKSRC}/CMakeLists.txt
> +.endif
> +
> +.include <bsd.port.post.mk>

Could you try and put the whole target definition inside the if so that
it does not needlessly create an empty target, like this:


=2Eif ${ARCH:Mmips*} || ${ARCH:Mpowerpc*} || ${ARCH} =3D=3D sparc64
post-patch:
	${REINPLACE_CMD} -e 's/ -Weverything//'  ${WRKSRC}/CMakeLists.txt
=2Eendif

--=20
Mathieu Arnold

--ykvbjypibdhyxvj7
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iQKTBAABCgB9FiEEOraXidLtEhBkQLpbOkUW81GDzkgFAlvcE99fFIAAAAAALgAo
aXNzdWVyLWZwckBub3RhdGlvbnMub3BlbnBncC5maWZ0aGhvcnNlbWFuLm5ldDNB
QjY5Nzg5RDJFRDEyMTA2NDQwQkE1QjNBNDUxNkYzNTE4M0NFNDgACgkQOkUW81GD
zkh7Ag/8CbytdirI7bOU82PCSa5SiOcOUHcSLmpXA+R0eKACMd9rPcttEF3g8nLJ
eDQOCjYbodQkzTcJzqbxYxESNf5dhI1EeORv36VbErj6qGVqwCj78x8K5PRStNpA
KFGK98JqXE20n9A3owrlF/4z4FaysJqFrUtOsKJkBJkhexWuVNr4HsqwZWI5G/Gb
STabqWtmK6hF8OaHNPyLCzr/8FXcd1JsiWPXTVv/CaaPGKWDIIABnXiY0iibAAXj
pmUP7MwxvPgat+rlc4EHTGl47EXCyfnrwYlesyyMBbMRZTKieWQhbG7qcN2wkV1Q
jCDe9mQ+jFFP/tmVu7wBln7AsFFI55rNwPFC/zwkPu5snMsjPxktYeOMtkrVW/AH
gaQ6OKmx1quM/Fo7fF8KT3YftdkUBeDIvQEMW0K3Ok5Kn+xwMISrTJPNr4IAdRHX
00L9iKZE6JwLXVXg97ZYe0ocUREpFFTKC1A4fxE7o+1YWp+DUXqkswrM9wFTGM2o
67jI0VgUz9zJ2soHC/wpxhtyZv/rAMYhv2Xo7FK7uK5qnOHyAmjIZvtUfwQmtGpo
wE5hxLFM5HcLvq100FWZZ3/UWhcBdtQSZ/mHQFUyKqnhuy0ywBb1trhB9H2yU89E
Tqh2092SfNlBp3eJSwXY+VGdRFzH3DFSB+wqbJ4MvL2z1wKnRfI=
=aubn
-----END PGP SIGNATURE-----

--ykvbjypibdhyxvj7--



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