Skip to content
This repository was archived by the owner on Apr 21, 2025. It is now read-only.

Add FIDL HLCPP tables.c file to the build #798

Merged
merged 1 commit into from
Dec 7, 2023
Merged

Conversation

mk12
Copy link
Contributor

@mk12 mk12 commented Dec 6, 2023

This is a continuation of #795. This tables.c file is currently empty, but we will migrate the fidlc coding tables to it. See https://fxrev.dev/953989 for the full soft transition plan.

@mk12
Copy link
Contributor Author

mk12 commented Dec 6, 2023

Note: Must wait for https://fxrev.dev/953989 to land and roll into flutter before merging this PR.

I changed it so that it will work both before and after the roll.

This is a continuation of flutter#795. This tables.c file is currently empty,
but we will migrate the fidlc coding tables to it. See
https://fxrev.dev/953989 for the full soft transition plan.
@jrwang jrwang merged commit bb212f1 into flutter:master Dec 7, 2023
@mk12 mk12 deleted the tables branch December 7, 2023 20:40
mk12 added a commit to mk12/buildroot that referenced this pull request Jan 26, 2024
Now that flutter#798 added the new tables.c file, and https://fxrev.dev/946309
moved the content from the old file to the new file, we can stop
generating the old file (i.e. stop passing --tables to fidlc).

See https://fxbug.dev/42115242 for context.
mk12 added a commit that referenced this pull request Jan 29, 2024
Now that #798 added the new tables.c file, and https://fxrev.dev/946309
moved the content from the old file to the new file, we can stop
generating the old file (i.e. stop passing --tables to fidlc).

See https://fxbug.dev/42115242 for context.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants