Bug 1009153

Summary:

Review Request: javolution - Real-time Java library

Product:

[Fedora] Fedora

Reporter:

Pete MacKinnon <pmackinn>

Component:

Package Review

Assignee:

gil cattaneo <puntogil>

Status:

CLOSED NEXTRELEASE

QA Contact:

Fedora Extras Quality Assurance <extras-qa>

Severity:

unspecified

Docs Contact:

Priority:

unspecified

Version:

rawhide

CC:

notting, puntogil

Target Milestone:

---

Flags:

puntogil: fedora-review+
kevin: fedora-cvs+

Target Release:

---

Hardware:

Unspecified

OS:

Unspecified

Whiteboard:

Fixed In Version:

Doc Type:

Bug Fix

Doc Text:

Story Points:

---

Clone Of:

Environment:

Last Closed:

2013-11-14 21:39:01 UTC

Type:

Bug

Regression:

---

Mount Type:

---

Documentation:

---

CRM:

Verified Versions:

Category:

---

oVirt Team:

---

RHEL 7.3 requirements from Atomic Host:

Cloudforms Team:

---

Target Upstream Version:

Embargoed:

Attachments:

Description	Flags
review notes	none
new review notes	none

Description Pete MacKinnon 2013-09-17 19:45:55 UTC

This project provides a Java library for real-time applications. It is maven-based and can be used to build multi-platform real-time applications. It forms an important dependency for a forthcoming review request (Hive).

Note that this review request is for version 5.5.1, the latest is actually 6.0 as of August 18. However, 6.0 is heavily dependent on OSGi. The version of OSGi currently available in Fedora is R4 (Felix & JBoss) which is incompatible with being built using the OpenJDK 7 javac. See the following:

http://stackoverflow.com/questions/10911231/how-to-compile-mavenized-osgi-4-3-bundle-with-openjdk-7
http://blog.osgi.org/2012/10/43-companion-code-for-java-7.html

Spec URL: http://pmackinn.fedorapeople.org/javolution/javolution.spec
SRPM URL: http://pmackinn.fedorapeople.org/javolution/javolution-5.5.1-0.1.fc19.src.rpm

Comment 1 gil cattaneo 2013-09-18 16:55:56 UTC

hi
there is a newer release in https://svn.java.net/svn/javolution~svn/tags/
regards

Comment 2 Pete MacKinnon 2013-09-18 16:57:31 UTC

Please read the comments in the Description.

Comment 3 gil cattaneo 2013-09-18 20:48:39 UTC

please, remove from spec file these 

BuildRequires: javapackages-tools
BuildRequires: maven
BuildRequires: maven-compiler-plugin
BuildRequires: maven-install-plugin
BuildRequires: maven-javadoc-plugin
BuildRequires: maven-source-plugin
BuildRequires: maven-surefire-plugin

all Group fiels        - no more required
all Requires           - added by java package tools (XMvn)

fix unowned directory
add %dir %{_javadir}/%{name}
in main package

if possible use pom macros
e.g.
%pom_remove_plugin javolution:colapi
%pom_xpath_remove "pom:project/pom:build/pom:extensions[pom:groupId = 'org.jvnet.wagon-svn' ]"
or
%pom_xpath_remove "pom:project/pom:build/pom:extensions"

fix license field
License:       BSD and Public domain

these are under  Public domain license

Public domain
-------------
/var/lib/mock/fedora-rawhide-i386/root/builddir/build/BUILD/javolution-5.5.1/src/main/java/org/xml/sax/AttributeList.java
/var/lib/mock/fedora-rawhide-i386/root/builddir/build/BUILD/javolution-5.5.1/src/main/java/org/xml/sax/DTDHandler.java
/var/lib/mock/fedora-rawhide-i386/root/builddir/build/BUILD/javolution-5.5.1/src/main/java/org/xml/sax/DocumentHandler.java
/var/lib/mock/fedora-rawhide-i386/root/builddir/build/BUILD/javolution-5.5.1/src/main/java/org/xml/sax/EntityResolver.java
/var/lib/mock/fedora-rawhide-i386/root/builddir/build/BUILD/javolution-5.5.1/src/main/java/org/xml/sax/ErrorHandler.java
/var/lib/mock/fedora-rawhide-i386/root/builddir/build/BUILD/javolution-5.5.1/src/main/java/org/xml/sax/HandlerBase.java
/var/lib/mock/fedora-rawhide-i386/root/builddir/build/BUILD/javolution-5.5.1/src/main/java/org/xml/sax/InputSource.java
/var/lib/mock/fedora-rawhide-i386/root/builddir/build/BUILD/javolution-5.5.1/src/main/java/org/xml/sax/Locator.java
/var/lib/mock/fedora-rawhide-i386/root/builddir/build/BUILD/javolution-5.5.1/src/main/java/org/xml/sax/Parser.java
/var/lib/mock/fedora-rawhide-i386/root/builddir/build/BUILD/javolution-5.5.1/src/main/java/org/xml/sax/SAXException.java
/var/lib/mock/fedora-rawhide-i386/root/builddir/build/BUILD/javolution-5.5.1/src/main/java/org/xml/sax/SAXParseException.java


