Bug 436239 - Review Request: joda-time - Java date and time API
Review Request: joda-time - Java date and time API
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
All Linux
medium Severity medium
: ---
: ---
Assigned To: Mamoru TASAKA
Fedora Extras Quality Assurance
: 227073 (view as bug list)
Depends On:
Blocks: 417511
  Show dependency treegraph
Reported: 2008-03-05 21:03 EST by Conrad Meyer
Modified: 2008-07-21 12:11 EDT (History)
5 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2008-04-07 17:50:27 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
mtasaka: fedora‑review+
kevin: fedora‑cvs+

Attachments (Terms of Use)
patch to prevent from junit downloading (1.58 KB, patch)
2008-03-16 13:11 EDT, Mamoru TASAKA
no flags Details | Diff

  None (edit)
Description Conrad Meyer 2008-03-05 21:03:47 EST
Spec URL: http://konradm.fedorapeople.org/fedora/SPECS/joda-time.spec
SRPM URL: http://konradm.fedorapeople.org/fedora/SRPMS/joda-time-1.5.2-1.fc8.src.rpm
Joda-Time provides a quality replacement for the Java date
and time classes. The design allows for multiple calendar
systems, while still providing a simple API. The 'default'
calendar is the ISO8601 standard which is used by XML. The
Gregorian, Julian, Buddhist, Coptic and Ethiopic systems
are also included, and we welcome further additions.
Supporting classes include time zone, duration, format
and parsing.
Comment 1 Conrad Meyer 2008-03-05 21:04:03 EST
*** Bug 227073 has been marked as a duplicate of this bug. ***
Comment 2 Mamoru TASAKA 2008-03-14 12:44:14 EDT
It seems that junit is needed for BuildRequires?
Comment 3 Conrad Meyer 2008-03-15 02:17:36 EDT
Yes, but the way their build process is set up it tries to fetch junit from the 
web instead of using the system junit -- or something. I'm not completely sure 
how to fix this yet, but I'm working on it.
Comment 4 Mamoru TASAKA 2008-03-16 13:11:19 EDT
Created attachment 298198 [details]
patch to prevent from junit downloading

I am not java expert (and my knowledge about java is very little).
however applying the patch attached and adding "BR: junit" seems
to pass rebuild.

Comment 6 Mamoru TASAKA 2008-03-17 03:52:56 EDT
For 1.5.2-2:

* Requires
  - For -javadocs subpackages:
Requires:       %{name}-%{version}-%{release}
    This must be %{name} = %{version}-%{release}

* src/testdata/
  - The files under src/testdata seem binary (although
    I can see almost all parts by less command).
    If they can safely be igored, please remove all of
    them at %prep (I don't know if they can be removed,
    however even if they are removed joda-time compiles:
    http://koji.fedoraproject.org/koji/taskinfo?taskID=519102 )

* tzdata
  - This package uses tzdata (please check files under
    src/java/org/joda/time/tz/src/) and the version is

    However from
    it seems current latest tzdata is 2008a (you can find 
    tzdata2008a.tar.gz from the URL above) and tzdata of Chile
    is revised.
    ! Note
      Current Fedora rawhide tzdata is also 2007k, however changelog
      shows Fedora tzdata maintainer patched for Chilean tzdata
      and essentially it is the same as 2008a.
    The history of joda-time shows the upstream does not always
    sync tzdata part with the URL above.

    - This package should have tzdataXXXXx.tar.gz as Source(1,
      for example) and override files under src/java/org/joda/time/tz/src/
      by the files in tzdata tarball.
      This means that you have to update joda-time every time
      tzdata is updated (usually you can notice by Fedora tzdata update).

      Also, in this case it is better that the EVR (Epoch-Version-Release)
      of tzdata refects the version of tzdata used for joda-time
      (say, 1.5.2-2.tzdata2008a.fc9?).

    - Or are there any way thatjoda-time use contents in tzdata rpm

* Documents
  - IMO it is better that the following files is added
    to %doc.

* javadoc
  - Mark both
%dir %{_javadocdir}/%{name}-%{version}
    as %doc.
  ! Note
    The above two lines can be replaced by
%doc %{_javadocdir}/%{name}-%{version}/
    The %files entry <a_directory>/ (not %dir <a_directory>)
    contains the directory itself and all files/directories/etc
    under the directory.
Comment 7 Mamoru TASAKA 2008-03-17 04:12:08 EDT
tzdata is Public Domain and you can freely use it.
Comment 8 Conrad Meyer 2008-03-17 17:40:14 EDT
I have incorporated all of your suggestions in to my local spec except for those
concerning tzdata (because I'm not done yet).

I have tried in a number of different ways to use the system tzdata, but cannot
seem to get that to work (it might work but needs love from someone who knows
the ins and outs of java/ant). I will go ahead and use 2008a from your link, and
include "tzdata2008a" in Release.
Comment 10 Mamoru TASAKA 2008-03-18 10:24:37 EDT
Okay. tzdata replacing seems to be working good.

One thing:
- From rpmlint:
$ rpmlint joda-time
joda-time.noarch: W: incoherent-version-in-changelog 1.5.2-3 1.5.2-3.tzdata2008a.fc9
  Please change %changelog to
* Mon Mar 17 2008 Conrad Meyer <konrad@tylerc.org> - 1.5.2-3.tz2008a
  before importing this to CVS.

             This package (joda-time) is APPROVED by me
Comment 11 Conrad Meyer 2008-03-18 14:19:01 EDT
OK. I'll wait on requesting CVS until we arrive at some conclusion about the 
status of java packaging.
Comment 12 Conrad Meyer 2008-03-19 16:59:38 EDT
Also: I'm not sure what the distinction is between tzdata and tzdata-java, but 
perhaps the latter could be used instead of shipping our own. I'll take a stab 
at it while we await FESCo's response.
Comment 13 Mamoru TASAKA 2008-04-04 00:30:59 EDT
Conrad, now
- Java packaging guidelines seems established
- Current tzdata is 2008b

So would you update this package again so that I can check
this package later?
Comment 14 Conrad Meyer 2008-04-04 00:37:55 EDT
(In reply to comment #13)
> Conrad, now
> - Java packaging guidelines seems established
> - Current tzdata is 2008b
> So would you update this package again so that I can check
> this package later?

Yep. Thanks for your patience!
Comment 15 Conrad Meyer 2008-04-05 15:40:59 EDT
New URLs:

Sorry for the delay, I've been busy these last few days.
Comment 16 Mamoru TASAKA 2008-04-06 03:15:25 EDT
Well, I just tried to rebuild but it failed (on dist-f9)

Comment 17 Conrad Meyer 2008-04-06 03:25:59 EDT
Bummer. I'd guess it's a GCJ problem (since it builds just fine in 

New spec and SRPM sans AOT bits:
Spec: http://konradm.fedorapeople.org/fedora/SPECS/joda-time.spec

Comment 18 Mamoru TASAKA 2008-04-06 12:57:28 EDT

           This package (joda-time) is APPROVED by me (again)
Comment 19 Conrad Meyer 2008-04-06 15:20:51 EDT
New Package CVS Request
Package Name: joda-time
Short Description: Java date and time API
Owners: konradm
Branches: F-8
Cvsextras Commits: yes
Comment 20 Kevin Fenzi 2008-04-06 23:51:40 EDT
cvs done.
Comment 21 Conrad Meyer 2008-07-21 04:35:21 EDT
Package Change Request
Package Name: joda-time
New Branches: EL-5
Comment 22 Kevin Fenzi 2008-07-21 12:11:31 EDT
cvs done.

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