RetroPie / RetroPie-Setup

Shell script to set up a Raspberry Pi/Odroid/PC with RetroArch emulator and various cores
Other
10.08k stars 1.39k forks source link

Directlaunch Runcommand Crash #1271

Closed HerbFargus closed 8 years ago

HerbFargus commented 8 years ago

Before we actively pursue setting certain systems (e.g. Kodi see https://github.com/RetroPie/RetroPie-Setup/pull/904) as directlaunch from emulationstation we need to

From testing kodi works fine when launched directly with kodi-standalone but if runcommand is used it crashes.

HerbFargus commented 8 years ago

Just for reference this is my kodi crash log:

http://pastebin.com/KHT2eTXZ

joolswills commented 8 years ago

I just tested and it seems fine here

launching with

  <system>
    <name>kodi</name>
    <fullname>Kodi</fullname>
    <command>/opt/retropie/supplementary/runcommand/runcommand.sh 0 kodi-standalone</command>
    <theme>kodi</theme>
    <directlaunch>true</directlaunch>
  </system>
joolswills commented 8 years ago
  <system>
    <name>kodi</name>
    <fullname>Kodi</fullname>
    <command>/opt/retropie/supplementary/runcommand/runcommand.sh 0 _SYS_ kodi</command>
    <theme>kodi</theme>
    <directlaunch>true</directlaunch>
  </system>

works too (and gives the ability to the user to control the commandline etc)

HerbFargus commented 8 years ago

No crash on exit? May have been addressed with your last modification to fb_restore. If you're happy I'm happy.

joolswills commented 8 years ago

must have been that. or something else. I will switch kodi to direct launch in a PR and we can test.

HerbFargus commented 8 years ago

Brilliant :)

HerbFargus commented 8 years ago

Seems resolved now.

HerbFargus commented 8 years ago

Upon further testing it seems there is an issue with directlaunch in that once implemented it will crash emulationstation when switching themes through the start menu in emulationstation.

To reproduce:

joolswills commented 8 years ago

This should be opened against ES then.

HerbFargus commented 8 years ago

opened here: https://github.com/RetroPie/EmulationStation/issues/37