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

Raphael Groner <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[email protected]
           See Also|                            |https://bugzilla.redhat.com
                   |                            |/show_bug.cgi?id=1215762



--- Comment #6 from Raphael Groner <[email protected]> ---
Review swap with py4j (bug #1215762)?

Some informal hints to the spec file:

> Group:                Amusements/Games
Remove this line, it's deprecated since F17.

> Requires(post): info
> Requires(preun): info
What's this for? Your info file should not harm if there's no info installed in
the system, what I guess does not happen to often in practical usage. I do not
know what install-info does special, maybe just copy the file in the right
folder and reference in files section. Better remove those lines with Requires.

> URL:          http://sites.google.com/site/mohammedisam2000/home/projects
> Source0:      
> http://sites.google.com/site/mohammedisam2000/home/projects/%{name}-%{version}.tar.gz
Maybe better:
URL:        http://sites.google.com/site/mohammedisam2000/home/projects
Source0:    %{url}/%{name}-%{version}.tar.gz

Please tell if you will fix the above issues, those are just SHOULD. Later I
can do the official fedora-review run.

-- 
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
[email protected]
https://admin.fedoraproject.org/mailman/listinfo/package-review

Reply via email to