cvs commit: ports/www/chromium Makefile

[b. f.]

On 4/8/11, Alexey Dokuchaev <danfe at freebsd.org> wrote:
> On Fri, Apr 08, 2011 at 12:24:57PM +0000, b. f. wrote:
>> I guess that we should just trust users to configure them properly.
>
> Yes, I also believe that if user wants to build a package for machine other
> than theirs, they are expected to know how these things work and what to do
> to get the work done.  From our side, we should provide easy and straight-
> forward means of such customizations.
>
> Summarizing our discussion, what do you think of the following?
>
> .if ! ${MACHINE_CPU:Msse2} || defined(PACKAGE_BUILDING) && ${ARCH} !=
> "amd64"
> GYP_DEFINES+=	disable_sse2=1
> .endif
>
> Default package would have reasonable defaults now, custom port build can
> be controlled by passing desired MACHINE_CPU.  Building SSE2-enabled
> package for i386 is a bit tricky, but cannot be possibly polluted by
> stale cached OPTIONS value.

Sorry, I tried to convey in my previous message that I was mistaken in
thinking that the earlier changes would cause problems for the default
i386 packages.  Packages for the machine hosting the build, and those
built for generic machines via the portbuild scripts, have the right
settings now, so I would leave the port as it is.  As you wrote, those
who are building packages for machines other than the host should
ensure that CPUTYPE has the desired value (they should also ensure
that all OPTIONS and other relevant variables in included Makefiles
have the desired values, so the argument about build pollution for
only this knob is a bit overstated), so I think that adding the change
above just creates an unnecessary obstacle to building SSE2-enable
packages on i386 when such a build is desired.

b.