[Bug 1366277] python-py-gfm - Github-Flavored Markdown for Python-Markdown

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

 



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

Zbigniew Jędrzejewski-Szmek <zbyszek@xxxxxxxxx> changed:

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



--- Comment #1 from Zbigniew Jędrzejewski-Szmek <zbyszek@xxxxxxxxx> ---
I'd suggest:
%global _description \
This is an implementation of GitHub-Flavored Markdown written as an extension \
...

%description %_description
%description -n python2-%{pypi_name} %_description
...


I'm not sure about the name: python-py-* suggests that this has something to do
with py.test and python-py. Maybe the name should just be python-gfm? This
matches the module name too.

? package name is OK
+ license is acceptable (BSD 3-clause)
+ license is specified correctly
+ latest version is packaged
+ modern python packaging template is used
+ %python_provide macro is used
+ P/R/BR look correct
+ no scriptlets necessary
+ %check is present
+ builds and installs OK
+ fedora-review and rpmlint are happy

rpmlint:
python-py-gfm.src: E: description-line-too-long C to the Python Markdown
library. It aims for maximal compatibility with GitHub's rendering.

OK, looks good, only thing is the name, maybe.

-- 
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://lists.fedoraproject.org/admin/lists/package-review@xxxxxxxxxxxxxxxxxxxxxxx




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