Add test to build crates used by r-a on stable by lqd · Pull Request #129529 · rust-lang/rust (original) (raw)

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 andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Conversation22 Commits7 Checks6 Files changed

Conversation

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.Learn more about bidirectional Unicode characters

[ Show hidden characters]({{ revealButtonHref }})

lqd

Member

@lqd lqd commented

Aug 24, 2024

• Loading

r? @Kobzol

I've opened other PRs for this one to work and they've landed already. I cherry-picked your commit, and added the last remaining pieces we needed I think.

@rustbot

Could not assign reviewer from: Kobzol.
User(s) Kobzol are either the PR author, already assigned, or on vacation, and there are no other candidates.
Use r? to specify someone else to assign.

@rustbot

This comment was marked as outdated.

@rustbot rustbot added A-run-make

Area: port run-make Makefiles to rmake.rs

S-waiting-on-review

Status: Awaiting review from the assignee but also interested parties.

T-compiler

Relevant to the compiler team, which will review and decide on the PR/issue.

labels

Aug 24, 2024

@lqd

I've also removed the cargo nightly feature / RUSTC_BOOTSTRAP

lqd

@lqd lqd mentioned this pull request

Aug 25, 2024

@Kobzol @lqd

@lqd

@lqd

otherwise the test would build in the source root's target folder

@lqd

@lqd

it'll be easier to see and update the list: the other cmd args can just be ignored

@lqd

lqd

@lqd

All the prerequisite PRs have landed.

@lqd lqd marked this pull request as ready for review

August 27, 2024 17:52

@rustbot

This PR modifies tests/run-make/. If this PR is trying to port a Makefile
run-make test to use rmake.rs, please update the
run-make port tracking issue
so we can track our progress. You can either modify the tracking issue
directly, or you can comment on the tracking issue and link this PR.

cc @jieyouxu

The run-make-support library was changed

cc @jieyouxu

Kobzol

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Left a few nits, otherwise LGTM. Thank you for pushing this through!

@lqd

@Kobzol

Thanks! You can r=me once CI is green.

@Kobzol

@bors

📌 Commit a178559 has been approved by Kobzol

It is now in the queue for this repository.

bors added a commit to rust-lang-ci/rust that referenced this pull request

Sep 4, 2024

@bors

Rollup of 8 pull requests

Successful merges:

r? @ghost @rustbot modify labels: rollup

bors added a commit to rust-lang-ci/rust that referenced this pull request

Sep 5, 2024

@bors

Rollup of 8 pull requests

Successful merges:

r? @ghost @rustbot modify labels: rollup

matthiaskrgr added a commit to matthiaskrgr/rust that referenced this pull request

Sep 5, 2024

@matthiaskrgr

Add test to build crates used by r-a on stable

r? @Kobzol

I've opened other PRs for this one to work and they've landed already. I cherry-picked your commit, and added the last remaining pieces we needed I think.

bors added a commit to rust-lang-ci/rust that referenced this pull request

Sep 5, 2024

@bors

…iaskrgr

Rollup of 6 pull requests

Successful merges:

Failed merges:

r? @ghost @rustbot modify labels: rollup

GuillaumeGomez added a commit to GuillaumeGomez/rust that referenced this pull request

Sep 6, 2024

@GuillaumeGomez

Add test to build crates used by r-a on stable

r? @Kobzol

I've opened other PRs for this one to work and they've landed already. I cherry-picked your commit, and added the last remaining pieces we needed I think.

bors added a commit to rust-lang-ci/rust that referenced this pull request

Sep 6, 2024

@bors

…llaumeGomez

Rollup of 7 pull requests

Successful merges:

r? @ghost @rustbot modify labels: rollup

matthiaskrgr added a commit to matthiaskrgr/rust that referenced this pull request

Sep 8, 2024

@matthiaskrgr

Add test to build crates used by r-a on stable

r? @Kobzol

I've opened other PRs for this one to work and they've landed already. I cherry-picked your commit, and added the last remaining pieces we needed I think.

bors added a commit to rust-lang-ci/rust that referenced this pull request

Sep 8, 2024

@bors

…iaskrgr

Rollup of 7 pull requests

Successful merges:

r? @ghost @rustbot modify labels: rollup

matthiaskrgr added a commit to matthiaskrgr/rust that referenced this pull request

Sep 8, 2024

@matthiaskrgr

Add test to build crates used by r-a on stable

r? @Kobzol

I've opened other PRs for this one to work and they've landed already. I cherry-picked your commit, and added the last remaining pieces we needed I think.

bors added a commit to rust-lang-ci/rust that referenced this pull request

Sep 8, 2024

@bors

…iaskrgr

Rollup of 7 pull requests

Successful merges:

r? @ghost @rustbot modify labels: rollup

workingjubilee added a commit to workingjubilee/rustc that referenced this pull request

Sep 9, 2024

@workingjubilee

Add test to build crates used by r-a on stable

r? @Kobzol

I've opened other PRs for this one to work and they've landed already. I cherry-picked your commit, and added the last remaining pieces we needed I think.

workingjubilee added a commit to workingjubilee/rustc that referenced this pull request

Sep 9, 2024

@workingjubilee

Add test to build crates used by r-a on stable

r? @Kobzol

I've opened other PRs for this one to work and they've landed already. I cherry-picked your commit, and added the last remaining pieces we needed I think.

bors added a commit to rust-lang-ci/rust that referenced this pull request

Sep 9, 2024

@bors

workingjubilee added a commit to workingjubilee/rustc that referenced this pull request

Sep 10, 2024

@workingjubilee

Add test to build crates used by r-a on stable

r? @Kobzol

I've opened other PRs for this one to work and they've landed already. I cherry-picked your commit, and added the last remaining pieces we needed I think.

bors added a commit to rust-lang-ci/rust that referenced this pull request

Sep 10, 2024

@bors

bors added a commit to rust-lang-ci/rust that referenced this pull request

Sep 10, 2024

@bors

bors added a commit to rust-lang-ci/rust that referenced this pull request

Sep 10, 2024

@bors

rust-timer added a commit to rust-lang-ci/rust that referenced this pull request

Sep 10, 2024

@rust-timer

Rollup merge of rust-lang#129529 - lqd:stable-new-solver, r=Kobzol

Add test to build crates used by r-a on stable

r? @Kobzol

I've opened other PRs for this one to work and they've landed already. I cherry-picked your commit, and added the last remaining pieces we needed I think.

@lqd lqd deleted the stable-new-solver branch

September 10, 2024 10:42

@Veykril

A very late thanks for this test btw :)

Labels

A-run-make

Area: port run-make Makefiles to rmake.rs

S-waiting-on-bors

Status: Waiting on bors to run and complete tests. Bors will change the label on completion.

T-compiler

Relevant to the compiler team, which will review and decide on the PR/issue.