Skip to content

Linux version (flatpak & snap) can not save files #498

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

Closed
konsumschaf opened this issue Nov 18, 2024 · 5 comments
Closed

Linux version (flatpak & snap) can not save files #498

konsumschaf opened this issue Nov 18, 2024 · 5 comments

Comments

@konsumschaf
Copy link

Description
The Linux Version is not able to save a file

To Reproduce
Open Powertab, File -> New, File -> Save, enter some name, get error message:

Error saving file: filesystem error: cannot rename: Ungültiger Link über Gerätegrenzen hinweg [/tmp/test.pt2.new] [/run/user/1000/doc/cef48855/test.pt2]

Additional Info
Tried to alter the permissions within flatpak (changed file permissions to HOST) - did not change the behaviour

Version
Power Tab Editor 2.0.20 (v2.0.20-0-geff05f0)
Kubuntu 22.04 and Kubuntu 22.10

@cameronwhite
Copy link
Member

Thanks for the report! I'll try to get a fix out for this ASAP
It looks like it might be caused by the temp folder appearing as a different filesystem volume when under flatpak, rather than a permission issue

  • The filesystem::rename() added in 62298b1 likely needs to fall back to doing a copy + remove if it fails
  • Rather than using the temp folder, under snap/flatpak it's also probably safer to use the application's user data folder which we're also using now for auto-backups (7280ee7)

@konsumschaf
Copy link
Author

Thanks a lot! I would like to test this, but the actions only offer Win & Mac build. Where can I get the Linux Builds?

@cameronwhite
Copy link
Member

Yeah, the Github builds just create Windows and Mac packages. If you're able to test with snap, I've pushed an updated package (version 2.0.21-beta1 to the latest/beta channel)

@konsumschaf
Copy link
Author

OK, thanks, I can confirm that saving is working now!

@cameronwhite
Copy link
Member

Great, thanks for testing!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants