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 Text Server related options to the build profiles editor. #63871
Conversation
bruvzg
commented
Aug 3, 2022
- Adds SCons options to disable Brotli and Graphite.
- Adds option categories to the build profiles editor.
- Adds options default state to the build profiles editor.
- Adds Text Server related options to the build profiles editor.
- Fix misplaced OpenGL/Vulkan SCons options (were shifted due to missing "rendering device" option).
b89b95d
to
2ba1f8c
Compare
modules/freetype/config.py
Outdated
from SCons.Script import BoolVariable, PathVariable, Variables, Help | ||
|
||
opts = Variables() | ||
opts.Add(BoolVariable("brotli", "Enable Brotli decompressor for WOFF2 fonts support", True)) | ||
|
||
opts.Update(env) | ||
Help(opts.GenerateHelpText(env)) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think updating the opts and help might mess with this hack repeated multiple times in the SConstruct
:
Line 307 in 2c0c76c
env_base["platform"] = selected_platform # Must always be re-set after calling opts.Update(). |
We should probably solve it properly as it's a hassle.
As a side note, PathVariable
doesn't seem used?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"Mono" module seems to be doing the same, without resetting env_base["platform"]
.
Maybe modules should have separate function to return options, like platforms.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think updating the opts and help might mess with this hack repeated multiple times
Does not seem to be any difference, but I have replaced it with the get_opts()
method, similar to the one used in platforms.
2ba1f8c
to
3969fc7
Compare
3969fc7
to
e011b70
Compare
Adds SCons options to disable Brotli and Graphite. Adds option categories to the build profiles editor. Adds options default state to the build profiles editor. Adds Text Server related options to the build profiles editor. Fix misplaced OpenGL/Vulkan SCons options.
e011b70
to
066ca97
Compare
Thanks! |