Skip to content

Commit

Permalink
two small tweaks
Browse files Browse the repository at this point in the history
  • Loading branch information
samansmink committed Nov 14, 2024
1 parent ca1ea12 commit f801a7a
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 2 deletions.
4 changes: 2 additions & 2 deletions makefiles/duckdb_extension_c.Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -64,8 +64,8 @@ ifeq ($(DUCKDB_PLATFORM),windows_amd64_mingw)
MINGW=1
endif
ifeq ($(MINGW),1)
EXTRA_COPY_STEP_DEBUG=$(PYTHON_VENV_BIN) -c "import shutil;shutil.copyfile('cmake_build/debug/Debug/lib$(EXTENSION_LIB_FILENAME)', '$(OUTPUT_LIB_PATH_DEBUG)')"
EXTRA_COPY_STEP_RELEASE=$(PYTHON_VENV_BIN) -c "import shutil;shutil.copyfile('cmake_build/release/Release/lib$(EXTENSION_LIB_FILENAME)', '$(OUTPUT_LIB_PATH_RELEASE)')"
EXTRA_COPY_STEP_DEBUG=$(PYTHON_VENV_BIN) -c "from pathlib import Path;Path('./cmake_build/debug/Debug').mkdir(parents=True, exist_ok=True);import shutil;shutil.copyfile('cmake_build/debug/lib$(EXTENSION_LIB_FILENAME)', '$(OUTPUT_LIB_PATH_DEBUG)')"
EXTRA_COPY_STEP_RELEASE=$(PYTHON_VENV_BIN) -c "from pathlib import Path;Path('./cmake_build/release/Release').mkdir(parents=True, exist_ok=True);import shutil;shutil.copyfile('cmake_build/release/lib$(EXTENSION_LIB_FILENAME)', '$(OUTPUT_LIB_PATH_RELEASE)')"
endif

#############################################
Expand Down
8 changes: 8 additions & 0 deletions makefiles/duckdb_extension_c_api.Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -124,6 +124,14 @@ ifeq ($(DUCKDB_PLATFORM),linux_amd64)
SKIP_TESTS=1
endif

# The mingw/rtools can not be tested using the python test runner unfortunately
ifeq ($(DUCKDB_PLATFORM),windows_amd64_rtools)
SKIP_TESTS=1
endif
ifeq ($(DUCKDB_PLATFORM),windows_amd64_mingw)
SKIP_TESTS=1
endif

ifeq ($(SKIP_TESTS),1)
TEST_RELEASE_TARGET=tests_skipped
TEST_DEBUG_TARGET=tests_skipped
Expand Down

0 comments on commit f801a7a

Please sign in to comment.