Skip to content

onedrive-gui: Update to v1.3.0#8525

Open
maikwoehl wants to merge 1 commit into
getsolus:mainfrom
maikwoehl:onedrive-gui-update
Open

onedrive-gui: Update to v1.3.0#8525
maikwoehl wants to merge 1 commit into
getsolus:mainfrom
maikwoehl:onedrive-gui-update

Conversation

@maikwoehl
Copy link
Copy Markdown
Contributor

@maikwoehl maikwoehl commented Apr 12, 2026

Summary

Resolves #8199.

Bugfixes:

  • Fix non-working sync after big delete
  • Fix missing resync dialog when full-resync is enabled in the config
  • Fix detection of already running onedrive process

Enhancements:

  • Option for autostart on login
  • File transfer UI shows timestamp in relative format
  • System tray icon shows synchronization status

Full release notes:

Test Plan

  • Launched application
  • Created profile
  • Onedrive daemon was detected
  • Exercised settings dialogs

Checklist

  • Package was built and tested against unstable
  • This change could gainfully be listed in the weekly sync notes once merged

**Summary**

This PR resolves software update request getsolus#8199.

Bugfixes:

- Fix non-working sync after big delete
- Fix missing resync dialog when full-resync is enabled in the config
- Fix detection of already running onedrive process

Enhancements:

- Option for autostart on login
- File transfer UI shows timestamp in relative format
- System tray icon shows synchronization status

Full release notes:

- [1.3.0](https://github.com/bpozdena/OneDriveGUI/releases/tag/v1.3.0)

Signed-off-by: Maik Wöhl <[email protected]>
@EbonJaeger
Copy link
Copy Markdown
Member

I'm not a big fan of going from multiple smaller, targeted patches to one big megapatch. Keeping changes to smaller patches that only do a single thing, one, is a lot easier to review, and two, is more maintainable because instead of having to rebase a single huge patch, you might only have to do a couple of small ones.

Copy link
Copy Markdown
Member

@EbonJaeger EbonJaeger left a comment

Choose a reason for hiding this comment

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

This should also have an AppStream metainfo file, since it is a desktop application that ships a .desktop file. If the upstream project doesn't have one, we'll have to create one; there are a bunch of packages in the repository that do this.

install -D -m00644 src/onedrive_gui/resources/OneDriveGUI.desktop $installdir/usr/share/applications/OneDriveGUI.desktop
install -D -m00644 src/onedrive_gui/resources/images/icons8-clouds-48.png $installdir/usr/share/icons/onedrive-gui.png
install -D -m00644 src/onedrive_gui/resources/images/OneDriveGUI.png $installdir/usr/share/icons/OneDriveGUI.png
install -D -m00644 src/onedrive_gui/resources/images/OneDriveGUI.ico $installdir/usr/share/icons/OneDriveGUI.ico
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Do we need an .ico file on Linux?

%python3_install
install -D -m00644 src/onedrive_gui/resources/OneDriveGUI.desktop $installdir/usr/share/applications/OneDriveGUI.desktop
install -D -m00644 src/onedrive_gui/resources/images/icons8-clouds-48.png $installdir/usr/share/icons/onedrive-gui.png
install -D -m00644 src/onedrive_gui/resources/images/OneDriveGUI.png $installdir/usr/share/icons/OneDriveGUI.png
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This is in the wrong place. It should be in ${installdir}/usr/share/icons/hicolor/<size>/apps/, where size is whatever size the icon is.

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

Labels

None yet

Projects

Status: Triage

Development

Successfully merging this pull request may close these issues.

onedrive-gui

2 participants