Subject: Re: CVS commit: pkgsrc/graphics/imlib2
To: None <jmmv@NetBSD.org>
From: Johnny Lam <jlam@NetBSD.org>
List: pkgsrc-changes
Date: 06/10/2005 16:06:45
jmmv@home.network wrote:
> On Fri, Jun 10, 2005 at 06:21:45PM +0000, Thomas Klausner wrote:
> 
>>Module Name:	pkgsrc
>>Committed By:	wiz
>>Date:		Fri Jun 10 18:21:45 UTC 2005
>>
>>Modified Files:
>>	pkgsrc/graphics/imlib2: options.mk
>>
>>Log Message:
>>Slightly different solution to previous:
>>just ignore mmx pkg_option if set on non-i386.
> 
> 
> I think that the best way to achieve this is to only add mmx to
> PKG_SUPPORTED_OPTIONS when on i386.  This way you can handle the
> PKG_OPTIONS:Mmmx unconditionally.  (FWIW, this is what I did in
> mplayer during the switch and proved to be very useful.)

FWIW, I agree with Julio as PKG_SUPPORTED_OPTIONS is naturally expected 
to contain options that are truly supported.  I think it's okay for the 
same package on different platforms to support different options.

	Cheers,

	-- Johnny Lam <jlam@NetBSD.org>