New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
py-pytest-html: Add version 3.2.0 #38989
Conversation
aea416c
to
4bbc331
Compare
4bbc331
to
0a91191
Compare
EDIT: The |
0a91191
to
7ef4082
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Confirmed the version sha256.
I think there is something shady going on with dependency ordering. First, there is a python module name collision between But then, I don't understand Spack's dependency ordering:
So I still get the ADDENDUM: I confirm sometimes the install-time test would pass, but then |
92f55cb
to
eb67905
Compare
eb67905
to
9e1effe
Compare
I didn't expect this fix to go down a rabbit hole... If there is no way to control PYTHONPATH order, I'm beginning to think it is safer to constrain to pytest@:7.1 pending pytest-html@4.0.0.
EDIT: 4.0.0rc4 is not released on pypi. |
Oh, that's fun. I think the reason that no one else has complained is that the I don't think there's going to be a reliable way to control the order of this, especially if you have a root package that depends on all 3 of these packages. So I agree with the decision to pin to pytest 7.1 and older until the pytest-html 4.0 release. |
I added the conflict. I wasn't sure about it until I found the install-test would pass but then $ spack install --fresh --test root py-pytest-html ^py-pytest@7.3.2
[... install ok ... test ok ...]
$ ( spack load py-pytest-html/...; pytest -VV )
[...]
ModuleNotFoundError: No module named 'py.xml'; 'py' is not a package I'll edit the PR description. |
* py-pytest-html: add 3.2.0 * py-pytest-html: Add py-py version requirement See https://github.com/pytest-dev/pytest-html/blob/v3.2.0/setup.py#L16 * py-pytest-html: Add dependencies from setup.py and pyproject.toml * py-pytest-html: Add git url * py-pytest-html: Add conflict with py-pytest@7.2: pending py-pytest-html@4
Add py-pytest-html 3.2.0.
Add conflict with
py-pytest@7.2:
, due to conflictingpy
python modules (see PR comments). A reliable fix should come with py-pytest-html 4 (not released yet).For the record, here is the errors you'd get with a conflicting
py
python module:Or