remove bundled library
already available in xml-commons-apis package or in OpenJDK rt.jar

/var/lib/mock/fedora-rawhide-i386/root/builddir/build/BUILD/javolution-5.5.1/src/main/java/org/xml/sax/Attributes.java
/var/lib/mock/fedora-rawhide-i386/root/builddir/build/BUILD/javolution-5.5.1/src/main/java/org/xml/sax/ContentHandler.java
/var/lib/mock/fedora-rawhide-i386/root/builddir/build/BUILD/javolution-5.5.1/src/main/java/org/xml/sax/SAXNotRecognizedException.java
/var/lib/mock/fedora-rawhide-i386/root/builddir/build/BUILD/javolution-5.5.1/src/main/java/org/xml/sax/SAXNotSupportedException.java
/var/lib/mock/fedora-rawhide-i386/root/builddir/build/BUILD/javolution-5.5.1/src/main/java/org/xml/sax/XMLFilter.java
/var/lib/mock/fedora-rawhide-i386/root/builddir/build/BUILD/javolution-5.5.1/src/main/java/org/xml/sax/XMLReader.java

fix wrong-file-end-of-line-encoding /usr/share/doc/javolution/LICENSE.txt
e.g.
sed -i 's/\r//' LICENSE.txt

not approved

Comment 4 gil cattaneo 2013-09-18 20:50:22 UTC

Created attachment 799593 [details]
review notes

Comment 5 gil cattaneo 2013-09-18 20:56:06 UTC

(In reply to gil cattaneo from comment #3)
> please, remove from spec file these 
> 
> BuildRequires: javapackages-tools
> BuildRequires: maven
> BuildRequires: maven-compiler-plugin
> BuildRequires: maven-install-plugin
> BuildRequires: maven-javadoc-plugin
> BuildRequires: maven-source-plugin
> BuildRequires: maven-surefire-plugin
> 
> all Group fiels        - no more required
> all Requires           - added by java package tools (XMvn)
> 
> fix unowned directory
> add %dir %{_javadir}/%{name}
> in main package
> 
> if possible use pom macros
> e.g.
> %pom_remove_plugin javolution:colapi
> %pom_xpath_remove "pom:project/pom:build/pom:extensions[pom:groupId =
> 'org.jvnet.wagon-svn' ]"
> or
> %pom_xpath_remove "pom:project/pom:build/pom:extensions"
> 
> fix license field
> License:       BSD and Public domain

should be correct as this last one
License:       BSD
> 
> these are under  Public domain license
> 
because you must remove all bundled libraries
https://fedoraproject.org/wiki/Packaging:No_Bundled_Libraries
> already available in xml-commons-apis package or in OpenJDK rt.jar

> /var/lib/mock/fedora-rawhide-i386/root/builddir/build/BUILD/javolution-5.5.1/
> src/main/java/org/xml/sax/AttributeList.java
> /var/lib/mock/fedora-rawhide-i386/root/builddir/build/BUILD/javolution-5.5.1/
> src/main/java/org/xml/sax/DTDHandler.java
> /var/lib/mock/fedora-rawhide-i386/root/builddir/build/BUILD/javolution-5.5.1/
> src/main/java/org/xml/sax/DocumentHandler.java
> /var/lib/mock/fedora-rawhide-i386/root/builddir/build/BUILD/javolution-5.5.1/
> src/main/java/org/xml/sax/EntityResolver.java
> /var/lib/mock/fedora-rawhide-i386/root/builddir/build/BUILD/javolution-5.5.1/
> src/main/java/org/xml/sax/ErrorHandler.java
> /var/lib/mock/fedora-rawhide-i386/root/builddir/build/BUILD/javolution-5.5.1/
> src/main/java/org/xml/sax/HandlerBase.java
> /var/lib/mock/fedora-rawhide-i386/root/builddir/build/BUILD/javolution-5.5.1/
> src/main/java/org/xml/sax/InputSource.java
> /var/lib/mock/fedora-rawhide-i386/root/builddir/build/BUILD/javolution-5.5.1/
> src/main/java/org/xml/sax/Locator.java
> /var/lib/mock/fedora-rawhide-i386/root/builddir/build/BUILD/javolution-5.5.1/
> src/main/java/org/xml/sax/Parser.java
> /var/lib/mock/fedora-rawhide-i386/root/builddir/build/BUILD/javolution-5.5.1/
> src/main/java/org/xml/sax/SAXException.java
> /var/lib/mock/fedora-rawhide-i386/root/builddir/build/BUILD/javolution-5.5.1/
> src/main/java/org/xml/sax/SAXParseException.java
>

Comment 6 Pete MacKinnon 2013-09-19 15:31:29 UTC

Feedback applied and some additional BR removed.

However, 
adding %dir %{_javadir}/%{name} is not required as far as I can tell. This is a single jar package and thus there is no %{_javadir}/%{name}.

New spec and latest SRPM (0.2) at same location.

Comment 7 gil cattaneo 2013-09-19 17:11:15 UTC

yes, i see sorry, 
but you should add also the new source rpm
regards

Comment 8 gil cattaneo 2013-09-19 17:16:13 UTC

please. you should remove
Group field in the main package
and superfluous 
BuildArch:     noarch
in javadoc sub package
regards

Comment 9 gil cattaneo 2013-09-19 17:30:46 UTC

Spec URL: http://pmackinn.fedorapeople.org/javolution/javolution.spec
SRPM URL: http://pmackinn.fedorapeople.org/javolution/javolution-5.5.1-0.2.fc19.src.rpm

Comment 10 Pete MacKinnon 2013-09-19 18:47:43 UTC

Spec URL: http://pmackinn.fedorapeople.org/javolution/javolution.spec
SRPM URL: http://pmackinn.fedorapeople.org/javolution/javolution-5.5.1-0.3.fc19.src.rpm

Comment #8 re javadoc arc applied but most other Java packages have

Group:         Development/Libraries

so no change there. Why do you think that's necessary?

Comment 11 gil cattaneo 2013-09-19 20:02:40 UTC

(In reply to Pete MacKinnon from comment #10)
> Spec URL: http://pmackinn.fedorapeople.org/javolution/javolution.spec
> SRPM URL:
> http://pmackinn.fedorapeople.org/javolution/javolution-5.5.1-0.3.fc19.src.rpm
> 
> Comment #8 re javadoc arc applied but most other Java packages have
> 
> Group:         Development/Libraries
> 
> so no change there. Why do you think that's necessary?

see https://fedoraproject.org/wiki/Packaging:Guidelines?rd=Packaging/Guidelines#Group_tag
if you want use the package also in EPEL is requires, otherwise not necessary

Comment 12 gil cattaneo 2013-09-19 20:06:16 UTC

> Comment #8 re javadoc arc applied but most other Java packages have
usually if the main package is noarch also the sub package/s is/are noarch

Comment 13 gil cattaneo 2013-09-19 20:28:06 UTC

Created attachment 800144 [details]
new review notes

Comment 14 gil cattaneo 2013-09-19 20:28:37 UTC

approved

Comment 15 Pete MacKinnon 2013-09-27 13:50:27 UTC

New Package SCM Request
=======================
Package Name: javolution
Short Description: A Java library for real-time applications
Owners: pmackinn
Branches: f19 f20 f21
InitialCC: java-sig

Comment 16 Kevin Fenzi 2013-09-27 18:44:05 UTC

Git done (by process-git-requests).

There's no such thing as a f21 branch yet...

Comment 17 Fedora Update System 2013-12-03 00:57:49 UTC

javolution-5.5.1-0.4.fc20 has been submitted as an update for Fedora 20.
https://admin.fedoraproject.org/updates/javolution-5.5.1-0.4.fc20

Comment 18 Fedora Update System 2013-12-14 03:50:35 UTC

javolution-5.5.1-0.4.fc20 has been pushed to the Fedora 20 stable repository.