Skip to content

Commit

Permalink
disable flight_sql support on windows after all
Browse files Browse the repository at this point in the history
it doesn't build on windows, fails without much info:
```
[75/110] Building CXX object src\arrow\flight\sql\CMakeFiles\arrow_flight_sql_shared.dir\Unity\unity_0_cxx.cxx.obj
FAILED: src/arrow/flight/sql/CMakeFiles/arrow_flight_sql_shared.dir/Unity/unity_0_cxx.cxx.obj
```
  • Loading branch information
h-vetinari committed Dec 10, 2022
1 parent 2798178 commit 16fd0e8
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 7 deletions.
1 change: 0 additions & 1 deletion recipe/bld-arrow.bat
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,6 @@ cmake -G "Ninja" ^
-DARROW_DATASET:BOOL=ON ^
-DARROW_FLIGHT:BOOL=ON ^
-DARROW_FLIGHT_REQUIRE_TLSCREDENTIALSOPTIONS:BOOL=ON ^
-DARROW_FLIGHT_SQL:BOOL=ON ^
-DARROW_GANDIVA:BOOL=ON ^
-DARROW_GCS:BOOL=ON ^
-DARROW_HDFS:BOOL=ON ^
Expand Down
10 changes: 4 additions & 6 deletions recipe/meta.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -134,21 +134,19 @@ outputs:
test:
commands:
{% set headers = [
"arrow/api.h", "arrow/flight/types.h", "arrow/flight/sql/api.h",
"gandiva/engine.h", "parquet/api/reader.h"
"arrow/api.h", "arrow/flight/types.h", "gandiva/engine.h", "parquet/api/reader.h"
] %}
{% set headers = headers + ["plasma/client.h"] %} # [unix]
{% set headers = headers + ["arrow/flight/sql/api.h", "plasma/client.h"] %} # [unix]
{% for each_header in headers %}
# headers
- test -f $PREFIX/include/{{ each_header }} || (echo "{{ each_header }} not found" && exit 1) # [unix]
- if not exist %LIBRARY_INC%\{{ "\\".join(each_header.split("/")) }} exit 1 # [win]
{% endfor %}

{% set libs = (cuda_compiler_version != "None") * ["arrow_cuda"] + [
"arrow", "arrow_dataset", "arrow_flight", "arrow_flight_sql",
"arrow_python", "gandiva", "parquet"
"arrow", "arrow_dataset", "arrow_flight", "arrow_python", "gandiva", "parquet"
] %}
{% set libs = libs + ["plasma"] %} # [unix]
{% set libs = libs + ["arrow_flight_sql", "plasma"] %} # [unix]
{% for each_lib in libs %}
# shared
- test -f $PREFIX/lib/lib{{ each_lib }}.so # [linux]
Expand Down

0 comments on commit 16fd0e8

Please sign in to comment.