On Tue, Feb 06, 2024 at 01:36:58AM +0000, Klemens Nanni wrote: > On Tue, Feb 06, 2024 at 02:28:34AM +0100, Tobias Heider wrote: > > Tested and found out it builds and runs fine on macppc. > > > > ok to enable it there? > > > > Index: Makefile > > =================================================================== > > RCS file: /cvs/ports/games/yquake2/Makefile,v > > retrieving revision 1.28 > > diff -u -p -r1.28 Makefile > > --- Makefile 26 Sep 2023 09:41:45 -0000 1.28 > > +++ Makefile 6 Feb 2024 01:17:26 -0000 > > @@ -1,4 +1,4 @@ > > -ONLY_FOR_ARCHS= i386 amd64 sparc64 > > +ONLY_FOR_ARCHS= i386 amd64 sparc64 macppc > > > > COMMENT= Yamagi Quake II > > N= yquake2 > > > > Do you want to stick to the whitelist or flip to NOT_FOR_ARCHS? > > I'm not interested in this game, just wondered how it came to be that > architectures support changed.
I wouldn't mind enabling it for all archs either. Not sure which arch it doesn't work on so NOT_FOR_ARCHS doesn't seem very useful. I guess if it works on 32 bit BE macppc there is a good chance that most others work too. > > revision 1.4 > date: 2016/06/28 20:50:22; author: awolk; state: Exp; lines: +6 -4; > commitid: OCjzSGt5M9IIpgwf; > Update games/yquake2 5.32 => 5.34 > > OK abieber@ > > Changes in the port: > [...] > Additional notes: > [...] > - upstream said in their changelog that they are changing their > platform policy 'Switch from an arch whitelist to an "all archs are > supported" approach.'. I saw in our tree a change by landry@ > specifically limiting the supported platform to the upstream > supported versions > > (http://cvsweb.openbsd.org/cgi-bin/cvsweb/ports/games/yquake2/Makefile?rev=1.2&content-type=text/x-cvsweb-markup). > I am not removing this restriction until I get some feedback from > people able to test other platforms than currently > listed in ONLY_FOR_ARCHS. >
