[Bug 1503175] Review Request: thonny - Python IDE for beginners

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

 



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

Ben Rosser <rosser.bjr@xxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |rosser.bjr@xxxxxxxxx



--- Comment #2 from Ben Rosser <rosser.bjr@xxxxxxxxx> ---
Hi Aivar,

Welcome to Fedora!

As Sergey says, the copy of the spec here and the one in the SRPM differ. In
general this is a bad idea, even though here the differences are
minor/cosmetic:

$ diff thonny.spec thonny/thonny.spec 
11d10
< 
15d13
< 
31d28
< # In order to get rid of rpmlint warning

I took a quick look at the spec; it generally looks pretty reasonable. I have a
few initial comments:

Source0:       
https://pypi.python.org/packages/01/ad/b9ce07063b9d6b9c5f3835b0256775feacd75de44d86f813924ee96d3f16/thonny-2.1.12.tar.gz

You can replace the "2.1.12" here by the "%{version}" macro; this way whenever
you bump the version of the package the Source URL will automatically be
updated, too. (It's easy to forget otherwise to change the version in the URL).

Also, if possible, it's better to use the "files.pythonhosted.org" URLs for
PyPI, as they lack the magic hashes and are generally simpler. e.g. something
like the following, where %pypi_name is "thonny":

Source0:       
https://files.pythonhosted.org/packages/source/t/%{pypi_name}/%{pypi_name}-%{version}.tar.gz

(See https://fedoraproject.org/wiki/Packaging:Python#Example_common_spec_file
for another example of this).

While it's certainly fine to have the man page and desktop file in the spec
like this, if you are the upstream maintainer, is it possible to include them
in the upstream releases in the future? Distributions other than Fedora would
likely benefit from them.

You should also look into creating and shipping an AppData file for the package
as described here: https://fedoraproject.org/wiki/Packaging:AppData.

-- 
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
To unsubscribe send an email to package-review-leave@xxxxxxxxxxxxxxxxxxxxxxx




[Index of Archives]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite Conditions]     [KDE Users]

  Powered by Linux