Skip to content

remove make autogen step#267

Merged
hasumikin merged 3 commits intomrubyc:masterfrom
hasumikin:fix-test
Feb 23, 2026
Merged

remove make autogen step#267
hasumikin merged 3 commits intomrubyc:masterfrom
hasumikin:fix-test

Conversation

@hasumikin
Copy link
Member

  • autogen files are created in rake, so making them in host container in advance is not necessary

hasumikin and others added 3 commits February 23, 2026 09:41
The src/ directory is mounted from the host in Docker-based CI, so it
is owned by the runner user and not writable by the container user.
Moving the output to build/ avoids permission conflicts because Docker
creates build/ fresh and owns it.

- Add -I$(BUILD_DIR) to CFLAGS so headers are found by the compiler.
- Add --output-dir option to make_method_table.rb.
- Accept an optional output path argument in generate_unicode_case_tables.rb.
- Simplify the autogen Make target: drop rm -f and mkdir -p $(BUILD_DIR).
- clean_all now just delegates to clean (autogen files are in build/).
- Update autogen.yml to copy from build/ when packaging release archives.

Co-Authored-By: Claude Sonnet 4.6 <[email protected]>
@hasumikin hasumikin merged commit 32c91fc into mrubyc:master Feb 23, 2026
9 of 24 checks passed
@hasumikin hasumikin deleted the fix-test branch February 23, 2026 01:22
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