Bug 542320

Summary:	Review Request: plexus-lang - Plexus language bundle access component
Product:	[Fedora] Fedora	Reporter:	Lubomir Rintel <lkundrak>
Component:	Package Review	Assignee:	Mat Booth <mat.booth>
Status:	CLOSED WONTFIX	QA Contact:	Fedora Extras Quality Assurance <extras-qa>
Severity:	medium	Docs Contact:
Priority:	low
Version:	rawhide	CC:	fedora-package-review, notting
Target Milestone:	---	Flags:	mat.booth: fedora-review-
Target Release:	---
Hardware:	All
OS:	Linux
Whiteboard:	F13FeatureIDEA
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2009-12-09 18:05:02 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:	542341

Description Lubomir Rintel 2009-11-29 09:51:51 UTC

SPEC: http://v3.sk/~lkundrak/SPECS/plexus-lang.spec
SRPM: http://v3.sk/~lkundrak/SRPMS/plexus-lang-1.2-0.1.20091129svn.fc12.src.rpm

Description:

The Plexus project seeks to create end-to-end developer tools for
writing applications. At the core is the container, which can be
embedded or for a full scale application server. There are many
reusable components for hibernate, form processing, jndi, i18n,
velocity, etc. This package contains a language bundle access
component.

Comment 1 Lubomir Rintel 2009-11-29 09:54:35 UTC

rpmlint unhappy:

plexus-lang.noarch: W: no-documentation
 -- none was shipped by upstream

plexus-lang.noarch: W: non-conffile-in-etc /etc/maven/fragments/plexus-lang
 -- I believe rpmlint is correct here (this should really go into %_datadir), unfortunately we can't do anything about, the correct fix would require a change of %update_maven_depmap. Tens of other packages use this.

Scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=1836063

Comment 2 Lubomir Rintel 2009-11-29 11:32:10 UTC

Removed redundant dependencies (urls remain the same, new scratch build http://koji.fedoraproject.org/koji/taskinfo?taskID=1836094)

Comment 3 Mat Booth 2009-12-01 13:27:36 UTC

I will take this one.

Comment 4 Mat Booth 2009-12-03 22:20:11 UTC

Hmm, this package looks very much plexus-i18n to me... Is this some kind of fork or just a newer version of plexus-i18n?

Plexus-lang provides identical copies of all the classes that plexus-i18n provides just under a different package name.

Comment 5 Lubomir Rintel 2009-12-09 11:58:24 UTC

Good catch, Mat. To be honest, I could not determine. I sent a mail to gui that seems to be a single commiter to plexus-lang and asked him to clarify the situation.

Comment 6 Lubomir Rintel 2009-12-09 18:05:02 UTC

Response from upstream:

"""
Please use plexus-i18n - the original one. Plexus-lang was a one off 
that kinda of stalled now, so if one is deprecated, that would be this one.
"""

Closing this.

Comment 7 Mat Booth 2009-12-09 18:29:48 UTC

Cool. Easiest review I've ever done. ;-)