[Bug 1665303] Review Request: rtv - A simple terminal viewer for Reddit (Reddit Terminal Viewer)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



https://bugzilla.redhat.com/show_bug.cgi?id=1665303



--- Comment #7 from Ben Rosser <rosser.bjr@xxxxxxxxx> ---
Sorry for the delay on this... I did look into updating praw to 3.6.2 from
3.6.0. I discovered that, despite praw3 claiming it doesn't work with decorator
>= 4.1, it actually works fine-- I was at least able to run the tests.

It looks like praw 4+ has been split into two packages, praw and "prawcore",
which would be a new additional dependency.

https://github.com/praw-dev/prawcore

I have not tested if rtv works with praw 4+.

I propose to do the following for now-- does this sound like a reasonable
course of action?

* Update the praw package to 3.6.2 and remove the unnecessary version
requirements, to at least make it install correctly.
* Unbundle praw in rtv to use praw 3.6.2.
* Package prawcore and update praw to the latest release. If it breaks rtv,
I'll either re-bundle praw 3 or work with upstream to support newer praw
releases.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
_______________________________________________
package-review mailing list -- package-review@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to package-review-leave@xxxxxxxxxxxxxxxxxxxxxxx
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: https://lists.fedoraproject.org/archives/list/package-review@xxxxxxxxxxxxxxxxxxxxxxx




[Index of Archives]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite Conditions]     [KDE Users]

  Powered by Linux