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

Revert "Define wrapper env vars as full paths" #45442

Merged
merged 3 commits into from Aug 21, 2018

Conversation

matthewbauer
Copy link
Member

This reverts commit 89efc27, reversing
changes made to d0f1102. This is from PR #44767.

This PR had many unintended side effects. It seems prudent to just revert, hopefully making way to do this some time later on when all of the issues are resolved.

@globin globin merged commit f7bc33a into staging Aug 21, 2018
@globin globin deleted the revert-wrapper-env-var-path branch August 21, 2018 23:14
@7c6f434c
Copy link
Member

How much breakage should this be shown to fix on master to be merged to master ahead of the other changes in staging? Or is staging merge including this imminent?

@vcunat
Copy link
Member

vcunat commented Aug 22, 2018

Rebuild-wise, excluding anything else will probably not help. Stability-wise, I don't know (I haven't looked at most of the other changes).

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

Successfully merging this pull request may close these issues.

None yet

6 participants