pyjnius build error on s390x about pyjnius HOT 10 CLOSED

Commenting out pxi files in setup.py works for me. Thanks

from pyjnius.

I believe the plan might be to remove those lines from pyjnius itself

We should do this in a new release asap, as it may prevent jnius installation on platforms without prebuilt binaries.

I have worked building my own jnius in the past, but hadn't seen this problem before. I guess something changed in cython in the last year or two, but not that recently.

What is the long term solution for this issue instead of manually commenting the pxi files?

I only figured this out myself when I posted it, it's now clear that include files should not be compiled.

from pyjnius.

Can you try commenting out all the .pxi files in FILES in setup.py, i.e.:

FILES = [
    # 'jni.pxi',
    # 'jnius_compat.pxi',
    # 'jnius_conversion.pxi',
    # 'jnius_export_class.pxi',
    # 'jnius_export_func.pxi',
    # 'jnius_jvm_android.pxi',
    # 'jnius_jvm_desktop.pxi',
    # 'jnius_jvm_dlopen.pxi',
    # 'jnius_localref.pxi',
    # 'jnius_nativetypes3.pxi',
    # 'jnius_proxy.pxi',
    'jnius.pyx',
    # 'jnius_utils.pxi',
]

from pyjnius.

The above should work, alternatively you can use Cython 0.29.36.

from pyjnius.

Just updating cython from 0.29.35 to 0.29.36 did not help surpass the error about pxi files. I had to comment out those lines in setup.py to surpass the error. Thanks

from pyjnius.

Can you try commenting out all the .pxi files in FILES in setup.py, i.e.:

FILES = [
    # 'jni.pxi',
    # 'jnius_compat.pxi',
    # 'jnius_conversion.pxi',
    # 'jnius_export_class.pxi',
    # 'jnius_export_func.pxi',
    # 'jnius_jvm_android.pxi',
    # 'jnius_jvm_desktop.pxi',
    # 'jnius_jvm_dlopen.pxi',
    # 'jnius_localref.pxi',
    # 'jnius_nativetypes3.pxi',
    # 'jnius_proxy.pxi',
    'jnius.pyx',
    # 'jnius_utils.pxi',
]

What is the long term solution for this issue instead of manually commenting the pxi files?
Is there a plan to fix this?

from pyjnius.

Just updating cython from 0.29.35 to 0.29.36 did not help surpass the error about pxi files. I had to comment out those lines in setup.py to surpass the error. Thanks

I'm surprised that didn't work, IMHO you perhaps had leftover .c files that you needed to get rid of?

What is the long term solution for this issue instead of manually commenting the pxi files? Is there a plan to fix this?

I believe the plan might be to remove those lines from pyjnius itself, probably ideally while getting rid of the preprocessor stuff (IF and DEF) causing warnings, because I suppose the need to list the pxi's here is (EDIT: or rather, was) related to that?

EDIT: "was" because I'm not entirely certain it causes issues to drop this with current-ish Cython versions, 3.0.0 and 0.29.36 ; at least it seems to build fine here but admittedly I didn't test the result.

from pyjnius.

I think this is now addressed by #669.
Would you be able to try to install the current Git please @ravigumm?

from pyjnius.

Fixed via #669 .

Please upgrade to 1.6.0 and open a new issue if the issue persists on your side.

from pyjnius.