Skip to content

Conversation

@alice-mkh
Copy link

They were removed in 3435b26

@Kekun
Copy link
Contributor

Kekun commented Aug 9, 2018

The GNOME Games Flatpak installation relies on these rules to be there so I can't approve that MR more.

@andres-asm
Copy link
Contributor

andres-asm commented Aug 9, 2018 via email

@Kekun
Copy link
Contributor

Kekun commented Aug 16, 2018

Any news? GNOME Games depends on this fix. :(

@andres-asm
Copy link
Contributor

You probably should add them too fr500/dosbox_svn instead, it's a faster core (and I can merge it there)

@alice-mkh
Copy link
Author

alice-mkh commented Dec 2, 2018

fr500/dosbox_svn seems to introduce SDL dependency, which isn't ideal for us. Any reason not to merge this, considering other PRs are being merged?

@andres-asm
Copy link
Contributor

I have no idea I have no control over libretro repos.
It introduces SVN because dosbox has always used SVN for many things (timers, threads, CD-ROM)

@alice-mkh
Copy link
Author

Ah, and you're the only person besides Adrien who received a notification. Sorry :(

As for dependency, aren't libretro cores meant to be self-contained? I.e. I don't know any other core that has external dependencies (not saying there aren't any, just I don't know any :D)


@twinaphex since you're the person who merged the last MR here, mind taking a look at this one?

@escalade
Copy link

escalade commented Dec 3, 2018

@Exalm

External libraries can be compiled in statically, lots of cores do this AFAIK.

@fr500

I think only SDL_sound (mp3/wav/aiff track support) is used for CDROM, right?

BTW there's a very good SDL2 port that I've been using for a long time here: https://github.com/duganchen

Sorry for the off topic, just thought it may be interesting.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants