Skip to content

Don't overwrite local changes when updating submodules#104865

Merged
bors merged 3 commits into
rust-lang:masterfrom
dotdot0:bootstrap
Dec 1, 2022
Merged

Don't overwrite local changes when updating submodules#104865
bors merged 3 commits into
rust-lang:masterfrom
dotdot0:bootstrap

Conversation

@dotdot0

@dotdot0 dotdot0 commented Nov 25, 2022

Copy link
Copy Markdown
Contributor

Fixes #103485

@rustbot

rustbot commented Nov 25, 2022

Copy link
Copy Markdown
Collaborator

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @Mark-Simulacrum (or someone else) soon.

Please see the contribution instructions for more information.

@rustbot rustbot added T-bootstrap Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap) S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Nov 25, 2022
@rust-log-analyzer

This comment has been minimized.

@jyn514

jyn514 commented Nov 26, 2022

Copy link
Copy Markdown
Member

r? @jyn514

@pratushrai0309 can you please fix the tidy error?

@rustbot rustbot assigned jyn514 and unassigned Mark-Simulacrum Nov 26, 2022
Comment thread src/bootstrap/lib.rs Outdated
@jyn514 jyn514 changed the title bootstrap submodule sync overwrites local changes --changes Don't overwrite local changes when updating submodules Nov 27, 2022
@rust-log-analyzer

This comment has been minimized.

@dotdot0

dotdot0 commented Nov 27, 2022

Copy link
Copy Markdown
Contributor Author

@jyn514 I think it's done

@jyn514

jyn514 commented Nov 30, 2022

Copy link
Copy Markdown
Member

@bors r+ rollup

@bors

bors commented Nov 30, 2022

Copy link
Copy Markdown
Collaborator

📌 Commit d742407 has been approved by jyn514

It is now in the queue for this repository.

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Nov 30, 2022
matthiaskrgr added a commit to matthiaskrgr/rust that referenced this pull request Nov 30, 2022
Don't overwrite local changes when updating submodules

Fixes rust-lang#103485
bors added a commit to rust-lang-ci/rust that referenced this pull request Dec 1, 2022
…iaskrgr

Rollup of 9 pull requests

Successful merges:

 - rust-lang#103065 (rustdoc-json: Document and Test that args can be patterns.)
 - rust-lang#104865 (Don't overwrite local changes when updating submodules)
 - rust-lang#104895 (Avoid Invalid code suggested when encountering unsatisfied trait bounds in derive macro code)
 - rust-lang#105063 (Rustdoc Json Tests: Don't assume that core::fmt::Debug will always have one item.)
 - rust-lang#105064 (rustdoc: add comment to confusing CSS `main { min-width: 0 }`)
 - rust-lang#105074 (Add Nicholas Bishop to `.mailmap`)
 - rust-lang#105081 (Add a regression test for rust-lang#104322)
 - rust-lang#105086 (rustdoc: clean up sidebar link CSS)
 - rust-lang#105091 (add Tshepang Mbambo to .mailmap)

Failed merges:

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit 3d64420 into rust-lang:master Dec 1, 2022
@rustbot rustbot added this to the 1.67.0 milestone Dec 1, 2022
bors added a commit to rust-lang-ci/rust that referenced this pull request Dec 4, 2022
Don't exit with an error if there are no changes to submodules

Fixes rust-lang#105215, which regressed in rust-lang#104865.
RalfJung pushed a commit to RalfJung/miri that referenced this pull request Dec 5, 2022
Don't exit with an error if there are no changes to submodules

Fixes rust-lang/rust#105215, which regressed in rust-lang/rust#104865.
@dotdot0 dotdot0 deleted the bootstrap branch December 16, 2022 04:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-bootstrap Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

bootstrap submodule sync overwrites local changes

6 participants