Skip to content

[cpullvm] Added build-essential to restore missing C++ headers on ubu…#373

Open
pranav4330 wants to merge 1 commit into
qualcomm:qualcomm-softwarefrom
pranav4330:install-libc++-dependencies
Open

[cpullvm] Added build-essential to restore missing C++ headers on ubu…#373
pranav4330 wants to merge 1 commit into
qualcomm:qualcomm-softwarefrom
pranav4330:install-libc++-dependencies

Conversation

@pranav4330

Copy link
Copy Markdown
Contributor

Ubuntu 24.04 ARM runner no longer includes default C++ headers (libstdc++-dev).
This causes missing header errors like during builds.
Adding build-essential restores the standard C++ toolchain and fixes the issue.

…ntu-arm-24.04 arm runner

Signed-off-by: Pranav Patil <pranpati@qti.qualcomm.com>
# Used by lldb
sudo apt-get install swig libedit-dev
sudo apt-get install swig libedit-dev build-essential

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

build-essential brings GCC and GCC libraries, no LLVM libraries which is the cause of builder failure.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pranav4330,

Go back to the code that installs:
sudo apt install -y libc++-dev libc++abi-dev

And then try to run these commands after the installation to understand better what is failing:
echo '#include ' > test.cpp
echo 'int main() { std::cout << "Hello, Clang on Ubuntu 24.04!" << std::endl; return 0; }' >> test.cpp
clang++ -std=c++17 -stdlib=libc++ test.cpp -o test_clang
./test_clang

If this test passes, then you successfully resolved the problem. Otherwise, it might give us more hints on what needs to be fixed.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You can also add -v to clang command line to see where clang is searching for C++ lib, and see if it matches what you installed.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Okay Thank you for the input, i will check on this once and update.

@jonathonpenix

Copy link
Copy Markdown
Contributor

I think this has been superseded by #362 right?

Might want to close this one/mark it as a draft just to avoid confusion

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Setting "Request changes" just to mark that #362 is the active review to fix this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants