[Bug 496524] Review Request: jorbis - Pure Java Ogg Vorbis Decoder

2009-05-08 Thread bugzilla
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=496524


Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

   Fixed In Version|0.0.17-2.fc10   |0.0.17-2.fc11




-- 
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@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 496524] Review Request: jorbis - Pure Java Ogg Vorbis Decoder

2009-05-08 Thread bugzilla
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=496524





--- Comment #11 from Fedora Update System upda...@fedoraproject.org  
2009-05-09 00:11:39 EDT ---
jorbis-0.0.17-2.fc11 has been pushed to the Fedora 11 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.

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


[Bug 496524] Review Request: jorbis - Pure Java Ogg Vorbis Decoder

2009-05-02 Thread bugzilla
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=496524





--- Comment #9 from Fedora Update System upda...@fedoraproject.org  
2009-05-02 12:27:13 EDT ---
jorbis-0.0.17-2.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.

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


[Bug 496524] Review Request: jorbis - Pure Java Ogg Vorbis Decoder

2009-05-02 Thread bugzilla
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=496524


Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
   Fixed In Version||0.0.17-2.fc9
 Resolution||NEXTRELEASE




-- 
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@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 496524] Review Request: jorbis - Pure Java Ogg Vorbis Decoder

2009-05-02 Thread bugzilla
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=496524





--- Comment #10 from Fedora Update System upda...@fedoraproject.org  
2009-05-02 12:42:38 EDT ---
jorbis-0.0.17-2.fc10 has been pushed to the Fedora 10 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.

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


[Bug 496524] Review Request: jorbis - Pure Java Ogg Vorbis Decoder

2009-05-02 Thread bugzilla
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=496524


Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

   Fixed In Version|0.0.17-2.fc9|0.0.17-2.fc10




-- 
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@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 496524] Review Request: jorbis - Pure Java Ogg Vorbis Decoder

2009-05-01 Thread bugzilla
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=496524


Dennis Gilmore den...@ausil.us changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #5 from Dennis Gilmore den...@ausil.us  2009-05-01 17:03:48 EDT 
---
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.

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


[Bug 496524] Review Request: jorbis - Pure Java Ogg Vorbis Decoder

2009-05-01 Thread bugzilla
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=496524





--- Comment #8 from Fedora Update System upda...@fedoraproject.org  
2009-05-01 18:45:52 EDT ---
jorbis-0.0.17-2.fc9 has been submitted as an update for Fedora 9.
http://admin.fedoraproject.org/updates/jorbis-0.0.17-2.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.

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


[Bug 496524] Review Request: jorbis - Pure Java Ogg Vorbis Decoder

2009-05-01 Thread bugzilla
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=496524





--- Comment #6 from Fedora Update System upda...@fedoraproject.org  
2009-05-01 18:43:56 EDT ---
jorbis-0.0.17-2.fc11 has been submitted as an update for Fedora 11.
http://admin.fedoraproject.org/updates/jorbis-0.0.17-2.fc11

-- 
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@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 496524] Review Request: jorbis - Pure Java Ogg Vorbis Decoder

2009-05-01 Thread bugzilla
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=496524





--- Comment #7 from Fedora Update System upda...@fedoraproject.org  
2009-05-01 18:44:56 EDT ---
jorbis-0.0.17-2.fc10 has been submitted as an update for Fedora 10.
http://admin.fedoraproject.org/updates/jorbis-0.0.17-2.fc10

-- 
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@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 496524] Review Request: jorbis - Pure Java Ogg Vorbis Decoder

2009-04-30 Thread bugzilla
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=496524


Mattias Ellert mattias.ell...@fysast.uu.se changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Comment #3 from Mattias Ellert mattias.ell...@fysast.uu.se  2009-04-30 
07:14:33 EDT ---
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.

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


[Bug 496524] Review Request: jorbis - Pure Java Ogg Vorbis Decoder

2009-04-30 Thread bugzilla
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=496524


Orcan 'oget' Ogetbil oget.fed...@gmail.com changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #4 from Orcan 'oget' Ogetbil oget.fed...@gmail.com  2009-04-30 
11:04:54 EDT ---
Thank you for the review!

New Package CVS Request
===
Package Name: jorbis
Short Description: Pure Java Ogg Vorbis Decoder
Owners: oget
Branches: F-9 F-10 F-11
InitialCC:

