Open Bug 1257421 Opened 4 years ago Updated 2 years ago
Add a combo function that does both set
_config and set _define in moz .configure
From bug 1257104 comment 15, about set_config and set_define: > Feels like we should grow a combo function that does both of these since we use it all over the place. That's actually tricky. set_define is a template defined in util.configure, and set_config is a function that is only available to @depends functions, and that is bound to a different instance of DependsOutput for each such function, so it can't be called from a template without being passed down to that template. A random thought of something ugly that would work, though, is something like: set_define('FOO', 'bar', also=set_config) which, despite the ugliness, actually has the advantage of still responding to grep set_define and grep set_config... Thoughts?
I think I'd be happier if we could work the mechanics to have `set_config_and_define('FOO', '1')`, but I can live with your proposal.
You need to log in before you can comment on or make changes to this bug.