Skip to content

Commit

Permalink
Merge pull request #10275 from Ethyling/conda-comp
Browse files Browse the repository at this point in the history
This PR enables the usage of conda compilers to build conda packages. This is required to use `mambabuild`

Authors:
   - Jordan Jacobelli (https://github.com/Ethyling)

Approvers:
   - AJ Schmidt (https://github.com/ajschmidt8)
   - GALI PREM SAGAR (https://github.com/galipremsagar)
   - https://github.com/jakirkham
  • Loading branch information
jjacobelli authored May 23, 2022
2 parents 6acf226 + 8634789 commit f10f380
Show file tree
Hide file tree
Showing 8 changed files with 50 additions and 15 deletions.
8 changes: 8 additions & 0 deletions conda/recipes/cudf/conda_build_config.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,8 @@
c_compiler_version:
- 9

cxx_compiler_version:
- 9

sysroot_version:
- "2.17"
7 changes: 4 additions & 3 deletions conda/recipes/cudf/meta.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -19,15 +19,16 @@ build:
script_env:
- VERSION_SUFFIX
- PARALLEL_LEVEL
- CC
- CXX
- CUDAHOSTCXX
# libcudf's run_exports pinning is looser than we would like
ignore_run_exports:
- libcudf

requirements:
build:
- {{ compiler('c') }}
- {{ compiler('cxx') }}
- sysroot_{{ target_platform }} {{ sysroot_version }}
host:
- protobuf
- python
- cython >=0.29,<0.30
Expand Down
8 changes: 8 additions & 0 deletions conda/recipes/cudf_kafka/conda_build_config.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,8 @@
c_compiler_version:
- 9

cxx_compiler_version:
- 9

sysroot_version:
- "2.17"
6 changes: 3 additions & 3 deletions conda/recipes/cudf_kafka/meta.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -17,15 +17,15 @@ build:
number: {{ GIT_DESCRIBE_NUMBER }}
string: py{{ py_version_numeric }}_{{ GIT_DESCRIBE_HASH }}_{{ GIT_DESCRIBE_NUMBER }}
script_env:
- CC
- CXX
- CUDAHOSTCXX
- PARALLEL_LEVEL
- VERSION_SUFFIX

requirements:
build:
- cmake >=3.20.1,!=3.23.0
- {{ compiler('c') }}
- {{ compiler('cxx') }}
- sysroot_{{ target_platform }} {{ sysroot_version }}
host:
- python
- cython >=0.29,<0.30
Expand Down
3 changes: 0 additions & 3 deletions conda/recipes/custreamz/meta.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -19,9 +19,6 @@ build:
script_env:
- VERSION_SUFFIX
- PARALLEL_LEVEL
- CC
- CXX
- CUDAHOSTCXX

requirements:
host:
Expand Down
3 changes: 0 additions & 3 deletions conda/recipes/dask-cudf/meta.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -19,9 +19,6 @@ build:
script_env:
- VERSION_SUFFIX
- PARALLEL_LEVEL
- CC
- CXX
- CUDAHOSTCXX

requirements:
host:
Expand Down
12 changes: 12 additions & 0 deletions conda/recipes/libcudf/conda_build_config.yaml
Original file line number Diff line number Diff line change
@@ -1,3 +1,15 @@
c_compiler_version:
- 9

cxx_compiler_version:
- 9

cuda_compiler:
- nvcc

sysroot_version:
- "2.17"

cmake_version:
- ">=3.20.1,!=3.23.0"

Expand Down
18 changes: 15 additions & 3 deletions conda/recipes/libcudf/meta.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -14,9 +14,6 @@ source:

build:
script_env:
- CC
- CXX
- CUDAHOSTCXX
- PARALLEL_LEVEL
- CMAKE_GENERATOR
- CMAKE_C_COMPILER_LAUNCHER
Expand All @@ -31,6 +28,10 @@ build:
requirements:
build:
- cmake {{ cmake_version }}
- {{ compiler('c') }}
- {{ compiler('cxx') }}
- {{ compiler('cuda') }} {{ cuda_version }}
- sysroot_{{ target_platform }} {{ sysroot_version }}
host:
- librmm {{ minor_version }}.*
- cudatoolkit {{ cuda_version }}.*
Expand All @@ -48,6 +49,8 @@ outputs:
string: cuda{{ cuda_major }}_{{ GIT_DESCRIBE_HASH }}_{{ GIT_DESCRIBE_NUMBER }}
run_exports:
- {{ pin_subpackage("libcudf", max_pin="x.x") }}
ignore_run_exports_from:
- {{ compiler('cuda') }}
requirements:
build:
- cmake {{ cmake_version }}
Expand Down Expand Up @@ -287,6 +290,8 @@ outputs:
build:
number: {{ GIT_DESCRIBE_NUMBER }}
string: {{ GIT_DESCRIBE_HASH }}_{{ GIT_DESCRIBE_NUMBER }}
ignore_run_exports_from:
- {{ compiler('cuda') }}
requirements:
build:
- cmake {{ cmake_version }}
Expand All @@ -308,9 +313,14 @@ outputs:
build:
number: {{ GIT_DESCRIBE_NUMBER }}
string: {{ GIT_DESCRIBE_HASH }}_{{ GIT_DESCRIBE_NUMBER }}
ignore_run_exports_from:
- {{ compiler('cuda') }}
requirements:
build:
- cmake {{ cmake_version }}
- {{ compiler('c') }}
- {{ compiler('cxx') }}
- sysroot_{{ target_platform }} {{ sysroot_version }}
host:
- {{ pin_subpackage('libcudf', exact=True) }}
run:
Expand All @@ -327,6 +337,8 @@ outputs:
build:
number: {{ GIT_DESCRIBE_NUMBER }}
string: cuda{{ cuda_major }}_{{ GIT_DESCRIBE_HASH }}_{{ GIT_DESCRIBE_NUMBER }}
ignore_run_exports_from:
- {{ compiler('cuda') }}
requirements:
build:
- cmake {{ cmake_version }}
Expand Down

0 comments on commit f10f380

Please sign in to comment.