Skip to content

Conversation

@SirTyson
Copy link
Contributor

Adds new core dependency required in stellar/stellar-core#5101

Copilot AI review requested due to automatic review settings January 23, 2026 19:25
@github-project-automation github-project-automation bot moved this to Backlog (Not Ready) in DevX Jan 23, 2026
@SirTyson SirTyson requested review from anupsdf and leighmcculloch and removed request for Copilot January 23, 2026 19:25
@leighmcculloch leighmcculloch merged commit 29f29b3 into stellar:main Jan 23, 2026
80 checks passed
@github-project-automation github-project-automation bot moved this from Backlog (Not Ready) to Done in DevX Jan 23, 2026
apt-get -y install iproute2 procps lsb-release \
git build-essential pkg-config autoconf automake libtool \
bison flex sed perl libpq-dev parallel \
bison flex sed perl libpq-dev parallel libgoogle-perftools-dev \
Copy link
Member

Choose a reason for hiding this comment

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

Is the dependency needed at runtime too or just build time?

If it's needed at runtime too it'll need to go into the dependencies file as well. Just fyi in case you see a failure at runtime. I wouldn't proactively add it there or anywhere else unless you see a failure.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'm like 90% sure we don't need it at runtime given how we link it to our build, but the core CI test on stellar/stellar-core#5101 should double check this for us anyway.

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

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants