[Bug 971236] Review Request: sugar-deducto - A learning activity aimed towards improving children's skills to deducing logic

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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

Parag AN(पराग) <panemade@xxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
                 CC|                            |panemade@xxxxxxxxx
           Assignee|nobody@xxxxxxxxxxxxxxxxx    |panemade@xxxxxxxxx
              Flags|                            |fedora-review?

--- Comment #1 from Parag AN(पराग) <panemade@xxxxxxxxx> ---
Review:-

+ Package builds in mock successfully

+ rpmlint on rpms gave
sugar-deducto.noarch: W: spelling-error %description -l en_US children's ->
children, childcare's, Nichiren's
sugar-deducto.src: W: spelling-error %description -l en_US children's ->
children, childcare's, Nichiren's
2 packages and 0 specfiles checked; 0 errors, 2 warnings.
==> this can be ignored

+ Source verified with upstream as (sha256sum)
srpm tarball  :
db7bdea1c281aa9f6a850f58edb01388efe24568336bfa733472b17c385ed53a
upstream tarball :
db7bdea1c281aa9f6a850f58edb01388efe24568336bfa733472b17c385ed53a

- License tag is not completely valid.

+ Rest looks as per packaging guidelines.

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

2) License tag should be GPLv3+ and MIT. 
Read
https://fedoraproject.org/wiki/Packaging:LicensingGuidelines#.22or_later_version.22_licenses

Also, Add a comment above license tag 
# sprites.py is in MIT and all other files in GPLv3+

-- 
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=QEWLpEo7dd&a=cc_unsubscribe
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review





[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]