largestgithubuseronearth / addwater

Installer for the Firefox GNOME theme
GNU General Public License v3.0
18 stars 2 forks source link

Unable to set flatpak theme #15

Open RedAuburn opened 10 hours ago

RedAuburn commented 10 hours ago

To Reproduce Have Firefox installed, both system (dnf) and flatpak. Change preference from base to flatpak try to apply the theme get a message "Installation failed."

Expected behavior theme is set for the flatpak install of firefox.

Please include the log file from the day it happened

harry@fedora:~/Documents$ flatpak run dev.qwery.AddWater 
/var/home/harry/.var/app/dev.qwery.AddWater/cache/add-water already exists. Skipped.
/var/home/harry/.var/app/dev.qwery.AddWater/cache/add-water/downloads already exists. Skipped.
/var/home/harry/.var/app/dev.qwery.AddWater/cache/add-water/logs already exists. Skipped.
All paths initialized.
-------------------------
[INFO] addwater.apps.firefox.firefox_details — 01:01 || Setting Firefox data path: /var/home/harry/.mozilla/firefox/
[INFO] addwater.apps.firefox.firefox_details — 01:01 || Valid path. Done.
[INFO] addwater.backend — 01:01 || Backend created for Firefox
[INFO] addwater.components.online — 01:01 || Checking for updates...
[INFO] addwater.components.online — 01:01 || no update available
[INFO] addwater.preferences — 01:01 || Preferences Window activated
[WARNING] addwater.preferences — 01:01 || Autofind paths disabled
[INFO] addwater.preferences — 01:01 || User specified path: /var/home/harry/.var/app/org.mozilla.firefox/.mozilla/firefox/
[INFO] addwater.apps.firefox.firefox_details — 01:01 || Setting Firefox data path: /var/home/harry/.var/app/org.mozilla.firefox/.mozilla/firefox/
[INFO] addwater.apps.firefox.firefox_details — 01:01 || Valid path. Done.
[INFO] addwater.backend — 01:02 || beginning installation...
[INFO] addwater.components.install — 01:02 || kicking off install...
[ERROR] addwater.components.install — 01:02 || Install failed. can't find theme path OR profile path.
[INFO] addwater.backend — 01:02 || install process completed

Technical Details (please complete the following information):

largestgithubuseronearth commented 6 hours ago

Sorry about that. After looking into it, it seems the Package switcher isn't properly resetting the interface. So when you click apply, it's still trying to install to the base package but failing because the paths don't match up.

I will work on this tonight and push a fix tomorrow morning.

Thanks for taking the time to report this.

largestgithubuseronearth commented 3 hours ago

Status update: The process is a bit more involved than I expected and will require refactoring a couple of important pieces of code. Expect an update within the next couple of days.

Sorry again, and thanks for your patience.