moonlight: request for comments and testing

hhromic

@LeSabotageur let me know how it goes with your controller, if the pending PR in moonlight solves your issue, definitively is worth trying to push it to be merged upstream =).

Regarding the generation of config files for games that have a semicolon in the name, I'm afraid I haven't been able to replicate the issue on my side. I do have games with these characters in the name and they are working fine here.

The closest I could get to replicate the same error you are getting was when there is no roms/steam folder previously created, so no config files could be created and I get the same error than you, no matter if they had semicolons.

I wonder if you have the latest version of RetroPie-Setup updated?
Maybe something got screwed up. When you have time (no rush!), could you please completely remove moonlight, the scriptmodule and the config files in roms/steam, then start over with building and generating the config files again?
Just to confirm it is not something spurious on your side?

Something like this should do:

cd $HOME/RetroPie-Setup
sudo ./retropie_packages.sh moonlight remove  # remove moonlight
rm -rf  scriptmodules/supplementary/moonlight.sh  # remove the scriptmodule
rm -rf $HOME/Retropie/roms/steam  # remove all the config files
patch -p1 < <(wget https://github.com/hhromic/RetroPie-Setup/compare/master...add-moonlight.diff -qO-)  # re-add the scriptmodule
sudo ./retropie_packages.sh moonlight clean  # clean the build directory
sudo ./retropie_packages.sh moonlight  # re-build, install and configure moonlight
sudo ./retropie_packages.sh moonlight gui  # start the configuration GUI

Thanks again for the testing! appreciated!

reesk92

@hhromic im not officially on the ora team anymore but i still maintain close ties with them and will help and assist where i can

reesk92

im not sure what your trying to do but as for having ffmpeg installed it is my understanding we have ffmpeg installed for video rendering of video snaps because omxplayer or whatever pi uses is unavailable for odroid so we use that atleast thats what i think we use it for

that or we use it for something to do with the lcd screen on the ogst case that our board supports

either way im fairly certain considering the minimal image we started with that we wouldnt install anything that wasnt absloutley neccesary to the running of the image

ill see what i can do for passing this along to someone who can better assist you

hhromic

@reesk92 many thanks on trying to help, really appreciated !

Just to clarify, I'm not a direct user of ORA, but I'm trying to help another user to get moonlight to build in his ORA ( @liquidsnake34 ).

What is happening is that @liquidsnake34 tried to build moonlight on his ORA installation and the build failed because moonlight references an updated ffmpeg API (3.1+) than what is apparnetly installed (2.8.6).

Apparently, moonlight doesn't need ffmpeg on ODROID as it can use the chipset directly like it does on the RPI. I say "apparently" because I can't test myself.

The options I can see are to (1) disable detecting/using ffmpeg in the moonlight makefiles or (2) update ffmpeg in ORA. The second option would be prefered if it is relatively easy to do, and that is what I don't know for sure and would require advise from ORA seniors :)
In case that the first option is necessary, then I would need to test if moonlight still runs fine without compiling ffmpeg support. According to what you just said, it might be necessary after all if it does hardware decoding.

Thanks a lot for wanting to help and if you are not the right person, I look forward to be refered to someone that can further help.

fnkngrv

If the version that is compiling is lower than what is needed most likely a kernel level issue. Someone would need to verify the latest version of ffmpeg supported on the 3.10 kernel. Whatever is showing as installed on the build is what compiled with the original RetroPie installation itself. RetroPie official has very basic support for the XU4 and we have not done anything specific to the installation of ffmpeg itself. I am assuming its installation is tied into VLC. We actually leverage mplayer for most video related media on the build. We are for the most part the most concerned with retrogaming systems. We have not spent any cycles on things like Moonlight, Attract Mode, or Kodi.

hhromic

@fnkngrv thanks for the heads-up!
I agree upgrading ffmpeg is not a straightforward action and probably it is best to leave it alone because of the hardware acceleration and the coupling with the kernel version/state.
I also understand ORA is all about the retrogaming more than side projects like moonlight, no problem.

I was researching further and indeed the best solution would be to disable ffmpeg support (and attempt to compile aginst it) in moonlight, which is only used for software-only decoding. Moonlight seems to support a number of embedded chipsets directly: Broadcom, Freescale, Amlogic and Rockchip.
I'm not sure which one is picked for XU4 (or if any supports XU4 directly) as I don't have that board and can't compile/test myself. I will try to debug with the user and see what can be done for moonlight to compile/work in XU4 natively (if possible).

Thanks again for the help and clarifications!

fnkngrv

The XU4 leverages the following:

Samsung Exynos 5422 4-cores ARM Cortex-A15 @2.0GHz
and 4-cores ARM Cortex-A7 @1.4GHz CPU

8 core BIG little architecture

mitu

@fnkngrv said in moonlight: request for comments and testing:

8 core BIG little architecture

You mean little endian ?

Swampy

First of all, thank you for your afford.

I have tested your script with a Raspberry Pi 3b+ and the recent RetroPie release.
It works.

I only realised a little problem:

If I start your script within RetroPie-Setup and try to pair, it doesn't show me the PIN for connecting to the remote host.
Only after I abort the dialogue box with the question for the PIN in the remote host, it shows me the PIN.

If I start it manually with "sudo ./retropie_packages.sh moonlight gui " it works.

mitu

@Swampy Try replacing, in the function exec_moonlight from the scriptmodule, the line:

sudo -u $user "$md_inst/moonlight.sh" "$@"

with:

sudo -u $user "$md_inst/moonlight.sh" "$@" > /dev/tty < /dev/tty

and see if starting the GUI from the setup script works for pairing.

hhromic

@Swampy thanks for testing! I'm happy the scriptmodule is working for you as well.

Interesting little bug you found there indeed. I admit I mostly test using retropie_packages.sh and I assumed the main GUI would just work, hence didn't notice that behaviour.

The suggestion from @mitu makes sense and should work. I can't test at the moment but also will check it and report back/fix the issue.

hhromic

I confirm the issue and implemented a fix now, thanks @Swampy for reporting and @mitu for the solution suggestion!

@Swampy , please update the scriptmodule and try again, should work fine now.

@mitu , I didn't modify the line you suggest because it interferes with other parts of the scriptmodule (e.g. the gen_configs function). Instead I put the redirections on the affected calls from the gui function (pair/unpair calls). Thanks for the pointer anyway, was helpful!

hhromic

Warning
The latest Nvidia GeForce Experience (GFE) driver released on Nov 19, 2018 just broke streaming foor the current version of moonlight. A fix is already proposed but not yet merged in upstream moonlight repository.
The issue is reported here: https://github.com/irtimmer/moonlight-embedded/issues/705

The affected GFE version is:
GFE 3.16.0.122 (gs_04_11_25137997, 7.1.411.0)

I will try to backport the fix until it is merged upstream.

hhromic

Update
A temporary patch to fix compatibility with GFE 3.16 is now included in the moonlight scriptmodule. Just repatch and rebuild moonlight and streaming should work fine in GFE 3.16 again.
I sent a PR now to upstream moonlight, hopefully it gets merged soon: https://github.com/irtimmer/moonlight-common-c/pull/1

hhromic

Update
The fix was solved upstream now (that was quick!). The temporary patch is now dropped as is no longer needed. Moonlight is again working fine with GFE 3.16

Swampy

It works.

Thank you for your fast solution.

hhromic

@Swampy glad to hear!

LeSabotageur

@hhromic I've finally got a go at it, but sadly I've still encountered errors at the generation of files (yeah, it's a bit long...) :

Generating config file for 'The Elder Scrolls V: Skyrim' ...
touch: setting times of '/home/pi/RetroPie/roms/steam/The Elder Scrolls V: Skyrim.ml': No such file or directory
sed: cannot stat /home/pi/RetroPie/roms/steam/The Elder Scrolls V: Skyrim.ml: No such file or directory
/home/pi/RetroPie-Setup/scriptmodules/inifuncs.sh: line 82: /home/pi/RetroPie/roms/steam/The Elder Scrolls V: Skyrim.ml: Invalid argument
touch: setting times of '/home/pi/RetroPie/roms/steam/The Elder Scrolls V: Skyrim.ml': No such file or directory
touch: setting times of '/home/pi/RetroPie/roms/steam/The Elder Scrolls V: Skyrim.ml': No such file or directory
sed: cannot stat /home/pi/RetroPie/roms/steam/The Elder Scrolls V: Skyrim.ml: No such file or directory
/home/pi/RetroPie-Setup/scriptmodules/inifuncs.sh: line 82: /home/pi/RetroPie/roms/steam/The Elder Scrolls V: Skyrim.ml: Invalid argument
chown: cannot access '/home/pi/RetroPie/roms/steam/The Elder Scrolls V: Skyrim.ml': No such file or directory
Generating config file for 'Hotline Miami 2: Wrong Number' ...
touch: setting times of '/home/pi/RetroPie/roms/steam/Hotline Miami 2: Wrong Number.ml': No such file or directory
sed: cannot stat /home/pi/RetroPie/roms/steam/Hotline Miami 2: Wrong Number.ml: No such file or directory
/home/pi/RetroPie-Setup/scriptmodules/inifuncs.sh: line 82: /home/pi/RetroPie/roms/steam/Hotline Miami 2: Wrong Number.ml: Invalid argument
touch: setting times of '/home/pi/RetroPie/roms/steam/Hotline Miami 2: Wrong Number.ml': No such file or directory
touch: setting times of '/home/pi/RetroPie/roms/steam/Hotline Miami 2: Wrong Number.ml': No such file or directory
sed: cannot stat /home/pi/RetroPie/roms/steam/Hotline Miami 2: Wrong Number.ml: No such file or directory
/home/pi/RetroPie-Setup/scriptmodules/inifuncs.sh: line 82: /home/pi/RetroPie/roms/steam/Hotline Miami 2: Wrong Number.ml: Invalid argument
chown: cannot access '/home/pi/RetroPie/roms/steam/Hotline Miami 2: Wrong Number.ml': No such file or directory
Generating config file for 'Metro: Last Light' ...
touch: setting times of '/home/pi/RetroPie/roms/steam/Metro: Last Light.ml': No such file or directory
sed: cannot stat /home/pi/RetroPie/roms/steam/Metro: Last Light.ml: No such file or directory
/home/pi/RetroPie-Setup/scriptmodules/inifuncs.sh: line 82: /home/pi/RetroPie/roms/steam/Metro: Last Light.ml: Invalid argument
touch: setting times of '/home/pi/RetroPie/roms/steam/Metro: Last Light.ml': No such file or directory
touch: setting times of '/home/pi/RetroPie/roms/steam/Metro: Last Light.ml': No such file or directory
sed: cannot stat /home/pi/RetroPie/roms/steam/Metro: Last Light.ml: No such file or directory
/home/pi/RetroPie-Setup/scriptmodules/inifuncs.sh: line 82: /home/pi/RetroPie/roms/steam/Metro: Last Light.ml: Invalid argument
chown: cannot access '/home/pi/RetroPie/roms/steam/Metro: Last Light.ml': No such file or directory
Generating config file for 'Metro: Last Light' ...
touch: setting times of '/home/pi/RetroPie/roms/steam/Metro: Last Light.ml': No such file or directory
sed: cannot stat /home/pi/RetroPie/roms/steam/Metro: Last Light.ml: No such file or directory
/home/pi/RetroPie-Setup/scriptmodules/inifuncs.sh: line 82: /home/pi/RetroPie/roms/steam/Metro: Last Light.ml: Invalid argument
touch: setting times of '/home/pi/RetroPie/roms/steam/Metro: Last Light.ml': No such file or directory
touch: setting times of '/home/pi/RetroPie/roms/steam/Metro: Last Light.ml': No such file or directory
sed: cannot stat /home/pi/RetroPie/roms/steam/Metro: Last Light.ml: No such file or directory
/home/pi/RetroPie-Setup/scriptmodules/inifuncs.sh: line 82: /home/pi/RetroPie/roms/steam/Metro: Last Light.ml: Invalid argument
chown: cannot access '/home/pi/RetroPie/roms/steam/Metro: Last Light.ml': No such file or directory
Generating config file for 'DuckTales: Remastered' ...
touch: setting times of '/home/pi/RetroPie/roms/steam/DuckTales: Remastered.ml': No such file or directory
sed: cannot stat /home/pi/RetroPie/roms/steam/DuckTales: Remastered.ml: No such file or directory
/home/pi/RetroPie-Setup/scriptmodules/inifuncs.sh: line 82: /home/pi/RetroPie/roms/steam/DuckTales: Remastered.ml: Invalid argument
touch: setting times of '/home/pi/RetroPie/roms/steam/DuckTales: Remastered.ml': No such file or directory
touch: setting times of '/home/pi/RetroPie/roms/steam/DuckTales: Remastered.ml': No such file or directory
sed: cannot stat /home/pi/RetroPie/roms/steam/DuckTales: Remastered.ml: No such file or directory
/home/pi/RetroPie-Setup/scriptmodules/inifuncs.sh: line 82: /home/pi/RetroPie/roms/steam/DuckTales: Remastered.ml: Invalid argument
chown: cannot access '/home/pi/RetroPie/roms/steam/DuckTales: Remastered.ml': No such file or directory
Press ENTER to continue...

But that's minor, and from what I've tested lately it works great.

Now I'm gonna check out that Steam Link integration and see how it fares compared to moonlight !

A Former User

@hhromic Just pulled in the patch on the latest versions of everything, and was able to build, and then set up via:

sudo ./retropie_packages.sh moonlight gui

It worked great (and also let me easily build the current upstream HEAD so that the latest GFE would work)!

I ran into 2 minor issues, which were user error, and possibly something to note in case someone else runs into it as well. I had initially set up moonlight via this script. I needed to do both of the following options from that script to have the scriptmodule versions show up in RetroPie instead:

3: Remove Launch Scripts
7: Refresh SYSTEMS Config File

Other than that, things are working great! A quick feature request would be to build moonlights configuration options into runcommand (bitrate, fps, width and height, etc.) . It would be reallyslick to be able to change those settings on the fly, to be able to find the best settings without having to leave your seat / controller.

hhromic

@LeSabotageur the issue you are experiencing is very puzzling, shouldn't happen at all. I would like to get to the bottom of your issue to make sure is not an overlooking in the scriptmodule :)

Can you please run these commands and report back the output for each?

$ git -C $HOME/RetroPie-Setup log --oneline -n 10
$ ls -la $HOME/RetroPie/roms
$ ls -la $HOME/RetroPie/roms/steam
$ /opt/retropie/supplementary/moonlight/moonlight.sh list

Thanks a lot for testing and giving positive feedback! Moonlight is a great game streaming client.

@jml , also thanks for the testing ! I'm glad you are happy with the scriptmodule state and enjoying moonlight so far. Indeed users should clean any previous moonlight scripts/installations.

Regarding your suggestion with runcommand, unfortunately it doesn't have any programming API or extensibility to implement such features from this scriptmodule. However, if you are not afraid of the console, what you can do it so generate a set of configurations for testing and put them in a subfolder inside the steam folder. For example something like this:

$ DIR=$HOME/RetroPie/roms/steam/testing
$ mkdir -p $DIR
$ for fps in 30 60; do for bitrate in 5000 10000 15000; do cat > $DIR/launch-$fps-$bitrate.ml << _EOF_; done; done
config = /opt/retropie/configs/all/moonlight/global.conf
fps = $fps
bitrate = $bitrate
app = Steam
_EOF_

If you get the idea, you can further create combinations for width/height parameters too. After this, you will have all these options launchable from EmulationStation for quick testing in a testing subfolder under the steam system, alongside your normal configs.