Hide Forgot
Hello, I know the package just got in: I'm really sorry to be a buzz kill :( I just tried the F16 build from koji: http://koji.fedoraproject.org/koji/buildinfo?buildID=311652 When I run it, according to the Readme file, I get the following error: 22:46 *** <(^) TWIRSSI v2.5.1 22:46 *** (_(\ http://twirssi.com/ for full docs 22:46 *** || ` Log in with /twitter_login, send updates with /tweet 22:46 *** Loading WWW::Shorten::TinyURL... 22:46 ERROR: Failed to load WWW::Shorten::TinyURL - either clear 22:46 ERROR: short_url_provider or install the CPAN module The readme does say that you need WWW::Shortner::TinyURL installed, but is the user expected to install it from CPAN? Shouldn't this be packaged for fedora and installed as a Requires? It would improve the user experience IMO. (Users dont like errors, it freaks us out ;)) Another missing Requires seems to be LWP::Protocol::https: 23:01 ERROR: Failed to get OAuth authorization_url: GET https://api.twitter.com/oauth/request_token failed: 501 Protocol scheme 'https' is not supported (LWP::Protocol::https not installed) at /usr/share/irssi/scripts/twirssi.pl line 896 This one's in the repos already, and just needs to be added as Requires: perl-LWP-Protocol-https It's working now, but I look forward to these nitpicks :) Thanks for packaging it up, Ankur
You're right Ankur, when I read the code it didn't look like WWW::Shorten::* was mandatory but I realized that's because URL shortening is an "optional" feature that happens to be turned on by default. I'll be putting in a review request for perl-WWW-Shorten shortly. And thanks for catching the dep on LWP::Protocol:https; one of those things that's not immediately apparent from the code.
Good catches, Ankur! I suppose I should try using the packages I review ;)
(In reply to comment #2) > Good catches, Ankur! > > I suppose I should try using the packages I review ;) Hehe, it isn't always possible :) (In reply to comment #1) > You're right Ankur, when I read the code it didn't look like WWW::Shorten::* > was mandatory but I realized that's because URL shortening is an "optional" > feature that happens to be turned on by default. I'll be putting in a review > request for perl-WWW-Shorten shortly. > > And thanks for catching the dep on LWP::Protocol:https; one of those things > that's not immediately apparent from the code. Sure :) I really look forward to the fixes. I'll give karma as soon as the packages hit -testing. Thanks, Ankur
twirssi-2.5.1-3.fc17 has been submitted as an update for Fedora 17. https://admin.fedoraproject.org/updates/twirssi-2.5.1-3.fc17
twirssi-2.5.1-3.fc16 has been submitted as an update for Fedora 16. https://admin.fedoraproject.org/updates/twirssi-2.5.1-3.fc16
twirssi-2.5.1-3.fc15 has been submitted as an update for Fedora 15. https://admin.fedoraproject.org/updates/twirssi-2.5.1-3.fc15
Package twirssi-2.5.1-3.fc17: * should fix your issue, * was pushed to the Fedora 17 testing repository, * should be available at your local mirror within two days. Update it with: # su -c 'yum update --enablerepo=updates-testing twirssi-2.5.1-3.fc17' as soon as you are able to. Please go to the following url: https://admin.fedoraproject.org/updates/FEDORA-2012-5551/twirssi-2.5.1-3.fc17 then log in and leave karma (feedback).
twirssi-2.5.1-3.fc17 has been pushed to the Fedora 17 stable repository. If problems still persist, please make note of it in this bug report.
twirssi-2.5.1-3.fc15 has been pushed to the Fedora 15 stable repository. If problems still persist, please make note of it in this bug report.
twirssi-2.5.1-3.fc16 has been pushed to the Fedora 16 stable repository. If problems still persist, please make note of it in this bug report.