neffo / bing-wallpaper-gnome-extension

GNOME shell extension that syncs your desktop & lock screen wallpaper to Microsoft Bing's Image of the Day.
https://extensions.gnome.org/extension/1262/bing-wallpaper-changer/
GNU General Public License v3.0
299 stars 56 forks source link

[BUG] Port extension to GNOME 45 #213

Closed neffo closed 1 year ago

neffo commented 1 year ago

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/

ttys3 commented 1 year ago

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

Spinnekop commented 1 year ago

Missing my daily bing!

neffo commented 1 year ago

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 commented 1 year ago

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 commented 1 year ago

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 commented 1 year ago

+1

neffo commented 1 year ago

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

syahny commented 1 year ago

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

IBBoard commented 1 year ago

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 commented 1 year ago

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 commented 1 year ago

I've tested https://github.com/neffo/bing-wallpaper-gnome-extension/pull/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 commented 1 year ago

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

neffo commented 1 year ago

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.

ivanjx commented 1 year ago

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

image

neffo commented 1 year ago
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 commented 1 year ago

works fine now. thanks @neffo

neffo commented 1 year ago

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 commented 1 year ago

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 commented 1 year ago

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 commented 1 year ago

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!

bklaas commented 1 year ago

Confirmed working here on Endeavour(Arch) running Gnome 45 :1st_place_medal: :+1: