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

[BUG] Port extension to GNOME 45 #213

Closed
neffo opened this issue Sep 7, 2023 · 21 comments · Fixed by #216 or #223
Closed

[BUG] Port extension to GNOME 45 #213

neffo opened this issue Sep 7, 2023 · 21 comments · Fixed by #216 or #223
Labels

Comments

@neffo
Copy link
Owner

neffo commented Sep 7, 2023

This GNOME 45 will likely break compatibility with earlier versions of GNOME, and supporting GNOME 45 will likely require some remediation to fix in this extension:

https://blogs.gnome.org/shell-dev/2023/09/02/extensions-in-gnome-45/

@neffo neffo added the bug label Sep 7, 2023
@ttys3
Copy link

ttys3 commented Sep 25, 2023

https://gjs.guide/extensions/upgrading/gnome-shell-45.html

@Spinnekop
Copy link

Missing my daily bing!

@neffo
Copy link
Owner Author

neffo commented Sep 25, 2023

Yeah, I've been putting this off (I'm on Ubuntu 23.04, so haven't moved to GNOME 45). Looking at it though, thanks for your patience!

@scottbeamer
Copy link

Yeah, I've been putting this off (I'm on Ubuntu 23.04, so haven't moved to GNOME 45). Looking at it though, thanks for your patience!

This is why the good lord invented virtual machines. :)

Ubuntu 23.10 Beta and Fedora 39 Beta are out, both have GNOME 45 and both are pretty stable.

@IBBoard
Copy link
Contributor

IBBoard commented Oct 9, 2023

openSUSE Tumbleweed is also running Gnome 45 and is basically permanently stable (and a rolling release, and has btrfs snapshotting built-in with a good management tool!)

@c3101
Copy link

c3101 commented Oct 11, 2023

+1

@neffo
Copy link
Owner Author

neffo commented Oct 11, 2023

Apologies all been swamped with work, but will get this done asap. The wallpapers must flow.

@syahny
Copy link

syahny commented Oct 13, 2023

It doesn't work on GNOME 45, I know you're busy, but please check. Please fix it. Thank you.

@IBBoard
Copy link
Contributor

IBBoard commented Oct 13, 2023

Hey @SeokYoung-Ahn - just so you know, that comes across as a rude and demanding comment given what has already been posted.

Three weeks ago the developer said they were looking at the problem, which shows that they know it is a problem.

Just yesterday they said that they had been busy with work and said that they'd fix it as soon as possible (asap).

Lots of open source developers do this as a hobby. Pushing for things to go faster does not encourage people who are giving their free time and it is disrespectful of that time. And when they've already said why there is a delay then it's doubly disrespectful of their time.

.

If anyone else is really desperate to get Bing wallpapers while Neff updates the extension then search for "bing wallpaper archive" in the search engine of your choice and you'll find plenty of wallpaper sites that are archiving them. You may want to have an ad blocker installed before you visit most of them, though!

@ttys3
Copy link

ttys3 commented Oct 13, 2023

currently I use a workaround :

download executable from https://github.com/ttys3/Bismuth/releases/download/v1.3.0/bismuth-Linux-x86_64-musl.tar.gz

install the binary exeutable to /usr/local/bin/bismuth

and install the user cron job:

git clone https://github.com/ttys3/Bismuth.git
cd contribute/systemd
./install.sh

not perfect, but things works.

update: systemd cron not run the second day issue has been resolved.

@greyltc
Copy link

greyltc commented Oct 19, 2023

I've tested #215 and it solves this issue for me. I've rolled that patch into my package in the AUR for the time being to get the wallpapers flowing for Arch Linux users.

@odror
Copy link

odror commented Oct 21, 2023

That workaround does not work for me on arch. Gnome still ignores the extension because it is not compatible with 45

@neffo
Copy link
Owner Author

neffo commented Oct 21, 2023

Ok, I have a working version now #216 . Massive thanks to @mcardillo55 for doing the heavy lifting with PR #215 . Please give him all the thanks for this. I've brought over some small fixes as well (from this branch #203 ), but please let me know how this works.

There are a lot of things I want to bring over from the other branch, but I want to get this version live ASAP. Thanks all for your patience.

@neffo neffo linked a pull request Oct 21, 2023 that will close this issue
@ivanjx
Copy link

ivanjx commented Oct 21, 2023

it runs on gnome 45 but i cant seem to make it download the images

image

@neffo
Copy link
Owner Author

neffo commented Oct 21, 2023

