https://bugzilla.redhat.com/show_bug.cgi?id=843678

Parag AN(पराग) <panem...@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
           Assignee|nob...@fedoraproject.org    |panem...@gmail.com
              Flags|                            |fedora-review?



--- Comment #10 from Parag AN(पराग) <panem...@gmail.com> ---
reviewing above srpm

1) use BR:python2-devel

2) remove follwing line in spec
rm -rf  {buildroot}

3)Don't use any backslash between %{buildroot} and %{_prefix}, so your %install
should look like
%{__python} ./setup.py install --prefix=%{buildroot}%{_prefix}

4) This package archive is really bad in file permissions. You can use
following %prep which should address most of rpmlint issues
%setup -q -n Castle.activity
chmod -x *.py
chmod -x data/*.dat
chmod -x  activity/activity.svg
chmod +x setup.py Castle.py

sed -i "s|\r||g" Castle.py
sed -i "s|\r||g" activity/activity.svg

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=rivnQgWAOo&a=cc_unsubscribe
_______________________________________________
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

Reply via email to