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

Add ci for DragonFly #45623

Merged
merged 1 commit into from Nov 13, 2017
Merged

Add ci for DragonFly #45623

merged 1 commit into from Nov 13, 2017

Conversation

mneumann
Copy link
Contributor

No description provided.

@rust-highfive
Copy link
Collaborator

r? @nikomatsakis

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

@kennytm kennytm added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Oct 30, 2017
@nikomatsakis
Copy link
Contributor

r? @alexcrichton

@alexcrichton
Copy link
Member

Thanks! Since this won't be currently activated, mind moving this to the disabled folder as well? (src/ci/docker/disabled)

@kennytm kennytm 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 Oct 31, 2017
@mneumann
Copy link
Contributor Author

mneumann commented Nov 2, 2017

@alexcrichton: Would be nice to enable this once I fix formatting :). Or, if ci for DragonFly is too time-consuming (whatever other reason), is there a possibility to upload "official" distributions for DragonFly that we build ourselves on our machines? This would make rustup find something when being installed.

Currently, when I run this builder and cross-compile Rust for DragonFly, I trigger an internal bug in rustc itself (sorry, no bug report yet).

@alexcrichton
Copy link
Member

Ah unfortunately we don't quite have the resources to commit to this right now, producing rustc for a platform requires quite a bit of compute power :(

@mneumann
Copy link
Contributor Author

mneumann commented Nov 3, 2017

@alexcrichton: ok, no problem. but, is there some way so that we can upload snapshots of rust to an "official" location? atm, we are hosting them ourselves. this especially does not play very well together with rustup. probably, it's easier for us all, to not integrate that well with DragonFly. We have rust in the ports, but being able to install rust nightly via rustup is something that is missing.

@alexcrichton
Copy link
Member

Unfortunately not easily right now :(

@Mark-Simulacrum
Copy link
Member

Can we get this merged into the disabled folder? That would set the groundwork for eventually enabling, which we can't do currently unfortunately.

@mneumann
Copy link
Contributor Author

@alexcrichton @Mark-Simulacrum ok, another try! I moved it into disabled and moved the patches (which broke "tidy") out into a separate file.

@alexcrichton
Copy link
Member

@bors: r+

@bors
Copy link
Contributor

bors commented Nov 12, 2017

📌 Commit 949abfe has been approved by alexcrichton

@kennytm kennytm 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-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Nov 12, 2017
@bors
Copy link
Contributor

bors commented Nov 12, 2017

⌛ Testing commit 949abfe with merge 7f43981...

bors added a commit that referenced this pull request Nov 12, 2017
@bors
Copy link
Contributor

bors commented Nov 13, 2017

☀️ Test successful - status-appveyor, status-travis
Approved by: alexcrichton
Pushing 7f43981 to master...

@bors bors merged commit 949abfe into rust-lang:master Nov 13, 2017
@mneumann mneumann deleted the dragonfly-ci branch November 17, 2017 12:21
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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants