[Bug 226180] Merge Review: mysql

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





--- Comment #3 from Orcan 'oget' Ogetbil   2009-01-17 
18:24:56 EDT ---
I can't find the koji entry for the new version.. I guess it is still being
worked on. 

fitzsim, can you provide a SRPM when you're 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.

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


[Bug 226180] Merge Review: mysql

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


Orcan 'oget' Ogetbil  changed:

   What|Removed |Added

 Status|ASSIGNED|NEW
 CC|a...@redhat.com,|
   |mcla...@redhat.com  |
 AssignedTo|oget.fed...@gmail.com   |nob...@fedoraproject.org




--- Comment #2 from Orcan 'oget' Ogetbil   2009-01-17 
18:03:48 EDT ---
Shoot! Wrong bug. Sorry for the mess.

I'll review this next.

-- 
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 226180] Merge Review: mysql

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


Orcan 'oget' Ogetbil  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||a...@redhat.com,
   ||mcla...@redhat.com,
   ||oget.fed...@gmail.com
 AssignedTo|nob...@fedoraproject.org|oget.fed...@gmail.com
   Flag||fedora-review?




--- Comment #1 from Orcan 'oget' Ogetbil   2009-01-17 
18:01:40 EDT ---
The full review arrived!

* Summary and especially the description are bizarre. Can you update them. You
can find these on the manpage:
Xinerama - API for Xinerama extension to X11 Protocol   

Xinerama is a simple library designed to interface the Xinerama Extension
for 
retrieving information about physical output devices which may be combined 
into a single logical X screen.

* rpmlint says
   libXinerama.src:18: W: unversioned-explicit-obsoletes XFree86-libs
   libXinerama.src:18: W: unversioned-explicit-obsoletes xorg-x11-libs
   libXinerama.src:32: W: unversioned-explicit-obsoletes XFree86-devel
   libXinerama.src:32: W: unversioned-explicit-obsoletes xorg-x11-devel
   libXinerama.x86_64: E: zero-length /usr/share/doc/libXinerama-1.0.3/AUTHORS
   libXinerama.x86_64: E: zero-length /usr/share/doc/libXinerama-1.0.3/README
   libXinerama.x86_64: W: obsolete-not-provided XFree86-libs
   libXinerama.x86_64: W: obsolete-not-provided xorg-x11-libs
   libXinerama-devel.x86_64: W: obsolete-not-provided XFree86-devel
   libXinerama-devel.x86_64: W: obsolete-not-provided xorg-x11-devel
The zero-length files are obviously not needed so they should be removed. The
obsoletes look very problematic. Can you fix those (or alternatively explain
them in the SPEC file as comments)?

* BR: libXau-devel is not needed. Afaict it is not used.

* BRs: libX11-devel pkgconfig and xorg-x11-proto-devel are not needed. They
will be picked up by libXext-devel.

* Packages containing pkgconfig(.pc) files must 'Requires: pkgconfig' (for
directory ownership and usability). This applies to the devel package.

! Try to make use of the %{name} macro (e.g. files sections).

* Parallel make must be supported whenever possible. If it is not supported,
this should be noted in the SPEC file as a comment.


Added Adam and Matthias to CC since they are the last two known maintainers.
Sorry if this was not desired.

-- 
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 226180] Merge Review: mysql

2007-02-09 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: mysql


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


Bug 226180 depends on bug 223713, which changed state.

Bug 223713 Summary: mysql: non-failsafe install-info use in scriptlets
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=223713

   What|Old Value   |New Value

 Resolution||RAWHIDE
 Status|NEW |CLOSED



-- 
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 226180] Merge Review: mysql

2007-01-31 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: mysql


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

  BugsThisDependsOn||223713




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