-
Notifications
You must be signed in to change notification settings - Fork 17
Seperate out AppImage build from Tauri #103
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
base: develop
Are you sure you want to change the base?
Conversation
the existing one is broken, and i think it might be tauri's fault.
tauri was overcomplicating this. let's make the appimage independent of that
this should build the appimage without tauri's involvement
installing by $USER caused issues when i did it so i've fixed it also added $PWD because ./ was lazy
lol woops this caused a typo that cascaded into an appimage that failed to find the executable
|
OH SHIT i left appimagetool in |
DecDuck
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'll pull and test if it builds locally on aarch64
|
more pressing, how do i sign these commits? i might have to literally redo all this work over that |
|
You can read about it here: https://docs.github.com/en/authentication/managing-commit-signature-verification/signing-commits |
|
it doesn't say how to sign existing commits, only new commits |
|
|
|
there we go! now everything's signed |
|
Okay, I had to fix some capitalization in the script, but works a treat on aarch64 Ubuntu (distrobox). |
|
Also I think you missed signing the first commit |
|
i did |
the existing one is broken, and i think it might be tauri's fault.
|
gonna tap out for the night, it's 1:30 am where i'm at, same as yesterday haha |
|
the newest build doesn't work on my steamdeck. a few libs are missing is all |
|
we might literally have better luck manually keeping track of which libs are needed for this rather than leaning on CLI tools |
|
alright, hopping back on this. i see there's some conflicts to resolve, so lemme do that first |
|
Huh. apparently, |
* fix: potential download fixes * fix: show installed games not on remote * fix: more download_logic error handling * partial: move to async * feat: interactivity improvements * feat: v2 download API * fix: download seek offsets * fix: clippy * fix: apply clippy suggestion * fix: performance improvements starting up download * fix: finished bucket file * fix: ui tweaks and fixes * fix: revert version to 0.3.2 * fix: clippy
|
I think this should do it (pushed the linking stuff), and I fixed the conflicts. |
|
so this works on my arch system, but not on my steam deck for some reason. |
|
removing all kinds of appindicator installed on my arch system, then running the appimage? just fine. |
|
Is this in desktop mode or "gaming mode" (idk what it's called) on the Steam Deck? What desktop do you use on your desktop and Steam Deck? |
|
on my desktop, i use xfce (and the x11 display server,) and in its desktop mode, the steam deck runs kde by default, with its x11 display server. |
|
i am so sorry for the wait on this, i literally just didn't notice |
|
Maybe it's Wayland vs not? I tried Tauri's build without stripping, can you try that? Should be attached to the v0.3.2 release. We may have to force either atayana-appindicator (newer) or libappindicator3 (older), whichever one isn't broken. |
|
it looks like you might not have understood what i said. steam's desktop mode uses x11, same as my main desktop. i haven't been using gaming mode to test it in a more consistent environment. remember, the specific error is that it's not finding libayatana-appindicator. |
|
The package looks for both appindicator libraries on the system, in this order:
My suggestion was to include the other one (libappindicator3) to see if that's what the Steam Deck wants. The Steam Deck may be out of date/KDE might be using an old version of the library. |
|
ohhh okay |
|
ok, so it finds the library, but then it segfaults: |
|
Was this built with this PR or the Tauri release? |
|
this PR |

tauri's shit, it makes me watch all the bad vine compilations with all the bad vines
i got a functional appimage out of doing it like this instead