Skip to content

devcontainer: Use libc++ when building Envoy#31872

Merged
yanavlasov merged 1 commit intoenvoyproxy:mainfrom
fredyw:devcontainer_fix
Jan 18, 2024
Merged

devcontainer: Use libc++ when building Envoy#31872
yanavlasov merged 1 commit intoenvoyproxy:mainfrom
fredyw:devcontainer_fix

Conversation

@fredyw
Copy link
Copy Markdown
Member

@fredyw fredyw commented Jan 17, 2024

Fixes #31856

Risk Level: low
Testing: manual
Docs Changes: n/a
Release Notes: n/a
Platform Specific Features: n/a

Fixes envoyproxy#31856

Signed-off-by: Fredy Wijaya <fredyw@google.com>
@repokitteh-read-only
Copy link
Copy Markdown

As a reminder, PRs marked as draft will not be automatically assigned reviewers,
or be handled by maintainer-oncall triage.

Please mark your PR as ready when you want it to be reviewed!

🐱

Caused by: #31872 was opened by fredyw.

see: more, trace.

@fredyw fredyw marked this pull request as ready for review January 17, 2024 18:00
@fredyw
Copy link
Copy Markdown
Member Author

fredyw commented Jan 17, 2024

/assign @phlax

@yanavlasov yanavlasov merged commit c0a4200 into envoyproxy:main Jan 18, 2024
@fredyw fredyw deleted the devcontainer_fix branch January 18, 2024 17:15
@ravenblackx ravenblackx added the backport/review Request to backport to stable releases label Jan 18, 2024
phlax pushed a commit to phlax/envoy that referenced this pull request Feb 6, 2024
Fixes envoyproxy#31856

Signed-off-by: Fredy Wijaya <fredyw@google.com>
Signed-off-by: Ryan Northey <ryan@synca.io>
@phlax
Copy link
Copy Markdown
Member

phlax commented Feb 6, 2024

backport added in #32211

(same q!) @ravenblackx does this need further backport?

@ravenblackx
Copy link
Copy Markdown
Contributor

Looks like a problem that was new in 1.29 so that should be far enough.

@phlax phlax removed the backport/review Request to backport to stable releases label Feb 6, 2024
phlax pushed a commit that referenced this pull request Feb 6, 2024
Fixes #31856

Signed-off-by: Fredy Wijaya <fredyw@google.com>
Signed-off-by: Ryan Northey <ryan@synca.io>
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.

DevContainer: build failing on main

4 participants