On Mon, 4 Oct 2021 23:55:06 -0600 Thomas Frohwein <tfrohw...@fastmail.com> wrote:
> On Mon, Oct 04, 2021 at 07:32:56PM +0000, Charlene Wendling wrote: > > Hi, > > > > > > > There is an existing port draft by bcallah@ and cwen@ in > > > openbsd-wip [2]. I've graciously adopted the patches to the > > > bundled physfs for big endian compatibility and to use > > > ~/.local/share/VVVVVV/ for the assets file data.zip. > > > > Actually, that patch may not be needed, because VVVVVV does not > > support big endian archs [0], as i've found out later. Yet i've > > built your port on macppc, and indeed colors are off, most sprites > > are not displayed, so i think it needs a 'NOT_FOR_ARCHS=$ > > {BE_ARCHS}' at least. > > > > > Some comments on where I diverge from the version in openbsd-wip: > > > [...] > > > - Compiles with std=c++98, so looks to me like base-gcc can be > > > included in COMPILER... > > > > Due to [0], it won't be built on sparc64, the last GCC arch > > we build packages for, so i think the COMPILER line can safely be > > removed. > > > > > [...] > > > > Charlène. > > > > [0] https://github.com/TerryCavanagh/VVVVVV/pull/461 > > > > Thanks, that makes sense. I added the NOT_FOR_ARCHS and removed the > patch. > > I left the COMPILER line with the comment as it's C++, in case that > will be relevant in the future. I think if there's a decision at some > point that this isn't needed with sparc64 in NOT_FOR_ARCHS, it can be > addressed then IMO. (Comments welcome on what direction this topic is > going...) > > I've added an updated tarball; I tested it again with the free assets > and still works. ok? OK cwen@ port-wise; it would be nice if someone is OK with the licensing terms though.