Skip to content

fix(cpp): pre-create .amp directory to allow clangd to find it #1025

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
rjaegers merged 3 commits into from
Nov 25, 2025
Merged

fix(cpp): pre-create .amp directory to allow clangd to find it #1025

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
dac77f7
fix: pre-create .amp directory to allow clangd to find it
rjaegers Nov 25, 2025
9bfa051
test: add test for clangd compile commands path
rjaegers Nov 25, 2025
4dd0b6a
test: update integration-tests.bats
rjaegers Nov 25, 2025
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
1 change: 1 addition & 0 deletions .devcontainer/cpp/Dockerfile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -114,6 +114,7 @@ RUN --mount=type=cache,target=/var/log,sharing=locked \
--slave /usr/bin/mull-reporter mull-reporter /usr/bin/mull-reporter-${CLANG_VERSION} \
--slave /usr/lib/mull-ir-frontend mull-ir-frontend /usr/lib/mull-ir-frontend-${CLANG_VERSION} \
&& update-alternatives --install /usr/bin/python python /usr/bin/python3 10 \
&& mkdir /root/.amp \
&& cp /etc/skel/.bashrc /root/.bashrc

# Set up package managers CPM and Conan
Expand Down
5 changes: 5 additions & 0 deletions test/cpp/integration-tests.bats
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -178,6 +178,11 @@ teardown() {
assert_output --partial "All checks completed, 0 errors"
}

@test "clangd should start with a specified compile commands path" {
run timeout 1s clangd --compile-commands-dir=/root/.amp
refute_output --partial "Path specified by --compile-commands-dir does not exist. The argument will be ignored."
}

@test "using lld as an alternative linker should result in working host executable" {
cmake --preset gcc
cmake --build --preset gcc-lld
Expand Down