-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
top file compilation fixes #26657
top file compilation fixes #26657
Conversation
@@ -2305,7 +2305,10 @@ def __gen_opts(self, opts): | |||
'state_auto_order', | |||
opts['state_auto_order']) | |||
opts['file_roots'] = mopts['file_roots'] | |||
opts['env_order'] = mopts['env_order'] | |||
opts['top_file_merging_strategy'] = mopts.get('top_file_merging_strategy', |
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.
@cachedout, is this the correct way to prefer master opts, with a fallback to minion opts, for example to handle the masterless case?
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 this will be just fine. Thanks.
This all looks great. Excellent work. @jfindlay |
@cachedout, please review.
Also see:
https://gist.github.com/jfindlay/b2e6807354b304d282d4
https://gist.github.com/jfindlay/99a84526ef1d4a954ad0
References:
#24878
#26307