Re: [UPDATE] openmsx to 17.0

On Fri, Aug 06 2021, Jeremie Courreges-Anglas <jca@wxcvbn.org> wrote:
> On Wed, Aug 04 2021, Ricardo Mestre <mestre.openbsd@sapo.pt> wrote:
>> Hi,
>>
>> I don't know what I'm doing but with the below I was able to run a few MSX games
>> with the new major release 17.0 (skipped 16.0) on amd64. Release notes below
>> [0][1].
>>
>> Comments? OK?
>
> Since I touched this last to add riscv64 support,
>
>> Index: patches/patch-build_cpu_py
>> ===================================================================
>> RCS file: /cvs/ports/emulators/openmsx/patches/patch-build_cpu_py,v
>> retrieving revision 1.1
>> diff -u -p -u -r1.1 patch-build_cpu_py
>> --- patches/patch-build_cpu_py 1 Aug 2021 00:18:05 -0000 1.1
>> +++ patches/patch-build_cpu_py 4 Aug 2021 12:27:14 -0000
>> @@ -3,9 +3,9 @@ $OpenBSD: patch-build_cpu_py,v 1.1 2021/
>> Index: build/cpu.py
>> --- build/cpu.py.orig
>> +++ build/cpu.py
>> -@@ -88,6 +88,12 @@ class PPC64(CPU):
>> - name = 'ppc64'
>> - bigEndian = True
>> +@@ -97,6 +97,12 @@ class RISCV64(CPU):
>> + name = 'riscv64'
>> + bigEndian = False
>>
>> +class RISCV64(CPU):
>> + '''64-bit RISC-V.
>
> This patch can be removed,

Confirmed.

> upstream had already committed the same, see
> https://github.com/openMSX/openMSX/commit/b06f755fa5654b3b17643feeaf74497a64755cf0
>
>> Index: patches/patch-build_detectsys_py
>> ===================================================================
>> RCS file: /cvs/ports/emulators/openmsx/patches/patch-build_detectsys_py,v
>> retrieving revision 1.6
>> diff -u -p -u -r1.6 patch-build_detectsys_py
>> --- patches/patch-build_detectsys_py 1 Aug 2021 00:18:05 -0000 1.6
>> +++ patches/patch-build_detectsys_py 4 Aug 2021 12:27:14 -0000
>> @@ -3,7 +3,7 @@ $OpenBSD: patch-build_detectsys_py,v 1.6
>> Index: build/detectsys.py
>> --- build/detectsys.py.orig
>> +++ build/detectsys.py
>> -@@ -48,6 +48,8 @@ def detectCPU():
>> +@@ -53,6 +53,8 @@ def detectCPU():
>> return 'sheb' if cpu.endswith('eb') else 'sh'
>> elif cpu == 'avr32':
>> return 'avr32'
>
> This one should stay, the release you're upgrading to doesn't have this
> commit:
> https://github.com/openMSX/openMSX/commit/321d1d2f634d10ef72dc71831588540a7c6de59e
> Please add this link as a comment in the patch.
>
> I'm running a build on riscv64, I'll let you know tomorrow if it goes well.

Builds fine, ok jca@ ports-wise

--
jca | PGP : 0x1524E7EE / 5135 92C1 AD36 5293 2BDF DDCC 0DFA 74AE 1524 E7EE