Bug 516074 - Review Request: jide-oss - Swing component library built on top of Java/Swing
Summary: Review Request: jide-oss - Swing component library built on top of Java/Swing
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Susi Lehtola
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: 513119
TreeView+ depends on / blocked
 
Reported: 2009-08-06 15:49 UTC by Hicham HAOUARI
Modified: 2009-08-10 21:45 UTC (History)
3 users (show)

Fixed In Version: 2.7.1-3.1181svn.fc10
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2009-08-10 21:42:55 UTC
Type: ---
Embargoed:
susi.lehtola: fedora-review+
kevin: fedora-cvs+


Attachments (Terms of Use)

Description Hicham HAOUARI 2009-08-06 15:49:00 UTC
Spec URL: http://hicham.fedorapeople.org/jide-oss.spec
SRPM URL: http://hicham.fedorapeople.org/jide-oss-2.7.1-1.20090805svn.fc11.src.rpm
Description: 
JIDE Common Layer is Swing component library built on top of Java/Swing.
It is also the foundation of other component products from JIDE.
This project has over 30 Swing components and over 100k lines of code.
It greatly enhanced the default component set provided by Swing and allow 
developers to focus on business logic layer instead of making components.

JIDE Common Layer was originally developed by JIDE Software developers
as a foundation in order to build other more advanced components.
In April of 2007, JIDE Software decided to open source this common layer
so that more and more developers can leverage them instead of wasting time
building them again.

rpmlint output : none

koji build page :
http://koji.fedoraproject.org/koji/taskinfo?taskID=1586658

Comment 1 Susi Lehtola 2009-08-06 16:07:33 UTC
Use the SVN revision in the tarball name and revision, e.g.

# SVN revision used
%global svnrel 1181

Release: 1.%{svnrel}svn%{?dist}

#This is a cvs snapshot, to create this tarball
# svn co -r %{svnrel} https://jide-oss.dev.java.net/svn/jide-oss/branches/trunk_2.7.1/ jide-oss --username guest
#create the tarball : tar -cjvf jide-oss-2.7.1.tar.bz2 jide-oss

**

Still, don't version the jar as it isn't versioned by upstream.
http://fedoraproject.org/wiki/Packaging/Java#Package_naming

Comment 2 Susi Lehtola 2009-08-06 16:10:59 UTC
Also, you can drop BR: dos2unix if you just use
https://fedoraproject.org/wiki/PackagingTricks#Remove_DOS_line_endings

Comment 3 Hicham HAOUARI 2009-08-06 16:52:20 UTC
the jar is versioned by upstream.

I updated the spec file.

Spec URL: http://hicham.fedorapeople.org/jide-oss.spec
SRPM URL:
http://hicham.fedorapeople.org/jide-oss-2.7.1-2.1181svn.fc11.src.rpm
Description: 
JIDE Common Layer is Swing component library built on top of Java/Swing.
It is also the foundation of other component products from JIDE.
This project has over 30 Swing components and over 100k lines of code.
It greatly enhanced the default component set provided by Swing and allow 
developers to focus on business logic layer instead of making components.

JIDE Common Layer was originally developed by JIDE Software developers
as a foundation in order to build other more advanced components.
In April of 2007, JIDE Software decided to open source this common layer
so that more and more developers can leverage them instead of wasting time
building them again.

rpmlint output : none

Comment 4 Susi Lehtola 2009-08-06 17:16:15 UTC
(In reply to comment #3)
> the jar is versioned by upstream.

Sorry, my bad :)

Comment 5 Hicham HAOUARI 2009-08-06 17:41:52 UTC
is the package ok now ?

Comment 6 Susi Lehtola 2009-08-06 18:34:19 UTC
You don't have to create an account to check out, you can use --username guest as implicitly pointed out in comment #1.

**

You should send the patches upstream, as they're probably needed by a lot of people.

**

rpmlint is clean.

existing class files and jars are correctly removed in %prep.

MUST: The package does not yet exist in Fedora. The Review Request is not a duplicate. OK
MUST: The spec file for the package is legible and macros are used consistently. OK
MUST: The package must be named according to the Package Naming Guidelines. OK
MUST: The spec file name must match the base package %{name}. OK
MUST: The package must be licensed with a Fedora approved license and meet the  Licensing Guidelines. OK

MUST: The License field in the package spec file must match the actual license. OK
- Excerpt from LICENSE.txt 
 JIDE Common Layer is a dual-licensed. The two licenses are GPL with classpath  
 exception and free commercial license.

 The first license is GPL with classpath exception. This is the same license 
 under which the Java platform is released. You can read the license agreement 
 from the link below.

 GPL v2: http://www.gnu.org/licenses/gpl.txt
 Classpath Exception: http://www.gnu.org/software/classpath/license.html
- SwingWorker is LGPLv2+ but that's compatible with GPLv2.

