[Bug 2017670] Review Request: python-tomli-w - A Python library for writing TOML

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

 



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

Miro Hrončok <mhroncok@xxxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |mhroncok@xxxxxxxxxx



--- Comment #1 from Miro Hrončok <mhroncok@xxxxxxxxxx> ---
spec sanity:


Consider using https://github.com/hukkin/tomli-w for URL, as this is the URL
this project lists in its metadata as homepage.


Consider constructing the GitHub tarball URL to include the name of the
project, it plays much more nicely in shared SOURCES folder:


  Source0:       
https://github.com/hukkin/tomli-w/archive/%{version}/tomli-w-%{version}.tar.gz

Or even:

  Source0:        %{url}/archive/%{version}/tomli-w-%{version}.tar.gz


%description should be word-wrapped to 79 characters. rpmlint will complain
otherwise.




Note that there is a build dependency loop with flit from
https://src.fedoraproject.org/rpms/python-flit/pull-request/16 -- is there a
way to build this without flit?


-- 
You are receiving this mail because:
You are always notified about changes to this product and component
You are on the CC list for the bug.
https://bugzilla.redhat.com/show_bug.cgi?id=2017670
_______________________________________________
package-review mailing list -- package-review@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to package-review-leave@xxxxxxxxxxxxxxxxxxxxxxx
Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: https://lists.fedoraproject.org/archives/list/package-review@xxxxxxxxxxxxxxxxxxxxxxx
Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure




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

  Powered by Linux