Re: Request for review for Poetry(Was: Re: Asking for help Poetry)
Hi Emmanuel,
I failed to build the poetry deb with the latest package source[1] for some
pytest cases[4][5] since I was aware of this thread
https://lists.debian.org/debian-python/2021/06/msg00017.html . Do you
suggest where I should report or record the error messages I found?
By following my build steps[2] I could:
- successfully build the source with the commit "Initial review pass for
python-poetry package"[3]
- fail to build the source with the commit "ignore tests that use tar.gz
file instead of skip them"[4]
- fail to build the latest source[1][5]
Would you mind elaborating a bit more regarding the packaging steps or the
current packaging status? I walked through the mailing list and the latest
update I found is this email
https://lists.debian.org/debian-python/2021/08/msg00025.html thread. It
must be me not following the latest status and missing some necessary
information.
I will try to fix these pytest errors during package building. It would be
much appreciated that you could let me know if you are aware of anything
that I should know for fixing these errors or regarding the packaging
status. Thank you!
[1] git clone g...@salsa.debian.org:python-team/packages/poetry.git
[2] (build environment: ubuntu 20.04 on amd64)
1. Use this script to prepare the build environment
https://gitlab.com/tai271828/tool-box/-/blob/dev/scripts/prepare-deb-build-poetry.sh
2. gbp buildpackage --git-ignore-new --git-builder=sbuild
--extra-package=/home/ubuntu/build-poetry-deb/DEPENDENCIES/python3-cleo_0.8.1-2_all.deb
--extra-package=/home/ubuntu/build-poetry-deb/DEPENDENCIES/python3-httpretty_1.1.4-1_all.deb
--extra-package=/home/ubuntu/build-poetry-deb/DEPENDENCIES/python3-pkginfo_1.4.2-3_all.deb
[3] The source of the commit 92c24884d3d1a67fa5de490520d1baf83dc208bf .
Besides, I performed a quick test of the deb I built by installing it on
Sid and trying some basic commands like "poetry init" and "poetry shell".
The basic function behaviors expectedly.
[4] The source of the commit c20a910ab5ca679a362f762f5c317ca3af0e0989 and
some of the error messages:
short test summary info
===
FAILED tests/test_factory.py::test_create_poetry - AssertionError: assert
['L...
FAILED tests/inspection/test_info.py::test_info_setup_complex_pep517_error
- ...
FAILED
tests/masonry/builders/test_editable_builder.py::test_builder_installs_proper_files_for_standard_packages
### 3 failed, 576 passed, 5 skipped, 21 deselected, 15 warnings in 25.73s
short test summary info
FAILED tests/test_factory.py::test_create_poetry - AssertionError: assert
['L...
FAILED
tests/masonry/builders/test_editable_builder.py::test_builder_installs_proper_files_for_standard_packages
2 failed, 577 passed, 5 skipped, 21 deselected, 14 warnings in 24.09s
-
dh_auto_test: error: pybuild --test --test-pytest -i python{version} -p
"3.10 3.9" returned exit code 13
make: *** [debian/rules:41: binary] Error 25
dpkg-buildpackage: error: debian/rules binary subprocess returned exit
status 2
[5] Some of the error messages from latest package source:
short test summary info
FAILED tests/inspection/test_info.py::test_info_setup_complex_pep517_error
- ...
1 failed, 587 passed, 5 skipped, 23 deselected, 15 warnings in 22.23s
-
588 passed, 5 skipped, 23 deselected, 14 warnings in 24.06s ==
==
dh_auto_test: error: pybuild --test --test-pytest -i python{version} -p
"3.10 3.9" returned exit code 13
make: *** [debian/rules:44: binary] Error 25
dpkg-buildpackage: error: debian/rules binary subprocess returned exit
status 2
Kind regards,
Tai
Re: Request for review for Poetry(Was: Re: Asking for help Poetry)
On Fri, Nov 19, 2021 at 03:54:48PM +0100, Taihsiang Ho (tai271828) wrote:
> [5] Some of the error messages from latest package source:
>
> short test summary info
>
> FAILED tests/inspection/test_info.py::test_info_setup_complex_pep517_error
> - ...
> 1 failed, 587 passed, 5 skipped, 23 deselected, 15 warnings in 22.23s
> -
>
> 588 passed, 5 skipped, 23 deselected, 14 warnings in 24.06s ==
> ==
> dh_auto_test: error: pybuild --test --test-pytest -i python{version} -p
> "3.10 3.9" returned exit code 13
> make: *** [debian/rules:44: binary] Error 25
>
> dpkg-buildpackage: error: debian/rules binary subprocess returned exit
> status 2
Yes, one test fails with 3.10 which isn't unexpected because the package
wasn't tested with it.
--
WBR, wRAR
signature.asc
Description: PGP signature
Re: Request for review for Poetry(Was: Re: Asking for help Poetry)
Hi Andrey,
Thank you for your information. I will have a look at the failure. If
anyone has been working on this error, it would be appreciated to let me
know how I can help.
Besides, I found this thread
https://lists.debian.org/debian-python/2021/11/msg00025.html soon after I
sent out my previous email. I believe I have followed up the latest process
and status of poetry packaging for now. My apologies for not checking the
email threads carefully enough.
-tai
On Fri, Nov 19, 2021 at 4:19 PM Andrey Rahmatullin wrote:
> On Fri, Nov 19, 2021 at 03:54:48PM +0100, Taihsiang Ho (tai271828) wrote:
> > [5] Some of the error messages from latest package source:
> >
> > short test summary info
> >
> > FAILED
> tests/inspection/test_info.py::test_info_setup_complex_pep517_error
> > - ...
> > 1 failed, 587 passed, 5 skipped, 23 deselected, 15 warnings in 22.23s
> > -
> >
> > 588 passed, 5 skipped, 23 deselected, 14 warnings in 24.06s ==
> > ==
> > dh_auto_test: error: pybuild --test --test-pytest -i python{version} -p
> > "3.10 3.9" returned exit code 13
> > make: *** [debian/rules:44: binary] Error 25
> >
> > dpkg-buildpackage: error: debian/rules binary subprocess returned exit
> > status 2
> Yes, one test fails with 3.10 which isn't unexpected because the package
> wasn't tested with it.
>
> --
> WBR, wRAR
>
status of python-cx-oracle in Debian?
Hello, I am currently in need of the cx_Oracle python module. I see the salsa repo [0] is gathering dust at the moment... Are you still working on packaging it for Debian? Would it be okay if I took over and worked on it under the Debian Python Team? BTW - I note that since it requires the oracle client, it should go to contrib, not main. Please let me know if you understand otherwise. Thanks, --Joe 0: https://salsa.debian.org/python-team/packages/python-cx-oracle
Re: status of python-cx-oracle in Debian?
Hello, Yo be honest I did not even recalled having uploaded this one to salsa and if d/control is not properly set, please consider its under DPMT control and feel free to take over from where I left it. Best regards, Adam. On November 19, 2021 5:14:14 PM GMT+01:00, Joe Nahmias wrote: >Hello, > >I am currently in need of the cx_Oracle python module. I see the salsa repo [0] >is gathering dust at the moment... Are you still working on packaging it for >Debian? Would it be okay if I took over and worked on it under the Debian >Python Team? > >BTW - I note that since it requires the oracle client, it should go to contrib, >not main. Please let me know if you understand otherwise. > >Thanks, >--Joe > >0: https://salsa.debian.org/python-team/packages/python-cx-oracle
On updating ipython to upstream 7.29.0
Hi,
I tried to update ipython to latest upstream. I fixed some things here
and there but there's still :
1) Building the doc, some files don't get found (several no such file
or directory docs/source/whatever/file) and a few syntax errors,
perhaps due to the previous issues -- strangely, that doesn't make the
build fail!
2) Running autopkgtest doesn't work (and that's why I pushed to salsa
but didn't upload):
ERRORS
ERROR collecting test session
_
/usr/lib/python3/dist-packages/_pytest/config/__init__.py:570: in
_importconftest
mod = import_path(conftestpath, mode=importmode)
/usr/lib/python3/dist-packages/_pytest/pathlib.py:544: in import_path
raise ImportPathMismatchError(module_name, module_file, path)
E _pytest.pathlib.ImportPathMismatchError: ('IPython.conftest',
'/tmp/autopkgtest.Peukid/build.VCv/src/IPython/conftest.py',
PosixPath('/tmp/autopkgtest.Peukid/build.VCv/src/debian/python3-
ipython/usr/lib/python3/dist-packages/IPython/conftest.py'))
=== warnings summary
===
../../../../usr/lib/python3/dist-packages/nose/importer.py:12
/usr/lib/python3/dist-packages/nose/importer.py:12:
DeprecationWarning: the imp module is deprecated in favour of
importlib; see the module's documentation for alternative uses
from imp import find_module, load_module, acquire_lock,
release_lock
IPython/paths.py:66
/tmp/autopkgtest.Peukid/build.VCv/src/IPython/paths.py:66:
UserWarning: IPython parent '/home/jpuydt' is not a writable location,
using a temp directory.
warn("IPython parent '{0}' is not a writable location,"
-- Docs: https://docs.pytest.org/en/stable/warnings.html
=== short test summary info
ERROR - _pytest.pathlib.ImportPathMismatchError: ('IPython.conftest',
'/tmp/...
Interrupted: 1 error during collection
= 2 warnings, 1 error in 0.81s
=
autopkgtest [18:48:54]: test command1: ---]
autopkgtest [18:48:54]: test command1: - - - - - - - - - - results - -
- - - - - - - -
command1 FAIL non-zero exit status 2
I'm not sure the failure isn't due to my running the autopkgtest with a
schroot sbuild...
I'll try to dig further, but in the meantime if someone has a clue, I'm
all ears!
Cheers,
J.Puydt
Re: Request for review for Poetry(Was: Re: Asking for help Poetry)
Hi Tai,
First of all thanks for the mail, and for the work in the poetry package.
Very
detailed mail, amazing :)
As mentioned Andrey, yes there's a test that failed with py3.10. The
package is
already in NEW, so we need to wait. In spite of this we can work to fix it.
I didn't investigate yet, So feel free to send a patch if you have one :).
For point [2] , call gbp with the b-depends are no longer needed, because
all of them
are already in Debian.
Cheers,
Emmanuel
On Fri, Nov 19, 2021 at 12:36 PM Taihsiang Ho (tai271828)
wrote:
> Hi Andrey,
>
> Thank you for your information. I will have a look at the failure. If
> anyone has been working on this error, it would be appreciated to let me
> know how I can help.
>
> Besides, I found this thread
> https://lists.debian.org/debian-python/2021/11/msg00025.html soon after I
> sent out my previous email. I believe I have followed up the latest process
> and status of poetry packaging for now. My apologies for not checking the
> email threads carefully enough.
>
> -tai
>
> On Fri, Nov 19, 2021 at 4:19 PM Andrey Rahmatullin
> wrote:
>
>> On Fri, Nov 19, 2021 at 03:54:48PM +0100, Taihsiang Ho (tai271828) wrote:
>> > [5] Some of the error messages from latest package source:
>> >
>> > short test summary info
>> >
>> > FAILED
>> tests/inspection/test_info.py::test_info_setup_complex_pep517_error
>> > - ...
>> > 1 failed, 587 passed, 5 skipped, 23 deselected, 15 warnings in 22.23s
>> > -
>> >
>> > 588 passed, 5 skipped, 23 deselected, 14 warnings in 24.06s ==
>> > ==
>> > dh_auto_test: error: pybuild --test --test-pytest -i python{version} -p
>> > "3.10 3.9" returned exit code 13
>> > make: *** [debian/rules:44: binary] Error 25
>> >
>> > dpkg-buildpackage: error: debian/rules binary subprocess returned exit
>> > status 2
>> Yes, one test fails with 3.10 which isn't unexpected because the package
>> wasn't tested with it.
>>
>> --
>> WBR, wRAR
>>
>
