Bug 474983 - Review Request: TVAnytimeAPI - A java API for parsing, manipulating and creating TV-Anytime metadata
Summary: Review Request: TVAnytimeAPI - A java API for parsing, manipulating and creat...
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
low
medium
Target Milestone: ---
Assignee: Mamoru TASAKA
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: tvbrowser
TreeView+ depends on / blocked
 
Reported: 2008-12-06 11:16 UTC by Sandro Mathys
Modified: 2009-01-27 01:49 UTC (History)
5 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2009-01-26 15:25:29 UTC
mtasaka: fedora-review+
kevin: fedora-cvs+


Attachments (Terms of Use)

Description Sandro Mathys 2008-12-06 11:16:09 UTC
Spec URL: http://red.fedorapeople.org/SRPMS/TVAnytimeAPI.spec
SRPM URL: http://red.fedorapeople.org/SRPMS/TVAnytimeAPI-1.3-0.1.fc10.src.rpm
Description: A java API for parsing, manipulating and creating TV-Anytime metadata.
TV-Anytime is an open standard for metadata describing TV and radio
programmes that is designed to support Personal Video Recoders (PVRs),
programme guides and related technologies.

This  is a dependency needed for tvbrowser, for which a review request exists already.

rpmlint on spec, srpm and noarch-rpms finishes checking without any warnings or
errors.

I'd still need a sponsor.

Comment 1 Sandro Mathys 2008-12-12 01:02:28 UTC
Successfully created a mock-(re)build of this version of this pkg.

Comment 2 Mamoru TASAKA 2008-12-15 15:05:19 UTC
(Removing NEEDSPONSOR)

Comment 3 Sandro Mathys 2008-12-16 04:29:04 UTC
Spec URL: http://red.fedorapeople.org/SRPMS/TVAnytimeAPI.spec
SRPM URL: http://red.fedorapeople.org/SRPMS/TVAnytimeAPI-1.3-1.fc10.src.rpm

New version including much of the experience I got when my jcalendar pkg was
reviewed.

Comment 5 Mamoru TASAKA 2008-12-19 18:47:32 UTC
Well, 

Many files under xml/ directory has the statement:
---------------------------------------------------------------
Use of this BBC data for non-TV-Anytime purposes is strictly forbidden
---------------------------------------------------------------

I guess this statement renders this software non-free.
spot, how do you think?

Comment 6 Tom "spot" Callaway 2008-12-19 18:55:04 UTC
At the very least, the xml data is non-free and not permissable in Fedora.

Comment 7 Sandro Mathys 2009-01-09 22:34:08 UTC
Spec URL: http://red.fedorapeople.org/SRPMS/TVAnytimeAPI.spec
SRPM URL: http://red.fedorapeople.org/SRPMS/TVAnytimeAPI-1.3-2.fc11.src.rpm

- Removed the xml/ subdir for legal problems.
- Also removed lib/* and docs/ because I was already modifying the source archive anyway.
- Added examples/ to the docs

Comment 8 Mamoru TASAKA 2009-01-11 14:15:32 UTC
Removing FE-Legal. Thank you, spot.

Comment 9 Mamoru TASAKA 2009-01-11 15:04:00 UTC
For 1.3-2:

* License
  - The License tag should be "LGPLv2+".

* javadoc directory
  - Would you explain why you don't simply use
    %{_javadocdir}/%{name} as javadoc installation
    directory and instead want to use %_javadocdir/%name-%version
    and create symlink?

    ! Note
      On rpm packaging please take much care of creating symlink
      pointing to directory, because with rpm restriction (or cpio?)
      once this is done it gets very difficult to change this
      symlink to actual directory or so.

Comment 10 Sandro Mathys 2009-01-11 22:50:03 UTC
Spec URL: http://red.fedorapeople.org/SRPMS/TVAnytimeAPI.spec
SRPM URL: http://red.fedorapeople.org/SRPMS/TVAnytimeAPI-1.3-2.fc11.src.rpm


* License corrected.

* Because I was told I should in another review: https://bugzilla.redhat.com/show_bug.cgi?id=475019#c6
...honestly, I've also just checked my /usr/share/javadoc/* and while there's not much in there, everything is using %name-%version with a symlink to it :)

Comment 11 Mamoru TASAKA 2009-01-14 15:40:17 UTC
Well, where is your new srpm? (please change the release number
every time you modify your spec file)

Comment 13 Mamoru TASAKA 2009-01-15 16:54:39 UTC
(In reply to comment #10)
> * Because I was told I should in another review:
> https://bugzilla.redhat.com/show_bug.cgi?id=475019#c6
> ...honestly, I've also just checked my /usr/share/javadoc/* and while there's
> not much in there, everything is using %name-%version with a symlink to it :)

I checked this by using repoquery, i.e. _all_ packages installing
javadoc files under /usr/share/javadoc/. 

It seems packages reviewed in earlier days seem to be creating such symlinks.
Actually this is the first package creating javadoc subpackage trying
to create such symlink I reviewed. However this is not a blocker.

-------------------------------------------------------
   This package (TVAnytimeAPI) is APPROVED by mtasaka
-------------------------------------------------------

Comment 14 Sandro Mathys 2009-01-15 22:25:28 UTC
New Package CVS Request
=======================
Package Name: TVAnytimeAPI
Short Description: A Java API for parsing, manipulating and creating TV-Anytime metadata
Owners: red
Branches: F-9 F-10
InitialCC:

Comment 15 Kevin Fenzi 2009-01-17 03:31:27 UTC
cvs done.

Comment 16 Mamoru TASAKA 2009-01-25 07:54:29 UTC
Please rebuild this package on koji and for F-10/9
submit requests to push the rebuilt packages into repositories.

Comment 17 Mamoru TASAKA 2009-01-26 15:25:29 UTC
Now closing, thank you.

Comment 18 Fedora Update System 2009-01-27 01:46:56 UTC
TVAnytimeAPI-1.3-3.fc10 has been pushed to the Fedora 10 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 19 Fedora Update System 2009-01-27 01:49:25 UTC
TVAnytimeAPI-1.3-3.fc9 has been pushed to the Fedora 9 stable repository.  If problems still persist, please make note of it in this bug report.


Note You need to log in before you can comment on or make changes to this bug.