Skip to content

Comments

Fix cxx_flags order#424

Merged
keith merged 1 commit intomasterfrom
ks/fix-cxx_flags-order
Aug 28, 2025
Merged

Fix cxx_flags order#424
keith merged 1 commit intomasterfrom
ks/fix-cxx_flags-order

Conversation

@keith
Copy link
Member

@keith keith commented Aug 28, 2025

Default flags should come before flags passed with --copt or copts = []

Broken by 7009b77 fixes #422

Default flags should come before flags passed with `--copt` or `copts = []`

Broken by 7009b77 fixes #422
@keith keith force-pushed the ks/fix-cxx_flags-order branch from 46064c4 to 8f6e199 Compare August 28, 2025 15:39
@keith keith merged commit 4797ada into master Aug 28, 2025
12 checks passed
@keith keith deleted the ks/fix-cxx_flags-order branch August 28, 2025 15:52
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.

bazel_dep(name = "apple_support", version = "1.23.0") leads to issues with std::lerp?

2 participants