[Bug 755065] Review Request: python-sqlalchemy0.7 - compat package epel 6

[bugzilla at redhat.com]

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

Steve Traylen <steve.traylen at cern.ch> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
                 CC|                            |steve.traylen at cern.ch
          QAContact|extras-qa at fedoraproject.org |steve.traylen at cern.ch
  Status Whiteboard|                            |BuildFails
               Flag|                            |fedora-review?

--- Comment #1 from Steve Traylen <steve.traylen at cern.ch> 2011-11-18 13:43:52 EST ---
Hi,

Build fails with mock... 
A mock of CentOS6 (OS, Updates & CS) +EPEL. repositories 


+ /usr/bin/python ./sqla_nose.py
Traceback (most recent call last):
  File "./sqla_nose.py", line 19, in <module>
    nose.main(addplugins=[NoseSQLAlchemy()])
TypeError: __init__() got an unexpected keyword argument 'addplugins'
error: Bad exit status from /var/tmp/rpm-tmp.Ya2ziP (%check)
RPM build errors:
    Bad exit status from /var/tmp/rpm-tmp.Ya2ziP (%check)
Child returncode was: 1
EXCEPTION: Command failed. See logs for output.
 # ['bash', '--login', '-c', 'rpmbuild -bb --target x86_64 --nodeps
builddir/build/SPECS/python-sqlalchemy0.7.spec']
Traceback (most recent call last):
  File "/usr/lib/python2.6/site-packages/mockbuild/trace_decorator.py", line
70, in trace
    result = func(*args, **kw)
  File "/usr/lib/python2.6/site-packages/mockbuild/util.py", line 328, in do
    raise mockbuild.exception.Error, ("Command failed. See logs for output.\n #
%s" % (command,), child.returncode)
Error: Command failed. See logs for output.
 # ['bash', '--login', '-c', 'rpmbuild -bb --target x86_64 --nodeps
builddir/build/SPECS/python-sqlalchemy0.7.spec']
LEAVE do --> EXCEPTION RAISED

Checking now in koji in case it's a RHEL vs CentOS:

http://koji.fedoraproject.org/koji/taskinfo?taskID=3524833

same result.

Also if it were me I would remove all the stuff that is never going 
to be used, i.e.

%if ! (0%{?fedora} > 12 || 0%{?rhel} > 5)
%{!?python_sitearch: %global python_sitearch %(%{__python} -c "from
distutils.sysconfig import get_python_lib; print get_python_lib(1)")}
%endif

%if 0%{?fedora} > 12 || 0%{?rhel} > 6
%global with_python3 1
%endif

and all the cases below that that use these, they are redundant.

Also there is no README.Fedora file suggesting how this package should
be used as is dictated in the alternate egg guidelines.

Steve.

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