svn commit: r316102 - in head/emulators/dolphin-emu-devel: . files

[Ganael LAPLANCHE]

On Fri, 19 Apr 2013 13:49:35 +0000, Alexey Dokuchaev wrote

Hi Alexey,

> I would say this is rather bogus BROKEN message: it's totally 
> unclear what exactly "i386-specific option" is: assembler 
> instruction, compiler option, or something else.

This is an old one that has been re-ordered, see :

http://www.freebsd.org/cgi/cvsweb.cgi/ports/emulators/dolphin-emu-devel/Makefile.diff?r1=1.2;r2=1.3

and to be honest, I am not sure whether Dolphin is still BROKEN or not
on PowerPC, as it should now be buildable on (e.g.) Android devices too.

> This is not only bad because perhaps NOT_FOR_ARCHS might suit 
> better in this case, but also because it does not encourage 
> some brave soul who might pass by this port to attempt to fix 
> it.  Vague hints tend to give these "oh well, maybe later" feelings.

To encourage people to test the port on other archs, maybe something like :

ONLY_FOR_ARCHS=         i386 amd64
ONLY_FOR_ARCHS_REASON=  Has only been tested on i386 and amd64

would be more explicit ?

Best regards,

--
Ganael LAPLANCHE <ganael.laplanche at martymac.org>
http://www.martymac.org | http://contribs.martymac.org
FreeBSD: martymac <martymac at FreeBSD.org>, http://www.FreeBSD.org