Bugzilla Bug 233318

XaoS-3.4 uses dev-lang/nasm for x86 and sci-libs/gsl for other platforms. If
--with-gsl is not given in amd64 and nasm is installed, it'll generate i386 asm
which will cause a build error.

So, --with-gsl --without-i386asm should be used for all platforms except x86.

On a separate note, the ebuild currently depends unconditionally on
x11-proto/xf86dgaproto. As far as I see, the only thing that depends on this is
the DGA driver which is not built by default. So it'd be nice if the dep were
controlled via an use flag or the dga driver were disabled/enabled in the
ebuild.

Also, $(use_with X) does nothing, it should be $(use_with X x).

Created an attachment (id=161660) [details]
xaos-3.4.ebuild.patch

Proposed patch for the ebuild.

Thank you very much for pointing this out. Wouldn't it be much
cleaner to just depend on gsl for all arches and disable the generation
of asm globally? I am not a big fan of these ARCH? type dependencies
unless absolutely necessary. Xaos doesn't seem the kind of application
that would benefit a lot from custom asm, but since I don't know xaos 
very well I may be totally wrong. What do you think?

Thanks,
Markus

I don't know the package either... so I have no idea if it's worth using the
x86 asm. And I'm on amd64, so I'll use gsl anyway ;)

(In reply to comment #3)
> I don't know the package either... so I have no idea if it's worth using the
> x86 asm. And I'm on amd64, so I'll use gsl anyway ;)
> 

Ok, I'll go with gsl only then. If somebody really
feels asm is necessary on x86 we can always consider adding it 
lateron.

Thanks,
Markus 

I just fixed this in cvs by adding sci-libs/gsl as mandatory
dependency and forcing its use.

Best,
Markus