Skip to content

Replay of #8637: Add the corresponding --action_env to .bazelrc from …#8812

Merged
martinwicke merged 1 commit intotensorflow:masterfrom
damienmg:configure
Mar 29, 2017
Merged

Replay of #8637: Add the corresponding --action_env to .bazelrc from …#8812
martinwicke merged 1 commit intotensorflow:masterfrom
damienmg:configure

Conversation

@damienmg
Copy link
Copy Markdown
Contributor

…./configure

The original PR got overriden by bc456e3

This will make the fetch phase reproducible so the refetch does
not happens unless people re-run configure (the environment variable
are stored in the .bazelrc file).

Fixes #8619.

…elrc from ./configure

The original PR got overriden by bc456e3

This will make the fetch phase reproducible so the refetch does
not happens unless people re-run configure (the environment variable
are stored in the .bazelrc file).

Fixes tensorflow#8619.
@damienmg damienmg requested a review from gunan March 29, 2017 17:24
@martinwicke
Copy link
Copy Markdown
Member

martinwicke commented Mar 29, 2017

What I don't see is how bc456e3 overwrite the original -- bc456e3's diff doesn't have any deleted lines in configure which echo >> bazel.rc

@martinwicke martinwicke merged commit 879eeb5 into tensorflow:master Mar 29, 2017
@damienmg
Copy link
Copy Markdown
Contributor Author

damienmg commented Mar 29, 2017 via email

@martinwicke
Copy link
Copy Markdown
Member

martinwicke commented Mar 29, 2017 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants