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

tauri: migrating to V2 #585

Draft
wants to merge 9 commits into
base: main
Choose a base branch
from
Draft

tauri: migrating to V2 #585

wants to merge 9 commits into from

Conversation

xTVaser
Copy link
Member

@xTVaser xTVaser commented Oct 5, 2024

Fixes #529 and #224
Fixes #581

May potentially solve #581 as it uses a newer version of webkit2gtk -- im currently convinced the UI rendering bug is the fault of webkitgtk2, and i tested last night that on newer versions (i used GNOME Web) it doesn't manifest.

Before merging:

This is blocked for the following reasons:

@trippjoe
Copy link
Member

Both linux related tauri bugs have been closed.

@Milkfat32
Copy link
Contributor

tauri-apps/tauri#10626 (bad webkit version out there? upstream ubuntu bug report https://bugs.launchpad.net/ubuntu/+source/webkit2gtk/+bug/2081166)

If the Linux version is to be released only as a Flatpak, this is not an issue as the webkit version is controlled in the Flatpak sandbox. This is done through the runtime sdk selection and not the system libraries.

@xTVaser
Copy link
Member Author

xTVaser commented Nov 18, 2024

It was an issue because you couldn't develop or debug the app on ubuntu linux.

Milkfat32 and others added 5 commits January 11, 2025 13:17
Updated tauri packages
Added tauri.linux.conf to disable tauri updater

Also went with default flatpak install directory data/ folder for
installation. Button on splash is replaced by Continue after locale
select.
Removed the Installation Directory setting for Linux build. I didn't
think it should be able to be changed? If someone changed it to some
directory in /home/ without giving access through flatseal/terminal it
would throw errors so I assumed this was the best solution

Added icons/desktop file since that should be upstream according to
flathub maintainers. Once v2 is released I will update the flathub repo,
since flathub bot builds from the git releases on this repo. The appdata
file does not work as intended yet in the ci, have only gotten that to
update properly on the flathub repo

Ping me on Discord and I will create a request to add Vaser/tripp as
maintainer, since I don't own the repo on flathub.

---------

Co-authored-by: Milkfat32 <[email protected]>
Comment on lines +1 to +7
{
"plugins": {
"updater": {
"active": false
}
}
}
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why we we disabling the updater in linux?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updates to flatpak is handled by the flatpak package manager. For diy distros like gentoo/arch this is done with flatpak update, for others this is usually in some software manager.

Comment on lines +4 to +8
"targets": [
"msi",
"app",
"dmg"
],
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

has no flatpak configuration, looks like we need to build the deb. CI will have to similarly be updated to accommodate the new release asset.

Copy link
Contributor

@Milkfat32 Milkfat32 Jan 12, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That should not be necessary. Flatpaks in flathub are built in the flathub repository, so the launcher has its own repository in flathub. I have a workflow that creates the offline mirrors for the rust and npm dependencies and I build from source. I believe building a flatpak file outside the repository simply creates a .flatpakref file that users have to manually install. In the flathub repository it shows up in software managers.

Copy link
Member Author

@xTVaser xTVaser Jan 12, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't really like the idea of us 100% depending on flathub / external repos to handle our linux distribution. From working on / observing PCSX2 they follow a model I much prefer, where they build the flatpak and publish it to flathub, that means the user can either download the flatpak directly from the pcsx2 website, or via flathub.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Understandable. I did not know it was possible to build and publish outside the Flathub repositories. It does seem like a lot of extra work though, and I am not entirely sure how publishing to Flathub works. It seems like you need some kind of access token, which I don't know how to get other than perhaps asking for it? The documentation on this is very sparse.

I don't think it is that big a benefit to download the flatpak from the website, as users are generally conditioned to only seek software from repositories or through software managers

In my opinion this should be considered for the future, like it also seems the PCSX2 devs did as they moved from the flathub github repositories to self publishing, as to not stall the v2 branch for too long as I would have to do a lot more digging and asking around

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.

UI moves when clicked on Migrate to Tauri 2.0
3 participants