Subject: Re: CVS commit: pkgsrc/graphics/imlib2
To: Thomas Klausner <wiz@netbsd.org>
From: None <jmmv@home.network>
List: pkgsrc-changes
Date: 06/10/2005 22:18:21
On Fri, Jun 10, 2005 at 10:02:53PM +0200, Thomas Klausner wrote:
> On Fri, Jun 10, 2005 at 09:37:37PM +0200, jmmv@home.network 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.)
> 
> I prefer this method, because otherwise PKG_SUPPORTED_OPTIONS is
> empty and the options framework doesn't like this.

Then the options framework ought to be fixed ;)

-- 
Julio M. Merino Vidal <jmmv84@gmail.com>
http://www.livejournal.com/users/jmmv/
The NetBSD Project - http://www.NetBSD.org/