Skip to content

[gyb] add the moral equivalent of an #include directive #8265

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

Merged
merged 2 commits into from
Mar 22, 2017

Conversation

dabrahams
Copy link
Contributor

The few places that are explicitly using execute_template could benefit

The few places that are explicitly using execute_template could benefit
@dabrahams
Copy link
Contributor Author

@swift-ci Please test

utils/gyb.py Outdated
self.local_bindings = local_bindings
self.line_directive = line_directive
self.line_directive = '// ###sourceLocation' if line_directive is None else line_directive
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Over 80 chars - will fail python linting

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks; I'll clean that up before merging it.

@dabrahams dabrahams changed the title [gyb] use line directives absent explicit request [gyb] add the moral equivalent of an #include directive Mar 22, 2017
@dabrahams
Copy link
Contributor Author

@swift-ci Please smoke test and merge

1 similar comment
@dabrahams
Copy link
Contributor Author

@swift-ci Please smoke test and merge

@swift-ci swift-ci merged commit 063e90d into master Mar 22, 2017
@dabrahams dabrahams deleted the gyb-always-line-directive branch March 22, 2017 19:40
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.

3 participants