[Bug 226048] Merge Review: libtheora

2009-11-17 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=226048


Bug 226048 depends on bug 478651, which changed state.

Bug 478651 Summary: Violates package naming guidelines
https://bugzilla.redhat.com/show_bug.cgi?id=478651

   What|Old Value   |New Value

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE



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

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


[Bug 226048] Merge Review: libtheora

2009-08-25 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=226048


Bug 226048 depends on bug 478651, which changed state.

Bug 478651 Summary: Violates package naming guidelines
https://bugzilla.redhat.com/show_bug.cgi?id=478651

   What|Old Value   |New Value

 Status|CLOSED  |ASSIGNED
 Resolution|NOTABUG |



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

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


[Bug 226048] Merge Review: libtheora

2009-01-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=226048


Debarshi Ray debarshi@gmail.com changed:

   What|Removed |Added

 Depends on||478651




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

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


[Bug 226048] Merge Review: libtheora

2009-01-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=226048





--- Comment #7 from Debarshi Ray debarshi@gmail.com  2009-01-02 15:11:08 
EDT ---
(In reply to comment #3)

Re-opening the review.

 Introducing an Epoch should be delayed as long as it makes sense.

We do need to fix this now. See: https://bugzilla.redhat.com/478651

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

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


[Bug 226048] Merge Review: libtheora

2009-01-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=226048


Debarshi Ray debarshi@gmail.com changed:

   What|Removed |Added

   Keywords||Reopened
 Status|CLOSED  |ASSIGNED
 CC||debarshi@gmail.com
 Resolution|RAWHIDE |




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

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


[Bug 226048] Merge Review: libtheora

2009-01-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=226048


Bug 226048 depends on bug 478651, which changed state.

Bug 478651 Summary: Violates package naming guidelines
https://bugzilla.redhat.com/show_bug.cgi?id=478651

   What|Old Value   |New Value

 Status|NEW |CLOSED
 Resolution||NOTABUG



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

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


[Bug 226048] Merge Review: libtheora

2008-04-18 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: libtheora


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Priority|normal  |medium
Product|Fedora Extras   |Fedora
Version|devel   |rawhide

[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2008-04-18 05:24 EST ---
I would like to request early branching for F-10 of libtheora

Package Change Request
==
Package Name: libtheora
New Branches: F-9


-- 
Configure bugmail: https://bugzilla.redhat.com/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 226048] Merge Review: libtheora

2008-04-18 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: libtheora


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2008-04-18 12:22 EST ---
cvs done.

-- 
Configure bugmail: https://bugzilla.redhat.com/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 226048] Merge Review: libtheora

2007-04-13 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: libtheora


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE
   Flag|fedora-review?  |fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2007-04-13 19:31 EST ---
I guess we'll have to wait until a stable version is out (hopefully it won't be
called 1.0).

Closing this bug, since in this case it's a review of an existing package and
there's no owner set

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


[Bug 226048] Merge Review: libtheora

2007-04-11 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: libtheora


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Severity|normal  |medium




--- Additional Comments From [EMAIL PROTECTED]  2007-04-11 07:13 EST ---
All the Bads and Maybes seem to be fixed in the current rawhide package, except
for the version. The only way to fix it that I can see would be to add an epoch,
since rpm considers 1.0alpha  1.0. Is this worth an epoch ?

-- 
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 226048] Merge Review: libtheora

2007-04-11 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: libtheora


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





--- Additional Comments From [EMAIL PROTECTED]  2007-04-11 11:29 EST ---
Introducing an Epoch should be delayed as long as it makes sense.  The next
upstream version might for example be 1.1 instead of 1.0 which would mean an
Epoch introduced today would have had only negative effects.

-- 
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 226048] Merge Review: libtheora

2007-03-19 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: libtheora


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   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 226048] Merge Review: libtheora

2007-03-18 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: libtheora


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2007-03-18 20:11 EST ---
Almost ready for approval (see Maybe and Bad section)

Good:
- Package name matches upstream
- Spec file name matches base package name
- License matches actual license, included in %doc, is FLOSS
- Spec written in American English, is legible
- Source matches upstream
- Package builds fine on all supported architectures
- Build requirements complete
- ldconfig called properly
- directory ownerships OK
- files listing: no duplicates
- file permissions OK
- Inter-dependencies OK

Maybe:
- Why --enable-static=yes ?

Bad:
- rpmlint:
  E: libtheora invalid-version 1.0alpha7
 according to the Naming Guidelines, the 'alpha' part should be
 part of the release tag
  W: libtheora-devel summary-ended-with-dot
  E: theora-tools binary-or-shlib-defines-rpath

  No documentation for libtheora-devel and theora-tools, but that's fine

- Source URL not provided. Also, use .tar.bz2 instead of .tar.gz?


-- 
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 226048] Merge Review: libtheora

2007-03-18 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: libtheora


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]




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