jorisvandenbossche commented on code in PR #37436: URL: https://github.com/apache/arrow/pull/37436#discussion_r1310473300
########## ci/appveyor-cpp-build.bat: ########## @@ -132,10 +132,17 @@ set ARROW_HOME=%CONDA_PREFIX%\Library @rem ARROW-3075; pkgconfig is broken for Parquet for now set PARQUET_HOME=%CONDA_PREFIX%\Library +@rem Move tzdata to a non-standard location to test the +@rem configurability of the timezone database path +mkdir %USERPROFILE%\Downloads\test\tzdata +move %USERPROFILE%\Downloads\tzdata %USERPROFILE%\Downloads\test\tzdata + python setup.py develop -q || exit /B set PYTHONDEVMODE=1 +@rem Configure the path of the timesone database to a new location +python -c "import pyarrow;import os;path = os.path.expandvars(r'%USERPROFILE%\Downloads\test\tzdata');pa.set_timezone_db_path(path)" Review Comment: I also think this will not persist to the actual pytest invocation (that's a new python session, so (py)arrow will be a fresh import) So we probably have to add a way to configure this with an env variable as well? Or a pytest config variable to pass to the `pytest` cmd? (do you know how R tests this?) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