Oct 21 20:29:41 selected image changed to: current
Oct 21 20:29:41 getCurrentImageIndex for 202310200700: 811
Oct 21 20:29:41 getCurrentImageIndex for 202310200700: 811
Oct 21 20:29:41 Downloading https://www.bing.com/th?id=OHR.PygmySloth_ROW1074539359_UHD.jpg to file:///mnt/2t_btr1/BingWallpaper/20231020-PygmySloth_ROW1074539359_UHD.jpg
Oct 21 20:29:41 Storing state as JSON: {"maxlongdate":202310200700,"title":"Pygmy three-toed sloth baby, Isla Escudo de Veraguas, Panama","explanation":"Bing Wallpaper of the Day for 2023-10-20","copyright":"© Suzi Eszterhas/Minden Pictures","longstartdate":"202310200700","imageinfolink":"https://www.bing.com/search?q=sloth+animal&form=hpcapt&filters=HpDate%3a%2220231020_0700%22","imageURL":"https://www.bing.com/th?id=OHR.PygmySloth_ROW1074539359_UHD.jpg","filename":"/mnt/2t_btr1/BingWallpaper/20231020-PygmySloth_ROW1074539359_UHD.jpg","favourite":false}
Oct 21 20:29:41 getCurrentImageIndex for 202310200700: 811
Oct 21 20:29:41 selected image changed to: PygmySloth_ROW1074539359
Oct 21 20:29:41 /th?id=OHR.PygmySloth_ROW1074539359
Oct 21 20:29:41 Downloading https://www.bing.com/th?id=OHR.PygmySloth_ROW1074539359_UHD.jpg to file:///mnt/2t_btr1/BingWallpaper/20231020-PygmySloth_ROW1074539359_UHD.jpg
Oct 21 20:29:41 Storing state as JSON: {"maxlongdate":202310200700,"title":"Pygmy three-toed sloth baby, Isla Escudo de Veraguas, Panama","explanation":"Bing Wallpaper of the Day for 2023-10-20","copyright":"© Suzi Eszterhas/Minden Pictures","longstartdate":"202310200700","imageinfolink":"https://www.bing.com/search?q=sloth+animal&form=hpcapt&filters=HpDate%3a%2220231020_0700%22","imageURL":"https://www.bing.com/th?id=OHR.PygmySloth_ROW1074539359_UHD.jpg","filename":"/mnt/2t_btr1/BingWallpaper/20231020-PygmySloth_ROW1074539359_UHD.jpg","favourite":false}
Oct 21 20:29:42 scale factor: 2
Oct 21 20:29:42 image set to : /mnt/2t_btr1/BingWallpaper/20231020-PygmySloth_ROW1074539359_UHD.jpg
Oct 21 20:29:42 Download successful

Confirmed, I broke something but should be fixed now @ivanjx , if you are grabbing directly form the PR do you want to try again?

@ivanjx
Copy link

ivanjx commented Oct 21, 2023

works fine now. thanks @neffo

@neffo
Copy link
Owner Author

neffo commented Oct 21, 2023

thanks @ivanjx

I'll push this version to EGO, noting it probably has some bugs lingering.

There is a rewrite of the shuffle code in another branch, but I'm not 100% happy with it. So will leave that as is.

@IBBoard
Copy link
Contributor

IBBoard commented Oct 30, 2023

I'm sure it'll fix itself when EGO approves the newest version, but is anyone else finding that Gnome Extensions keeps "upgrading" to the old version of the extension (which then fails to load).

Annoyingly, it seems to do it without asking and just says "extensions will be upgraded when you log in again".

@neffo
Copy link
Owner Author

neffo commented Oct 30, 2023

Yeah it'll keep downgrading, presumably to remove?

If you set the version to the same as the current approved (45) within metadata.json it may work without downgrading.

@IBBoard
Copy link
Contributor

IBBoard commented Oct 30, 2023

If you set the version to the same as the current approved (45) within metadata.json it may work without downgrading.

Yep, that seems to have done it!

Quick pointer for others, though: If it has already given you the "will be upgraded" message then log out, log in, let it "upgrade", re-extract, edit the version, log out and in to enable it, and it now thinks it has v45 but it's the version that works.

If you don't do that then you'll edit it to v45 and then it'll replace everything when you log out and back in!

@neffo neffo reopened this Nov 2, 2023
@neffo neffo linked a pull request Nov 6, 2023 that will close this issue
13 tasks
@neffo neffo closed this as completed in #223 Nov 6, 2023
@bklaas
Copy link

bklaas commented Nov 6, 2023

Confirmed working here on Endeavour(Arch) running Gnome 45
🥇 👍

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