Bug 485424

- please add a comment stating why we're only building the feature we are
- please add information about steps to get us to a full BIRT here:  http://fedoraproject.org/wiki/Eclipse
- please add a comment above the one patch stating why we're doing this
- let's change the variable "qualifier" to "rhinoqualifier" to avoid confusion
- should we be using the same qualifier as BIRT like we do with Mylyn?

Lines beginning with X need attention; those beginning with * are okay:

* verify the final provides and requires of the binary RPMs
X make sure lines are <= 80 characters
  - please add some line continuations to fix this where possible
* package successfully compiles and builds
* BuildRequires are proper
* macros fine
* package is named appropriately
* it is legal for Fedora to distribute this
* license field matches the actual license.
* license is open source-compatible.
* specfile name matches %{name}
* md5sum matches upstream
* skim the summary and description for typos, etc.
* summary and description good
* correct buildroot
* %{?dist} used correctly
* license text included in package and marked with %doc
* packages meets FHS (http://www.pathname.com/fhs/)
* rpmlint on <this package>.srpm gives no output
* changelog format okay
* Summary tag does not end in a period
* no PreReq
* specfile is legible
* specfile written in American English
* no -doc sub-package necessary
* not native, so no rpath, static linking, etc.
* no config files
* not a GUI app
* no -devel necessary
* install section begins with rm -rf $RPM_BUILD_ROOT or %{buildroot}
* no translations so no locale handling
* no Requires(pre,post)
* package not relocatable
* package contains code
* package owns all directories and files
* no %files duplicates
* file permissions fine
* %clean present
* %doc files do not affect runtime
* not a web app
* package includes license text in the package and marks it with %doc
* run rpmlint on the binary RPMs => no output

$ rpmlint ../RPMS/noarch/eclipse-birt-2.3.1-1.fc10.noarch.rpm
eclipse-birt.noarch: W: dangling-relative-symlink /usr/share/eclipse/dropins/birt/eclipse/plugins/batik-xml.jar ../../../../../java/batik/batik-xml.jar
eclipse-birt.noarch: W: dangling-relative-symlink /usr/share/eclipse/dropins/birt/eclipse/plugins/batik-parser.jar ../../../../../java/batik/batik-parser.jar
eclipse-birt.noarch: W: dangling-relative-symlink /usr/share/eclipse/dropins/birt/eclipse/plugins/fop.jar ../../../../../java/fop.jar
eclipse-birt.noarch: W: dangling-relative-symlink /usr/share/eclipse/dropins/birt/eclipse/plugins/batik-svg-dom.jar ../../../../../java/batik/batik-svg-dom.jar
eclipse-birt.noarch: W: dangling-relative-symlink /usr/share/eclipse/dropins/birt/eclipse/plugins/org.apache.xml.resolver_1.2.0.jar ../../../../../java/xml-commons-resolver.jar
eclipse-birt.noarch: W: dangling-relative-symlink /usr/share/eclipse/dropins/birt/eclipse/plugins/javax.xml_1.3.4.jar ../../../../../java/xml-commons-apis.jar
eclipse-birt.noarch: W: dangling-relative-symlink /usr/share/eclipse/dropins/birt/eclipse/plugins/xml-commons-apis-ext.jar ../../../../../java/xml-commons-apis-ext.jar
eclipse-birt.noarch: W: dangling-relative-symlink /usr/share/eclipse/dropins/birt/eclipse/plugins/batik-dom.jar ../../../../../java/batik/batik-dom.jar
eclipse-birt.noarch: W: dangling-relative-symlink /usr/share/eclipse/dropins/birt/eclipse/plugins/org.apache.xerces_2.9.0.jar ../../../../../java/xerces-j2.jar
eclipse-birt.noarch: W: dangling-relative-symlink /usr/share/eclipse/dropins/birt/eclipse/plugins/batik-svggen.jar ../../../../../java/batik/batik-svggen.jar
eclipse-birt.noarch: W: dangling-relative-symlink /usr/share/eclipse/dropins/birt/eclipse/plugins/batik-awt-util.jar ../../../../../java/batik/batik-awt-util.jar
eclipse-birt.noarch: W: dangling-relative-symlink /usr/share/eclipse/dropins/rhino/eclipse/plugins/org.mozilla.rhino_1.6.7.v20080214/lib/js.jar ../../../../../../../java/js.jar
eclipse-birt.noarch: W: dangling-relative-symlink /usr/share/eclipse/dropins/birt/eclipse/plugins/batik-swing.jar ../../../../../java/batik/batik-swing.jar
eclipse-birt.noarch: W: dangling-relative-symlink /usr/share/eclipse/dropins/birt/eclipse/plugins/batik-css.jar ../../../../../java/batik/batik-css.jar
eclipse-birt.noarch: W: dangling-relative-symlink /usr/share/eclipse/dropins/birt/eclipse/plugins/org.apache.commons.codec_1.3.0.jar ../../../../../java/commons-codec.jar
eclipse-birt.noarch: W: dangling-relative-symlink /usr/share/eclipse/dropins/birt/eclipse/plugins/batik-extension.jar ../../../../../java/batik/batik-extension.jar
eclipse-birt.noarch: W: dangling-relative-symlink /usr/share/eclipse/dropins/birt/eclipse/plugins/javax.wsdl_1.5.0.jar ../../../../../java/wsdl4j.jar
eclipse-birt.noarch: W: dangling-relative-symlink /usr/share/eclipse/dropins/birt/eclipse/plugins/batik-util.jar ../../../../../java/batik/batik-util.jar
eclipse-birt.noarch: W: dangling-relative-symlink /usr/share/eclipse/dropins/birt/eclipse/plugins/batik-bridge.jar ../../../../../java/batik/batik-bridge.jar
eclipse-birt.noarch: W: dangling-relative-symlink /usr/share/eclipse/dropins/birt/eclipse/plugins/sac.jar ../../../../../java/sac.jar
eclipse-birt.noarch: W: dangling-relative-symlink /usr/share/eclipse/dropins/birt/eclipse/plugins/batik-transcoder.jar ../../../../../java/batik/batik-transcoder.jar
eclipse-birt.noarch: W: dangling-relative-symlink /usr/share/eclipse/dropins/birt/eclipse/plugins/org.apache.xml.serializer_2.7.1.jar ../../../../../java/xalan-j2-serializer.jar
eclipse-birt.noarch: W: dangling-relative-symlink /usr/share/eclipse/dropins/birt/eclipse/plugins/batik-gui-util.jar ../../../../../java/batik/batik-gui-util.jar
1 packages and 0 specfiles checked; 0 errors, 23 warnings.

These are fine as they're all relative symlinks.

* I verified that it installs and that the chart feature is picked up by plugins requiring it

Spec URL:
http://akurtakov.fedorapeople.org/eclipse-birt.spec
SRPM URL:
http://akurtakov.fedorapeople.org/eclipse-birt-2.3.1-2.fc10.src.rpm


(In reply to comment #1)
> - please add a comment stating why we're only building the feature we are
Fixed. It was already done but more details added.
> - please add information about steps to get us to a full BIRT here: 
> http://fedoraproject.org/wiki/Eclipse
Fixed. Same as previous.
> - please add a comment above the one patch stating why we're doing this
Fixed.
> - let's change the variable "qualifier" to "rhinoqualifier" to avoid confusion
Fixed.
> - should we be using the same qualifier as BIRT like we do with Mylyn?
No. Different BIRT plugins hava different qualifiers so it's not applicable.
> 
> Lines beginning with X need attention; those beginning with * are okay:
> 
> * verify the final provides and requires of the binary RPMs
> X make sure lines are <= 80 characters
>   - please add some line continuations to fix this where possible
Fixed.

Thanks.  This package is approved.

New Package CVS Request
=======================
Package Name: eclipse-birt
Short Description:  Eclipse-based reporting system
Owners: akurtakov
Branches: 
InitialCC: akurtakov

cvs done.

Builded in rawhide.

Builded in rawhide.

Close.