Bug 226338 - Merge Review: PyQt
Merge Review: PyQt
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
All Linux
medium Severity medium
: ---
: ---
Assigned To: Rex Dieter
Fedora Package Reviews List
Depends On:
  Show dependency treegraph
Reported: 2007-01-31 15:45 EST by Nobody's working on this, feel free to take it
Modified: 2012-09-04 12:09 EDT (History)
4 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2008-01-30 12:40:16 EST
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
rdieter: fedora‑review+
limburgher: fedora‑cvs+

Attachments (Terms of Use)

  None (edit)
Description Nobody's working on this, feel free to take it 2007-01-31 15:45:54 EST
Fedora Merge Review: PyQt

Initial Owner: than@redhat.com
Comment 1 Rex Dieter 2007-02-12 11:54:44 EST
PyQt looks good, simple and clean, APPROVED.

A few small SHOULD items to consider post-merge:
* -BuildRequires: libGL-devel libGLU-devel
  qt-devel Req's these now.
* +BuildRequires: qscintilla-devel
  to build/include qscintilla extention (provided now by Extras' PyQt-qscintilla)
Comment 2 Michał Bentkowski 2007-02-12 12:25:40 EST
Source0 URL no longer exists, should be fixed.
Comment 3 Rex Dieter 2007-02-26 11:50:14 EST
Good catch, use (something like)
Comment 4 Rex Dieter 2007-05-16 14:29:13 EDT
Than, are you going to implement items from comment #1 (post merge), so we can
put PyQt-qscintilla to rest?  

If so either preserve the -qscintilla/-qscintilla-devel subpkgs, or add
Obsoletes: %{name}-qscintilla > %{version}-%{release}
Provides:  %{name}-qscintilla = %{version}-%{release}
and add to -devel
Obsoletes: %{name}-qscintilla-devel > %{version}-%{release}
Provides:  %{name}-qscintilla-devel = %{version}-%{release}
Comment 5 Ngo Than 2007-05-18 06:55:29 EDT
it seems the CVS has problem i cannot connect to CVS server. I will add the 
change in CVS when CVS works again.
Comment 6 Rex Dieter 2007-06-20 10:33:46 EDT
Than, can you incorporate the qscintilla bits soon?
Comment 7 Rex Dieter 2007-10-09 11:16:29 EDT
ping (3+ mos later...)
Comment 8 Rex Dieter 2008-01-30 12:40:16 EST
all done, merged.
Comment 9 Jonathan Underwood 2012-09-04 09:22:25 EDT
Package Change Request
Package Name: PyQt
New Branches: el6
Owners: jgu
Comment 10 Gwyn Ciesla 2012-09-04 09:42:47 EDT
Are the existing maintainers on board with the EL-6 branch?
Comment 11 Rex Dieter 2012-09-04 09:50:24 EDT
no objection
Comment 12 Jonathan Underwood 2012-09-04 11:00:43 EDT
Thanks Rex - this is to solve BZ #847506 - ScientificPython-qt (built for EL6) requires PyQt.
Comment 13 Jonathan Underwood 2012-09-04 11:02:06 EDT
Jon - did you mean to cancel the CVS flag?
Comment 14 Gwyn Ciesla 2012-09-04 12:09:00 EDT
Git done (by process-git-requests).

At that time I did, yes. :)

Note You need to log in before you can comment on or make changes to this bug.