[Review] Wizz RSS
John Vrbanac
john.vrbanac at linux.com
Sun Jul 15 19:44:30 UTC 2012
Hello all, I have reviewed Wizz RSS for issues. I have fixed a number of
packaging and media location issues within the following branch:
https://code.launchpad.net/~john.vrbanac/ubuntu-app-reviews/wizzrss
While the application installs to extras, currently, the application
will not open without my branch changes. See my comment to the author
regarding this.
Question for author:
I did have a question about the feedparser python library that is
included with the app. I know that there is a different (newer?) version
in the ubuntu archive. I was wondering if there was a particular reason
for including this file and not using the version in the archives?
Comment to author:
I noticed that in WizzrssWindow you were trying to grab your images
directly from a directory in /usr/share/wizzrss/media/... . Because the
application is being installed to /opt/extras.ubuntu.com/ those files
did not exist in /usr/share anymore. As a result the application
wouldn't even start because it couldn't find those files, I went ahead
and fixed the references to use a modified get_media_file() helper
function instead of hard coded paths. You'll find those changes in my
branch as well.
Cheers!
--
John Vrbanac
More information about the App-review-board
mailing list