Re: Review requests: four python packages to update mkdocs

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

 



On 02. 09. 21 11:33, Miro Hrončok wrote:
On 02. 09. 21 11:31, Sandro Mani wrote:

On 02.09.21 11:16, Miro Hrončok wrote:
On 02. 09. 21 10:14, Sandro Mani wrote:
On 02.09.21 02:13, Miro Hrončok wrote:
On 02. 09. 21 0:51, Sandro Mani wrote:
Hi

To update to the current version of mkdocs, I'd need these four dependencies reviewed:

- python-mkdocs-redirects: https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=2000347 - python-pyyaml_env_tag: https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=2000348 - python-ghp-import: https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=2000349 - python-mergedeep: https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=2000350

All are simple python packages.

Happy to review in exchange.

If you use the new Python macros/guidelines, I will gladly review the packages, no need to review packages in exchange.

https://fedoraproject.org/wiki/Changes/PythonPackagingGuidelines202x

Read this as my extended offer to review any package that uses the new Python macros as long as I am technically able to.

Hmm in the case of python-pyyaml_env_tag, using

[...]

%prep %autosetup -p1 -n %{srcname}-%{version} %generate_buildrequires %pyproject_buildrequires -t %build %pyproject_wheel %install %pyproject_install %pyproject_save_files %{srcname} %check %pytest

[...]

I get

[...] ERROR: tox config file (either pyproject.toml, tox.ini, setup.cfg) not found [...]

But there is a pyproject.toml file in the source archive. Any ideas how to debug this? FWIW, this is the src.rpm: https://smani.fedorapeople.org/review/python-pyyaml_env_tag-0.1-2.fc36.src.rpm

I see that the tox error wording is a tad confusing.
pyproject.toml is there, but there is no tox configuration in it.
It seems that upstream does not use tox, so neither can downstream.

So how can I make %pyproject_wheel use setup.py? Delete the pyproject.toml file?

pyproject.toml sets peotry as build backend, not setuptools -- there is no need to use setup.py?. To avoid the error, don't use the -t (tox) option for %pyproject_buildrequires.

I've meant flit, not peotry.

Here is a spec file that works:

https://churchyard.fedorapeople.org/SRPMS/python-pyyaml-env-tag.spec

--
Miro Hrončok
--
Phone: +420777974800
IRC: mhroncok
_______________________________________________
devel mailing list -- devel@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to devel-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/devel@xxxxxxxxxxxxxxxxxxxxxxx
Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure




[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [Fedora Announce]     [Fedora Users]     [Fedora Kernel]     [Fedora Testing]     [Fedora Formulas]     [Fedora PHP Devel]     [Kernel Development]     [Fedora Legacy]     [Fedora Maintainers]     [Fedora Desktop]     [PAM]     [Red Hat Development]     [Gimp]     [Yosemite News]

  Powered by Linux