Skip to content
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

Jython compatibility #3111

Closed
frenzymadness opened this issue Jan 12, 2018 · 8 comments
Closed

Jython compatibility #3111

frenzymadness opened this issue Jan 12, 2018 · 8 comments
Labels
type: bug problem that needs to be addressed type: infrastructure improvement to development/releases/CI structure

Comments

@frenzymadness
Copy link

Hello.

I've tried to use pytest with Jython but it fails. I have an empty docker container based on Fedora 27 and Jython 2.7.1 (latest stable) installed from Fedora repository and pytest installed from PyPI. The problem I have doesn't come from pytest but it is caused by some incompatibility in the attrs module.

The question is whether Jython can be still considered as a supported platform and the issue is only a bug in pytest dependencies.

I've filled issue for the attrs module as well: python-attrs/attrs#321

Thank you for all your work and have a nice day.

@pytestbot
Copy link
Contributor

GitMate.io thinks the contributors most likely able to help are @RonnyPfannschmidt, and @nicoddemus.

@pytestbot pytestbot added the type: bug problem that needs to be addressed label Jan 12, 2018
@nicoddemus
Copy link
Member

Hi @frenzymadness,

Jython currently is not part of our CI process so I don't consider it completely supported, but pytest does have some code to deal with Jython specifics.

Not sure what's the opinion of other maintainers: should we strive to make Jython a first-class platform to be supported by including it in CI or should we remove its support officially at some point in the future?

@fabioz
Copy link
Contributor

fabioz commented Jul 31, 2018

As a note, the current version (3.7.0) is no longer installing with pip on Jython because of scandir (I think it's a new dependency as it was certainly working until last week -- pip install pytest==3.6 still works though).

Report on scandir: benhoyt/scandir#105

@nicoddemus
Copy link
Member

@fabioz thanks for the info.

scandir is a dependency of pathlib2, which we now use on Python<3.4.

Thanks for the upstream report. 👍

@RonnyPfannschmidt
Copy link
Member

as far as i understood scandir cannot be supported on jython unless jython adds support in some way

@RonnyPfannschmidt
Copy link
Member

wondering how hard it would be o add automatic wheels for it with manylinux/appveyor

@RonnyPfannschmidt
Copy link
Member

benhoyt/scandir#106 should fix it

@nicoddemus
Copy link
Member

Now that benhoyt/scandir#106 has been merged we can close this. Thanks again @frenzymadness for the report.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug problem that needs to be addressed type: infrastructure improvement to development/releases/CI structure
Projects
None yet
Development

No branches or pull requests

5 participants