[Bug 452251] Review Request: libmatthew-java - collection of java libraries

2008-06-28 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: libmatthew-java - collection of java libraries


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution||CURRENTRELEASE
   Fixed In Version||0.7.1-1.fc9




-- 
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, or are watching someone who is.

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


[Bug 452251] Review Request: libmatthew-java - collection of java libraries

2008-06-28 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: libmatthew-java - collection of java libraries


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





--- Additional Comments From [EMAIL PROTECTED]  2008-06-28 18:16 EST ---
libmatthew-java-0.7.1-1.fc9 has been pushed to the Fedora 9 stable repository.  
If problems still persist, please make note of it in this bug report.

-- 
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, or are watching someone who is.

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


[Bug 452251] Review Request: libmatthew-java - collection of java libraries

2008-06-27 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: libmatthew-java - collection of java libraries


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2008-06-27 13:00 EST ---
cvs done.

-- 
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, or are watching someone who is.

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


[Bug 452251] Review Request: libmatthew-java - collection of java libraries

2008-06-27 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: libmatthew-java - collection of java libraries


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





--- Additional Comments From [EMAIL PROTECTED]  2008-06-27 16:25 EST ---
libmatthew-java-0.7.1-1.fc9 has been submitted as an update for Fedora 9

-- 
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, or are watching someone who is.

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


[Bug 452251] Review Request: libmatthew-java - collection of java libraries

2008-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: libmatthew-java - collection of java libraries


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





--- Additional Comments From [EMAIL PROTECTED]  2008-06-26 11:58 EST ---
New Package CVS Request
===
Package Name: libmatthew-java
Short Description: collection of java libraries
Owners: omajid
Branches: F-9
InitialCC:
Cvsextras Commits: yes


-- 
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, or are watching someone who is.

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


[Bug 452251] Review Request: libmatthew-java - collection of java libraries

2008-06-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: libmatthew-java - collection of java libraries


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO|177841  |
  nThis||




-- 
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, or are watching someone who is.

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


[Bug 452251] Review Request: libmatthew-java - collection of java libraries

2008-06-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: libmatthew-java - collection of java libraries


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





--- Additional Comments From [EMAIL PROTECTED]  2008-06-25 10:33 EST ---
Updated:

Spec URL: http://omajid.fedorapeople.org/libmatthew-java/libmatthew-java.spec
SRPM URL:
http://omajid.fedorapeople.org/libmatthew-java/libmatthew-java-0.7.1-1.fc9.src.rpm

replaced ld with gcc; passed %{optflags} to cflags, ppflags, ldflags and 
gcjflags

-- 
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, or are watching someone who is.

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


[Bug 452251] Review Request: libmatthew-java - collection of java libraries

2008-06-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: libmatthew-java - collection of java libraries


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-review+, fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2008-06-25 11:06 EST ---
Just one more change request.  It may produce the same result, but the more
conventional way to link a shared object is to pass -shared to gcc.  So I'd
recommend replacing:

-Wl,-soname=$@

in soname.patch with

-shared

You should work with upstream to clean up these build scripts.  Most importantly
they should not be invoking the linker directly, but there's also the make
variables: specifying CFLAGS on the make line should add to, not override, the
tools options.  So if upstream is fixed you should only need:

make %{?_smp_mflags} \
CFLAGS='%{optflags}'\
GCJFLAGS='%{optflags}' \
LDFLAGS='%{optflags}' \
PPFLAGS='%{optflags}'

Please make the -shared change when you commit the package.

Approved.

-- 
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, or are watching someone who is.

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


[Bug 452251] Review Request: libmatthew-java - collection of java libraries

2008-06-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: libmatthew-java - collection of java libraries


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





--- Additional Comments From [EMAIL PROTECTED]  2008-06-25 16:35 EST ---
Updated

Spec URL: http://omajid.fedorapeople.org/libmatthew-java/libmatthew-java.spec
SRPM URL:
http://omajid.fedorapeople.org/libmatthew-java/libmatthew-java-0.7.1-1.fc9.src.rpm

