From 70aea0cd7dd2a65ef8f49d799a76b0e5ee9c2ffe Mon Sep 17 00:00:00 2001 From: Jonathan Keane Date: Thu, 29 Dec 2022 10:47:53 -0600 Subject: [PATCH 1/2] Remove the virtualenv first --- r/tests/testthat/test-python.R | 3 +++ 1 file changed, 3 insertions(+) diff --git a/r/tests/testthat/test-python.R b/r/tests/testthat/test-python.R index cf10579e2de..0a2a985c96d 100644 --- a/r/tests/testthat/test-python.R +++ b/r/tests/testthat/test-python.R @@ -25,6 +25,9 @@ test_that("install_pyarrow", { # PyArrow doesn't support Python 3.6 or earlier skip_on_python_older_than("3.7") + # try and cleanup the testing virtual env if it already exists + try(reticulate::virtualenv_remove("arrow-test")) + venv <- try(reticulate::virtualenv_create("arrow-test")) # Bail out if virtualenv isn't available skip_if(inherits(venv, "try-error")) From 52f8814d940a82a019bae47c26cfce12b7adff51 Mon Sep 17 00:00:00 2001 From: Jonathan Keane Date: Thu, 29 Dec 2022 13:38:19 -0600 Subject: [PATCH 2/2] skipon 10.13 --- r/tests/testthat/test-python.R | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/r/tests/testthat/test-python.R b/r/tests/testthat/test-python.R index 0a2a985c96d..da1a4336b4a 100644 --- a/r/tests/testthat/test-python.R +++ b/r/tests/testthat/test-python.R @@ -24,9 +24,9 @@ test_that("install_pyarrow", { skip_if_not_installed("reticulate") # PyArrow doesn't support Python 3.6 or earlier skip_on_python_older_than("3.7") - - # try and cleanup the testing virtual env if it already exists - try(reticulate::virtualenv_remove("arrow-test")) + # skip on 10.13 because we do not ship binaries for pyarrow for macos 10.13 anymore + on_macos <- tolower(Sys.info()[["sysname"]]) %in% "darwin" + skip_if(on_macos && numeric_version(Sys.info()["release"]) < "18.0.0", "No pyarrow binaries are available for macOS 10.13") venv <- try(reticulate::virtualenv_create("arrow-test")) # Bail out if virtualenv isn't available