Skip to content

Commit eaca53c

Browse files
cxzl25dongjoon-hyun
authored andcommitted
ORC-1954: Fix CI asan-test
### What changes were proposed in this pull request? ### Why are the changes needed? https://github.com/apache/orc/actions/runs/16255164323 ``` [Invalid workflow file: .github/workflows/asan_test.yml#L33](https://github.com/apache/orc/actions/runs/16255164323/workflow) You have an error in your yaml syntax on line 33 ``` ### How was this patch tested? GA ### Was this patch authored or co-authored using generative AI tooling? No Closes #2326 from cxzl25/ORC-1954. Authored-by: sychen <sychen@ctrip.com> Signed-off-by: Dongjoon Hyun <dongjoon@apache.org>
1 parent a4ff1c8 commit eaca53c

2 files changed

Lines changed: 4 additions & 4 deletions

File tree

.github/workflows/asan_test.yml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -48,9 +48,9 @@ jobs:
4848
- name: Checkout
4949
uses: actions/checkout@v4
5050
- name: Install dependencies
51-
run: |
52-
sudo apt-get update
53-
sudo apt-get install -y build-essential cmake libpthread-stubs0-dev
51+
run: |
52+
sudo apt-get update
53+
sudo apt-get install -y build-essential cmake libpthread-stubs0-dev
5454
- name: Configure and Build with ASAN and UBSAN
5555
env:
5656
CC: ${{ matrix.cc }}

CMakeLists.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -181,7 +181,7 @@ endif()
181181

182182
# Configure Undefined Behavior Sanitizer if enabled
183183
if (ENABLE_UBSAN)
184-
if (CMAKE_CXX_COMPILER_ID MATCHES "GNU|Clang")
184+
if (CMAKE_CXX_COMPILER_ID MATCHES "Clang")
185185
set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -fsanitize=undefined -fno-sanitize=alignment,vptr,function -fno-sanitize-recover=all")
186186
set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} -fsanitize=undefined -fno-sanitize=alignment,vptr,function -fno-sanitize-recover=all")
187187
message(STATUS "Undefined Behavior Sanitizer enabled")

0 commit comments

Comments
 (0)