Skip to content

Fix Objective-C source routing in Ninja backend#97

Merged
Sunrisepeak merged 1 commit into
mainfrom
codex/fix-objc-source-rule
Jun 1, 2026
Merged

Fix Objective-C source routing in Ninja backend#97
Sunrisepeak merged 1 commit into
mainfrom
codex/fix-objc-source-rule

Conversation

@Sunrisepeak
Copy link
Copy Markdown
Member

Summary

  • route .m Objective-C sources through the C/Objective-C ninja rule instead of cxx_object
  • add a focused ninja backend regression test that verifies .m uses c_object and package-owned cflags
  • bump mcpp to 0.0.41 and document the fix

Validation

  • mcpp test -- --gtest_filter=NinjaBackend.ObjectiveCSourceUsesCObjectRuleAndCFlags (red before the fix, green after)
  • MCPP=$(realpath "$(find target -type f -name mcpp -printf '%T@ %p\n' | sort -rn | head -1 | cut -d' ' -f2)") timeout 600 bash tests/e2e/26_c_language_support.sh
  • mcpp test
  • git diff --check

@Sunrisepeak Sunrisepeak force-pushed the codex/fix-objc-source-rule branch from 2ae1885 to 94a0549 Compare June 1, 2026 02:13
@Sunrisepeak Sunrisepeak merged commit 41d5f96 into main Jun 1, 2026
3 checks passed
@Sunrisepeak Sunrisepeak deleted the codex/fix-objc-source-rule branch June 1, 2026 02:35
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.

1 participant