Looking forward for this port. :)
Elias.
2018-05-02 12:55 GMT-03:00 Ryan Freeman <r...@slipgate.org>:
> On Mon, Apr 30, 2018 at 07:23:11PM -0700, Thomas Frohwein wrote:
>> Please find attached this port of the 3DS emulator citra. It can run homebrew
>> and other 3DS roms. Comes with a commandline binary (citra) and a Qt one
>> (citra-qt). Like many other emulators, it requires USE_WXNEEDED to run
>> unfortunately. Based on my testing, the emulation speed seems good for 2D
>> games, but 3D games run a little sluggish on this i7-6820HQ with Intel HD
>> Graphics 530.
>
> Hey Thomas,
>
> As per #openbsd-gaming, I got this built and tested on my amd64 desktop
> yesterday. Builds and packages fine and runs things. Notes inline below.
>
>>
>> A few notes on the port:
>>
>> - versioning - right now based on numbering of nightly (670). Alternatives
>> might
>> be the date of the nightly, e.g. 20180427. Let me know if there's a
>> preference.
>> - listed license information on all relevant builtin libs. Let me know if
>> this
>> is too detailed.
>
> I think this license listing you have is excellent, very easy to discern where
> licenses are on the components.
>
>> - I tried to disable builtin dependencies where possible, like cryptopp and
>> enet.
>> - I can't get soundtouch from ports to work because of a type incompatibility
>> short* vs. float* (same problem exists for dolphin emulator). Apparently
>> these ports are set up for integer samples, while our ports version is not
>> (additional information can be found here:
>> https://www.surina.net/soundtouch/README.html
>> under 3.1 Supported sample data formats for anyone who might be luckier
>> than
>> me in troubleshooting this).
>> - The inclusion of submodules follows the way it's been done with
>> emulators/ppsspp.
>> - I disabled WebService which is for telemetry and pulls in additional
>> dependencies.
>>
>> I put my fastmail email address as maintainer email. I'm planning to switch
>> all
>> the ports with my ymail address to this one over time. Will keep ymail
>> address
>> still available for the lifecycle of 6.3.
>>
>> All tests pass in 'make test'.
>
> You might want to run fmt -72 on the pkg/DESCR file.
> There might also be desire to have the rmdir commands one-per-line rather than
> giving a single rmdir command several arguments with \ to split lines in the
> post-extract part.
>
> Outside of that it looks good from what I can see, thanks for the port!
>
> -ryan
>