https://bugzilla.redhat.com/show_bug.cgi?id=1293630
Bug ID: 1293630 Summary: Review Request: wallch - A wallpaper changer Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nobody@fedoraproject.org Reporter: jjelen@redhat.com QA Contact: extras-qa@fedoraproject.org CC: package-review@lists.fedoraproject.org
Spec URL: https://jjelen.fedorapeople.org/wallch.spec SRPM URL: https://jjelen.fedorapeople.org/wallch-4.15-2.fc24.src.rpm Description: Wallch is a wallpaper changer. It doesn't simply change your desktop background with the wallpapers that you have in your hard disk, though. While it does that well by monitoring the folder that you have selected for new or deleted pictures, it has lots of features, like Picture of the day, Live Earth, Wallpaper Clocks and Live Website!
Fedora Account System Username: jjelen
Copr repo: https://copr.fedoraproject.org/coprs/jjelen/Wallch/
https://bugzilla.redhat.com/show_bug.cgi?id=1293630
--- Comment #1 from philip.worrall@googlemail.com --- Created attachment 1158061 --> https://bugzilla.redhat.com/attachment.cgi?id=1158061&action=edit generate tarball from launchpad
https://bugzilla.redhat.com/show_bug.cgi?id=1293630
philip.worrall@googlemail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |philip.worrall@googlemail.c | |om
--- Comment #2 from philip.worrall@googlemail.com --- Hello Jakub,
I am NOT an official packager but i have tried to do an informal review.
The application installs but just running it for a few minutes I noticed a couple of key bugs. For example the wallpaper of the day does not work because some of the websites images are fetched from now use https. Furthermore, I couldn't get wallpaper clocks to work either.
I think the upstream have stopped doing regular releases but you can find a more recent (non-released) revision in launchpad where a couple of these issues have been fixed. I started to modify the specfile (adding a generate-tarball script to checkout the latest revision from launchpad) but it wont build until your patches are rebased on the later revision i.e. the dependency on unity needs removing. I started to have a look at this but its not trivial because a lot of files have been moved around.
I have attached the script i wrote, id be happy to take another look if you revised your patches against the newer revision of this package.
https://bugzilla.redhat.com/show_bug.cgi?id=1293630
--- Comment #3 from Jakub Jelen jjelen@redhat.com --- Hello, thank you for comments. I didn't move on with the package since submitting review request, but I was using it on my own from copr (with more or less problems). I also contributed the change with conditionally removed Unity and qt4 compatibility (before I build it with qt5), which apply on the master well: https://code.launchpad.net/~wallch/wallpaper-changer/trunk/+activereviews
But it didn't get through to the "master" yet. They also get to official Ubuntu repositories and have a lot of bugs there. I built latest revision from bzr and updated the copr. But I am not sure about the release numbering when doing bazaar snapshots.
Also there was problem on Fedora 24 with missing cmath include (solved now)
I also updated the SPEC and SRPM files attached:
Spec URL: https://jjelen.fedorapeople.org/wallch.spec SRPM URL: https://jjelen.fedorapeople.org/wallch-4.15.r213-2.fc25.src.rpm
https://bugzilla.redhat.com/show_bug.cgi?id=1293630
--- Comment #4 from philip.worrall@googlemail.com --- Created attachment 1158537 --> https://bugzilla.redhat.com/attachment.cgi?id=1158537&action=edit wallch wikipedia-direct
https://bugzilla.redhat.com/show_bug.cgi?id=1293630
philip.worrall@googlemail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |jjelen@redhat.com Flags| |needinfo?(jjelen@redhat.com | |)
--- Comment #5 from philip.worrall@googlemail.com --- Hi Jakub,
I tried the new version but im still having problems getting the picture of the day.
After going through the code the potd are scraped from a dropbox file. the problem is that if there is no entry for todays date (as it is showing now https://dl.dropboxusercontent.com/u/257493884/potd) then it will complain there is no image.
Please see attached patch which adds functions to fetch the potd directly from wikipedia if the dropbox link is out-of-date i.e. no entry for today.
Also as i dont have a LP account please feel free to send this upstream.
https://bugzilla.redhat.com/show_bug.cgi?id=1293630
--- Comment #6 from philip.worrall@googlemail.com --- Upstream bug: https://bugs.launchpad.net/wallpaper-changer/+bug/1553897
https://bugzilla.redhat.com/show_bug.cgi?id=1293630
Jakub Jelen jjelen@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flags|needinfo?(jjelen@redhat.com | |) |
--- Comment #7 from Jakub Jelen jjelen@redhat.com --- Thank you for the patch. It looks nice, but I believe loading images directly from wikipedia is exactly the case the authors are trying to avoid, as described in the POTD page [1].
Problem is that the method of downloading the image once and uploading it to imgur is probably not as reliable as expected and this can not be solved by the end-user application.
As another possibility, would be to run another mirror with independent way of downloading and re-uploading pictures, but it would require also having the backend part opensource and run it in some other service (I can put it on mine VPS or somewhere else). I will propose this to the authors, but I am not sure when I will get a time to do that. It will depend on the complexity of the task itself.
Anyway POTD is now working fine.
[1] http://melloristudio.com/wallch/picture_of_the_day
https://bugzilla.redhat.com/show_bug.cgi?id=1293630
--- Comment #8 from philip.worrall@googlemail.com --- Oh I didn't see that page.
But remember the patch will only use wikipedia directly IIF the imageurl does not have an entry for today, so like a fallback case to prevent raising an error to the user. For today for example, its using imageurl because links were added for the 18th and 19th of May.
Writing the script should be easy assuming there is somewhere to host it and add a cronjob. Let me know if you need help with the former.
package-review@lists.fedoraproject.org