MUST: The sources used to build the package must match the upstream source, as provided in the spec URL. OK
MUST: The package MUST successfully compile and build into binary rpms. OK
MUST: The spec file MUST handle locales properly. N/A
MUST: Optflags are used and time stamps preserved. OK
MUST: A package must own all directories that it creates or require the package that owns the directory. OK
MUST: Files only listed once in %files listings. OK
MUST: Debuginfo package is complete. N/A
MUST: Permissions on files must be set properly. OK
MUST: Clean section exists. OK

MUST: Large documentation files must go in a -doc subpackage. NEEDSWORK
- There is a docs/JIDE_Common_Layer_Developer_Guide.pdf, which should go into -doc. (There's a .doc file too, but IMHO the pdf is better.)

MUST: All relevant items are included in %doc. Items in %doc do not affect runtime of application. OK
MUST: Desktop files are installed properly. N/A
MUST: No file conflicts with other packages and no general names. OK
MUST: Buildroot cleaned before install. OK
SHOULD: %{?dist} tag is used in release. OK

SHOULD: If the package does not include license text(s) as separate files from upstream, the packager should query upstream to include it. NEEDSWORK
- There is a http link to the license in LICENSE.txt, though, so this is not necessary.

SHOULD: The package builds in mock. OK

**

The package has been

APPROVED


Before CVS import add the -doc package. (Remember, no need to define it explicitly as noarch since the main package already is noarch.)

Comment 7 Hicham HAOUARI 2009-08-07 00:50:04 UTC
the doc package should include the pdf file only?

Comment 8 Susi Lehtola 2009-08-07 05:04:49 UTC
(In reply to comment #7)
> the doc package should include the pdf file only?  

Yes, unless there's some other (biggish) documentation files I didn't find.

I don't think there's is any sense in including the .doc in addition to the .pdf if their contents is identical - it's easier to get the .pdf displayed correctly.

Comment 9 Hicham HAOUARI 2009-08-07 12:20:09 UTC
which location should i use for the doc package?

Comment 10 Susi Lehtola 2009-08-07 12:23:12 UTC
What do you mean by location?

%files doc
%defattr(-,root,root,-)
%doc docs/JIDE_Common_Layer_Developer_Guide.pdf

Comment 11 Hicham HAOUARI 2009-08-07 12:45:32 UTC
updated the spec file.

Spec URL: http://hicham.fedorapeople.org/jide-oss.spec
SRPM URL:
http://hicham.fedorapeople.org/jide-oss-2.7.1-3.1181svn.fc11.src.rpm

Comment 12 Susi Lehtola 2009-08-07 13:07:36 UTC
Summary:	User documentation for %{name}
                ^^^^

Isn't this more like *developer* documentation? :)

Comment 13 Hicham HAOUARI 2009-08-07 13:16:09 UTC
updated the spec file.

Spec URL: http://hicham.fedorapeople.org/jide-oss.spec

Comment 14 Susi Lehtola 2009-08-07 13:28:16 UTC
Yes, yes, import the package already!

And update the other spec files so I can review them.

Comment 15 Hicham HAOUARI 2009-08-07 14:09:46 UTC
New Package CVS Request
=======================
Package Name: jide-oss
Short Description: Swing component library built on top of Java/Swing
Owners: hicham
Branches: F-11
InitialCC: hicham

Comment 16 Susi Lehtola 2009-08-07 18:04:45 UTC
Why no F-10 branch?

Comment 17 Hicham HAOUARI 2009-08-07 18:11:19 UTC
New Package CVS Request
=======================
Package Name: jide-oss
Short Description: Swing component library built on top of Java/Swing
Owners: hicham
Branches: F-10 F-11
InitialCC: hicham

Comment 18 Kevin Fenzi 2009-08-07 19:59:11 UTC
cvs done.

Comment 19 Fedora Update System 2009-08-08 00:37:04 UTC
jide-oss-2.7.1-3.1181svn.fc11 has been submitted as an update for Fedora 11.
http://admin.fedoraproject.org/updates/jide-oss-2.7.1-3.1181svn.fc11

Comment 20 Fedora Update System 2009-08-08 00:38:37 UTC
jide-oss-2.7.1-3.1181svn.fc10 has been submitted as an update for Fedora 10.
http://admin.fedoraproject.org/updates/jide-oss-2.7.1-3.1181svn.fc10

Comment 21 Fedora Update System 2009-08-08 19:23:26 UTC
jide-oss-2.7.1-3.1181svn.fc11 has been pushed to the Fedora 11 testing repository.  If problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update jide-oss'.  You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F11/FEDORA-2009-8370

Comment 22 Fedora Update System 2009-08-08 19:24:22 UTC
jide-oss-2.7.1-3.1181svn.fc10 has been pushed to the Fedora 10 testing repository.  If problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update jide-oss'.  You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F10/FEDORA-2009-8376

Comment 23 Fedora Update System 2009-08-10 21:42:51 UTC
jide-oss-2.7.1-3.1181svn.fc11 has been pushed to the Fedora 11 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 24 Fedora Update System 2009-08-10 21:45:07 UTC
jide-oss-2.7.1-3.1181svn.fc10 has been pushed to the Fedora 10 stable repository.  If problems still persist, please make note of it in this bug report.


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