[Bug 34] Review request: xmltv - A set of utilities to manage your TV viewing
https://bugzilla.rpmfusion.org/show_bug.cgi?id=34 --- Comment #24 from Richard hobbes1...@gmail.com 2012-08-19 14:02:24 CEST --- (In reply to comment #23) There is still no spec update. https://dl.dropbox.com/u/34775202/xmltv.spec -- Configure bugmail: https://bugzilla.rpmfusion.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the assignee for the bug.
[Bug 34] Review request: xmltv - A set of utilities to manage your TV viewing
https://bugzilla.rpmfusion.org/show_bug.cgi?id=34 --- Comment #25 from Nicolas Chauvet kwiz...@gmail.com 2012-08-19 14:25:56 CEST --- (In reply to comment #24) (In reply to comment #23) There is still no spec update. https://dl.dropbox.com/u/34775202/xmltv.spec I haven't received the related src.rpm so I cannot grab the missing patch. +# This is for tv_grab_jp which is currently disabled in source +#BuildRequires: perl(Text::Kakasi) Why this isn't properly conditionalized between fedora and el ? This perl module is available in fedora. -- Configure bugmail: https://bugzilla.rpmfusion.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the assignee for the bug.
[Bug 34] Review request: xmltv - A set of utilities to manage your TV viewing
https://bugzilla.rpmfusion.org/show_bug.cgi?id=34 --- Comment #26 from Richard hobbes1...@gmail.com 2012-08-19 14:42:02 CEST --- (In reply to comment #25) (In reply to comment #24) (In reply to comment #23) There is still no spec update. https://dl.dropbox.com/u/34775202/xmltv.spec I haven't received the related src.rpm so I cannot grab the missing patch. https://dl.dropbox.com/u/34775202/xmltv-0.5.63-1.fc16.src.rpm +# This is for tv_grab_jp which is currently disabled in source +#BuildRequires: perl(Text::Kakasi) Why this isn't properly conditionalized between fedora and el ? This perl module is available in fedora. Because the grabber is disabled upstream there's no reason to pull in the build requirement. If it's enabled at a later date then I would uncomment and conditionalize. -- Configure bugmail: https://bugzilla.rpmfusion.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the assignee for the bug.
[Bug 34] Review request: xmltv - A set of utilities to manage your TV viewing
https://bugzilla.rpmfusion.org/show_bug.cgi?id=34 --- Comment #27 from Nicolas Chauvet kwiz...@gmail.com 2012-08-19 16:42:52 CEST --- (In reply to comment #26) (In reply to comment #25) (In reply to comment #24) (In reply to comment #23) There is still no spec update. https://dl.dropbox.com/u/34775202/xmltv.spec I haven't received the related src.rpm so I cannot grab the missing patch. https://dl.dropbox.com/u/34775202/xmltv-0.5.63-1.fc16.src.rpm Ok, you have been added to the ACL for the current branches. -- Configure bugmail: https://bugzilla.rpmfusion.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the assignee for the bug.
[Bug 34] Review request: xmltv - A set of utilities to manage your TV viewing
https://bugzilla.rpmfusion.org/show_bug.cgi?id=34 --- Comment #23 from Nicolas Chauvet kwiz...@gmail.com 2012-08-18 10:02:29 CEST --- There is still no spec update. -- Configure bugmail: https://bugzilla.rpmfusion.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the assignee for the bug.
[Bug 34] Review request: xmltv - A set of utilities to manage your TV viewing
https://bugzilla.rpmfusion.org/show_bug.cgi?id=34 --- Comment #22 from Richard hobbes1...@gmail.com 2012-08-08 02:02:06 CEST --- Got a response from upstream on bad FSF addresses. They have been fixed. -- Configure bugmail: https://bugzilla.rpmfusion.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the assignee for the bug.
[Bug 34] Review request: xmltv - A set of utilities to manage your TV viewing
https://bugzilla.rpmfusion.org/show_bug.cgi?id=34 --- Comment #21 from Richard hobbes1...@gmail.com 2012-07-30 17:59:47 CEST --- Nicolas, I'm ready to maintain xmltv unless you have some additional requirements. Do I need to update the CVS request? -- Configure bugmail: https://bugzilla.rpmfusion.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the assignee for the bug.
[Bug 34] Review request: xmltv - A set of utilities to manage your TV viewing
https://bugzilla.rpmfusion.org/show_bug.cgi?id=34 --- Comment #12 from Richard hobbes1...@gmail.com 2012-07-24 15:48:04 CEST --- I just wanted to ping this CVS request. Is there something that needs to be done before it can be completed? This is the last requirement for the EL-6 branch of MythTV. -- Configure bugmail: https://bugzilla.rpmfusion.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the assignee for the bug.
[Bug 34] Review request: xmltv - A set of utilities to manage your TV viewing
https://bugzilla.rpmfusion.org/show_bug.cgi?id=34 --- Comment #13 from Nicolas Chauvet kwiz...@gmail.com 2012-07-24 16:20:37 CEST --- (In reply to comment #12) I just wanted to ping this CVS request. Is there something that needs to be Im' not sure to understand this request. So let's have some minimal formalism. As a CVS admin process holder, I need to have the agreement from the primary maintainer. That's the same with fedora and having an empty EL-6 branch doesn't mean a different process. As a primary maintainer of xmltv, I've requested to update to the latest version to verify the future maintainer capabilities. I use the very same process to allow co-maintainers on the package I'm responsible. This is how I got ACL for vlc, and later ffmpeg, despite at that time; the previous maintainer started to apply my patches without allowing me any ACL as a first step. So RPM Fusion might be a smaller crew, there is no reason to have less formalism than in Fedora. I really hopes to allow more maintainers to gain access, that's doesn't mean to bypass this very trivial process. -- Configure bugmail: https://bugzilla.rpmfusion.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the assignee for the bug.
[Bug 34] Review request: xmltv - A set of utilities to manage your TV viewing
https://bugzilla.rpmfusion.org/show_bug.cgi?id=34 --- Comment #14 from Richard hobbes1...@gmail.com 2012-07-24 17:43:54 CEST --- You didn't list any specific actions needed but I have done the following locally: - Updated to the latest released version. - Updated the noask patch. - Added a new binary and man page to %files. - Removed some optional spec stuff (BuildRoot:, %clean, %defattr...) - Fixed a bad end-of-line encoding - Reported a couple of bad FSF addresses upstream. https://sourceforge.net/tracker/?func=detailaid=3547942group_id=39046atid=424135 Are there other actions you're looking for? Or specific packages this needs to be tested against? -- Configure bugmail: https://bugzilla.rpmfusion.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the assignee for the bug.
[Bug 34] Review request: xmltv - A set of utilities to manage your TV viewing
https://bugzilla.rpmfusion.org/show_bug.cgi?id=34 --- Comment #15 from Richard hobbes1...@gmail.com 2012-07-24 18:07:49 CEST --- I've run into an issue that EL6 does not provide perl(Text::Kakasi). In looking into this, I've found that none of the resulting rpms require it either so I'm not sure if it's really needed or not. I commented it out and the EL6 build did not appear to be affected. -- Configure bugmail: https://bugzilla.rpmfusion.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the assignee for the bug.
[Bug 34] Review request: xmltv - A set of utilities to manage your TV viewing
https://bugzilla.rpmfusion.org/show_bug.cgi?id=34 --- Comment #16 from Nicolas Chauvet kwiz...@gmail.com 2012-07-24 18:12:04 CEST --- (In reply to comment #14) You didn't list any specific actions needed but I have done the following This was done to Ken Dreyer in another report for some reason. -- Configure bugmail: https://bugzilla.rpmfusion.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the assignee for the bug.
[Bug 34] Review request: xmltv - A set of utilities to manage your TV viewing
https://bugzilla.rpmfusion.org/show_bug.cgi?id=34 --- Comment #17 from Nicolas Chauvet kwiz...@gmail.com 2012-07-24 20:31:57 CEST --- (In reply to comment #16) (In reply to comment #14) You didn't list any specific actions needed but I have done the following This was done to Ken Dreyer in another report for some reason. https://bugzilla.rpmfusion.org/show_bug.cgi?id=2397 (In reply to comment #14) ... Are there other actions you're looking for? Or specific packages this needs to be tested against? That would be all. -- Configure bugmail: https://bugzilla.rpmfusion.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the assignee for the bug.
[Bug 34] Review request: xmltv - A set of utilities to manage your TV viewing
https://bugzilla.rpmfusion.org/show_bug.cgi?id=34 --- Comment #18 from Richard hobbes1...@gmail.com 2012-07-24 20:44:16 CEST --- (In reply to comment #17) (In reply to comment #16) (In reply to comment #14) You didn't list any specific actions needed but I have done the following This was done to Ken Dreyer in another report for some reason. https://bugzilla.rpmfusion.org/show_bug.cgi?id=2397 Yes, those comments look familiar, I just couldn't remember what BZ they were on. I guess one question I have is if perl-Text-Kakasi needs to be optional or not. It doesn't appear to affect building any of the grabbers (config output is the same with or without) and it doesn't show up in any of the package requirements. Why would we just not drop the BuildRequires: all together? -- Configure bugmail: https://bugzilla.rpmfusion.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the assignee for the bug.
[Bug 34] Review request: xmltv - A set of utilities to manage your TV viewing
https://bugzilla.rpmfusion.org/show_bug.cgi?id=34 --- Comment #19 from Richard hobbes1...@gmail.com 2012-07-24 23:15:18 CEST --- Ok, I figured out what's going on. the perl(Text:Kakasi) requirement is for tv_grab_jp (Japanese) but it's currently disabled in the source because the site that supplies the data is currently blocking xmltv. Because of that I have commented out the requirement in my version of the spec since it will not do anything anyway. If that's figured out in a later version it can be re-enabled. Is there anything else that needs to be completed before the cvs request can be processed? -- Configure bugmail: https://bugzilla.rpmfusion.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the assignee for the bug.
[Bug 34] Review request: xmltv - A set of utilities to manage your TV viewing
https://bugzilla.rpmfusion.org/show_bug.cgi?id=34 --- Comment #20 from Richard hobbes1...@gmail.com 2012-07-25 00:34:17 CEST --- Ok, added perl(DateTime::Format::ISO8601) and now tv_grab_na_dtv get's built and installed. -- Configure bugmail: https://bugzilla.rpmfusion.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the assignee for the bug.
[Bug 34] Review request: xmltv - A set of utilities to manage your TV viewing
https://bugzilla.rpmfusion.org/show_bug.cgi?id=34 Ken Dreyer ktdre...@ktdreyer.com changed: What|Removed |Added Blocks||2053 -- Configure bugmail: https://bugzilla.rpmfusion.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the assignee for the bug.
[Bug 34] Review request: xmltv - A set of utilities to manage your TV viewing
https://bugzilla.rpmfusion.org/show_bug.cgi?id=34 Ken Dreyer ktdre...@ktdreyer.com changed: What|Removed |Added CC||ktdre...@ktdreyer.com Blocks||33 --- Comment #10 from Ken Dreyer ktdre...@ktdreyer.com 2012-07-01 14:40:43 CEST --- Package Change Request == Package Name: mythtv New Branches: EL-6 Updated EPEL Owners: hobbes1...@gmail.com ktdre...@ktdreyer.com -- Configure bugmail: https://bugzilla.rpmfusion.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the assignee for the bug.
[Bug 34] Review request: xmltv - A set of utilities to manage your TV viewing
https://bugzilla.rpmfusion.org/show_bug.cgi?id=34 --- Comment #11 from Ken Dreyer ktdre...@ktdreyer.com 2012-07-01 14:41:11 CEST --- Package Change Request == Package Name: xmltv New Branches: EL-6 Updated EPEL Owners: hobbes1...@gmail.com ktdre...@ktdreyer.com -- Configure bugmail: https://bugzilla.rpmfusion.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the assignee for the bug.
[Bug 34] Review request: xmltv - A set of utilities to manage your TV viewing
http://bugzilla.rpmfusion.org/show_bug.cgi?id=34 Thorsten Leemhuis fed...@leemhuis.info changed: What|Removed |Added Blocks||4 -- Configure bugmail: http://bugzilla.rpmfusion.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the assignee for the bug.
[Bug 34] Review request: xmltv - A set of utilities to manage your TV viewing
http://bugzilla.rpmfusion.org/show_bug.cgi?id=34 NicolasChauvet [EMAIL PROTECTED] changed: What|Removed |Added Blocks|4 | Status|ASSIGNED|RESOLVED Resolution||FIXED --- Comment #9 from NicolasChauvet [EMAIL PROTECTED] 2008-10-31 12:14:17 --- package imported. -- Configure bugmail: http://bugzilla.rpmfusion.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the assignee for the bug.
[Bug 34] Review request: xmltv - A set of utilities to manage your TV viewing
http://bugzilla.rpmfusion.org/show_bug.cgi?id=34 NicolasChauvet [EMAIL PROTECTED] changed: What|Removed |Added Depends on||33 Blocks|2 |4 --- Comment #7 from NicolasChauvet [EMAIL PROTECTED] 2008-10-20 10:07:15 --- Package CVS request == Package Name: xmltv Short Description: A set of utilities to manage your TV viewing Owners: kwizart Branch: devel F-9 F-8 EL-5 InitialCC: -- License tag: free -- Configure bugmail: http://bugzilla.rpmfusion.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the assignee for the bug.
[Bug 34] Review request: xmltv - A set of utilities to manage your TV viewing
http://bugzilla.rpmfusion.org/show_bug.cgi?id=34 Xavier Lamien [EMAIL PROTECTED] changed: What|Removed |Added Depends on|33 | --- Comment #8 from Xavier Lamien [EMAIL PROTECTED] 2008-10-20 10:45:21 --- cvs done. -- Configure bugmail: http://bugzilla.rpmfusion.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the assignee for the bug.
[Bug 34] Review request: xmltv - A set of utilities to manage your TV viewing
http://bugzilla.rpmfusion.org/show_bug.cgi?id=34 --- Comment #5 from NicolasChauvet [EMAIL PROTECTED] 2008-10-15 15:27:07 --- (In reply to comment #4) * missing desktop file for xmltv-gui The Fedora guidelines are pretty strict on this, so even if the use case is limited here, it'd be better to have one. https://fedoraproject.org/wiki/Packaging/Guidelines#Desktop_files Either http://wiki.xmltv.org/favicon.ico or http://wiki.xmltv.org/logo.png can be used for the icon. So..., I have investigated. The normal use would be to have: tv_check --scan --listings /someplace/XMLTV_grabbed.xml --shows ~/.xmltv/shows.xml (it would need to be --configured before). XMLTV_grabbed.xml is usually generated from tv_grab_yourlocale but there is no sure way where to expect this file.(It depend on where the related apps store the file, but system wide TV.xml file isn't common). Also shows.xml are user dependant, but might be different than the current users. (for example if user schlobinux want to check shows from mythtv or freevo within the schlobinux X session). I would add that .desktop files aren't designed to support parsing of two files until now, so it doesn't worth the case to use this as it will lead to hardcode the path, whereas it can change often and for different uses. * Islandic grabber is not built. Adding the following fixes it : BuildRequires: perl(XML::DOM) BuildRequires: perl(XML::LibXSLT) Fixed and mock tested in Rawhide and F-8 (it works). -- Configure bugmail: http://bugzilla.rpmfusion.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the assignee for the bug.
[Bug 34] Review request: xmltv - A set of utilities to manage your TV viewing
http://bugzilla.rpmfusion.org/show_bug.cgi?id=34 --- Comment #3 from NicolasChauvet [EMAIL PROTECTED] 2008-10-14 21:10:30 --- SRPM: http://rpms.kwizart.net/fedora/reviews/xmltv/xmltv-0.5.53-1.fc8.kwizart.src.rpm SPEC: http://rpms.kwizart.net/fedora/reviews/xmltv/xmltv.spec Summary: A set of utilities to manage your TV viewing - Update to 0.5.53 - Remove -gui requirement on main - filter perl-Tk dependency on perl-XMLTV - Re-enable make test * I don't think that creating a .desktop file is that necessary. I don't use that script often (never) anyway. But splitting it in a -gui subpackage will prevent to requires perl-Tk and Thus, huge gui dependencies. * Duplicate requires can probably be fixed by using the same value when use a perl package. That been said i cannot see any side effect and it will remains developpers care to request the same version. The others points should have been fixed. perl-Log-TraceMessages has been pushed to stable for F-8/F-9 today which is the last dependency that was missing. It is already available for Rawhide, right now. -- Configure bugmail: http://bugzilla.rpmfusion.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the assignee for the bug.
[Bug 34] Review request: xmltv - A set of utilities to manage your TV viewing
http://bugzilla.rpmfusion.org/show_bug.cgi?id=34 --- Comment #2 from NicolasChauvet [EMAIL PROTECTED] 2008-08-08 16:21:34 --- Suggested additional BR: perl(Log::TraceMessages) (for debug purposes, from the README file) Anyone can take this (and dependencies ?) https://bugzilla.redhat.com/show_bug.cgi?id=458449 -- Configure bugmail: http://bugzilla.rpmfusion.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the assignee for the bug.