[Bug 529084] Review Request: javatar - Java tar archive io package

bugzilla at redhat.com bugzilla at redhat.com
Mon Oct 26 23:37:31 UTC 2009


Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=529084





--- Comment #2 from Orion Poplawski <orion at cora.nwra.com>  2009-10-26 19:37:30 EDT ---
Spec URL: http://www.cora.nwra.com/~orion/fedora/javatar.spec
SRPM URL: http://www.cora.nwra.com/~orion/fedora/javatar-2.5-2.fc12.src.rpm

* Mon Oct 26 2009 Orion Poplawski <orion at cora.nwra.com> - 2.5-2
- Remove existing class files
- Package javadocs
- Make sure debuginfo has source files

(In reply to comment #1)
> In the build log, I see this:
> 
> deploy:
>       [jar] Building jar:
> /builddir/build/BUILD/javatar-2.5/tar-2.5/jars/tar.jar
>       [jar] Manifest warning: "Name" attributes should not occur in the main
> section and must be the first element in all other sections: "Name: "Java Tar""
> 
> Is this caused by the sed invocation on the manifest file?  If so, is that
> invocation correct?

The sed simply deletes the existing Class-Path: entry.  I've got no idea about
this.  I'll post the to the fedora java devel list.

>  Also, the source archive contains .class files in
> classes/com/ice/tar.  Would you mind deleting those in %prep, just to be sure
> they don't affect compilation?

Done.

> The javadoc documentation is neither rebuilt nor packaged.  Would you consider
> invoking "ant javadoc", and putting the contents of doc/api into a -javadoc
> subpackage?

It is actually getting built, just not packaged.  Fixed.

> Here is the output of rpmlint:
> 
> javatar-debuginfo.x86_64: E: debuginfo-without-sources
> 
> The warnings are just part of life with GCJ and can be ignored.  The error is a
> problem.  Why are there no sources in the debuginfo package?

I think aot-compile-rpm gets confused with the copy of source files.  Removing
that seems to have fixed it.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.




More information about the package-review mailing list