Spec URL: http://mwiriadi.fedorapeople.org/packages/drpython/drpython.spec SRPM URL: http://mwiriadi.fedorapeople.org/packages/drpython/drpython-165-1.fc8.src.rpm Description: DrPython is a highly customizable cross-platform ide to aid programming in Python.
builds cleanly in mock (fedora-8-i386) rpmlint output: drpython.noarch: W: file-not-utf8 /usr/share/doc/drpython-165/History.txt drpython.noarch: W: file-not-utf8 /usr/share/doc/drpython-165/Changelog.txt drpython.src: W: mixed-use-of-spaces-and-tabs (spaces: line 2, tab: line 11) The utf-8 issues need not be fixed - but please fix the mixed spaces and tabs in the .spec file. The Icon entry in the .desktop should be without the file extension (ie. drpython instead of drpython.png). It might be nice to include a small shell script called drpython which could be installed in /usr/bin to start drpython. The icon drpython.png must be installed to /usr/share/pixmaps to be found by the .desktop file. So all in all it looks quite good. The issues that must be fixed are: 1) the mixed use of tabs and spaces in the .spec file 2) the icon must be installed so that it shows up in the menus And of course it would be nice if some of the other minor issues could be fixed as well.
All fixed hopefully. I've cleaned up the spec file compared to the previous one using a for loop. I have renamed the patch as well mainly for clarity on my sake for my sources directory. http://mwiriadi.fedorapeople.org/packages/drpython/drpython-165-2.fc8.src.rpm http://mwiriadi.fedorapeople.org/packages/drpython/drpython.spec http://mwiriadi.fedorapeople.org/packages/drpython/drpython-setup.py.postinst.patch http://mwiriadi.fedorapeople.org/packages/drpython/drpython.desktop http://mwiriadi.fedorapeople.org/packages/drpython/drpython.sh
This looks very nice now. You still have to fix the mixed use of tabs and spaces in the .spec file - it is the last thing keeping rpmlint from being quiet. Fix that and I will approve this package.
Sorry updated it http://mwiriadi.fedorapeople.org/packages/drpython/drpython.spec http://mwiriadi.fedorapeople.org/packages/drpython/drpython-165-3.fc8.src.rpm
This looks fine. Accepted.
New Package CVS Request ======================= Package Name: drpython Short Description: DrPython is a highly customizable cross-platform ide Owners: mwiriadi Branches: F-8 devel InitialCC: mwiriadi Cvsextras Commits: yes
CVS Done
Package Change Request ====================== Package Name: drpython New Branches: FC-7