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

Stop copying generated files into $BUILD_DIR #771

Merged
merged 2 commits into from
Jul 19, 2022

Conversation

brentleyjones
Copy link
Contributor

With recent changes we no longer need to copy generated input files, and can instead reference them directly, by changing the gen_dir symlink to point to $BAZEL_OUT.

This change replaces the "one time setup" phase of the installer with an always run script that creates the parent directories of generated files in $BAZEL_OUT. This has the same result as before: Xcode no longer needs to be reopened after a build resolve red generated files.

With recent changes we no longer need to copy generated input files, and can instead reference them directly, by changing the `gen_dir` symlink to point to `$BAZEL_OUT`.

This change replaces the "one time setup" phase of the installer with an always run script that creates the parent directories of generated files in `$BAZEL_OUT`. This has the same result as before: Xcode no longer needs to be reopened after a build resolve red generated files.
Copy link
Contributor

@jpsim jpsim left a comment

Choose a reason for hiding this comment

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

Seems to work well in my local testing

@brentleyjones brentleyjones enabled auto-merge (squash) July 19, 2022 21:55
@brentleyjones brentleyjones merged commit efc211b into main Jul 19, 2022
@brentleyjones brentleyjones deleted the bj/stop-copying-generated-files-into-build_dir branch July 19, 2022 21:57
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