[Bug 193804] Review Request: glitz

2008-07-07 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: glitz


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

Product|Fedora Extras   |Fedora
Version|devel   |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, 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 193804] Review Request: glitz

2007-06-08 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: glitz


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Severity|normal  |medium
   Priority|normal  |medium




-- 
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 193804] Review Request: glitz

2006-06-08 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: glitz


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution||NEXTRELEASE




-- 
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 193804] Review Request: glitz

2006-06-07 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: glitz


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO|163778  |163779
  nThis||




--- Additional Comments From [EMAIL PROTECTED]  2006-06-07 11:53 EST ---
APPROVED.


-- 
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 193804] Review Request: glitz

2006-06-07 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: glitz


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





--- Additional Comments From [EMAIL PROTECTED]  2006-06-07 11:26 EST ---
Updated package.

SPEC files: http://www.ece.ucdavis.edu/~ewwork/repo/testing/SPECS/glitz.spec
SRPM file:
http://www.ece.ucdavis.edu/~ewwork/repo/testing/SRPMS/glitz-0.5.6-3.src.rpm

-- 
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 193804] Review Request: glitz

2006-06-07 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: glitz


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED] |[EMAIL PROTECTED]
 CC|[EMAIL PROTECTED] |
OtherBugsDependingO|163776  |163778
  nThis||




--- Additional Comments From [EMAIL PROTECTED]  2006-06-07 05:55 EST ---
Some issues:
- glitz-devel contains pkg-config files, it therefore should
Requires: pkgconfig

- glitz-glx-devel contains pkg-config files, it therefore should
Requires: pkgconfig

- glitz-glx-devel depends on glitz-devel, it therefore must
Requires: glitz-devel
or
Requires: glitz-devel = %{version}-%{release}

- man3 manpages are programming manuals.
They should be part of the *-devel packages.


-- 
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 193804] Review Request: glitz

2006-06-05 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: glitz


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





--- Additional Comments From [EMAIL PROTECTED]  2006-06-05 15:37 EST ---
Updated the SPEC with your recommendations.

SPEC file: http://www.ece.ucdavis.edu/~ewwork/repo/testing/SPECS/glitz.spec
SRPM file:
http://www.ece.ucdavis.edu/~ewwork/repo/testing/SRPMS/glitz-0.5.6-2.src.rpm


-- 
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 193804] Review Request: glitz

2006-06-02 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: glitz


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||163776
  nThis||




-- 
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 193804] Review Request: glitz

2006-06-01 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: glitz


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





--- Additional Comments From [EMAIL PROTECTED]  2006-06-02 00:53 EST ---
I will be gone this weekend, but I will work on these issues in the near future
and update the packages for re-evaluation.  Thanks for the feedback.

-- 
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 193804] Review Request: glitz

2006-06-01 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: glitz


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





--- Additional Comments From [EMAIL PROTECTED]  2006-06-01 22:44 EST ---
Some comments:
* IMO, it should be considered to split this package into 2 subpackages: glitz
and glitz-glx, because the package internally consists of libglitz and several
backends, with libglitz-glx being one them.

Splitting it would allow more fine-grained deps, and thereby would help avoid
unnecessary deps of apps/libs based on glitz.

* The deps of the devel package don't seem to be correct:
- libglitz doesn't seem to depend on any X or GL library.
- libglitz-glx seems to depend libX11-devel and libGL-devel, but not on further
libX*-devel, in particular it doesn't not depend on libXt-devel.

* The package ships manpages, but doesn't install them. May-be it's worth
considering to install them.

* The configure script carries several options defaulting to true:
  --disable-glx   Disable glitz's GLX backend
  --disable-agl   Disable glitz's AGL backend
  --enable-eglEnable glitz's EGL backend
  --disable-wgl   Disable glitz's WGL backend
This will render building this package outside of a chroot-environment
non-deterministic, should one of these GL backends land in Fedora.
I.e. I recommend to add 
--enable-glx --disable-agl --disable-egl --disable-wgl to %configure

-- 
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 193804] Review Request: glitz

2006-06-01 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: glitz


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||192436
  nThis||




-- 
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 193804] Review Request: glitz

2006-06-01 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: glitz


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   External Bug||https://bugzilla.redhat.com/
  Reference||bugzilla/show_bug.cgi?id=192
   ||436
   External Bug|https://bugzilla.redhat.com/|
  Reference|bugzilla/show_bug.cgi?id=192|
   |432 |




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