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

Beta backport PR #72767 #72842

Closed
wants to merge 3 commits into from
Closed

Conversation

pnkfelix
Copy link
Member

@pnkfelix pnkfelix commented May 31, 2020

Beta backport of PR #72767

I am in the midst of testing the backported version locally, but the cherry-picks were all clean so I'm guessing/hoping this will be trivial

(and I want to discourage further use of it if possible.)
…ths.

Such splits arise from metadata refs into libstd.

This way, we can (in a follow on commit) continue to emit the virtual name into
things like the like the StableSourceFileId that ends up in incremetnal build
artifacts, while still using the devirtualized file path when we want to access
the file.

Note that this commit is intended to be a refactoring; the actual fix to the bug
in question is in a follow-on commit.
@pnkfelix
Copy link
Member Author

r? @Mark-Simulacrum

@Mark-Simulacrum
Copy link
Member

Hm okay, I was thinking to just do this on Monday as part of the stable rollup but if we can get it landed separately that seems even better, since it does seem like it could potentially not pass tests or whatever.

@bors r+ p=1

@bors
Copy link
Contributor

bors commented May 31, 2020

📌 Commit 0d19fb3 has been approved by Mark-Simulacrum

@bors bors added the S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. label May 31, 2020
@bors
Copy link
Contributor

bors commented Jun 1, 2020

⌛ Testing commit 0d19fb3 with merge 5b9dcf6fa07e6c22b54282ad0fd50009ba77fbdc...

@bors
Copy link
Contributor

bors commented Jun 1, 2020

💥 Test timed out

@bors bors added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Jun 1, 2020
@Mark-Simulacrum
Copy link
Member

Included in #72883

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants