Skip to content

Cleanup debuginfo generation a bit #68959

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

Closed
wants to merge 4 commits into from

Conversation

bjorn3
Copy link
Member

@bjorn3 bjorn3 commented Feb 8, 2020

This removes an no longer necessary hack. This also uses CrateDebugContext in favor of CodegenCx in debuginfo generation.

@rust-highfive
Copy link
Contributor

r? @estebank

(rust_highfive has picked a reviewer for you, use r? to override)

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Feb 8, 2020
@rust-highfive
Copy link
Contributor

The job x86_64-gnu-llvm-7 of your PR failed (pretty log, raw log). Through arcane magic we have determined that the following fragments from the build log may contain information about the problem.

Click to expand the log.
2020-02-08T17:14:48.1320188Z ========================== Starting Command Output ===========================
2020-02-08T17:14:48.1341636Z [command]/bin/bash --noprofile --norc /home/vsts/work/_temp/0a522f6d-2d91-406c-bc61-1ac7ef346503.sh
2020-02-08T17:14:48.1551794Z 
2020-02-08T17:14:48.1613500Z ##[section]Finishing: Disable git automatic line ending conversion
2020-02-08T17:14:48.1619786Z ##[section]Starting: Checkout rust-lang/rust@refs/pull/68959/merge to s
2020-02-08T17:14:48.1621451Z Task         : Get sources
2020-02-08T17:14:48.1621480Z Description  : Get sources from a repository. Supports Git, TfsVC, and SVN repositories.
2020-02-08T17:14:48.1621510Z Version      : 1.0.0
2020-02-08T17:14:48.1621554Z Author       : Microsoft
---
2020-02-08T17:14:49.0057694Z ##[command]git remote add origin https://github.com/rust-lang/rust
2020-02-08T17:14:49.0134702Z ##[command]git config gc.auto 0
2020-02-08T17:14:49.0211378Z ##[command]git config --get-all http.https://github.com/rust-lang/rust.extraheader
2020-02-08T17:14:49.0427355Z ##[command]git config --get-all http.proxy
2020-02-08T17:14:49.0609620Z ##[command]git -c http.extraheader="AUTHORIZATION: basic ***" fetch --force --tags --prune --progress --no-recurse-submodules --depth=2 origin +refs/heads/*:refs/remotes/origin/* +refs/pull/68959/merge:refs/remotes/pull/68959/merge

I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact @TimNN. (Feature Requests)

@mark-i-m
Copy link
Member

mark-i-m commented Feb 8, 2020

cc @eddyb I think

@bors
Copy link
Collaborator

bors commented Feb 9, 2020

☔ The latest upstream changes (presumably #68975) made this pull request unmergeable. Please resolve the merge conflicts.

@JohnCSimon JohnCSimon added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Feb 16, 2020
@JohnCSimon
Copy link
Member

ping from triage: @bjorn3 can you address the failed checks?

@bjorn3
Copy link
Member Author

bjorn3 commented Feb 22, 2020

This changes several files that will also be changed by #68965.

@rustbot modify labels: +S-blocked

@rustbot rustbot added the S-blocked Status: Blocked on something else such as an RFC or other implementation work. label Feb 22, 2020
@Dylan-DPC-zz Dylan-DPC-zz removed the S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. label Mar 24, 2020
@Dylan-DPC-zz
Copy link

This is blocked on another blocked pr, and neither of them seem to be moving in the near future, so closing this one. You can create a new PR later if you wish once there's some progress. Thanks

@Dylan-DPC-zz Dylan-DPC-zz added S-blocked-closed and removed S-blocked Status: Blocked on something else such as an RFC or other implementation work. labels Aug 8, 2020
@jyn514 jyn514 added S-blocked Status: Blocked on something else such as an RFC or other implementation work. and removed S-blocked-closed labels Mar 10, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-blocked Status: Blocked on something else such as an RFC or other implementation work.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants