[Bug 1910392] Review Request: python-build - Simple, correct PEP517 package builder

2022-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1910392

Fabian Affolter  changed:

   What|Removed |Added

  Flags|needinfo?(mail@fabian-affol |
   |ter.ch) |




-- 
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=1910392
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
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@lists.fedoraproject.org
Do not reply to spam, report it: 
https://pagure.io/fedora-infrastructure/new_issue


[Bug 1910392] Review Request: python-build - Simple, correct PEP517 package builder

2021-06-27 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1910392

Lumír Balhar  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution|--- |DUPLICATE
Last Closed|2021-02-27 00:45:28 |2021-06-27 09:44:16



--- Comment #13 from Lumír Balhar  ---
Thanks, Miro! My review request is here:
https://bugzilla.redhat.com/show_bug.cgi?id=1976483

*** This bug has been marked as a duplicate of bug 1976483 ***


-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
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@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 1910392] Review Request: python-build - Simple, correct PEP517 package builder

2021-06-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1910392



--- Comment #12 from Miro Hrončok  ---
https://fedoraproject.org/wiki/Policy_for_stalled_package_reviews?rd=Extras/Policy/StalledReviews#Submitter_not_responding

> When the submitter of a review ticket has not responded to comments for one 
> month

Last "response" is 2021-02-27.

> a comment is added to the ticket indicating that the review is stalled and 
> that a response is needed soon.

That was technically at 2021-06-11.

> If there is no response within one week, the ticket is closed

We are there. I won't close this just yet, but Lumír if you sumbit another
review request for this, feel free to mark this one as a duplicate.


-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
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@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 1910392] Review Request: python-build - Simple, correct PEP517 package builder

2021-06-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1910392

Lumír Balhar  changed:

   What|Removed |Added

 CC||lbal...@redhat.com



--- Comment #11 from Lumír Balhar  ---
Hello, Fabian.

I'd like to package this tool to Fedora. If you are no longer interested in
this review, could you please close it? I'll then open a new one.

You can see my progress in:
https://copr.fedorainfracloud.org/coprs/lbalhar/build/builds/


-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
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@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 1910392] Review Request: python-build - Simple, correct PEP517 package builder

2021-06-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1910392

Ben Beasley  changed:

   What|Removed |Added

  Flags||needinfo?(mail@fabian-affol
   ||ter.ch)



--- Comment #10 from Ben Beasley  ---
Are you still planning to package this?


-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
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@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 1910392] Review Request: python-build - Simple, correct PEP517 package builder

2021-03-30 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1910392

Miro Hrončok  changed:

   What|Removed |Added

 CC||mhron...@redhat.com



--- Comment #9 from Miro Hrončok  ---
%generate_buildrequires
%pyproject_buildrequires -x test

%build
%py3_build

%install
%py3_install


Note that mixing %pyproject_buildrequires with %py3_build/%py3_install is
generally not a supported way of building Python packages. Please, either use
the "old" macros, or the "new", described in
https://src.fedoraproject.org/rpms/pyproject-rpm-macros


-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
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@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 1910392] Review Request: python-build - Simple, correct PEP517 package builder

2021-02-27 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1910392

Fabian Affolter  changed:

   What|Removed |Added

   Keywords|Reopened|




-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
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@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 1910392] Review Request: python-build - Simple, correct PEP517 package builder

2021-02-27 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1910392

Fabian Affolter  changed:

   What|Removed |Added

 Status|CLOSED  |NEW
 Blocks|201449 (FE-DEADREVIEW)  |
 Resolution|NOTABUG |---
   Keywords||Reopened





Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=201449
[Bug 201449] FE-DEADREVIEW -- Reviews stalled due to lack of submitter response
should be blocking this bug.
-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
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@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 1910392] Review Request: python-build - Simple, correct PEP517 package builder

2021-01-27 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1910392



--- Comment #7 from c...@musicinmybrain.net ---
Created attachment 1751328
  --> https://bugzilla.redhat.com/attachment.cgi?id=1751328&action=edit
python-build-0.1.0-mypy-importlib-metadata.patch

Patch for mypy/typing issue


-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
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@lists.fedoraproject.org


[Bug 1910392] Review Request: python-build - Simple, correct PEP517 package builder

2021-01-27 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1910392



--- Comment #6 from c...@musicinmybrain.net ---
Created attachment 1751327
  --> https://bugzilla.redhat.com/attachment.cgi?id=1751327&action=edit
