Bug 754246

Summary: Review Request: TV-Browser - A TV Browsing application
Product: [Fedora] Fedora Reporter: Sven Baus <s.baus86>
Component: Package ReviewAssignee: Nobody's working on this, feel free to take it <nobody>
Status: CLOSED NOTABUG QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: rawhideCC: che666, eldermarco, notting, opensource, package-review, red, tcallawa
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
URL: http://www.tv-browser.org
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2013-05-01 21:54:20 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 201449    

Description Sven Baus 2011-11-15 20:29:01 UTC
SPEC: http://dl.dropbox.com/u/3351272/Stuff/tvbrowser/SPECS/TV-Browser.spec
SOURCE: http://downloads.sourceforge.net/project/tvbrowser/TV-Browser%20Releases%20%28Java%206%20and%20higher%29/3.1/tvbrowser_3.1_src.zip
SRPM: http://dl.dropbox.com/u/3351272/Stuff/tvbrowser/SRPMS/TV-Browser-3.1-1.fc14.src.rpm

This is just a first version of the package, since I know, that there is a lot to do. I just started this review request, in order to see, if there is somethine speaking against this package (forbidden items or so), because it's just my third package I build. 

I also need a sponsor ;).

TODO:
 - check the package (no rpmlint or mock has been done yet)
 - desktop file

I would like to see, if there are some legal concerns, or if I can go on :D!

Greets
Sven

Comment 1 Rudolf Kastl 2011-11-16 15:19:11 UTC
Well you made your life rather easy but unfortunately what you are doing is packaging prebuilt binaries that are built with provided libs instead of building against fedora provided libraries with the fedora provided java.

todo:

* package the src zip (upstream provides one)
* remove lib/*
* patch the build.xml or create symlinks to the appropriate libraries in lib/
* build it with ant
* package the results in a working manner

thanks for your effort in getting that nice application into the distribution.

Comment 2 Till Maas 2011-11-19 20:33:52 UTC
Blocking FE-Legal to get the legal opinion requested in the description of the bug.

Comment 3 Tom "spot" Callaway 2011-11-23 20:44:32 UTC
I'm not exactly sure what legal concerns there are here. I haven't looked at the source code at all, but from the TV-Browser website, it seems to be acceptable for Fedora (they secure permission before providing listing information).

Obviously, it needs to be built from source, have any bundled jars removed, and be under acceptable Free Software licenses, but that's no different from any other Fedora package. Lifting FE-Legal.

Comment 4 Sven Baus 2011-11-26 18:54:55 UTC
(In reply to comment #1)

> * package the src zip (upstream provides one)
> * remove lib/*
> * patch the build.xml or create symlinks to the appropriate libraries in lib/
> * build it with ant
> * package the results in a working manner


So I have removed the lib/* directory, but I don't know, how to go on, because inside the lib/* directory are some files, that I couldn't find in the standard fedora setup (/usr/share/java). 

Could someone please help me?

Comment 5 Thomas Spura 2011-12-04 12:19:23 UTC
*** Bug 472144 has been marked as a duplicate of this bug. ***

Comment 6 Sven Baus 2011-12-06 20:46:33 UTC
(In reply to comment #5)
> *** Bug 472144 has been marked as a duplicate of this bug. ***

Thanks, this helps me a bit. Now I could go a step further. 
Trying to build all the libs, I get the following output by some:
/usr/bin/build-jar-repository: error: Could not find jakarta-commons-codec Java extension for this JVM
/usr/bin/build-jar-repository: error: Could not find jakarta-commons-lang Java extension for this JVM
/usr/bin/build-jar-repository: error: Could not find jakarta-commons-net Java extension for this JVM
/usr/bin/build-jar-repository: error: Could not find jakarta-commons-compress Java extension for this JVM
/usr/bin/build-jar-repository: error: Some specified jars were not found for this jvm

I couldn't find for example "jakarta-commons-codec" in the repository. Isn't it available anymore? How should I go on?

Greets from germany
Sven

Comment 7 Sven Baus 2011-12-12 13:32:28 UTC
Hy there,
i habe been able to find almost everything to build the /lib/* directory, but what I couldn't find is: substance.

Does anyone know, if there is a package for this GUI Framework?!
Would be fine if I got some help ;).

Greets
Sven

Comment 8 Sven Baus 2011-12-28 14:22:37 UTC
Hello,

seems like we can close this review request, since TV-Browser needs Java Substance and I have no idea, how I should get this into fedora. 

Sorry :(.
Greets
Sven

Comment 9 Rudolf Kastl 2012-01-12 14:57:58 UTC
i am still busy this and next week but then i have time and i am willing to help you to get this software built correctly because i think it would be a nice addition to fedora. is there any way to contact you? are you on freenode irc?

kind regards,
Rudolf Kastl

Comment 10 Sven Baus 2012-01-12 20:47:55 UTC
Thanks for your help ;). I'm curently working on the substance package as you might see :D. The best way to contact me is by mail, because currently I'm a bit full with work and also in private life. Also I'm available via ICQ, if this would be an option.

Greetings,
Sven

Comment 11 Elder Marco 2012-02-01 01:54:02 UTC
Just a comment: The name of this package could be tvbrowser instead of Tv-Browser. 

http://fedoraproject.org/wiki/Packaging:NamingGuidelines#Case_Sensitivity

Comment 12 Rudolf Kastl 2012-03-23 08:20:45 UTC
sven could you please publish your latest spec file + patches (if needed) so i can take a closer look? sorry for the delays, but i have been rather busy.

Comment 13 Sven Baus 2012-03-24 13:35:30 UTC
(In reply to comment #12)
> sven could you please publish your latest spec file + patches (if needed) so i
> can take a closer look? sorry for the delays, but i have been rather busy.

SPEC: http://dl.dropbox.com/u/3351272/Stuff/tvbrowser/SPECS/TV-Browser.spec

I'm working on the dependencies, but I'm also as you are, a bit busy ;).

Comment 14 Jason Tibbitts 2013-05-01 21:54:20 UTC
Since the review ticket for a dependency was closed due to inactivity, I'll close this as well.