Skip to content

Register new snapshots (270a677) #23153

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

Merged
merged 1 commit into from
Mar 9, 2015
Merged

Register new snapshots (270a677) #23153

merged 1 commit into from
Mar 9, 2015

Conversation

Manishearth
Copy link
Member

Needed so that #21824 can land

@rust-highfive
Copy link
Contributor

r? @nikomatsakis

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

@Manishearth
Copy link
Member Author

I was unable to get a bitrig snapshot, it seems like the bitrig build is broken.

checking whether GCC or Clang is our host compiler... clang
checking build system type... Invalid configuration `x86_64-unknown-bitrig': system `bitrig' not recognized
configure: error: /bin/sh /home/builder/rust-buildbot/slave/snap3-bitrig-64/build/src/llvm/autoconf/config.sub x86_64-unknown-bitrig failed

What should I do in this case? Wait for it to be fixed, or land this directly?

cc @dhuseby @semarie

@Manishearth
Copy link
Member Author

(Also, I haven't yet tested this locally -- the build is ongoing -- PRd so that I can get Dave's input)

@Manishearth
Copy link
Member Author

This passes make check-stage2 (and stage1)

@semarie
Copy link
Contributor

semarie commented Mar 8, 2015

@dhuseby I think you hit a parsing problem in ksh (/bin/sh). see http://marc.info/?l=openbsd-bugs&m=142571265625168&w=2

the configure isn't well interperted by the shell.

it is #22474 that spot the problem.

@alexcrichton
Copy link
Member

@bors: r+ f81f8d8

Manishearth added a commit to Manishearth/rust that referenced this pull request Mar 8, 2015
bors added a commit that referenced this pull request Mar 9, 2015
@bors
Copy link
Collaborator

bors commented Mar 9, 2015

⌛ Testing commit f81f8d8 with merge 91bdf23...

@bors bors merged commit f81f8d8 into rust-lang:master Mar 9, 2015
@dhuseby
Copy link

dhuseby commented Mar 9, 2015

I determined that the configure was barfing part way, but didn't have the time to figure out the fix. thanks for the info, it's late sunday here in PST so i'll take a crack at it tomorrow. i have a build slave set up, i just need to get the configure to work and the snapshots should start rolling in.

@dhuseby
Copy link

dhuseby commented Mar 9, 2015

@Manishearth

"configure: error: /bin/sh /home/builder/rust-buildbot/slave/snap3-bitrig-64/build/src/llvm/autoconf/config.sub x86_64-unknown-bitrig failed"

that looks like you don't have all of the patches.

@dhuseby
Copy link

dhuseby commented Mar 9, 2015

I can manually build a snapshot for the new set and upload it while working on fixing the configure issue.

@semarie
Copy link
Contributor

semarie commented Mar 9, 2015

@dhuseby the problem is an invalid parse of configure by /bin/sh (as previously said). A quick correction is to change the comment about pandoc to remove the quote, or move the comment out of $(...).

@dhuseby
Copy link

dhuseby commented Mar 9, 2015

k, thanks

@Manishearth Manishearth deleted the snap branch March 9, 2015 18:41
@dhuseby
Copy link

dhuseby commented Mar 10, 2015

@alexcrichton I just manually built the latest bitrig snapshot while we sort out the buildbot issues: https://github.com/dhuseby/rust-cross-bitrig/blob/master/snapshots/rust-stage0-2015-03-07-270a677-bitrig-x86_64-4b2f11a96b1b5b3782d74bda707aca33bc179880.tar.bz2

will you upload that so i can patch up the src/snapshots.txt file?

@alexcrichton
Copy link
Member

I've uploaded the snapshot, thanks for making one!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants