Bug 1061747 - Review Request: collections-generic - A generics-enabled version of Apache Commons-Collections
Summary: Review Request: collections-generic - A generics-enabled version of Apache Co...
Keywords:
Status: CLOSED WONTFIX
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Haïkel Guémar
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: bigdata-review 1069366 1071456
TreeView+ depends on / blocked
 
Reported: 2014-02-05 14:34 UTC by Robert Rati
Modified: 2014-04-09 14:12 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2014-04-09 14:08:51 UTC
Type: ---
Embargoed:
karlthered: fedora-review?


Attachments (Terms of Use)

Description Robert Rati 2014-02-05 14:34:12 UTC
Spec URL: http://rrati.fedorapeople.org/collections-generic.spec
SRPM URL: http://rrati.fedorapeople.org/collections-generic-4.01-1.fc19.src.rpm
Description: A generics-enabled version of Jakarta Commons-Collections.
Fedora Account System Username: rrati

Comment 1 Mikolaj Izdebski 2014-02-06 14:43:46 UTC
Please change the description and use Apache instead of Jakarta.  Jakarta project [1] has been retired long time ago and COmmons is now a top-level apache project.

[1] https://jakarta.apache.org/

Comment 2 Mikolaj Izdebski 2014-02-06 14:48:20 UTC
Also, given that this project is a fork of Apache Commons (already packaged) and it wasn't updated in a while (last svn activity is 2007-02-12) I would recommend not to package it, but use Apache Commons instead.

Comment 3 Robert Rati 2014-02-06 14:49:15 UTC
updated in place and replaced Jakarta with Apache

Comment 4 Haïkel Guémar 2014-04-06 18:15:12 UTC
I understand Mikolaj concerns, but from a quick code review, apache commons collections 4.0 could not be used as a drop-in replacement of collections-generic.
Maybe, it won't be much complicated to move to latest ACC release (which now supports java generics) but it will require more scrutiny.

Since we already package multiple releases of java packages for other and it's a proper fork (and not bundling), couldn't we settle it by opening a ticket upstream considering to move to ACC 4.0 ? 
Getting Oozie in is critical for the Cloud product big data image.

Comment 5 Haïkel Guémar 2014-04-08 15:50:43 UTC
If nobody complains, I'll move forward this review this week.

Comment 6 Robert Rati 2014-04-08 17:35:48 UTC
I'd hold off on this review.  The package has compilation issues on java8 and I am working on porting jung/oozie to commons-collections4.  If both of those efforts are successful then this package won't be needed, and imo that's the ideal path.  jung is the most dependent package, and I'm close to porting it to commons-collections4.  I hope to complete jung in the next few days.

Comment 7 Robert Rati 2014-04-09 14:08:51 UTC
I was able to move jung and oozie to commons-collections4, so this package is no longer needed.  Closing the review request

Comment 8 Haïkel Guémar 2014-04-09 14:12:26 UTC
awesome, thanks !


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