also includes a patch (sent it upstream too) to match 
make %{?_smp_mflags} \
CFLAGS='%{optflags}'\
GCJFLAGS='%{optflags}' \
LDFLAGS='%{optflags}' \
PPFLAGS='%{optflags}'


-- 
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, or are watching someone who is.

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


[Bug 452251] Review Request: libmatthew-java - collection of java libraries

2008-06-24 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: libmatthew-java - collection of java libraries


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||452688
  nThis||




-- 
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, or are watching someone who is.

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


[Bug 452251] Review Request: libmatthew-java - collection of java libraries

2008-06-24 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: libmatthew-java - collection of java libraries


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





--- Additional Comments From [EMAIL PROTECTED]  2008-06-24 11:43 EST ---
Clipping error on rpmlint output:

$ rpmlint RPMS/i386/libmatthew-java-0.7.1-1.fc9.i386.rpm 
libmatthew-java.i386: W: no-soname /usr/lib/libcgi-java.so
libmatthew-java.i386: W: no-soname /usr/lib/libunix-java.so

This warning must be fixed.

-- 
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, or are watching someone who is.

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


[Bug 452251] Review Request: libmatthew-java - collection of java libraries

2008-06-24 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: libmatthew-java - collection of java libraries


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2008-06-24 11:41 EST ---
- MUST: rpmlint must be run on every package. The output should be posted...

$ rpmlint SRPMS/libmatthew-java-0.7.1-1.fc9.src.rpm
$ rpmlint RPMS/i386/libmatthew-java-0.7.1-1.fc9.i386.rpm
-soname /usr/lib/libcgi-java.so
-soname /usr/lib/libunix-java.so
$ rpmlint RPMS/i386/libmatthew-java-javadoc-0.7.1-1.fc9.i386.rpm
$ rpmlint RPMS/i386/libmatthew-java-debuginfo-0.7.1-1.fc9.i386.rpm

- MUST: The package must be named according to the Package Naming Guideli...

OK

- MUST: The spec file name must match the base package %{name}, in the fo...

OK

- MUST: The package must meet the Packaging Guidelines .

OK

- MUST: The package must be licensed with a Fedora approved license and m...

OK

- MUST: The License field in the package spec file must match the actual ...

OK

- MUST: If (and only if) the source package includes the text of the lice...

OK

- MUST: The spec file must be written in American English.

OK

- MUST: The spec file for the package MUST be legible. If the reviewer is...

OK

- MUST: The sources used to build the package must match the upstream sou...

I'm not sure what this comment means:

# debian version is newer than upstream

Source must come from the actual upstream, not from another
distribution.  If Debian's version is better than upstream then add
another patch with those changes.

- MUST: The package must successfully compile and build into binary rpms ...

OK

- MUST: If the package does not successfully compile, build or work on an...

OK

- MUST: All build dependencies must be listed in BuildRequires, except fo...

OK

- MUST: The spec file MUST handle locales properly. This is done by using...

OK

- MUST: Every binary RPM package which stores shared library files (not j...

The JNI objects should go in %{_libdir}/%{name}, not %{_libdir}.  See:

http://fedoraproject.org/wiki/Packaging/Java#Packaging_JAR_files_that_use_JNI

- MUST: If the package is designed to be relocatable, the packager must s...

OK

- MUST: A package must own all directories that it creates. If it does no...

OK

- MUST: A package must not contain any duplicate files in the %files list...

OK

- MUST: Permissions on files must be set properly. Executables should be ...

OK

- MUST: Each package must have a %clean section, which contains rm -rf %{...

OK

- MUST: Each package must consistently use macros, as described in the [w...

OK

- MUST: The package must contain code, or permissable content. This is de...

OK

- MUST: Large documentation files should go in a -doc subpackage. (The de...

OK

- MUST: If a package includes something as %doc, it must not affect the r...

OK

- MUST: Header files must be in a -devel package.

OK

- MUST: Static libraries must be in a -static package.

OK

- MUST: Packages containing pkgconfig(.pc) files must 'Requires: pkgconfi...

OK

- MUST: If a package contains library files with a suffix (e.g. libfoo.so...

See JNI comment above.

- MUST: In the vast majority of cases, devel packages must require the ba...

OK

- MUST: Packages must NOT contain any .la libtool archives, these should ...

OK

- MUST: Packages containing GUI applications must include a %{name}.deskt...

OK

- MUST: Packages must not own files or directories already owned by other...

OK

- MUST: At the beginning of %install, each package MUST run rm -rf %{buil...

OK

- MUST: All filenames in rpm packages must be valid UTF-8.

OK

- SHOULD: If the source package does not include license text(s) as a sep...

OK

- SHOULD: The description and summary sections in the package spec file s...

s/java/Java/ in the summary tag.

Usually description is in paragraph form, not bullet points.  I'll
leave this up to you though.

- SHOULD: The reviewer should test that the package builds in mock. See [...

A Rawhide i386 mock build fails:

+ /usr/lib/rpm/find-debuginfo.sh --strict-build-id
/builddir/build/BUILD/libmatthew-java-0.7.1
extracting debug info from
/var/tmp/libmatthew-java-0.7.1-1.fc10-root-mockbuild/usr/lib/libcgi-java.so
*** ERROR: No build ID note found in
/var/tmp/libmatthew-java-0.7.1-1.fc10-root-mockbuild/usr/lib/libcgi-java.so
RPM build errors:
error: Bad exit status from /var/tmp/rpm-tmp.14107 (%install)
Bad exit status from /var/tmp/rpm-tmp.14107 (%install)
Child returncode was: 1

- SHOULD: The package should compile and build into binary rpms on all su...

rpmbuild works on i386.

- SHOULD: The reviewer should test that the package functions as 

[Bug 452251] Review Request: libmatthew-java - collection of java libraries

2008-06-24 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: libmatthew-java - collection of java libraries


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





--- Additional Comments From [EMAIL PROTECTED]  2008-06-24 15:00 EST ---
Updated:

Spec URL: http://omajid.fedorapeople.org/libmatthew-java/libmatthew-java.spec
SRPM URL:
http://omajid.fedorapeople.org/libmatthew-java/libmatthew-java-0.7.1-1.fc9.src.rpm

fixed soname warning, jni object locations, added --build-id flag to build on
rawhide and removed explicit dependency of javadoc on base package

-- 
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, or are watching someone who is.

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


[Bug 452251] Review Request: libmatthew-java - collection of java libraries

2008-06-24 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: libmatthew-java - collection of java libraries


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





--- Additional Comments From [EMAIL PROTECTED]  2008-06-24 17:30 EST ---
$ rpmlint RPMS/i386/libmatthew-java-0.7.1-1.fc9.i386.rpm

mock build succeeds on fedora-rawhide-i386.

The upstream project shouldn't be using ld directly to link the JNI libraries. 
Can you patch it to use gcc instead?  This may allow you to not specify
--build-id.  Also, the contents of RPM_OPT_FLAGS (see rpm --eval %{optflags})
are not being passed to cpp, cc and ld.  They need to be.

-- 
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, or are watching someone who is.

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


[Bug 452251] Review Request: libmatthew-java - collection of java libraries

2008-06-23 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: libmatthew-java - collection of java libraries


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





--- Additional Comments From [EMAIL PROTECTED]  2008-06-23 17:10 EST ---
After talking to Thomas Fitzsimmons, modified spec file (and added a patch to
srpm) to build and run using openjdk 

-- 
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, or are watching someone who is.

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


[Bug 452251] Review Request: libmatthew-java - collection of java libraries

2008-06-20 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: libmatthew-java - collection of java libraries


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||177841
  nThis||
Summary|Review Request: libmatthew- |Review Request: libmatthew-
   |java - collection of java   |java - collection of java
   |libraries   |libraries




--- Additional Comments From [EMAIL PROTECTED]  2008-06-20 11:15 EST ---
Sorry, but I forgot to mention that this is my first package and I am looking
for a sponsor. Thanks

-- 
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, or are watching someone who is.

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