Bug 629660 - Review Request: apache-poi - The Java API for Microsoft Documents
Summary: Review Request: apache-poi - The Java API for Microsoft Documents
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Terje Røsten
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: 629664
TreeView+ depends on / blocked
 
Reported: 2010-09-02 16:01 UTC by Orion Poplawski
Modified: 2010-12-21 10:21 UTC (History)
6 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2010-12-21 10:21:40 UTC
Type: ---
Embargoed:
terje.rosten: fedora-review+


Attachments (Terms of Use)

Description Orion Poplawski 2010-09-02 16:01:28 UTC
Spec URL: http://www.cora.nwra.com/~orion/fedora/apache-poi.spec
SRPM URL: http://www.cora.nwra.com/~orion/fedora/apache-poi-3.7-0.2.beta2.fc13.src.rpm

Description:
The Apache POI Project's mission is to create and maintain Java APIs for
manipulating various file formats based upon the Office Open XML standards
(OOXML) and Microsoft's OLE 2 Compound Document format (OLE2). In short, you
can read and write MS Excel files using Java. In addition, you can read and
write MS Word and MS PowerPoint files using Java. Apache POI is your Java
Excel solution (for Excel 97-2008). We have a complete API for porting other
OOXML and OLE2 formats and welcome others to participate.

OLE2 files include most Microsoft Office files such as XLS, DOC, and PPT as
well as MFC serialization API based file formats. The project provides APIs
for the OLE2 Filesystem (POIFS) and OLE2 Document Properties (HPSF).

Office OpenXML Format is the new standards based XML file format found in
Microsoft Office 2007 and 2008. This includes XLSX, DOCX and PPTX. The
project provides a low level API to support the Open Packaging Conventions
using openxml4j.

For each MS Office application there exists a component module that attempts
to provide a common high level Java API to both OLE2 and OOXML document
formats. This is most developed for Excel workbooks (SS=HSSF+XSSF). Work is
progressing for Word documents (HWPF+XWPF) and PowerPoint presentations
(HSLF+XSLF).

The project has recently added support for Outlook (HSMF). Microsoft opened
the specifications to this format in October 2007. We would welcome
contributions.

There are also projects for Visio (HDGF) and Publisher (HPBF). 

http://koji.fedoraproject.org/koji/taskinfo?taskID=2442725

Currently most builds fail in koji due to OOM our of heap space errors.  I've filed: https://fedorahosted.org/fedora-infrastructure/ticket/2372

Comment 1 Terje Røsten 2010-10-27 18:43:16 UTC
Package looks clean, however the beta2 release is gone, could you update to beta3 and I will do
a full review.

Some initial comments:
o some of the buildrequires should also be requires?

Comment 2 Orion Poplawski 2010-10-27 20:54:33 UTC
* Wed Oct 27 2010 Orion Poplawski <orion.com> - 3.7-0.3.beta3
- Update to 3.7 beta3
- Add more Requires

http://www.cora.nwra.com/~orion/fedora/apache-poi.spec
http://www.cora.nwra.com/~orion/fedora/apache-poi-3.7-0.3.beta3.fc14.src.rpm

http://koji.fedoraproject.org/koji/taskinfo?taskID=2558622

Comment 3 Terje Røsten 2010-10-28 19:23:28 UTC
MUST Items:

+ rpmlint output ok

apache-poi.src:12: W: macro-in-comment %{version}
apache-poi.src:12: W: macro-in-comment %{reldate}
apache-poi.src: W: invalid-url Source1: http://www.ecma-international.org/publications/files/ECMA-ST/Office Open XML 1st edition\
 Part 4 (PDF).zip HTTP Error 404: Not Found
apache-poi-javadoc.noarch: W: spelling-error Summary(en_US) Javadocs -> Java docs, Java-docs, Javanese

Only harmless things here.

+ named and versioned according to the Package Naming Guidelines
+ spec file name matches base package name
+ complies with all the legal guidelines:
  + License: Apache 2.0 valid, matches actual license
  ? No known patent problems
  + No emulator, no firmware, no binary-only or prebuilt components
+ LICENSE packaged as %doc
+ source matches upstream:
  sha1sum:
28913957f7e98a37c1e42441c828f066659bfae9  poi-src-3.7-beta3-20100924.tar.gz
28913957f7e98a37c1e42441c828f066659bfae9  poi-src-3.7-beta3-20100924.tar.gz.pkg
1b3ddfa3ad13a9138f5399bdc4c10b54dd9df9d0  Office Open XML 1st edition Part 4 (PDF).zip
1b3ddfa3ad13a9138f5399bdc4c10b54dd9df9d0  Office Open XML 1st edition Part 4 (PDF).zip.pkg

+ builds on at least one arch (f15 Koji scratch build)
+ no known non-working arches, so no ExcludeArch needed
+ no missing BuildRequires (builds in mock)
+ no translations, so translation/locale guidelines don't apply
+ no shared libraries, so no ldconfig calls neeed
+ no duplicated system libraries
+ package not relocatable
! directory ownership correct
 - apache-poi-manual seems strange? No useful files included?
