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

[Eitan Adler]

On 19 April 2013 11:15, Ganael LAPLANCHE <martymac at freebsd.org> wrote:
> 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 ?

No.

ONLY_FOR_* means that the port is *never intended* to work on any
other platform.  BROKEN ports are, in theory, periodically tested.



-- 
Eitan Adler
Source, Ports, Doc committer
Bugmeister, Ports Security teams