Spec file demonstrating review suggestions

This spec file should implement all of the suggestions from the review. Feel
free to use it, or do things your own way.


-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
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@lists.fedoraproject.org


[Bug 1910392] Review Request: python-build - Simple, correct PEP517 package builder

2021-01-27 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1910392

c...@musicinmybrain.net changed:

   What|Removed |Added

  Flags||needinfo?(mail@fabian-affol
   ||ter.ch)



--- Comment #5 from c...@musicinmybrain.net ---
Package Review
==

Legend:
[x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated
[ ] = Manual review needed


Issues:
===
- Package installs properly.
  Note: Installation errors (see attachment)
  See: https://docs.fedoraproject.org/en-US/packaging-guidelines/

  The actual error looks like:
  Error:
   Problem: conflicting requests
 - nothing provides python3.9dist(mypy) = 0.790 needed by
python3-build+typing-0.1.0-2.fc34.noarch

  This is because the typing extension requires mypy==0.790. Consider something
  like this in %prep:

# Loosen exact-version mypy requirement for typing extra:
sed -r -i 's/(mypy)==/\1>=/' setup.cfg

  Note that only Fedora 34 provides a mypy >= 0.790. If you want to package for
  older Fedoras, you need to lower or eliminate the minimum version and then
  test it. For example:

# Drop exact-version mypy requirement for typing extra
sed -r -i 's/(mypy)==.*/\1/' setup.cfg

  If you do that (and maybe if you don’t) you could change 
%pyproject_buildrequires -x test
  to
%pyproject_buildrequires -x test,typing
  and then, in %check:

%{python3} -m mypy %{buildroot}%{python3_sitelib}/{pypi_name}

  to verify it works as expected.

- Once you do the above, you will get an error from mypy like:

Module 'importlib' has no attribute 'metadata'

  I have filed an upstream bug https://github.com/pypa/build/issues/211 and
  attached a patch corresponding to my accepted PR
  https://github.com/pypa/build/pull/213.

- You can build the docs even without the furo theme.
%pyproject_buildrequires -x test,typing,doc
  and in %prep,
# Build docs with alternative theme since python-furo is not packaged
# (https://bugzilla.redhat.com/show_bug.cgi?id=1910798):
sed -r -i "s/(html_theme[[:blank:]]*=[[:blank:]]*['])furo'/\1alabaster'/" \
docs/source/conf.py
sed -r -i '/^[[:blank:]]*furo[[:blank:]]*$/d' \
setup.cfg docs/requirements.txt
  then add the appropriate -doc subpackage.

- The %python_provides macro is obsolete. Remove it entirely; or, if you are
  packaging for Fedora 32, you may replace it with:

# Obsolete after Fedora 32 EOL:
%py_provides python3-%{pypi_name}

  See:
 
https://docs.fedoraproject.org/en-US/packaging-guidelines/Python/#_the_py_provides_macro

  EPEL does need python_provides, but lacks support for the
  pyproject-rpm-macros, so I know you are not targeting it with this spec file.

- You should run the test suite by adding the following under the %check
section:

%tox

  For the generated BR’s, you will now need:

%pyproject_buildrequires -t -x typing,doc

  to get the BR’s from the tox configuration, and to include tox itself.

  You will have to skip one test. I am honestly not sure what is going on here.

# _ test_create_isolated_build_host_with_no_pip
__
# [gw2] linux -- Python 3.9.1 /usr/bin/python3
# 
# tmp_path =
PosixPath('/tmp/pytest-of-mockbuild/pytest-0/popen-gw2/test_create_isolated_build_hos0')
# capfd = <_pytest.capture.CaptureFixture object at 0x7fea79428670>
# mocker = 
# 
# @pytest.mark.isolated
# def test_create_isolated_build_host_with_no_pip(tmp_path, capfd,
mocker):
# mocker.patch.object(build.env, 'pip', None)
# expected = {'pip', 'greenlet', 'readline', 'cffi'} if
platform.python_implementation() == 'PyPy' else {'pip'}
# 
# with build.env.IsolatedEnvBuilder() as isolated_env:
# cmd = [isolated_env.executable, '-m', 'pip', 'list',
'--format', 'json']
# packages = {p['name'] for p in
json.loads(subprocess.check_output(cmd, universal_newlines=True))}
# >   assert packages == expected
# E   AssertionError: assert {'build', 'pip'} == {'pip'}
# E Extra items in the left set:
# E 'build'
# E Use -v to get the full diff
# 
# tests/test_env.py:57: AssertionError
export PYTEST_ADDOPTS='-k "not
test_create_isolated_build_host_with_no_pip"'

= MUST items =

Generic:
[x]: Package is licensed with an open-source compatible license and meets
 other legal requirements as defined in the legal section of Packaging
 Guidelines.
[x]: License field in the package spec file matches the actual license.
 Note: Checking patched sources after %prep for licenses. Licenses
 found: "Unknown or generated", "*No copyright* Expat License". 37
 files have unknown license. Detailed output of licensecheck in
 /home/re

[Bug 1910392] Review Request: python-build - Simple, correct PEP517 package builder

2021-01-27 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1910392

c...@musicinmybrain.net changed:

   What|Removed |Added

   Assignee|nob...@fedoraproject.org|c...@musicinmybrain.net
  Flags||fedora-review?




-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
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@lists.fedoraproject.org


[Bug 1910392] Review Request: python-build - Simple, correct PEP517 package builder

2021-01-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1910392



--- Comment #4 from Fabian Affolter  ---
(In reply to code from comment #2)
> This would be a great candidate for using generated BR’s:
> 
> %generate_buildrequires
> %pyproject_buildrequires -x test
> 
> %prep
> 
> -
> 
> It saves you the trouble of manually enumerating, and possibly missing, BR’s
> (and re-doing it for every upstream update), and it automatically puts in
> version requirements too.
> 
> See https://src.fedoraproject.org/rpms/pyproject-rpm-macros for more.

Changed

(In reply to code from comment #3)
> For the typing extra, this should have:
> 
> %{?python_extras_subpkg:%python_extras_subpkg -n python3-%{pypi_name} -i
> %{python3_sitelib}/*.egg-info typing}
> 
> Details at
> https://docs.fedoraproject.org/en-US/packaging-guidelines/Python/
> #_python_extras.

Added

Koji scratch build:
https://koji.fedoraproject.org/koji/taskinfo?taskID=60540015

%changelog
* Sun Jan 24 2021 Fabian Affolter  - 0.1.0-2
- Switch to pyproject-rpm-macros (#1910392)

Spec URL: https://fab.fedorapeople.org/packages/SRPMS/python-build.spec
SRPM URL:
https://fab.fedorapeople.org/packages/SRPMS/python-build-0.1.0-2.fc33.src.rpm


-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
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@lists.fedoraproject.org


[Bug 1910392] Review Request: python-build - Simple, correct PEP517 package builder

2021-01-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1910392



--- Comment #3 from c...@musicinmybrain.net ---
For the typing extra, this should have:

%{?python_extras_subpkg:%python_extras_subpkg -n python3-%{pypi_name} -i
%{python3_sitelib}/*.egg-info typing}

Details at
https://docs.fedoraproject.org/en-US/packaging-guidelines/Python/#_python_extras.


-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
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@lists.fedoraproject.org


[Bug 1910392] Review Request: python-build - Simple, correct PEP517 package builder

2021-01-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1910392



--- Comment #2 from c...@musicinmybrain.net ---
This would be a great candidate for using generated BR’s:

-

%package -n python3-%{pypi_name}
Summary:%{summary}

BuildRequires:  python3-devel
BuildRequires:  python3dist(setuptools)
BuildRequires:  pyproject-rpm-macros

%description -n python3-%{pypi_name}
build will invoke the PEP 517 hooks to build a distribution package. It is a
simple build tool and does not perform any dependency management.

%generate_buildrequires
%pyproject_buildrequires -x test

%prep

-

It saves you the trouble of manually enumerating, and possibly missing, BR’s
(and re-doing it for every upstream update), and it automatically puts in
version requirements too.

See https://src.fedoraproject.org/rpms/pyproject-rpm-macros for more.


-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
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@lists.fedoraproject.org


[Bug 1910392] Review Request: python-build - Simple, correct PEP517 package builder

2021-01-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1910392

c...@musicinmybrain.net changed:

   What|Removed |Added

 CC||c...@musicinmybrain.net
   Doc Type|--- |If docs needed, set a value



--- Comment #1 from c...@musicinmybrain.net ---
Weird that it built in Koji, since it fails to build in mock for me. Looks like
it’s missing a BR on python3dist(pep517).

The following is excerpted from the output of “fedora-review -b 1910392”:

+ /usr/bin/pytest -v tests -k 'not test_create_isolated_build_host_with_no_pip
and not test_build_isolated   and not test_build_no_isolation_check_deps_empty
and not test_build_no_isolation_with_check_deps   and not
test_build_raises_build_exception and not
test_build_raises_build_backend_exception' --ignore
tests/test_projectbuilder.py
= test session starts
==
platform linux -- Python 3.9.1, pytest-6.0.2, py-1.10.0, pluggy-0.13.1 --
/usr/bin/python3
cachedir: .pytest_cache
rootdir: /builddir/build/BUILD/build-0.1.0, configfile: setup.cfg
plugins: forked-1.3.0, xdist-2.2.0, cov-2.10.1, mock-3.4.0
collecting ... collected 0 items / 4 errors
 ERRORS

__ ERROR collecting tests/test_env.py
__
ImportError while importing test module
'/builddir/build/BUILD/build-0.1.0/tests/test_env.py'.
Hint: make sure your test modules/packages have valid Python names.
Traceback:
/usr/lib64/python3.9/importlib/__init__.py:127: in import_module
return _bootstrap._gcd_import(name[level:], package, level)
tests/test_env.py:12: in 
import build.env
../../BUILDROOT/python-build-0.1.0-1.fc34.x86_64/usr/lib/python3.9/site-packages/build/__init__.py:16:
in 
import pep517.wrappers
E   ModuleNotFoundError: No module named 'pep517'
__ ERROR collecting tests/test_integration.py
__
ImportError while importing test module
'/builddir/build/BUILD/build-0.1.0/tests/test_integration.py'.
Hint: make sure your test modules/packages have valid Python names.
Traceback:
/usr/lib64/python3.9/importlib/__init__.py:127: in import_module
return _bootstrap._gcd_import(name[level:], package, level)
tests/test_integration.py:16: in 
import filelock
E   ModuleNotFoundError: No module named 'filelock'
_ ERROR collecting tests/test_main.py
__
ImportError while importing test module
'/builddir/build/BUILD/build-0.1.0/tests/test_main.py'.
Hint: make sure your test modules/packages have valid Python names.
Traceback:
/usr/lib64/python3.9/importlib/__init__.py:127: in import_module
return _bootstrap._gcd_import(name[level:], package, level)
tests/test_main.py:10: in 
import build
../../BUILDROOT/python-build-0.1.0-1.fc34.x86_64/usr/lib/python3.9/site-packages/build/__init__.py:16:
in 
import pep517.wrappers
E   ModuleNotFoundError: No module named 'pep517'
 ERROR collecting tests/test_module.py
_
ImportError while importing test module
'/builddir/build/BUILD/build-0.1.0/tests/test_module.py'.
Hint: make sure your test modules/packages have valid Python names.
Traceback:
/usr/lib64/python3.9/importlib/__init__.py:127: in import_module
return _bootstrap._gcd_import(name[level:], package, level)
tests/test_module.py:3: in 
import build
../../BUILDROOT/python-build-0.1.0-1.fc34.x86_64/usr/lib/python3.9/site-packages/build/__init__.py:16:
in 
import pep517.wrappers
E   ModuleNotFoundError: No module named 'pep517'
=== short test summary info

ERROR tests/test_env.py
ERROR tests/test_integration.py
ERROR tests/test_main.py
ERROR tests/test_module.py
!!! Interrupted: 4 errors during collection

== 4 errors in 0.32s
===
error: Bad exit status from /var/tmp/rpm-tmp.zWPJDu (%check)
Bad exit status from /var/tmp/rpm-tmp.zWPJDu (%check)


-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
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@lists.fedoraproject.org


[Bug 1910392] Review Request: python-build - Simple, correct PEP517 package builder

2020-12-24 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1910392

Fabian Affolter  changed:

   What|Removed |Added

 Blocks||1905174





Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1905174
[Bug 1905174] F34FailsToInstall: python3-check-manifest
-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
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@lists.fedoraproject.org


[Bug 1910392] Review Request: python-build - Simple, correct PEP517 package builder

2020-12-23 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1910392

Fabian Affolter  changed:

   What|Removed |Added

 Blocks||1893382





Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1893382
[Bug 1893382] python-check-manifest-0.45 is available
-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
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@lists.fedoraproject.org