[Bug 34] Review request: xmltv - A set of utilities to manage your TV viewing

2012-07-24 Thread RPM Fusion Bugzilla
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

2012-07-24 Thread RPM Fusion Bugzilla
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

2012-07-24 Thread RPM Fusion Bugzilla
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

2012-07-24 Thread RPM Fusion Bugzilla
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

2012-07-24 Thread RPM Fusion Bugzilla
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

2012-07-24 Thread RPM Fusion Bugzilla
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

2012-07-24 Thread RPM Fusion Bugzilla
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 2429] New: Review request: gens-gs - Sega Genesis, Sega CD, and Sega 32X emulator

2012-07-24 Thread RPM Fusion Bugzilla
https://bugzilla.rpmfusion.org/show_bug.cgi?id=2429

 Bug #: 2429
   Summary: Review request: gens-gs - Sega Genesis, Sega CD, and
Sega 32X emulator
Classification: Unclassified
   Product: Package Reviews
   Version: Current
  Platform: All
OS/Version: GNU/Linux
Status: NEW
  Severity: normal
  Priority: P5
 Component: Review Request
AssignedTo: rpmfusion-package-rev...@rpmfusion.org
ReportedBy: alexjn...@hotmail.com
CC: rpmfusion-package-rev...@rpmfusion.org


Gens/GS is a Sega Mega Drive emulator derived from Gens and maintained by
GerbilSoft. Project goals include clean source code, combined features from
various developments of Gens, and improved platform portability.

Why it's not in Fedora: Emulators of video game systems seem to have legal
problems.

This should be eligible for free, rather than non-free. I don't quite
understand why gens is in non-free, but gens-gs is GPLv2+ and LGPLv2+,
requiring nothing that is non-free, or at least to my knowledge.

SPEC URL:
http://dl.dropbox.com/u/42480493/gens-gs.spec
SOURCE RPM URL:
http://dl.dropbox.com/u/42480493/gens-gs-2.16.7-1.fc17.src.rpm

RPMLINT OUTPUT (all seem negligible):

gens-gs.i686: E: incorrect-fsf-address
/usr/share/doc/gens-gs-2.16.7/COPYING.txt
gens-gs.i686: W: no-manual-page-for-binary mdp_test
gens-gs.i686: W: no-manual-page-for-binary gens
gens-gs-debuginfo.i686: E: incorrect-fsf-address
/usr/src/debug/gens-gs-r7/src/mdp/render/scale2x/scale2x.h
gens-gs-debuginfo.i686: E: incorrect-fsf-address
/usr/src/debug/gens-gs-r7/src/mdp/render/scale4x/scalebit_4x.c
gens-gs-debuginfo.i686: E: incorrect-fsf-address
/usr/src/debug/gens-gs-r7/src/mdp/render/scale2x/scalebit_2x.c
gens-gs-debuginfo.i686: E: incorrect-fsf-address
/usr/src/debug/gens-gs-r7/src/mdp/render/scale2x/scalebit_2x.h
gens-gs-debuginfo.i686: E: incorrect-fsf-address
/usr/src/debug/gens-gs-r7/src/mdp/render/scale4x/scalebit_4x_mmx.h
gens-gs-debuginfo.i686: E: incorrect-fsf-address
/usr/src/debug/gens-gs-r7/src/mdp/render/scale4x/scalebit_4x_mmx.c
gens-gs-debuginfo.i686: E: incorrect-fsf-address
/usr/src/debug/gens-gs-r7/src/mdp/render/scale2x/scale2x.c
gens-gs-debuginfo.i686: E: incorrect-fsf-address
/usr/src/debug/gens-gs-r7/src/mdp/render/scale3x/scale3x.c
gens-gs-debuginfo.i686: E: incorrect-fsf-address
/usr/src/debug/gens-gs-r7/src/mdp/render/scale3x/scalebit_3x.h
gens-gs-debuginfo.i686: E: incorrect-fsf-address
/usr/src/debug/gens-gs-r7/src/mdp/render/scale3x/scale3x.h
gens-gs-debuginfo.i686: E: incorrect-fsf-address
/usr/src/debug/gens-gs-r7/src/mdp/render/scale2x/scalebit_2x_mmx.h
gens-gs-debuginfo.i686: E: incorrect-fsf-address
/usr/src/debug/gens-gs-r7/src/mdp/render/scale2x/scalebit_2x_mmx.c
gens-gs-debuginfo.i686: E: incorrect-fsf-address
/usr/src/debug/gens-gs-r7/src/mdp/render/scale3x/scalebit_3x.c
gens-gs-debuginfo.i686: E: incorrect-fsf-address
/usr/src/debug/gens-gs-r7/src/mdp/render/scale4x/scalebit_4x.h
gens-gs-debuginfo.i686: E: incorrect-fsf-address
/usr/src/debug/gens-gs-r7/src/mdp/render/scale4x/scale2x.h
gens-gs-debuginfo.i686: E: incorrect-fsf-address
/usr/src/debug/gens-gs-r7/src/mdp/render/scale4x/scale2x.c
3 packages and 0 specfiles checked; 17 errors, 2 warnings.

-- 
Configure bugmail: https://bugzilla.rpmfusion.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
You are the assignee for the bug.


[Bug 2429] Review request: gens-gs - Sega Genesis, Sega CD, and Sega 32X emulator

2012-07-24 Thread RPM Fusion Bugzilla
https://bugzilla.rpmfusion.org/show_bug.cgi?id=2429

Jeremy Newton alexjn...@hotmail.com changed:

   What|Removed |Added

 Blocks||2

-- 
Configure bugmail: https://bugzilla.rpmfusion.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
You are the assignee for the bug.


[Bug 34] Review request: xmltv - A set of utilities to manage your TV viewing

2012-07-24 Thread RPM Fusion Bugzilla
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

2012-07-24 Thread RPM Fusion Bugzilla
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.