Skip to content

ARROW-17172: [C++][Python] test_cython_api fails on windows #14133

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.

Sign up for GitHub

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

Merged
AlenkaF merged 5 commits into from
Sep 15, 2022
Merged

ARROW-17172: [C++][Python] test_cython_api fails on windows #14133

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
bf2287f
Try setting CONDA_DLL_SEARCH_MODIFICATION_ENABLE=1 and remove mark fr…
AlenkaF Sep 15, 2022
59c7abb
Remove CONDA_DLL_SEARCH_MODIFICATION_ENABLE
AlenkaF Sep 15, 2022
6ff7bd5
Restore change
AlenkaF Sep 15, 2022
9932546
Add message about CONDA_DLL_SEARCH_MODIFICATION_ENABLE
AlenkaF Sep 15, 2022
2628b61
Apply pitrou's suggestion for the comment
AlenkaF Sep 15, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 4 additions & 0 deletions ci/appveyor-cpp-setup.bat
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -55,6 +55,10 @@ mamba update -q -y -c conda-forge --all || exit /B
@rem Create conda environment
@rem

@rem Workaround for ARROW-17172
@rem This seems necessary for test_cython.py to succeed, otherwise
@rem the extension module being built would fail loading in a subprocess.
set CONDA_DLL_SEARCH_MODIFICATION_ENABLE=1
set CONDA_PACKAGES=

if "%ARROW_BUILD_GANDIVA%" == "ON" (
Expand Down
2 changes: 0 additions & 2 deletions python/pyarrow/tests/test_cython.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -81,8 +81,6 @@ def check_cython_example_module(mod):
mod.cast_scalar(scal, pa.list_(pa.int64()))


@pytest.mark.skipif(sys.platform == "win32",
reason="ARROW-17172: currently fails on windows")
@pytest.mark.cython
def test_cython_api(tmpdir):
"""
Expand Down