-- 
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@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 496524] Review Request: jorbis - Pure Java Ogg Vorbis Decoder

2009-04-29 Thread bugzilla
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=496524


Mattias Ellert mattias.ell...@fysast.uu.se changed:

   What|Removed |Added

 AssignedTo|nob...@fedoraproject.org|mattias.ell...@fysast.uu.se
   Flag||fedora-review?




-- 
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@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 496524] Review Request: jorbis - Pure Java Ogg Vorbis Decoder

2009-04-29 Thread bugzilla
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=496524


Mattias Ellert mattias.ell...@fysast.uu.se changed:

   What|Removed |Added

 Status|NEW |ASSIGNED




--- Comment #1 from Mattias Ellert mattias.ell...@fysast.uu.se  2009-04-29 
07:57:15 EDT ---
Fedora review jorbis-0.0.17-1.fc10.src.rpm 2009-04-29

rpmlint output:

[ell...@ellert jorbis]$ rpmlint 1256512409979681881233819/result/jorbis/*.rpm
jorbis.spec 
jorbis.src:156: W: libdir-macro-in-noarch-package (main package) %dir
%{_libdir}/gcj/%{name}
jorbis.src:157: W: libdir-macro-in-noarch-package (main package)
%{_libdir}/gcj/%{name}/%{name}-%{version}.jar.*
jorbis.src:170: W: libdir-macro-in-noarch-package player
%{_libdir}/gcj/%{name}/%{name}-player*.jar.*
jorbis.src:178: W: libdir-macro-in-noarch-package comment
%{_libdir}/gcj/%{name}/%{name}-comment*.jar.*
jorbis.spec:156: W: libdir-macro-in-noarch-package (main package) %dir
%{_libdir}/gcj/%{name}
jorbis.spec:157: W: libdir-macro-in-noarch-package (main package)
%{_libdir}/gcj/%{name}/%{name}-%{version}.jar.*
jorbis.spec:170: W: libdir-macro-in-noarch-package player
%{_libdir}/gcj/%{name}/%{name}-player*.jar.*
jorbis.spec:178: W: libdir-macro-in-noarch-package comment
%{_libdir}/gcj/%{name}/%{name}-comment*.jar.*
6 packages and 1 specfiles checked; 0 errors, 8 warnings.

* OK
! needs attention

* rpmlint warnings are bogus - rpmlint does not understand the
  conditional BuildArch.

* Package is named according to the guidelines

* The specfile is named after the package

* The package follows the guidelines for a native Java package

* Package license is Fedora approved (LGPLv2+)

* The README file says that the license is LGPL with no version, but
  that is equivalent to LGPLv2+ according to the Fedora License list

* The license file (COPYING.LIB) in the sources is installed as %doc

* The specfile is written in legible English

* Sources matches upstream:

6141edfb16d34412a86c95a97fd4b10f  jorbis-0.0.17.zip
6141edfb16d34412a86c95a97fd4b10f  SRPM/jorbis-0.0.17.zip

* Package builds in mock (Fedora 10)

* BuildRequires are sane

* No shared libraries

* Package owns the directories it creates

! The jars in -comment and -player are also in main (duplicate files)

* Permissions are sane and %files has %defattr

* %clean clears buildroot

* Consistent use of macros

* Documentation in -javadoc package

* %doc is not runtime essential

* subpackages require main with fully versioned dependencies

* Package does not own other's directories

* %install clears buildroot

* Installed filenames are valid UTF-8

-- 
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@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 496524] Review Request: jorbis - Pure Java Ogg Vorbis Decoder

2009-04-29 Thread bugzilla
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=496524





--- Comment #2 from Orcan 'oget' Ogetbil oget.fed...@gmail.com  2009-04-29 
13:35:30 EDT ---
Ah, * is greedy :)

Spec URL: http://oget.fedorapeople.org/review/jorbis.spec
SRPM URL: http://oget.fedorapeople.org/review/jorbis-0.0.17-2.fc10.src.rpm

Changelog: 0.0.17-2
- Fix duplicate files issue

-- 
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@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 496524] Review Request: jorbis - Pure Java Ogg Vorbis Decoder

2009-04-19 Thread bugzilla
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=496524


Orcan 'oget' Ogetbil oget.fed...@gmail.com changed:

   What|Removed |Added

 Blocks||496525




-- 
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@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review