Bug 243955

Looks OK for me.
Rpmlint is quiet.
No other problems found.

I have missed to check noarch rpm. There was many errors. They have been fixed
by me here:
http://www.salstar.sk/pub/fedora/SPECS/python-xlrd.spec
http://www.salstar.sk/pub/fedora/SRPMS/7/python-xlrd-0.6.1-2.fc7.src.rpm

* Fri Sep 7 2007 Jan ONDREJ (SAL) <ondrejj(at)salstar.sk> 0.6.1-2
- namewithoutprefix removed, too complicated and not required
- added directory xlrd
- permissions for python scripts set to 644
- files converted from DOS line-feeds to UNIX format
- "#!/usr/bin/env python" added to beginning runxlrd script  
- removed doc and examples from site-packages/xlrd directory 

I have installed and tested this package on my example xls file. Works as expected.

Anybody can approve this package?

Sean, can you review my changes and package functionality?

I hope to get to it this week.  Sorry, I've been swamped but it's on my list to do.

The changes in the above spec file look good to me, and rpmlint is not reporting
any errors.  There are some warnings about doc files not being UTF-8 though.

* Fri Sep 14 2007 Jan ONDREJ (SAL) <ondrejj(at)salstar.sk> 0.6.1-3
- doc files converted to UTF-8
- removed HISTORY.html README.html because they are also in xlrd/doc/

Please retest again.
http://www.salstar.sk/pub/fedora/SPECS/python-xlrd.spec
http://www.salstar.sk/pub/fedora/SRPMS/7/python-xlrd-0.6.1-3.fc7.src.rpm

rpmlint is completely quiet now.

These changes look fine.  I've rebuilt the packages with the above changes and
put them at:

Spec URL: ftp://ftp.tummy.com/pub/tummy/RPMS/SRPMS/python-xlrd.spec
SRPM URL: ftp://ftp.tummy.com/pub/tummy/RPMS/SRPMS/python-xlrd-0.6.1-3.fc7.src.rpm

Can we get a review?

Thanks,
Sean

OK, looks OK and I don't see nothing more to enhance.

APPROVED.

You may request for a CVS module to be created.
You may add me to owners, if you need help to build new releases in future.

If you have questions, please ask me.

This package is already approved more than 3 weeks.
If you do not respond, I will request another review and I will maintain this
package in Fedora.

There are missing eggs for Fedora>8. Updated spec file and source:
  http://www.salstar.sk/pub/fedora/SPECS/python-xlrd.spec
  http://www.salstar.sk/pub/fedora/SRPMS/8/python-xlrd-0.6.1-4.fc8.src.rpm

No response from original author more than a month. I am removing my approval to
leave somebody else to make another review. After approval I want to be package
maintainer in Fedora.

Fair enough.  I spent most of November and December out of town at a client
site, so I haven't had much time to look at it until today.  Thanks for offering
to take the package.

1.)

 38 # add shebang and remove .py file
 39 (
 40   echo '#!/usr/bin/env python'
 41   cat $RPM_BUILD_ROOT/usr/bin/runxlrd.py
 42 ) >> $RPM_BUILD_ROOT/usr/bin/runxlrd
 43 rm -rf $RPM_BUILD_ROOT/usr/bin/runxlrd.py* \

Please replace "/usr/bin" with %{_bindir} (and eventually with "/usr/bin/env
python" with %{__python} in case you consider %{__python} macro to be acceptably
usable in specs -- ans you seem to do)

2.)

 58 %attr(755,root,root) /usr/bin/*

Likewise

3.)

The package should contain the text of the license

New SPEC file uploaded.

%changelog
* Thu Jan 3 2008 Jan ONDREJ (SAL) <ondrejj(at)salstar.sk> - 0.6.1-5
- updated /usr/bin/env by bindir macro

License is contained in licences.py and mentioned in README.

Thanks for the package.
I believe all outstanding issues to be addressed.

APPROVED

New Package CVS Request
=======================
Package Name: python-xlrd
Short Description: Library to extract data from Microsoft Excel (tm) spreadsheet
files
Owners: ondrejj,jafo
Branches: F-7 F-8 EL-4 EL-5
Cvsextras Commits: yes

cvs done.

python-xlrd-0.6.1-5.fc7 has been pushed to the Fedora 7 testing repository.  If problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update python-xlrd'

python-xlrd-0.6.1-5.fc8 has been pushed to the Fedora 8 testing repository.  If problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update python-xlrd'

python-xlrd-0.6.1-5.fc7 has been pushed to the Fedora 7 stable repository.  If problems still persist, please make note of it in this bug report.

python-xlrd-0.6.1-5.fc8 has been pushed to the Fedora 8 stable repository.  If problems still persist, please make note of it in this bug report.