Our perl-DateTime package bundles DateTime::Locale and DateTime::TimeZone. To simplify maintenance, I would like to unbundle them and maintain them as separate packages. (DateTime::TimeZone gets updated fairly frequently, but it's not possible to get automatic notifications from Upstream Release Monitoring, for example). It's also worth noting that neither suse nor debian bundle these modules.
The original rationale for bundling was due to circular (build) dependencies. This no longer applies for DateTime::Locale (DateTime::Locale does not require DateTime). For DateTime::TimeZone, the circular dependencies can temporarily be avoided using perl_bootstrap macro and appropriate filtering.
I've filed bug 730656 and bug 730657 for review of new perl-DateTime-Locale and perl-DateTime-TimeZone packages.
I've prepared an updated spec that only includes DateTime. And since upstream doesn't appear to have used 4 digits in the version number for a couple of years, now, I think it's worthwhile bumping epoch again and reverting to upstream's 2 digit version number.
Spec URL: http://iarnell.home.xs4all.nl/review/perl-DateTime.spec
SRPM URL: http://iarnell.home.xs4all.nl/review/perl-DateTime-0.70-1.fc17.src.rpm
(In reply to comment #0)
> I've filed bug 730656 and bug 730657 for review of new perl-DateTime-Locale and
> perl-DateTime-TimeZone packages.
I'll take a look at those.
Unless Steve shows up, I would also appreciate a review of the changes here too. It's not strictly required, but I basically rebuilt the spec from scratch, so I think having a second pair of eyes take a look would be sensible.
(In reply to comment #3)
> Thanks, Petr.
> Unless Steve shows up, I would also appreciate a review of the changes here
> too. It's not strictly required, but I basically rebuilt the spec from scratch,
> so I think having a second pair of eyes take a look would be sensible.
Sure, no problem :)
the new package looks quite good to me. However, I'd like to add perl(base), perl(constant) and perl(XSLoader) to BR list, and perl(XSLoader) as a runtime dep as well. Again, those are separate packages and XSLoader isn't detected by RPM.
You might want to filter some deps out as well, e.g.:
perl(DateTime::Locale) >= 0.40
perl(DateTime::Locale) >= 0.41
perl(DateTime::TimeZone) >= 0.59
perl(DateTime::TimeZone) >= 1.09
You could also enable more tests but I don't think that's necessary.
Thanks again, Petr. I'll tweak the dependencies.
I've requested branches for f14, f15, and f16, but will take it slowly and wait before thinking about pushing this to f14 and f15.
Updated in rawhide.
Taking as it's not really a review and I'm the one working on it.
perl-DateTime-TimeZone-1.35-3.fc16,perl-DateTime-0.70-2.fc16,perl-DateTime-Locale-0.45-1.fc16 has been submitted as an update for Fedora 16.
perl-DateTime-TimeZone-1.35-3.fc16, perl-DateTime-0.70-2.fc16, perl-DateTime-Locale-0.45-1.fc16 has been pushed to the Fedora 16 stable repository. If problems still persist, please make note of it in this bug report.
Updates were also pushed for f14/f15.