Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
11 changes: 0 additions & 11 deletions .github/workflows/build-native.yml
Original file line number Diff line number Diff line change
Expand Up @@ -72,17 +72,6 @@ jobs:
with:
workspaces: crates/codegraph-core

# Force clang for the host x86_64-linux-gnu build. With gcc (the runner's
# default `cc`) the bundled C parsers in tree-sitter-hcl produce a binary
# whose HCL extractor silently drops files (#1054). Clang produces a
# working binary.
- name: Use clang for x86_64-linux-gnu
if: matrix.target == 'x86_64-unknown-linux-gnu'
run: |
clang --version
echo "CC=clang" >> "$GITHUB_ENV"
echo "CXX=clang++" >> "$GITHUB_ENV"

- name: Install cross-compilation tools (aarch64-gnu)
if: matrix.target == 'aarch64-unknown-linux-gnu'
run: |
Expand Down
14 changes: 0 additions & 14 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -61,20 +61,6 @@ jobs:
with:
workspaces: crates/codegraph-core

# NOTE: this step was added in #1059 on the assumption that gcc-vs-clang
# was responsible for the HCL .tf drop in #1054, but the real root cause
# was a tree-sitter ABI mismatch (HCL grammar shipped ABI 15 while the
# runtime was pinned at ABI 14). That is fixed by the tree-sitter 0.25
# bump in this PR, so this clang override is now redundant. Kept in place
# here to keep this PR scoped to the ABI fix; a follow-up revert removes
# the step entirely.
- name: Use clang on Linux
if: matrix.os == 'ubuntu-latest'
run: |
clang --version
echo "CC=clang" >> "$GITHUB_ENV"
echo "CXX=clang++" >> "$GITHUB_ENV"

- name: Install napi-rs CLI
run: npm install -g @napi-rs/cli@3

Expand Down
21 changes: 0 additions & 21 deletions .github/workflows/publish.yml
Original file line number Diff line number Diff line change
Expand Up @@ -44,15 +44,6 @@ jobs:
with:
workspaces: crates/codegraph-core

# Match the build-native job: gcc-built tree-sitter parsers drop files
# on Linux x86_64 (#1054). Use clang here too so preflight parity tests
# exercise the same binary characteristics as the published artifact.
- name: Use clang for native build
run: |
clang --version
echo "CC=clang" >> "$GITHUB_ENV"
echo "CXX=clang++" >> "$GITHUB_ENV"

- name: Install napi-rs CLI
run: npm install -g @napi-rs/cli@3

Expand Down Expand Up @@ -187,18 +178,6 @@ jobs:
with:
workspaces: crates/codegraph-core

# Force clang for the host x86_64-linux-gnu build. With gcc (the runner's
# default `cc`) the bundled C parsers in tree-sitter-hcl produce a binary
# whose HCL extractor silently drops files (#1054), triggering an
# expensive WASM backfill on every build call. Clang produces a working
# binary. Verified locally: gcc 14.2.0 drops .tf files, clang does not.
- name: Use clang for x86_64-linux-gnu
if: matrix.target == 'x86_64-unknown-linux-gnu'
run: |
clang --version
echo "CC=clang" >> "$GITHUB_ENV"
echo "CXX=clang++" >> "$GITHUB_ENV"

- name: Install cross-compilation tools (aarch64-gnu)
if: matrix.target == 'aarch64-unknown-linux-gnu'
run: |
Expand Down
Loading