Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add customize spoof build vars support, minor opts #10

Merged
merged 11 commits into from
Jul 18, 2024
Merged

Add customize spoof build vars support, minor opts #10

merged 11 commits into from
Jul 18, 2024

Conversation

pomelohan
Copy link

pomelohan added 10 commits July 18, 2024 13:23
Signed-off-by: GarfieldHan <[email protected]>
Signed-off-by: GarfieldHan <[email protected]>
Signed-off-by: GarfieldHan <[email protected]>
1. Meet latest glaze's requirements, and upgrade it
2. Get rid of glaze_write_json to avoid using iostream, which made binary size increased. Use C methods instead.

Signed-off-by: GarfieldHan <[email protected]>
Reduce binary size by configuring right flags with it.

Signed-off-by: GarfieldHan <[email protected]>
Increase building time

Signed-off-by: GarfieldHan <[email protected]>
Mostly caused by Dobby and there is no need to take care of them.

Signed-off-by: GarfieldHan <[email protected]>
@pomelohan pomelohan marked this pull request as draft July 18, 2024 06:03
@pomelohan pomelohan marked this pull request as ready for review July 18, 2024 06:04
@chiteroman
Copy link

Nice :o

Co-authored-by: Wang Han <[email protected]>
Signed-off-by: GarfieldHan <[email protected]>
@5ec1cff 5ec1cff merged commit 4bbf952 into 5ec1cff:master Jul 18, 2024
1 check passed
@osm0sis
Copy link

osm0sis commented Jul 18, 2024

@pomelohan can you make the json entries capitalized? That way the format matches all other current ones and it'll be interoperable, plus might save some code where the Build field names are also already capitals.

@pomelohan
Copy link
Author

@pomelohan can you make the json entries capitalized? That way the format matches all other current ones and it'll be interoperable, plus might save some code where the Build field names are also already capitals.

You can modify and submit a PR by yourself since this PR has been merged 😃

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.

4 participants