https://bugzilla.redhat.com/show_bug.cgi?id=1190510 Carlos Morel-Riquelme <empateinfinito@xxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |empateinfinito@xxxxxxxxx --- Comment #1 from Carlos Morel-Riquelme <empateinfinito@xxxxxxxxx> --- Hi Fernado, this is a informal review. well you have some warnings in the spec file. rpmlint Checking: gitshelf-0.0.4-1.fc22.noarch.rpm gitshelf-0.0.4-1.fc22.src.rpm gitshelf.noarch: W: spelling-error Summary(en_US) repos -> ropes, reps, repose gitshelf.noarch: W: spelling-error %description -l en_US repos -> ropes, reps, repose gitshelf.noarch: W: spelling-error %description -l en_US config -> con fig, con-fig, configure gitshelf.noarch: W: spelling-error %description -l en_US repo -> rope, rep, reps gitshelf.noarch: W: invalid-license ASL 2 gitshelf.noarch: W: no-manual-page-for-binary gitshelf gitshelf.src: W: spelling-error Summary(en_US) repos -> ropes, reps, repose gitshelf.src: W: spelling-error %description -l en_US repos -> ropes, reps, repose gitshelf.src: W: spelling-error %description -l en_US config -> con fig, con-fig, configure gitshelf.src: W: spelling-error %description -l en_US repo -> rope, rep, reps gitshelf.src: W: invalid-license ASL 2 2 packages and 0 specfiles checked; 0 errors, 11 warnings. ------------------------------------------------------------------------------- 1) w: invalid-license ASL 2 - the correct way is "ASL 2.0" - please see the wiki https://fedoraproject.org/wiki/Licensing:Main?rd=Licensing#Package_Licensing_Guidelines 2) w: spelling-error - please change "repo" to "repository" - please change "config" to "configure" 3) w: no-manual-page-for-binary - pass this. 4) update this - release version to "2" - change version in your changelog ------------------------------------------------------------------------------ finally your spec file should look this %global pypi_name gitshelf Name: gitshelf Version: 0.0.4 Release: 2%{?dist} Summary: Manage a shelf full of git repositories License: ASL 2.0 URL: https://github.com/gitshelf/gitshelf Source0: https://pypi.python.org/packages/source/g/%{pypi_name}/%{pypi_name}-%{version}.tar.gz BuildArch: noarch BuildRequires: python-devel BuildRequires: python-setuptools %description gitshelf - a shelf full of git repositories. Manage a collection of git repository that, you don't want to manage as sub-modules. Created to be used to manage a set of salt states, formula & pillars in a controlled fashion, the default configuration file is a YAML file it's can also be loaded as a pillar (for whatever reason). Using the branch parameter, you can pin a repository to a branch, sha1, tag, whatever you need ... %prep %setup -q -n %{pypi_name}-%{version} # Remove bundled egg-info rm -rf %{pypi_name}.egg-info %build %{__python2} setup.py build %install %{__python2} setup.py install --skip-build --root %{buildroot} %files %doc README.md %{_bindir}/* %{python2_sitelib}/%{pypi_name} %{python2_sitelib}/%{pypi_name}-%{version}-py?.?.egg-info %changelog * Mon Feb 9 2015 Fernando Espinoza <espasmo@xxxxxxxxxxxxxxxxx> - 0.0.4-2 * Sun Feb 8 2015 Fernando Espinoza <espasmo@xxxxxxxxxxxxxxxxx> - 0.0.4-3 - Initial Packaging ------------------------------------------------------------------------------ -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component _______________________________________________ package-review mailing list package-review@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/package-review