flathub / org.DolphinEmu.dolphin-emu

https://flathub.org/apps/details/org.DolphinEmu.dolphin-emu
17 stars 18 forks source link

Wrong version number on beta branches #211

Closed ApexBelowZero closed 2 months ago

ApexBelowZero commented 2 months ago

When installing from the beta-2407-115 branch (the latest one), the shortened commit hash string "0c18393" is used for the application version instead of "2407-115". This makes it impossible for clients of the same revision (or anyone else except those who installed the same beta via flatpack) to connect/host via NetPlay.

ApexBelowZero commented 2 months ago

IMG_20240731_171031_427 20240731_213410

How it should look - how it actually is. If I'm installing it on the steamdeck incorrectly, I would really appreciate some instructions.

cpba commented 2 months ago

It's not a steamdeck specific issue, I've noticed this too when trying out the beta builds. I wanted to look into it but haven't found the time yet.

I the mean time, I recommend you use the stable branch for netplay. This issue isn't present there and it's much easier to sync versions with other people as it doesn't update constantly like the de releases do.

El dj., 1 d’ag. 2024, 6:46, Aleksandr Belozyorov @.***> va escriure:

IMG_20240731_171031_427.jpg (view on web) https://github.com/user-attachments/assets/369c325a-9bcd-415d-86bb-4598b76c843a 20240731_213410.jpg (view on web) https://github.com/user-attachments/assets/1cb028ba-07ef-4250-9bee-b037396a2443

How it should look - how it actually is. If I'm installing it on the steamdeck incorrectly, I would really appreciate some instructions.

— Reply to this email directly, view it on GitHub https://github.com/flathub/org.DolphinEmu.dolphin-emu/issues/211#issuecomment-2262022998, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACZPHZDAY7TMRLHNXOZ2EWDZPG4TVAVCNFSM6AAAAABLZMDNG6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENRSGAZDEOJZHA . You are receiving this because you are subscribed to this thread.Message ID: @.***>

ApexBelowZero commented 2 months ago

After updating from flatpack, unfortunately, you can't roll back to a stable version, because retroachievments bans the old version after switching to a newer one:

https://retroachievements.org/viewtopic.php?t=27283&c=232820#232820 https://retroachievements.org/viewtopic.php?t=27292&c=232939#232939

So everyone, who has updated dolphin to beta version from flatpack will lose an abiity to recieve achievments on previous stable one.

It's not a steamdeck specific issue, I've noticed this too when trying out the beta builds. I wanted to look into it but haven't found the time yet. I the mean time, I recommend you use the stable branch for netplay. This issue isn't present there and it's much easier to sync versions with other people as it doesn't update constantly like the de releases do. El dj., 1 d’ag. 2024, 6:46, Aleksandr Belozyorov @.> va escriure: IMG_20240731_171031_427.jpg (view on web) https://github.com/user-attachments/assets/369c325a-9bcd-415d-86bb-4598b76c843a 20240731_213410.jpg (view on web) https://github.com/user-attachments/assets/1cb028ba-07ef-4250-9bee-b037396a2443 How it should look - how it actually is. If I'm installing it on the steamdeck incorrectly, I would really appreciate some instructions. — Reply to this email directly, view it on GitHub <#211 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACZPHZDAY7TMRLHNXOZ2EWDZPG4TVAVCNFSM6AAAAABLZMDNG6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENRSGAZDEOJZHA . You are receiving this because you are subscribed to this thread.Message ID: @.>

cpba commented 2 months ago

Do you need both achievements and netplay at the same time?

If you don't need both at the same time, remember that you can install both the stable and beta releases and switch between them at any moment with this command:

flatpak make-current org.DolphinEmu.dolphin-emu stable

or if you want to go back to beta:

flatpak make-current org.DolphinEmu.dolphin-emu beta

cpba commented 2 months ago

I did a small test in #213 to check whether the downstream patches that we carry for xdg compatibility were causing the issue with the version string, but it still happens without them so the patches aren't to blame.

Must be something else in the build process, but I don't have time to look into it right now.

ColinKinloch commented 2 months ago

The problem is that SCM_DESC_STR / DOLPHIN_WC_DESCRIBE is taken from git describe --always --long which prints {tag}-{port tag commit count}. The issue is that flatpak-builder does a shallow clone of the target commit which means there is no tag.

The solution for the dev branch is to add disable-shallow-clone: true to the dolphin git source. However the dolphin repo has a lot of commits so this takes a long time. An alternative would be to manually set the version number for each dev build or just wait for achievements to become stable.

I personally intend to ignore the problem until it goes away. 🏃

cpba commented 2 months ago

I had no idea you lost the tag when doing a shallow clone.

We're already patching the version number so I figured a bit more wouldn't hurt.

@ApexBelowZero I'm merging a patch to produce the version string that you want, it should be up on flathub shortly. Feel free to reopen if it doesn't actually fix your issue.