Skip to content

Commit b0643ec

Browse files
authored
Merge pull request #102 from mathworks/ninja
Enable Ninja build on Windows
2 parents 3808eaa + d737dbe commit b0643ec

File tree

4 files changed

+30
-6
lines changed

4 files changed

+30
-6
lines changed

.github/workflows/build_and_test.yml

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -43,14 +43,18 @@ jobs:
4343
uses: actions/checkout@v3
4444
with:
4545
path: opentelemetry-matlab
46+
- name: Install ninja-build
47+
run: choco install ninja
4648
- name: Install MATLAB
4749
uses: matlab-actions/setup-matlab@v2
4850
with:
4951
products: MATLAB_Compiler
5052
- name: Build OpenTelemetry-Matlab
5153
working-directory: opentelemetry-matlab
54+
shell: cmd
5255
run: |
53-
cmake -S . -B build -DCMAKE_BUILD_TYPE=Release -DCMAKE_INSTALL_PREFIX=${{ env.OPENTELEMETRY_MATLAB_INSTALL }}
56+
call "C:\Program Files\Microsoft Visual Studio\2022\Enterprise\VC\Auxiliary\Build\vcvarsall.bat" x64
57+
cmake -S . -B build -G Ninja -DCMAKE_CXX_COMPILER="cl.exe" -DCMAKE_C_COMPILER="cl.exe" -DCMAKE_BUILD_TYPE=Release -DCMAKE_INSTALL_PREFIX=${{ env.OPENTELEMETRY_MATLAB_INSTALL }}
5458
cmake --build build --config Release --target install
5559
- name: Run tests
5660
env:

.github/workflows/publish_mltbx.yml

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -43,14 +43,18 @@ jobs:
4343
uses: actions/checkout@v3
4444
with:
4545
path: opentelemetry-matlab
46+
- name: Install ninja-build
47+
run: choco install ninja
4648
- name: Install MATLAB
4749
uses: matlab-actions/setup-matlab@v2
4850
with:
4951
products: MATLAB_Compiler
5052
- name: Build OpenTelemetry-Matlab
5153
working-directory: opentelemetry-matlab
54+
shell: cmd
5255
run: |
53-
cmake -S . -B build -DCMAKE_BUILD_TYPE=Release -DCMAKE_INSTALL_PREFIX=${{ env.OPENTELEMETRY_MATLAB_INSTALL }}
56+
call "C:\Program Files\Microsoft Visual Studio\2022\Enterprise\VC\Auxiliary\Build\vcvarsall.bat" x64
57+
cmake -S . -B build -G Ninja -DCMAKE_CXX_COMPILER="cl.exe" -DCMAKE_C_COMPILER="cl.exe" -DCMAKE_BUILD_TYPE=Release -DCMAKE_INSTALL_PREFIX=${{ env.OPENTELEMETRY_MATLAB_INSTALL }}
5458
cmake --build build --config Release --target install
5559
- name: Compress into single artifact
5660
working-directory: ${{ github.workspace }}

CMakeLists.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -186,7 +186,7 @@ if(NOT DEFINED OTEL_CPP_INSTALLED_DIR)
186186
PREFIX ${OTEL_CPP_PREFIX}
187187
UPDATE_DISCONNECTED 1
188188
PATCH_COMMAND ${patch_command}
189-
CMAKE_ARGS -DCMAKE_POSITION_INDEPENDENT_CODE=ON -DWITH_OTLP_HTTP=${WITH_OTLP_HTTP} -DWITH_OTLP_GRPC=${WITH_OTLP_GRPC} -DBUILD_TESTING=OFF -DWITH_BENCHMARK=OFF -DOPENTELEMETRY_INSTALL=ON -DCMAKE_TOOLCHAIN_FILE=${CMAKE_TOOLCHAIN_FILE} -DCMAKE_CXX_STANDARD=${OTEL_CPP_CXX_STANDARD} -DVCPKG_INSTALLED_DIR=${VCPKG_INSTALLED_DIR}
189+
CMAKE_ARGS -DCMAKE_POSITION_INDEPENDENT_CODE=ON -DCMAKE_CXX_COMPILER=${CMAKE_CXX_COMPILER} -DCMAKE_C_COMPILER=${CMAKE_C_COMPILER} -DCMAKE_BUILD_TYPE=${CMAKE_BUILD_TYPE} -DWITH_OTLP_HTTP=${WITH_OTLP_HTTP} -DWITH_OTLP_GRPC=${WITH_OTLP_GRPC} -DBUILD_TESTING=OFF -DWITH_BENCHMARK=OFF -DOPENTELEMETRY_INSTALL=ON -DCMAKE_TOOLCHAIN_FILE=${CMAKE_TOOLCHAIN_FILE} -DCMAKE_CXX_STANDARD=${OTEL_CPP_CXX_STANDARD} -DVCPKG_INSTALLED_DIR=${VCPKG_INSTALLED_DIR}
190190
BUILD_BYPRODUCTS ${OTEL_CPP_LIBRARIES}
191191
INSTALL_DIR ${OTEL_CPP_PREFIX}
192192
INSTALL_COMMAND ${CMAKE_COMMAND} --install . --prefix ${OTEL_CPP_PREFIX}

vcpkg.json

Lines changed: 19 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,23 @@
77
"nlohmann-json"
88
],
99
"features": {
10-
"otlp-http": {"description": "Otlp HTTP Exporter", "dependencies": ["curl"]},
11-
"otlp-grpc": {"description": "Otlp gRPC Exporter", "dependencies": ["grpc", "abseil", "c-ares", "re2", "openssl", "upb"]}
12-
}
10+
"otlp-http": {
11+
"description": "Otlp HTTP Exporter",
12+
"dependencies": [
13+
"curl"
14+
]
15+
},
16+
"otlp-grpc": {
17+
"description": "Otlp gRPC Exporter",
18+
"dependencies": [
19+
"grpc",
20+
"abseil",
21+
"c-ares",
22+
"re2",
23+
"openssl",
24+
"upb"
25+
]
26+
}
27+
},
28+
"builtin-baseline": "638b1588be3a265a9c7ad5b212cef72a1cad336a"
1329
}

0 commit comments

Comments
 (0)