[Bug 225807] Merge Review: glib-java

2007-04-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: Merge Review: glib-java


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NOTABUG




-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

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


[Bug 225807] Merge Review: glib-java

2007-04-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: Merge Review: glib-java


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|CLOSED  |ASSIGNED
   Keywords||Reopened
 Resolution|NOTABUG |




-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

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


[Bug 225807] Merge Review: glib-java

2007-04-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: Merge Review: glib-java


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Keywords|Reopened|




-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

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


[Bug 225807] Merge Review: glib-java

2007-04-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: Merge Review: glib-java


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Severity|normal  |medium
   Priority|normal  |medium

[EMAIL PROTECTED] changed:

   What|Removed |Added

   Priority|medium  |urgent




--- Additional Comments From [EMAIL PROTECTED]  2007-04-20 13:49 EST ---
All fixed, sorry for the delay.
And similar bugs found in my other packages fixed, too.
Could you please look at this again?
(And to the other 6 java-gnome pkgs and to grep, if you have spare time...)
Thanks!

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

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


[Bug 225807] Merge Review: glib-java

2007-03-14 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: Merge Review: glib-java


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
   Flag||fedora-review?




-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

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


[Bug 225807] Merge Review: glib-java

2007-03-14 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: Merge Review: glib-java


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





--- Additional Comments From [EMAIL PROTECTED]  2007-03-14 18:30 EST ---
1) rpmlint is not silent on source package:
W: glib-java rpm-buildroot-usage %prep rm -rf $RPM_BUILD_ROOT
$RPM_BUILD_ROOT should not be touched during %build or %prep stage, as it
will break short circuiting.

W: glib-java mixed-use-of-spaces-and-tabs (spaces: line 15, tab: line 26)
The specfile mixes use of spaces and tabs for indentation, which is a
cosmetic annoyance.  Use either spaces or tabs for indentation, not both.

2) rpmlint is not silent on binary package as well:
[EMAIL PROTECTED] redhat]$ rpmlint -i RPMS/i386/glib-java-*
W: glib-java-devel summary-ended-with-dot Compressed Java source files for
glib-java.
Summary ends with a dot.

E: glib-java zero-length /usr/share/doc/glib-java-0.2.6/NEWS
W: glib-java devel-file-in-non-devel-package /usr/include/glib-java/jg_jnu.h
A development file (usually source code) is located in a non-devel
package. If you want to include source code in your package, be sure to
create a development package.

3) incorrect BuildRoot tag.
It should be (in decreasing order of preference) one of these:
%(mktemp -ud %{_tmppath}/%{name}-%{version}-%{release}-XX)
%{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)
%{_tmppath}/%{name}-%{version}-%{release}-root

4) INSTALL should not be installed

5) Check Requires and BuildRequires whether all versions are necessary.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

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