Skip to content

fixing configure so that it works on bitrig/openbsd - #23236

Merged
bors merged 1 commit into
rust-lang:masterfrom
dhuseby:master
Mar 10, 2015
Merged

fixing configure so that it works on bitrig/openbsd#23236
bors merged 1 commit into
rust-lang:masterfrom
dhuseby:master

Conversation

@dhuseby

@dhuseby dhuseby commented Mar 9, 2015

Copy link
Copy Markdown

this fixes the issue the bitrig autobuilder is running into.

@rust-highfive

Copy link
Copy Markdown
Contributor

r? @huonw

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

@alexcrichton

Copy link
Copy Markdown
Member

@bors: r+ d0ae7ca rollup

Manishearth added a commit to Manishearth/rust that referenced this pull request Mar 10, 2015
 this fixes the issue the bitrig autobuilder is running into.
@bors
bors merged commit d0ae7ca into rust-lang:master Mar 10, 2015
@Manishearth

Copy link
Copy Markdown
Member

@semarie

semarie commented Mar 10, 2015

Copy link
Copy Markdown
Contributor

hum...make (BSD version from base) instead of gmake (GNU version from ports) ?

@dhuseby

dhuseby commented Mar 10, 2015

Copy link
Copy Markdown
Author

yup. it needs to run gmake. when I build on bitrig, it properly detects the presence of gmake and uses that instead of make. so maybe this is a problem with the build scripts?

@semarie

semarie commented Mar 10, 2015

Copy link
Copy Markdown
Contributor

@dhuseby a patch is coming in rust-buildbot

@dhuseby

dhuseby commented Mar 10, 2015

Copy link
Copy Markdown
Author

@semarie thanks. which PR?

@semarie

semarie commented Mar 10, 2015

Copy link
Copy Markdown
Contributor

@dhuseby

dhuseby commented Mar 10, 2015

Copy link
Copy Markdown
Author

@semarie why doesn't this affect openbsd as well?

@dhuseby

dhuseby commented Mar 10, 2015

Copy link
Copy Markdown
Author

oh, nvrmnd. it does, but you don't have a build bot.

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