Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CLI idle exit with success #142

Merged
merged 1 commit into from Nov 30, 2022

Conversation

DeadStarlin
Copy link
Contributor

This one is tricky, cause when idling, and you break with SIGINT CTRL+C, you get message that pipe broke - even you should exit successfully. I tried to find issue and looks like some issues not catching correctly SIGINT is fixed by new method and to not have message just need to exit successfully and game exists by itself without error. Cause read_count was called and EXIT successfully in handle_sigint_cli not had chance to execute. Maybe you have another idea how to fix it.
It does not break anything and on SIGINT won't show popup with error. Game is quitted and there are no processes left.

@PaulCombal
Copy link
Owner

I can't test right now, what OS are you using and are you positive this works?

@DeadStarlin
Copy link
Contributor Author

Right now Linux Debian (PopOs distro).

@PaulCombal
Copy link
Owner

Alright I'll believe you on that one, hope it'll work for everyone

@PaulCombal PaulCombal merged commit 9bc4d99 into PaulCombal:dev Nov 30, 2022
@PaulCombal
Copy link
Owner

Do you have other PRs pending?

@DeadStarlin DeadStarlin deleted the SIGINTExitWithError branch December 1, 2022 17:28
@DeadStarlin
Copy link
Contributor Author

Do you have other PRs pending?

No, nothing more now. If I'll have free time, I'll look into "next release" to-do list.

@PaulCombal
Copy link
Owner

Ok, then don't forget to @ me when you think you're done so that I will merge everything into master :)

@DeadStarlin
Copy link
Contributor Author

Ok, then don't forget to @ me when you think you're done so that I will merge everything into master :)

@PaulCombal I think for now you can move code to master. If you can't build the release, I can do it on my side and upload it, so you can attach it to releases.

@PaulCombal
Copy link
Owner

Yess, can you build an Appimage? I'll pin it to the releases tab if that works on my end

@DeadStarlin
Copy link
Contributor Author

SamRewritten-b50966a-x86_64.zip
Here is AppImage - I had to zip it cause GitHub won't allow me to attach it otherwise.

@PaulCombal
Copy link
Owner

Doesn't run on Ubuntu 20.04:

/tmp/.mount_SamRewWjfjhQ/usr/bin/samrewritten: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.32' not found (required by /tmp/.mount_SamRewWjfjhQ/usr/bin/samrewritten)
/tmp/.mount_SamRewWjfjhQ/usr/bin/samrewritten: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.34' not found (required by /tmp/.mount_SamRewWjfjhQ/usr/bin/samrewritten)
/tmp/.mount_SamRewWjfjhQ/usr/bin/samrewritten: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.33' not found (required by /tmp/.mount_SamRewWjfjhQ/usr/bin/samrewritten)
/tmp/.mount_SamRewWjfjhQ/usr/bin/samrewritten: /lib/x86_64-linux-gnu/libstdc++.so.6: version `GLIBCXX_3.4.29' not found (required by /tmp/.mount_SamRewWjfjhQ/usr/bin/samrewritten)
/tmp/.mount_SamRewWjfjhQ/usr/bin/samrewritten: /lib/x86_64-linux-gnu/libstdc++.so.6: version `CXXABI_1.3.13' not found (required by /tmp/.mount_SamRewWjfjhQ/usr/bin/samrewritten)
/tmp/.mount_SamRewWjfjhQ/usr/bin/samrewritten: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.33' not found (required by /tmp/.mount_SamRewWjfjhQ/usr/bin/../lib/libcurl.so.4)
/tmp/.mount_SamRewWjfjhQ/usr/bin/samrewritten: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.34' not found (required by /tmp/.mount_SamRewWjfjhQ/usr/bin/../lib/libcurl.so.4)
/tmp/.mount_SamRewWjfjhQ/usr/bin/samrewritten: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.33' not found (required by /tmp/.mount_SamRewWjfjhQ/usr/bin/../lib/libgtk-3.so.0)
/tmp/.mount_SamRewWjfjhQ/usr/bin/samrewritten: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.34' not found (required by /tmp/.mount_SamRewWjfjhQ/usr/bin/../lib/libgdk-3.so.0)
/tmp/.mount_SamRewWjfjhQ/usr/bin/samrewritten: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.34' not found (required by /tmp/.mount_SamRewWjfjhQ/usr/bin/../lib/libgmodule-2.0.so.0)
/tmp/.mount_SamRewWjfjhQ/usr/bin/samrewritten: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.33' not found (required by /tmp/.mount_SamRewWjfjhQ/usr/bin/../lib/libssh.so.4)
/tmp/.mount_SamRewWjfjhQ/usr/bin/samrewritten: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.33' not found (required by /tmp/.mount_SamRewWjfjhQ/usr/bin/../lib/libpsl.so.5)
/tmp/.mount_SamRewWjfjhQ/usr/bin/samrewritten: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.33' not found (required by /tmp/.mount_SamRewWjfjhQ/usr/bin/../lib/libcrypto.so.3)
/tmp/.mount_SamRewWjfjhQ/usr/bin/samrewritten: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.34' not found (required by /tmp/.mount_SamRewWjfjhQ/usr/bin/../lib/libcrypto.so.3)
/tmp/.mount_SamRewWjfjhQ/usr/bin/samrewritten: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.33' not found (required by /tmp/.mount_SamRewWjfjhQ/usr/bin/../lib/libgssapi_krb5.so.2)
/tmp/.mount_SamRewWjfjhQ/usr/bin/samrewritten: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.33' not found (required by /tmp/.mount_SamRewWjfjhQ/usr/bin/../lib/libldap-2.5.so.0)
/tmp/.mount_SamRewWjfjhQ/usr/bin/samrewritten: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.34' not found (required by /tmp/.mount_SamRewWjfjhQ/usr/bin/../lib/libldap-2.5.so.0)
/tmp/.mount_SamRewWjfjhQ/usr/bin/samrewritten: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.34' not found (required by /tmp/.mount_SamRewWjfjhQ/usr/bin/../lib/libzstd.so.1)
/tmp/.mount_SamRewWjfjhQ/usr/bin/samrewritten: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.34' not found (required by /tmp/.mount_SamRewWjfjhQ/usr/bin/../lib/libcairo.so.2)
/tmp/.mount_SamRewWjfjhQ/usr/bin/samrewritten: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.34' not found (required by /tmp/.mount_SamRewWjfjhQ/usr/bin/../lib/libepoxy.so.0)
/tmp/.mount_SamRewWjfjhQ/usr/bin/samrewritten: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.33' not found (required by /tmp/.mount_SamRewWjfjhQ/usr/bin/../lib/libxkbcommon.so.0)
/tmp/.mount_SamRewWjfjhQ/usr/bin/samrewritten: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.32' not found (required by /tmp/.mount_SamRewWjfjhQ/usr/bin/../lib/libunistring.so.2)
/tmp/.mount_SamRewWjfjhQ/usr/bin/samrewritten: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.34' not found (required by /tmp/.mount_SamRewWjfjhQ/usr/bin/../lib/libunistring.so.2)
/tmp/.mount_SamRewWjfjhQ/usr/bin/samrewritten: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.33' not found (required by /tmp/.mount_SamRewWjfjhQ/usr/bin/../lib/libgnutls.so.30)
/tmp/.mount_SamRewWjfjhQ/usr/bin/samrewritten: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.34' not found (required by /tmp/.mount_SamRewWjfjhQ/usr/bin/../lib/libgnutls.so.30)
/tmp/.mount_SamRewWjfjhQ/usr/bin/samrewritten: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.33' not found (required by /tmp/.mount_SamRewWjfjhQ/usr/bin/../lib/libkrb5.so.3)
/tmp/.mount_SamRewWjfjhQ/usr/bin/samrewritten: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.34' not found (required by /tmp/.mount_SamRewWjfjhQ/usr/bin/../lib/libkrb5.so.3)
/tmp/.mount_SamRewWjfjhQ/usr/bin/samrewritten: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.33' not found (required by /tmp/.mount_SamRewWjfjhQ/usr/bin/../lib/libk5crypto.so.3)
/tmp/.mount_SamRewWjfjhQ/usr/bin/samrewritten: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.33' not found (required by /tmp/.mount_SamRewWjfjhQ/usr/bin/../lib/libkrb5support.so.0)
/tmp/.mount_SamRewWjfjhQ/usr/bin/samrewritten: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.34' not found (required by /tmp/.mount_SamRewWjfjhQ/usr/bin/../lib/libkrb5support.so.0)
/tmp/.mount_SamRewWjfjhQ/usr/bin/samrewritten: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.34' not found (required by /tmp/.mount_SamRewWjfjhQ/usr/bin/../lib/libsasl2.so.2)
/tmp/.mount_SamRewWjfjhQ/usr/bin/samrewritten: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.33' not found (required by /tmp/.mount_SamRewWjfjhQ/usr/bin/../lib/libdbus-1.so.3)
/tmp/.mount_SamRewWjfjhQ/usr/bin/samrewritten: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.34' not found (required by /tmp/.mount_SamRewWjfjhQ/usr/bin/../lib/libdbus-1.so.3)
/tmp/.mount_SamRewWjfjhQ/usr/bin/samrewritten: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.33' not found (required by /tmp/.mount_SamRewWjfjhQ/usr/bin/../lib/libsystemd.so.0)
/tmp/.mount_SamRewWjfjhQ/usr/bin/samrewritten: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.32' not found (required by /tmp/.mount_SamRewWjfjhQ/usr/bin/../lib/libsystemd.so.0)
/tmp/.mount_SamRewWjfjhQ/usr/bin/samrewritten: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.34' not found (required by /tmp/.mount_SamRewWjfjhQ/usr/bin/../lib/libsystemd.so.0)
/tmp/.mount_SamRewWjfjhQ/usr/bin/samrewritten: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.32' not found (required by /tmp/.mount_SamRewWjfjhQ/usr/bin/../lib/liblzma.so.5)
/tmp/.mount_SamRewWjfjhQ/usr/bin/samrewritten: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.34' not found (required by /tmp/.mount_SamRewWjfjhQ/usr/bin/../lib/liblzma.so.5)
/tmp/.mount_SamRewWjfjhQ/usr/bin/samrewritten: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.33' not found (required by /tmp/.mount_SamRewWjfjhQ/usr/bin/../lib/libcap.so.2)
/tmp/.mount_SamRewWjfjhQ/usr/bin/samrewritten: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.33' not found (required by /tmp/.mount_SamRewWjfjhQ/usr/bin/../lib/libgcrypt.so.20)

I guess we need to fix the packaging before releasing..

@DeadStarlin
Copy link
Contributor Author

DeadStarlin commented Dec 5, 2022

Hmm, I just checked content of package (since it's zipped) and got all dependencies there in /usr/lib... I will set up a clean VM with Ubuntu to check what is going on.

Update - I installed fresh Ubuntu 22 LTS on VM, installed Steam and fuse with lib to run appimage, and it works. Maybe GitHub did something to file: https://www44.zippyshare.com/v/entzQ1xo/file.html

image

image

@PaulCombal
Copy link
Owner

It has to do with the version of glibc installed on the host system. I don't have extensive knowledge about how it works when it comes to building and distributing AppImages but if yours only works on newer systems only, then I think it's better to pin it in a popular issue. Let's make a release only when the CI/CD pipelines are correctly set up.

@PaulCombal PaulCombal mentioned this pull request Dec 6, 2022
@RealCelticGamer
Copy link

SamRewritten-b50966a-x86_64.zip Here is AppImage - I had to zip it cause GitHub won't allow me to attach it otherwise.

Will this work on the Steam Deck?

@ZeraX7
Copy link

ZeraX7 commented Mar 28, 2024

SamRewritten-b50966a-x86_64.zip Here is AppImage - I had to zip it cause GitHub won't allow me to attach it otherwise.

Hi. This used to work flawlessly on Steam Deck until yesterday. Now it doesn't work at all.
Could you please make a new Appimage? I'm not really good with Linux.

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.

None yet

4 participants