Skip to content

Commit 730909a

Browse files
committed
Merge branch-24.10 into branch-25.02
2 parents 5ae8975 + 7abfdf7 commit 730909a

File tree

7 files changed

+34
-23
lines changed

7 files changed

+34
-23
lines changed

ci/conda/README.md

+4-4
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ To build the Conda packages, it's recommended to run the provided scripts from a
44

55
```bash
66
cd ${MRC_ROOT}
7-
docker buildx build --target developement -t mrc-conda-build .
7+
docker buildx build --target development -t mrc-conda-build .
88
```
99

1010
This will create the image `mrc-conda-build` that can be used to build MRC conda packages. When running this container, is recommended to set the environment variable `CONDA_PKGS_DIRS` to a path mounted on the host to speed up the build process. Without this variable set, the packages needed during the build will need to be re-downloaded each time the container is run.
@@ -16,14 +16,14 @@ To build and save the MRC conda package, run the following:
1616
```bash
1717
docker run --rm -ti -v $PWD:/work \
1818
-e CONDA_PKGS_DIRS=/work/.cache/conda_pkgs \
19-
-e CONDA_ARGS="--output-folder=/work/.conda-bld" \
19+
-e CONDA_ARGS="--output-folder=/work/.tmp/.conda-bld" \
2020
mrc-conda-build ./ci/conda/recipes/run_conda_build.sh
2121
```
2222

23-
This will save the conda packages to `${MRC_ROOT}/.conda-bld`. To install from this location, use the following:
23+
This will save the conda packages to `${MRC_ROOT}/.tmp/.conda-bld`. To install from this location, use the following:
2424

2525
```bash
26-
conda install -c file://${MRC_ROOT}/.conda-bld mrc
26+
conda install -c file://${MRC_ROOT}/.tmp/.conda-bld mrc
2727
```
2828

2929
## Uploading the Conda Package

cpp/mrc/CMakeLists.txt

+3-2
Original file line numberDiff line numberDiff line change
@@ -259,9 +259,10 @@ include(GNUInstallDirs)
259259

260260
install(
261261
TARGETS libmrc
262-
DESTINATION ${lib_dir}
263262
EXPORT ${PROJECT_NAME}-exports
264-
COMPONENT Core
263+
COMPONENT core
264+
LIBRARY
265+
DESTINATION ${lib_dir}
265266
FILE_SET public_headers
266267
)
267268

protos/CMakeLists.txt

+2-1
Original file line numberDiff line numberDiff line change
@@ -94,8 +94,9 @@ add_dependencies(${PROJECT_NAME}_style_checks mrc_protos-headers-target)
9494
install(
9595
TARGETS mrc_protos mrc_architect_protos
9696
EXPORT ${PROJECT_NAME}-exports
97+
COMPONENT core
9798
PUBLIC_HEADER
98-
DESTINATION "${CMAKE_INSTALL_INCLUDEDIR}/${PROJECT_NAME}/protos"
99+
DESTINATION "${CMAKE_INSTALL_INCLUDEDIR}/${PROJECT_NAME}/protos"
99100
)
100101

101102
list(REMOVE_ITEM CMAKE_MODULE_PATH ${MRC_PROTO_MODULE_PATH_EXTENSIONS})

python/CMakeLists.txt

+4-8
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
# SPDX-FileCopyrightText: Copyright (c) 2021-2023, NVIDIA CORPORATION & AFFILIATES. All rights reserved.
1+
# SPDX-FileCopyrightText: Copyright (c) 2021-2024, NVIDIA CORPORATION & AFFILIATES. All rights reserved.
22
# SPDX-License-Identifier: Apache-2.0
33
#
44
# Licensed under the Apache License, Version 2.0 (the "License");
@@ -15,13 +15,9 @@
1515

1616
list(APPEND CMAKE_MESSAGE_CONTEXT "python")
1717

18-
find_package(CUDAToolkit REQUIRED)
19-
20-
2118
# Ensure python is configured
2219
morpheus_utils_python_configure()
2320

24-
2521
morpheus_utils_print_python_info()
2622

2723
# Create the mrc python package
@@ -30,9 +26,9 @@ morpheus_utils_create_python_package(mrc)
3026
# Add a few additional files to be copied
3127
file(GLOB pymrc_test_files "${CMAKE_CURRENT_SOURCE_DIR}/tests/*.py")
3228
morpheus_utils_add_python_sources(
33-
"pytest.ini"
34-
"tests/string_reader_input.txt"
35-
${pymrc_test_files}
29+
"pytest.ini"
30+
"tests/string_reader_input.txt"
31+
${pymrc_test_files}
3632
)
3733

3834
# Build the pymrc library

python/mrc/_pymrc/CMakeLists.txt

+20-5
Original file line numberDiff line numberDiff line change
@@ -14,9 +14,23 @@
1414

1515
include(GenerateExportHeader)
1616

17-
find_package(Python 3.8 REQUIRED COMPONENTS Development Interpreter)
18-
find_package(pybind11 REQUIRED)
19-
find_package(prometheus-cpp REQUIRED)
17+
rapids_find_package(Python 3.8 REQUIRED
18+
COMPONENTS
19+
Development
20+
Interpreter
21+
BUILD_EXPORT_SET ${PROJECT_NAME}-python-exports
22+
INSTALL_EXPORT_SET ${PROJECT_NAME}-python-exports
23+
)
24+
25+
rapids_find_package(pybind11 REQUIRED
26+
BUILD_EXPORT_SET ${PROJECT_NAME}-python-exports
27+
INSTALL_EXPORT_SET ${PROJECT_NAME}-python-exports
28+
)
29+
30+
rapids_find_package(prometheus-cpp REQUIRED
31+
BUILD_EXPORT_SET ${PROJECT_NAME}-python-exports
32+
INSTALL_EXPORT_SET ${PROJECT_NAME}-python-exports
33+
)
2034

2135
# Keep all source files sorted!!!
2236
add_library(pymrc
@@ -120,9 +134,10 @@ include(GNUInstallDirs)
120134

121135
install(
122136
TARGETS pymrc
123-
DESTINATION ${lib_dir}
124137
EXPORT ${PROJECT_NAME}-python-exports
125-
COMPONENT Python
138+
COMPONENT python
139+
LIBRARY
140+
DESTINATION ${lib_dir}
126141
FILE_SET public_headers
127142
)
128143

python/mrc/_pymrc/tests/CMakeLists.txt

-2
Original file line numberDiff line numberDiff line change
@@ -15,8 +15,6 @@
1515

1616
list(APPEND CMAKE_MESSAGE_CONTEXT "tests")
1717

18-
find_package(pybind11 REQUIRED)
19-
2018
add_subdirectory(coro)
2119

2220
# Keep all source files sorted!!!

0 commit comments

Comments
 (0)