Date: Sun, 22 Aug 2010 11:25:42 +0000 From: Alexey Dokuchaev <danfe@FreeBSD.org> To: Gerald Pfeifer <gerald@FreeBSD.org> Cc: cvs-ports@FreeBSD.org, cvs-all@FreeBSD.org, ports-committers@FreeBSD.org Subject: Re: cvs commit: ports/lang/gcc45 Makefile distinfo Message-ID: <20100822112542.GB58812@FreeBSD.org> In-Reply-To: <201008201617.o7KGHVAG037310@repoman.freebsd.org> References: <201008201617.o7KGHVAG037310@repoman.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On Fri, Aug 20, 2010 at 04:17:31PM +0000, Gerald Pfeifer wrote: > Replace NOT_FOR_ARCHS by ONLY_FOR_ARCHS. [1] > > Suggested by: linimon I don't really see the point. Ideally, any port (including gcc) should work across all architectures. If the port inherently cannot work on some arch (meaning it is not simply BROKEN there, but cannot be possibly fixed without e.g. total rewrite, or if it is binary-only, etc.), it can be marked IGNORE (old way) or (ONLY|NOT)_FOR_ARCHS (new way). Common sense would suggest that the one that requires shorter list of words should be picked; other thing to consider is will the list grow (in case of binary port, NOT_FOR_ARCHS is obviously not the best choice in this regard), or it would rather shrink (desired situation for open source software, since ideally it should work everywhere). Looking at committed change, it appears that list length remained the same (three words); alpha was dropped (which is correct since we do not support it these days). At this point NOT_FOR_ARCHS wins 2:3, but since nothing was said about about other Tier 2 arches (arm and sun4v), it is hard to tell if usage of ONLY_FOR_ARCHS is justified by n:m ratio. Now, since GCC is open source software, I believe it could/should be fixed to work on every Tier 1/2 arch; which means that ONLY_FOR_ARCHS will grow. In this light, the change is controversial at best. I'd like to hear Mark's rationale for [1]. ./danfe
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20100822112542.GB58812>