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

Update to cargo 0.19.0-nightly (994e552 2017-04-25) #16601

Merged
merged 2 commits into from Apr 28, 2017
Merged

Update to cargo 0.19.0-nightly (994e552 2017-04-25) #16601

merged 2 commits into from Apr 28, 2017

Conversation

@SimonSapin
Copy link
Member

SimonSapin commented Apr 25, 2017

This should be unblocked now: #15852 (comment)


This change is Reviewable

@highfive
Copy link

highfive commented Apr 25, 2017

Heads up! This PR modifies the following files:

  • @wafflespeanut: python/servo/command_base.py, python/servo/devenv_commands.py
@SimonSapin SimonSapin force-pushed the cargoup branch from d4b8af2 to f267ecb Apr 25, 2017
@SimonSapin
Copy link
Member Author

SimonSapin commented Apr 25, 2017

@bors-servo r=nox

@bors-servo
Copy link
Contributor

bors-servo commented Apr 25, 2017

📌 Commit f267ecb has been approved by nox

@highfive highfive assigned nox and unassigned KiChjang Apr 25, 2017
@bors-servo
Copy link
Contributor

bors-servo commented Apr 25, 2017

Testing commit f267ecb with merge aa77c73...

bors-servo added a commit that referenced this pull request Apr 25, 2017
Update to cargo 0.19.0-nightly (994e552 2017-04-25)

This should be unblocked now: #15852 (comment)

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/16601)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Apr 25, 2017

💔 Test failed - linux-dev

@SimonSapin
Copy link
Member Author

SimonSapin commented Apr 25, 2017

error: compile-fail/privatize.rs:8: unexpected "error": '8:1: 8:33: multiple matching crates for `deny_public_fields` [E0464]'

error: compile-fail/privatize.rs:8: unexpected "error": '8:1: 8:33: can't find crate for `deny_public_fields` [E0463]'

I can reproduce locally, and mach clean works around the issue. I wonder if we should run that on the builders and move on, or try to find a proper fix…

@SimonSapin
Copy link
Member Author

SimonSapin commented Apr 25, 2017

This is because compile tests use rustc with -L target/debug and -L target/debug/deps and rely of finding dependencies in these directories by name, whereas Cargo uses --extern name=path/to/something.rlib with exact file names. When upgrading Cargo, some hash changes, new .rlib files are created, and old ones are not removed by default.

Finding the proper paths to use with --extern in compile test seems like a lot of work and not worth it. So it’s got with the exceptional clean. I’d rather not SSH into each builder to do that and hope that homu and buildbot’s scheduling will work in my favor, and at the same time I’d rather not go back to cleaning on every single build.

So it sounds like we need something like Gecko’s CLOBBER file. ./mach clean would be run automatically as part of build jobs if the CLOBBER file has changed since the last build.

@bors-servo
Copy link
Contributor

bors-servo commented Apr 27, 2017

Testing commit f267ecb with merge 7368589fed4bbff3596af1d6c5488fed5a86c0e4...

@bors-servo
Copy link
Contributor

bors-servo commented Apr 27, 2017

💔 Test failed - arm32

@jdm
Copy link
Member

jdm commented Apr 28, 2017

@bors-servo: retry

@bors-servo
Copy link
Contributor

bors-servo commented Apr 28, 2017

Testing commit f267ecb with merge 4a6b939...

bors-servo added a commit that referenced this pull request Apr 28, 2017
Update to cargo 0.19.0-nightly (994e552 2017-04-25)

This should be unblocked now: #15852 (comment)

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/16601)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Apr 28, 2017

💔 Test failed - mac-dev-unit

@jdm
Copy link
Member

jdm commented Apr 28, 2017

@bors-servo
Copy link
Contributor

bors-servo commented Apr 28, 2017

Testing commit f267ecb with merge f128781...

bors-servo added a commit that referenced this pull request Apr 28, 2017
Update to cargo 0.19.0-nightly (994e552 2017-04-25)

This should be unblocked now: #15852 (comment)

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/16601)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Apr 28, 2017

☀️ Test successful - android, arm32, arm64, linux-dev, linux-rel-css, linux-rel-wpt, mac-dev-unit, mac-rel-css, mac-rel-wpt1, mac-rel-wpt2, windows-msvc-dev
Approved by: nox
Pushing f128781 to master...

@bors-servo bors-servo merged commit f267ecb into master Apr 28, 2017
3 of 4 checks passed
3 of 4 checks passed
dependency-ci Failed dependency checks
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
homu Test successful
Details
@SimonSapin SimonSapin deleted the cargoup branch Apr 28, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

6 participants
You can’t perform that action at this time.