Skip to content

Commit 6bcc513

Browse files
committed
Test without windows
1 parent 835775c commit 6bcc513

File tree

3 files changed

+6
-4
lines changed

3 files changed

+6
-4
lines changed

.github/workflows/build-wheels.yml

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@ jobs:
3333
- [ubuntu-24.04-arm, manylinux_aarch64]
3434
- [macos-13, macosx_x86_64]
3535
- [macos-14, macosx_arm64]
36-
- [windows-2022, win_amd64]
36+
# - [windows-2022, win_amd64]
3737
python: ["cp39", "cp310", "cp311", "cp312", "cp313"]
3838

3939
steps:
@@ -49,7 +49,8 @@ jobs:
4949
- name: Build wheels
5050
uses: pypa/[email protected]
5151
env:
52-
CI_WHEEL_BUILD: true
52+
CIBW_ENVIRONMENT:
53+
CI_WHEEL_BUILD=true
5354
CIBW_BUILD_VERBOSITY: 3
5455
CIBW_ENVIRONMENT_PASS_LINUX: SETUPTOOLS_SCM_PRETEND_VERSION_FOR_TILEDB S3_BUCKET TILEDB_TOKEN TILEDB_NAMESPACE RUNNER_TEMP
5556
CIBW_ENVIRONMENT_MACOS: >

tiledb/CMakeLists.txt

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,8 @@ install(TARGETS main DESTINATION tiledb)
4545

4646
# Handle TileDB shared library installation and RPATH setup
4747
if(TILEDB_DOWNLOADED)
48-
if(ENV{CI_WHEEL_BUILD})
48+
message(STATUS "CI_WHEEL_BUILD is set to: '$ENV{CI_WHEEL_BUILD}'")
49+
if(ENV{CI_WHEEL_BUILD} STREQUAL "true")
4950
# For CI builds - install to external directory
5051
set(TILEDB_EXTERNAL_DIR "$ENV{RUNNER_TEMP}/tiledb-external")
5152
message(STATUS "CI Build - installing TileDB to: ${TILEDB_EXTERNAL_DIR}")

tiledb/libtiledb/CMakeLists.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,7 @@ endif()
6060
install(TARGETS libtiledb DESTINATION tiledb)
6161

6262
if(TILEDB_DOWNLOADED)
63-
if(ENV{CI_WHEEL_BUILD})
63+
if(ENV{CI_WHEEL_BUILD} STREQUAL "true")
6464
# CI builds - shared library is in external directory
6565
set(TILEDB_EXTERNAL_DIR "$ENV{RUNNER_TEMP}/tiledb-external")
6666
if(APPLE)

0 commit comments

Comments
 (0)