You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Built-in variables, for example: $(buildir), $(arch), $(plat) ...
With the growing number of modules in the scripting scope, there are now hundreds of modules and tens of thousands of module APIs, it is not possible to support built-in variables for all parameters of all script module APIs. This leads to a lot of confusion.
Some of the current interfaces that support built-in variables in scripts:
print
os.execv/os.runv ...
os.cp/os.mv/...
In the script domain, we can call the module interface directly to get the values of these variables, e.g. config.plat(), os.projectdir() ...
Therefore, I think it would be clearer for users to just keep support for built-in variables in the description scope.
Whether or not to remove support for built-in variables in script scope?
Remove support for built-in variables in script scope, support it only in the description scope.
73%
Does not remove built-in variables in script scope.
reacted with thumbs up emoji reacted with thumbs down emoji reacted with laugh emoji reacted with hooray emoji reacted with confused emoji reacted with heart emoji reacted with rocket emoji reacted with eyes emoji
-
Built-in variables, for example:
$(buildir)
,$(arch)
,$(plat)
...With the growing number of modules in the scripting scope, there are now hundreds of modules and tens of thousands of module APIs, it is not possible to support built-in variables for all parameters of all script module APIs. This leads to a lot of confusion.
Some of the current interfaces that support built-in variables in scripts:
In the script domain, we can call the module interface directly to get the values of these variables, e.g.
config.plat()
,os.projectdir()
...Therefore, I think it would be clearer for users to just keep support for built-in variables in the description scope.
19 votes ·
Beta Was this translation helpful? Give feedback.
All reactions