Skip to content

Bump cc dependency - #130720

Closed
arttet wants to merge 1 commit into
rust-lang:masterfrom
arttet:fix/profiler_builtins
Closed

Bump cc dependency#130720
arttet wants to merge 1 commit into
rust-lang:masterfrom
arttet:fix/profiler_builtins

Conversation

@arttet

@arttetarttet commented Sep 22, 2024

Copy link
Copy Markdown
Contributor

Closes#130719

try-job: x86_64-msvc

@rustbot

Copy link
Copy Markdown
Collaborator

r? @Mark-Simulacrum

rustbot has assigned @Mark-Simulacrum.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

@rustbotrustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Sep 22, 2024
@rust-log-analyzer

This comment has been minimized.

@arttet
arttetforce-pushed the fix/profiler_builtins branch from e28d8a0 to 6792b19CompareSeptember 22, 2024 20:41
@rustbot

Copy link
Copy Markdown
Collaborator

These commits modify the library/Cargo.lock file. Unintentional changes to library/Cargo.lock can be introduced when switching branches and rebasing PRs.

If this was unintentional then you should revert the changes before this PR is merged.
Otherwise, you can ignore this comment.

@rust-log-analyzer

This comment has been minimized.

@arttet
arttetforce-pushed the fix/profiler_builtins branch 2 times, most recently from a109243 to 1265643CompareSeptember 23, 2024 10:28
@rustbotrustbot added A-testsuite Area: The testsuite used to check the correctness of rustc T-bootstrap Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap) labels Sep 23, 2024
@rustbot

Copy link
Copy Markdown
Collaborator

The list of allowed third-party dependencies may have been modified! You must ensure that any new dependencies have compatible licenses before merging.

cc @davidtwco, @wesleywiser

@arttet

Copy link
Copy Markdown
ContributorAuthor

r? @onur-ozkan

@onur-ozkan

Copy link
Copy Markdown
Contributor

Library team should review this.

r? library

@rustbot

Copy link
Copy Markdown
Collaborator

Failed to set assignee to library: invalid assignee

Note: Only org members with at least the repository "read" role, users with write permissions, or people who have commented on the PR may be assigned.

@onur-ozkan

Copy link
Copy Markdown
Contributor

r? libs

@rustbotrustbot added the T-libs Relevant to the library team, which will review and decide on the PR/issue. label Sep 23, 2024
@rustbotrustbot assigned Amanieu and unassigned onur-ozkanSep 23, 2024
@Amanieu

Copy link
Copy Markdown
Member

@bors r+

@bors

bors commented Sep 23, 2024

Copy link
Copy Markdown
Collaborator

📌 Commit 1265643 has been approved by Amanieu

It is now in the queue for this repository.

@borsbors 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 Sep 23, 2024
bors added a commit to rust-lang-ci/rust that referenced this pull request Sep 24, 2024
@bors

bors commented Sep 24, 2024

Copy link
Copy Markdown
Collaborator

⌛ Testing commit 1265643 with merge d969b05...

@rust-log-analyzer

This comment has been minimized.

@bors

bors commented Sep 24, 2024

Copy link
Copy Markdown
Collaborator

💔 Test failed - checks-actions

@borsbors removed the S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. label Sep 24, 2024
@arttet

Copy link
Copy Markdown
ContributorAuthor

If this fails again you should try only bumping to 1.0.100, to get a better idea of whether this is due to a specific cc change or just cache-related.

I really thought it's related to cc. But I've built the compiler on Windows several times successfully. Moreover, I have merged one of the PR.

@rust-log-analyzer

This comment has been minimized.

@bors

bors commented Sep 26, 2024

Copy link
Copy Markdown
Collaborator

💔 Test failed - checks-actions

@borsbors 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 Sep 26, 2024
@arttetarttet mentioned this pull request Sep 26, 2024
bors added a commit to rust-lang-ci/rust that referenced this pull request Sep 26, 2024
Bump cc dependency
We cannot merge the [PR](rust-lang#130720)
Let's try merging an older version for a change.
rust-lang#130720 (comment)
try-job: x86_64-msvc
bors added a commit to rust-lang-ci/rust that referenced this pull request Sep 26, 2024
Bump cc dependency
We cannot merge the [PR](rust-lang#130720)
Let's try merging an older version for a change.
rust-lang#130720 (comment)
try-job: x86_64-msvc
@workingjubilee

Copy link
Copy Markdown
Member

@bors rollup=iffy

@workingjubilee

Copy link
Copy Markdown
Member

Oh, I somehow missed this was already

@bors rollup=never

bors added a commit to rust-lang-ci/rust that referenced this pull request Sep 26, 2024
[experiment] Update cc to 1.0.100
See rust-lang#130720, rust-lang#130898; try with the minimal version bump.
try-job: x86_64-msvc
bors added a commit to rust-lang-ci/rust that referenced this pull request Sep 26, 2024
Bump cc dependency
We cannot merge the [PR](rust-lang#130720)
Let's try merging an older version for a change.
rust-lang#130720 (comment)
try-job: x86_64-msvc
bors added a commit to rust-lang-ci/rust that referenced this pull request Sep 27, 2024
[experiment] Update cc to 1.0.100
See rust-lang#130720, rust-lang#130898; try with the minimal version bump.
try-job: x86_64-msvc
bors added a commit to rust-lang-ci/rust that referenced this pull request Sep 27, 2024
[experiment] Update cc to 1.0.100
See rust-lang#130720, rust-lang#130898; try with the minimal version bump.
try-job: x86_64-msvc
bors added a commit to rust-lang-ci/rust that referenced this pull request Sep 27, 2024
[experiment] Update cc to 1.0.100
See rust-lang#130720, rust-lang#130898; try with the minimal version bump.
try-job: x86_64-msvc
bors added a commit to rust-lang-ci/rust that referenced this pull request Sep 27, 2024
[experiment] Update `cc` CI tests
See rust-lang#130720, rust-lang#130898; try with the minimal version bump.
try-job: x86_64-msvc
bors added a commit to rust-lang-ci/rust that referenced this pull request Sep 27, 2024
[experiment] Update `cc` CI tests
See rust-lang#130720, rust-lang#130898; try with the minimal version bump.
try-job: x86_64-msvc
workingjubilee added a commit to workingjubilee/rustc that referenced this pull request Sep 27, 2024
…oratrieb
Partially update `library/Cargo.lock`
Run `cargo update` in library but exclude updates to `cc` and to `compiler_builtins`. Exclusions were done because `cc` seems to have some issues updating [1], and `compiler_builtins` needs to be updated on its own.
Partially supersedes rust-lang#129538.
[1]: rust-lang#130720
try-job: x86_64-msvc
rust-timer added a commit to rust-lang-ci/rust that referenced this pull request Sep 27, 2024
Rollup merge of rust-lang#130892 - tgross35:library-cargo-update, r=Noratrieb
Partially update `library/Cargo.lock`
Run `cargo update` in library but exclude updates to `cc` and to `compiler_builtins`. Exclusions were done because `cc` seems to have some issues updating [1], and `compiler_builtins` needs to be updated on its own.
Partially supersedes rust-lang#129538.
[1]: rust-lang#130720
try-job: x86_64-msvc
@arttet

Copy link
Copy Markdown
ContributorAuthor

@bors try

@bors

bors commented Sep 27, 2024

Copy link
Copy Markdown
Collaborator

⌛ Trying commit 522a53d with merge ddfac70...

@tgross35

Copy link
Copy Markdown
Member

FYI 1.1.22 was already tested, see the results at #130926 (comment)

@bors

bors commented Sep 27, 2024

Copy link
Copy Markdown
Collaborator

☀️ Try build successful - checks-actions
Build commit: ddfac70 (ddfac706ad8c2f0ad73a05566c4a86fd64e248e7)

@tgross35

Copy link
Copy Markdown
Member

#130926 was merged (🎉) so this should be all set

@arttet

Copy link
Copy Markdown
ContributorAuthor

Thank you!

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

Labels

A-testsuiteArea: The testsuite used to check the correctness of rustcS-waiting-on-reviewStatus: Awaiting review from the assignee but also interested parties.T-bootstrapRelevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap)T-libsRelevant to the library team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Unknown architecture for macOS target when we build profiler_builtins

10 participants

@arttet@rustbot@rust-log-analyzer@onur-ozkan@Amanieu@bors@ChrisDenton@tgross35@workingjubilee@Mark-Simulacrum