[Bug 288495] [NEW PORT] emulators/Ymir: Sega Saturn emulator

https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=288495

--- Comment #7 from Stefan Schlosser <bsdcode@disroot.org> ---
(In reply to Vladimir Druzenko from comment #3)
> Why name with capital letter "Ymir"? Why not "ymir"?
It is constantly capitalized by Upstream, so "Ymir" is the official name. But
I'm not objecting against "ymir" for the portname, actually I'd also prefer it
to "Ymir".

After rereading "5.2.5. Package Naming Conventions" of the Porter's Handbook,
especially the examples listed, I see that "ymir" would be the more correct
portname. I will change this in the patch.


(In reply to Vladimir Druzenko from comment #4)
> Can port use devel/libfmt and x11-toolkits/imgui from ports?
From my initial comment:
> 4) The git submodule for the "imgui" library is kept instead of unbundling with
> x11-toolkits/imgui because the SDL3 backend is required and the port currently doesn't
> provide this backend. I recently submitted a patch to include the backend (bug #288454),
> but there's also the problem that the SDL2 and SDL3 backends can't be enabled at the same
> time. So the port would have to make a decision to enable one of two backends by default,
> but which one? Maybe it would be better to have one flavor per backend of
> x11-toolkits/imgui. Long story short: I guess the imgui submodule will stay for longer.

(In reply to Vladimir Druzenko from comment #5)
> 3. If it require zenity, then use SINGLE instead of RADIO:
> OPTIONS_DEFAULT=        LTO ZENITY
> OPTIONS_SINGLE=         DIALOG
> OPTIONS_SINGLE_DIALOG=  ZENITY ZENITY4
The intention is to explicitly not depend on a specific zenity version, because
a user could have the other version already installed and installation of Ymir
would then be in conflict with the currently installed zenity version.
Therefore ZENITY is not enabled by default and it's a RADIO options group. The
pkg-message instructs the user to install one of the zenity packages. A similar
thing was done in graphics/vhs with the browser driver.

-- 
You are receiving this mail because:
You are the assignee for the bug.