+ no duplicate files in %files
+ permissions correct, defattr used correctly
+ macros used where possible
+ no non-code content
+ no large documentation files, so no -doc package needed
+ no %doc files required at runtime
+ no devel symlinks which would need to be in a -devel subpackage
+ all filenames are valid UTF-8
+ other packaging guidelines:
  + complies with the FHS
  + proper changelog, tags, BuildRoot, BuildRequires, Summary, Description
  ! no macros in Summary and Description
  + no non-UTF-8 characters
  + all relevant documentation included as %doc
  + no conflicts

SHOULD Items:
+ license already included upstream
+ no translations for description and summary provided by upstream
+ scriptlets are sane
+ no file dependencies

Java

+ jar naming ok
+ directory structure ok
+ buildrequires and requires
+ no pre-built JAR files ok
+ ant ok


Summary: 
 o could you comment on the patent situation, this being MS relates stuff.
 o manual package seems strange as is
 o macros used in descriptions, please avoid that.

Unsure: 
I see the package don't include pom files, should they be included?

Comment 4 Orion Poplawski 2010-11-01 17:33:33 UTC
- We have lots of packages (e.g. OpenOffice) that can read and create Microsoft file formats.  I don't think there are any patent issues here.

- Descriptions are straight from http://fedoraproject.org/wiki/Packaging/Java#ant_2  I see nothing wrong with having %{name} in the descriptions.

- Fixed manual package

http://www.cora.nwra.com/~orion/fedora/apache-poi.spec
http://www.cora.nwra.com/~orion/fedora/apache-poi-3.7-0.4.beta3.fc14.src.rpm

Thanks!

Comment 5 Tom "spot" Callaway 2010-11-01 19:20:35 UTC
I need to run this past Red Hat Legal. Blocking FE-Legal.

Comment 6 Tom "spot" Callaway 2010-11-08 18:11:04 UTC
There are no legal concerns with apache-poi at this time. Lifting FE-Legal.

Comment 7 Terje Røsten 2010-11-08 19:20:28 UTC
Thanks Tom!

I think some kind information is generated without expanding macros. Such will be wrong
when using %{name} in  %description section. I can't find the reference. Well, well, 
do as you wish.

My last concern is about POM files. Any input here? I have package 
which is built by maven and depend om poi, I then had to add a custom depmap for poi.

Any way, I leave that to you. 

The package apache-poi is APPROVED.

Comment 8 Orion Poplawski 2010-11-08 19:38:38 UTC
I'm happy to add a pom if you can point me to a good one for poi.  Thanks!

New Package SCM Request
=======================
Package Name: apache-poi
Short Description: The Java API for Microsoft Documents
Owners: orion
Branches: f14 f13 el6 el5
InitialCC:

Comment 9 Terje Røsten 2010-11-08 19:47:00 UTC
This one might help: http://repo2.maven.org/maven2/org/apache/poi/poi/3.7-beta3/poi-3.7-beta3.pom

Comment 10 Orion Poplawski 2010-11-08 22:03:18 UTC
http://www.cora.nwra.com/~orion/fedora/apache-poi.spec
http://www.cora.nwra.com/~orion/fedora/apache-poi-3.7-0.5.beta3.fc14.src.rpm

* Mon Nov 8 2010 Orion Poplawski <orion.com> - 3.7-0.5.beta3
- Add pom

Can you give that a try?

Comment 11 Terje Røsten 2010-11-09 18:33:46 UTC
There are too many changes in rawhide for my package to build (not related to poi), however the POM and depmap looks good. Let's go for it.  

BTW: should jakarta-commons-logging deps be switched to apache-commons-logging?

Comment 12 Orion Poplawski 2010-11-09 20:29:25 UTC
(In reply to comment #11)
> There are too many changes in rawhide for my package to build (not related to
> poi), however the POM and depmap looks good. Let's go for it.  

Great.  Just waiting for the package to be created then.

> BTW: should jakarta-commons-logging deps be switched to apache-commons-logging?

Yup, done.

Comment 13 Kevin Fenzi 2010-11-10 03:14:09 UTC
Git done.

Comment 14 Terje Røsten 2010-12-06 08:28:51 UTC
Would be nice to get this closed. Orion, please import.

Comment 15 Orion Poplawski 2010-12-06 15:34:28 UTC
Sorry, just forgot to close the bug.  Imported and built.

Comment 16 Terje Røsten 2010-12-06 19:53:03 UTC
git seems fine, however I can't find any useful builds in koji?

Comment 17 Orion Poplawski 2010-12-06 20:54:36 UTC
Oh, right, I'm waiting for the build system to be able to handle files with spaces in it.  According to https://fedorahosted.org/fedora-packager/ticket/85, this should be soon.

Comment 18 Alexander Kurtakov 2010-12-21 10:21:40 UTC
URL encoded source works. Build done:
http://koji.fedoraproject.org/koji/buildinfo?buildID=210900


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