Bug 438138 (tzdata-java-epel) - Review Request: tzdata-java - Time zone data for Java language
Summary: Review Request: tzdata-java - Time zone data for Java language
Keywords:
Status: CLOSED NEXTRELEASE
Alias: tzdata-java-epel
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Marek Mahut
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: openjdk-epel
TreeView+ depends on / blocked
 
Reported: 2008-03-19 11:56 UTC by Lubomir Kundrak
Modified: 2008-04-08 10:08 UTC (History)
3 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2008-04-08 10:08:27 UTC
Type: ---
Embargoed:
mmahut: fedora-review+
kevin: fedora-cvs+


Attachments (Terms of Use)

Description Lubomir Kundrak 2008-03-19 11:56:19 UTC
Spec URL: http://people.redhat.com/lkundrak/SPECS/tzdata-java.spec
SRPM URL:
http://people.redhat.com/lkundrak/mock-results/tzdata-java-2007k-0.1.el5.noarch/tzdata-java-2007k-0.1.el5.src.rpm
mock:
http://people.redhat.com/lkundrak/mock-results/tzdata-java-2007k-0.1.el5.noarch/
Description: Time zone data for Java language

Compiled time zone data files for use with Java
programming language.

---
This package just makes sense for EPEL, so that we can get OpenJDK there.

It is superseded by subpackage of tzdata in Fedora, so will probably not live
longer than until next major RHEL release.

So from some point of view, it just just a hack.

Comment 1 Lubomir Kundrak 2008-03-19 11:58:47 UTC
This package's content is completely euqal to tzdata-java from Rawhide.

Problemetic points:

javazic.tar.gz -- no source URL for that. The same with tzdata package. Google
for its origin :) Java ZIC was not yet officially released in a polished and
packaged form yer.

General ugliness (replacing sun. with rht., etc.), shared with tzdata package :o)



Comment 2 Marek Mahut 2008-03-20 17:23:34 UTC
This hack looks clean. Lubomir explained to me the Java warnings, however as
this is not permanent package and will vanish after RHEL6 release I don't see
any problem with this.

APPROVED.

Comment 3 Lubomir Kundrak 2008-03-27 20:30:19 UTC
New Package CVS Request
=======================
Package Name: tzdata-java
Short Description: Time zone data for Java language
Owners: lkundrak
Branches: EL-4 EL-5
Cvsextras Commits: yes

Comment 4 Kevin Fenzi 2008-04-03 17:58:38 UTC
Is the License tag right here? The main 'tzdata' package which uses the same
upstream as this has a different tag... 

Also, can you check and make sure this conforms to the just approved java
guidelines?



Comment 5 Kevin Fenzi 2008-04-03 20:16:21 UTC
Per IRC conversation with Lubomir, the main tzdata package has an incorrect
license tag, and he's going to file a bug on it. 

cvs done.

Comment 6 Jason Tibbitts 2008-04-03 20:33:56 UTC
Well, when I did the merge review for tzdata we talked about the license;
perhaps someone could let me know what is incorrect about the discussion in bug
226509.  Bottom line: there is LGPL code in the source package but that code is
only used in building and none of it is present in the binary package; all
that's left is public domain data.  Since the License: tag refers only to the
final binary package, that should be correct.

Comment 7 Lubomir Kundrak 2008-04-04 07:58:50 UTC
(In reply to comment #6)
> Since the License: tag refers only to the
> final binary package, that should be correct.

Does it? The source rpm packages also contain this license tag, and the indeed
don't contain only public domain code. Is that rule documented somewhere?


Comment 8 Jason Tibbitts 2008-04-04 20:43:48 UTC
Please see http://fedoraproject.org/wiki/Packaging/LicensingGuidelines

"
The License: field refers to the licenses of the contents of the binary rpm.
When in doubt, ask.
"



Comment 9 Lubomir Kundrak 2008-04-08 10:08:27 UTC
Jason: Thanks, fixed.
Imported and built, Thanks Marek, Thanks Kevin.


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