Date: Fri, 8 Apr 2011 03:54:30 +0000 From: Alexey Dokuchaev <danfe@FreeBSD.org> To: bf1783@gmail.com Cc: Rene Ladan <rene@freebsd.org>, cvs-ports@freebsd.org, cvs-all@freebsd.org, ports-committers@freebsd.org Subject: Re: cvs commit: ports/www/chromium Makefile Message-ID: <20110408035430.GA56549@FreeBSD.org> In-Reply-To: <BANLkTik4HrVvVNU_nBtH8Oyt=81uKjn6og@mail.gmail.com> References: <201104072239.p37Md9oh031253@repoman.freebsd.org> <BANLkTikmUZsHNjw2Jpe8Y7v5CD6WkZGtoQ@mail.gmail.com> <20110408022459.GA40411@FreeBSD.org> <20110408023021.GA47293@FreeBSD.org> <BANLkTik4HrVvVNU_nBtH8Oyt=81uKjn6og@mail.gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
On Fri, Apr 08, 2011 at 03:47:07AM +0000, b. f. wrote: > On 4/8/11, Alexey Dokuchaev <danfe@freebsd.org> wrote: > > On Fri, Apr 08, 2011 at 02:24:59AM +0000, Alexey Dokuchaev wrote: > >> I've been thinking about this as well. However, instead of bringing back > >> this OPTION, I'd suggest adding "|| defined(PACKAGE_BUILDING)" to that > >> .if to ensure that packages are can be safely run on pre-SSE2 CPUs. > > > > Diff attached, seems to DTRT. > > Someone may want to build SSE2-enabled packages for i386 in a > tinderbox or on a package-building machine, and the option allows them > to do this, but this change does not. Also, with this change, you're > telling the configure script to disable SSE2 when PACKAGE_BUILDING is > defined even on amd64, where SSE2 is always available. So I don't > think it DTRT. One can always pass desired CPUTYPE is they want to build their package with specific target. As for amd64, here I agree, checking for ARCH makes sence. I don't mind this heuristics, as long as there is no SSE option. OPTIONS like this are bad since they are cached and can stay unnoticed for long periods of time. On the other hand, Chromium is web browser, which it most likely is going to be used on users' desktop, where SSE2 is probably feels like home for a while already. :-) ./danfe
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20110408035430.GA56549>