Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 22 Jul 2016 14:27:50 -0600
From:      Russell Haley <russ.haley@gmail.com>
To:        Jan Beich <jbeich@vfemail.net>
Cc:        olli hauer <ohauer@gmx.de>, freebsd-ports@freebsd.org
Subject:   Re: Passing target to source Makefile
Message-ID:  <20160722202750.4894802.64998.8848@gmail.com>
In-Reply-To: <37n2-9hp4-wny@vfemail.net>
References:  <CABx9NuQWqC2%2Bx82rT2zUa=qKiaC-EjUtfY7tess5NVpn8n-cgg@mail.gmail.com> <3b9c2f92-ef57-3e06-c110-8148dce16ba5@gmx.de> <CABx9NuQyh8iirhiPQHqZYAmTU5E%2BdEp6_kN2U0WKOU9F_Tuuqw@mail.gmail.com> <37n2-9hp4-wny@vfemail.net>

next in thread | previous in thread | raw e-mail | index | archive | help
Sorry for the top post. Thank you for taking the time to clarify.=A0

Russ

Sent=A0from=A0my=A0BlackBerry=A010=A0smartphone=A0on=A0the=A0Koodo=A0networ=
k.
=A0 Original Message =A0
From: Jan Beich
Sent: Friday, July 22, 2016 2:54 AM
To: Russell Haley
Cc: olli hauer; freebsd-ports@freebsd.org
Subject: Re: Passing target to source Makefile

Russell Haley <russ.haley@gmail.com> writes:

> 1) It does not use the recommended "freebsd" target. While the port
> overlays many of the same options, the system doesn't seem to respect
> the targets and also performs the installation without specifically
> calling the "install" target.

Try adding ALL_TARGET=3Dfreebsd. INSTALL_TARGET already defaults to
"install" and then the port appends extra steps via post-install.

$ make -V ALL_TARGET -C lang/lua53
all

> The current supported version of Lua is 5.3. While I can see the need
> to install past versions using explicit naming, it is my opinion that
> the current version should be installed as Lua, not luaXX.

Say, a port looks for Lua 5.1 headers. If Lua 5.3 is installed without
suffix the port may find wrong headers first. And fixing include order
isn't always trivial with complex build systems. But -isystem as used by
USES=3Dlocalbase partially alleviates that.

> 4) The current Makefile applies -fPIC to some of the platforms, but
> not armv6.

https://svnweb.freebsd.org/changeset/ports/409263

> - Update to the latest Lua revision

Do you mean Mk/bsd.default-versions.mk? lang/lua53 is already at 5.3.3.

> - Use clang and bmake instead of forcing gcc/gmake.

bmake isn't available on FreeBSD 9.3-RELEASE which isn't going to be
EOL'd until the next year. If you mean devel/bmake then I don't see the
benefit compared to devel/gmake - both are extra dependencies.

Also see https://svnweb.freebsd.org/changeset/ports/285048

> Please also note that I have made an upstream request to the Lua
> community to consider applying a patch to the freebsd target in their
> source/Makefile. The patch can be found here:
> http://pastebin.com/cUCtWb1w. I don't expect this to happen quickly
> however.

"freebsd" target is only slightly different from "linux" one. There's no
support to build shared library upstream, so -fPIC isn't required unless
you need to link liblua.a into another shared library.



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