ScummVM 2.1.0

hhromic

@GeekOB thanks for reaching out to ScummVM devs and finding out what branch we should be using.
I guess the scriptmodule was using the wrong branch before as well (branch-2-0).

So just to be clear, the officially endorsed branch to use for stable is branch-2-1-0 then?

EDIT: I also noticed they have tags in their repository, e.g. v2.1.0. Could you kindly ask the developers what is preferred for checking out the source? the tag v2.1.0 or the branch branch-2-1-0? The tag and the branch are not in sync in the repository. Thanks!

liquidpolicenaut

Hey guys!

I just wanted to say I also tried updating Scummvm to 2.1 on my Retropie via source and, for the most part, it seemed to work great except for two problems. The one small issue was that the game Beavis and Butthead will not longer load correctly. It is recognized by Scummvm just like before in the menu but when I go to launch it, it always kicks back to my main menu...

The main problem is that the D-pad on my controller is not recognized anymore. To clarify, if I use any controller that has ONLY a D-pad and no analog sticks, the d-pad doesn't work. I have tried 3 different controllers and none worked. Interestingly though, any controller I tried WITH analog sticks worked perfectly well... Both the analog sticks and the D-pad!

Any thoughts?

mitu

@liquidpolicenaut Which version of RetroPie are you using ? Which version of ScummVM are you using - SDL2 or SDL1 ?

liquidpolicenaut

@mitu Oh, sorry I forgot to mention! I'm running Retropie 4.3.11 and Scummvm 2.0.1 SDL2. Just as a side note...I also tried this on a newer Retropie build (4.4 something) and had the same results..

mitu

@liquidpolicenaut I'm on 4.5.1 and I'm also using the SDL2 version - but it's 2.1.0 (the version in this topic), not 2.0.1. I have no problem running Beavis and Butt-head in Virtual Stupidity with this version. Can't say much about the joystick part, since I don't use it to play ScummVM.

Can you post a log file - /dev/shm/runcommand.log - from running the game, after it crashes ?

liquidpolicenaut

@mitu Sorry, I meant it was 2.1 that I'm running

GeekOB

@liquidpolicenaut if you mean version 2.1.1pre like the one that's available by binary download, as I stated above it's not behaving as expected. @hhromic is already in contact with the ScummVM dev team and he'll work on a new strategy for releasing stable versions of this amazing port.

BuZz

I have a commit being pushed in a moment to fix the cloud support in the binaries. Just a missing dependency. Binaries will be uploaded soon after.

The rest I shall leave to @hhromic - it's in safe hands :-)

mitu

@GeekOB No, the game still works on the 2.1.1pre version - as the one shipped with binary download.
@liquidpolicenaut did you re-scan your games after you updated your binary ? Without a log file it's difficult to find why the game crashes.

GeekOB

Thanks @mitu, I stand corrected.
Are there any news on the decision of which branch(es) will be used for ScummVM @hhromic ?

hhromic

@GeekOB I'm working on it, please be patient and soon I will post updates :)

hhromic

@GeekOB can you please test the following PR on your side?
https://github.com/RetroPie/RetroPie-Setup/pull/2889

It applies the changes requested by Eugene from ScummVM, namely use the stable v2.1.0 tag for checking out the code and disabling the experimental engines.

Thanks!

GeekOB

@hhromic happy to, what do I need to do to test this?

hhromic

@GeekOB a simple way is to do this:

Go to your RetroPie-Setup directory. Make sure you don't have any pending changes or modifications.
Checkout a new branch for testing: git checkout -b scummvm-test

Apply the PR as a patch:

git am < <(curl -sSL https://patch-diff.githubusercontent.com/raw/RetroPie/RetroPie-Setup/pull/2889.patch)

Re-build and install ScummVM from source as normal.
Test ScummVM.
To revert the patch, checkout back to master: git checkout master and delete the testing branch: git branch -D scummvm-test.

GeekOB

I can confirm that version 2.1.0 is working properly after building from the testing branch.
Let me know if I can help with any additional testing @hhromic
Thanks!

hhromic

@GeekOB thanks for confirming!
I'll green-light the PR then to be merged soon.

GeekOB

Thanks @hhromic !!
What branch will the build from source option use?
Are you planning to introduce any "bleeding edge" variants in the experimental packages section?

GeekOB

I have updated to 2.1.0 using the latest binary. Thanks @hhromic and @BuZz !
I also went ahead and tested the "Update from source" option and received the following error messages:

Log started at: Thu  7 Nov 08:41:35 EST 2019

RetroPie-Setup version: 4.5.2 (5941e53d)
System: Raspbian GNU/Linux 9.11 (stretch) - Linux retropie 4.14.98-v7+ #1200 SMP Tue Feb 12 20:27:48 GMT 2019 armv7l GNU/Linux

= = = = = = = = = = = = = = = = = = = = =
Installing dependencies for 'scummvm' : ScummVM
= = = = = = = = = = = = = = = = = = = = =

/home/pi/RetroPie-Setup/tmp/build/scummvm /home/pi

= = = = = = = = = = = = = = = = = = = = =
Getting sources for 'scummvm' : ScummVM
= = = = = = = = = = = = = = = = = = = = =

git clone --recursive --depth 1 --branch v2.1.0 "https://github.com/scummvm/scummvm.git" "/home/pi/RetroPie-Setup/tmp/build/scummvm"
Cloning into '/home/pi/RetroPie-Setup/tmp/build/scummvm'...
error: RPC failed; curl 56 GnuTLS recv error (-9): A TLS packet with unexpected length was received.
fatal: The remote end hung up unexpectedly
fatal: early EOF
fatal: index-pack failed
fatal: Cannot change to '/home/pi/RetroPie-Setup/tmp/build/scummvm': No such file or directory
fatal: Cannot change to '/home/pi/RetroPie-Setup/tmp/build/scummvm': No such file or directory
HEAD is now in branch '' at commit ''
can't find file to patch at input line 3
Perhaps you used the wrong -p or --strip option?
The text leading up to this was:
--------------------------
|--- a/configure
|+++ b/configure
--------------------------
No file to patch.  Skipping patch.
1 out of 1 hunk ignored
/home/pi
Error running 'git clone --recursive --depth 1 --branch v2.1.0 https://github.com/scummvm/scummvm.git /home/pi/RetroPie-Setup/tmp/build/scummvm' - returned 128
scummvm patch /home/pi/RetroPie-Setup/scriptmodules/emulators/scummvm/01_rpi_enable_scalers.diff failed to apply

Log ended at: Thu  7 Nov 08:42:10 EST 2019
Total running time: 0 hours, 0 mins, 35 secs

I wanted to give you a heads up so you can fix it when you have a chance.
Thank you once again :)

mitu

@GeekOB said in ScummVM 2.1.0:

error: RPC failed; curl 56 GnuTLS recv error (-9): A TLS packet with unexpected length was received.

Looks like a connection problem on your end. Have you retried to do the source installation ?
Since the binaries are now following a stable branch and they've been recently build, you'll be installing the same version as the binary update.

GeekOB

Understood @mitu , thanks. I'll retest the source and let you know.
The version makes sense as it's very recent, not sure if the ScummVM team will be making major changes to that branch, we'll see.