[Bug 439627] Review Request: gluegen - Java/JNI glue code generator to call out to ANSI C

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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=439627





--- Comment #12 from D Haley <mycae@xxxxxxxxx>  2009-03-06 02:16:41 EDT ---
A few off-the-cuff comments:

I think:
># vs -d :pserver:username@xxxxxxxxxxxxxxxx:/cvs checkout gluegen

Should be:
# cvs -d :pserver:username@xxxxxxxxxxxxxxxx:/cvs checkout gluegen

The error:
>gluegen.i386: W: no-documentation
Means that you should have the doc macro. Normally you would include things
such as a changelog, licence files and readme stuff, or other such things:
Its easy fixed just use something like:
%doc LICENSE.txt README CHANGELOG
or similar in your %files section.

>gluegen.i386: W: non-standard-group Libraries

Change:
Group:  Libraries

To:
Group:  Development/Libraries

For each instance

>gluegen.i386: W: class-path-in-manifest /usr/share/java/gluegen-1.0.jar
Looks like your jar file has a MANIFEST in it. You need to sort this out, not
sure exactly what is the best way.

>gluegen.i386: W: no-soname /usr/lib/libgluegen-rt.so
You may need to tell GCC what soname to use. See bug 464781 comment 27 

>gluegen-manual.i386: W: spurious-executable-perm
Sounds like "/usr/share/doc/gluegen-manual-1.0/manual/index.html"  in your rpm
has bad permissions. You may need to chmod it.

>gluegen.i386: W: unstripped-binary-or-object /usr/lib/libgluegen-rt.so
.so files need to be executable for rpmbuild to strip them (I fell into this
trap).


Can this be re-opened?

-- 
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.

_______________________________________________
Fedora-package-review mailing list
Fedora-package-review@xxxxxxxxxx
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]