Skip to content
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

Incremental swift compilation #2781

Closed
keith opened this issue Apr 5, 2017 · 1 comment
Closed

Incremental swift compilation #2781

keith opened this issue Apr 5, 2017 · 1 comment
Labels
category: rules > ObjC / iOS / J2ObjC P3 We're not considering working on this, but happy to review a PR. (No assignee) type: feature request

Comments

@keith
Copy link
Member

keith commented Apr 5, 2017

Through some cursory tests to integrate bazel into our all swift project, we noticed that bazel doesn't currently support incremental swift compilation.

We've taken a stab at adding this by adding the -incremental and -emit-dependencies compiler flags and adding the dependency information to the output_file_map.json.

If I then run the commands that are being ran by bazel manually, I get incremental compilation, but running from bazel, it does not work incrementally. It seems as though bazel is removing the previously generated files after each change.

Is there a reason the initial Swift support didn't support incremental compilation? Also are there any plans to add this? I would be happy to submit a PR for the changes we've made if you can point me in the right direction on the problems we've ran in to.

Thanks!

@keith
Copy link
Member Author

keith commented Apr 17, 2017

Closing for bazelbuild/rules_apple#21

@keith keith closed this as completed Apr 17, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: rules > ObjC / iOS / J2ObjC P3 We're not considering working on this, but happy to review a PR. (No assignee) type: feature request
Projects
None yet
Development

No branches or pull requests

2 participants