Skip to content

Make header generation rename the file after format#28

Merged
saki7 merged 5 commits intomainfrom
preprocess-rename-after-format
Feb 16, 2026
Merged

Make header generation rename the file after format#28
saki7 merged 5 commits intomainfrom
preprocess-rename-after-format

Conversation

@yaito3014
Copy link
Member

Currently, some Windows workflows fail due to permission problem with tuple_impl.hpp.
This PR attempts to fix the issue by not creating and formatting the file but formatting and renaming the file.

@yaito3014 yaito3014 requested a review from saki7 February 14, 2026 19:36
@yaito3014 yaito3014 added the CI Build system issues label Feb 14, 2026
@saki7
Copy link
Member

saki7 commented Feb 14, 2026

I think the real problem is that alloy build is mistakenly enabled in X4 modification; it should only build X4 on such cases. So the fix for the root issue would need modification to ci.yml.

I'm not sure why the "permission denied" error is not occurring in local build (where it builds both X4 and alloy by default) but I think the above fix would eliminate the error on CI.

@yaito3014 yaito3014 force-pushed the preprocess-rename-after-format branch from f3cef5f to b419bda Compare February 16, 2026 05:35
@yaito3014
Copy link
Member Author

I completely forgot to add Iris::Alloy as dependency for MSVC build (see 400f9e4)

@saki7
Copy link
Member

saki7 commented Feb 16, 2026

I completely forgot to add Iris::Alloy as dependency for MSVC build (see 400f9e4)

oh

Copy link
Member

@saki7 saki7 left a comment

Choose a reason for hiding this comment

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

Super Thanks

@saki7 saki7 merged commit 7949000 into main Feb 16, 2026
25 checks passed
@saki7 saki7 deleted the preprocess-rename-after-format branch February 16, 2026 11:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI Build system issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants