Spec URL: http://hadoop.ru/rpm/python-aaargh.spec SRPM URL: http://hadoop.ru/rpm/python-aaargh-0.7.1-1.fc21.src.rpm Description: Aaargh is a Python module that makes building friendly command line applications really easy. Applications built with *Aaargh* provide a single executable with a subcommand for each exposed Python function. Each subcommand may have its own command line arguments. This is similar to the way version control systems provide multiple commands using a single entry point. Fedora Account System Username: vittyvk
Hi Vitaly, You can remove the "-n %{name}-%{version}" part in %setup, that's the default. The %description for the Python3 version contains a too long line. Actually it overflows of just *one* character, I guess it's caused by the '3' of Python3. ;-)
(In reply to Adrien Vergé from comment #1) > Hi Vitaly, > Thanks for your review! > You can remove the "-n %{name}-%{version}" part in %setup, that's the > default. I'm not using "%{name}-%{version}" but "%{modname}-%{version}" where "modname" stands for "aaargh" (without 'python-' prefix). I need to do that due to the fact that aaargh (as other python modules) is distributed in 'aaargh-VERSION.tar.gz' form. > > The %description for the Python3 version contains a too long line. Actually > it overflows of just *one* character, I guess it's caused by the '3' of > Python3. ;-) Fixed! SRPM and SPEC updated.
Hi Vitaly: Please use the versioned macros for python2, to prepare for the cutover to python3 as the default, per https://fedoraproject.org/wiki/Packaging:Python#Macros (for example, "%{__python2}" instead of "%{__python}"). And hey, if you're up for a review swap, I've got bug 1089553 in the hopper waiting for a reviewer :)
> Fixed! SRPM and SPEC updated. Please update %changelog even for review updates. > %define modname aaargh Please use %global. https://fedoraproject.org/wiki/Packaging:Guidelines?rd=Packaging/Guidelines#.25global_preferred_over_.25define > %doc LICENSE.rst Please add README.rst too. Upstream provides tests, so a %check section would be good. Always remember to pass the built packages through rpmlint to fix them as much as possible. Right now it says: python-aaargh.noarch: W: spelling-error %description -l en_US subcommand -> sub command, sub-command, commandant python-aaargh.noarch: W: incoherent-version-in-changelog 0.7-1 ['0.7.1-1.fc20', '0.7.1-1'] python3-aaargh.noarch: W: spelling-error %description -l en_US subcommand -> sub command, sub-command, commandant 2 packages and 0 specfiles checked; 0 errors, 3 warnings. Also, a personal suggestion: is there any reason for all the `with_python3` stuff? If not, just simplify your life: always build both packages. Please note that this is an unofficial review.
I've lost my interest in putting this package in Fedora as I don't use it anymore. Thanks for reviews anyway!