Skip to content

rustbuild: Implement DESTDIR support #38654

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
Jan 12, 2017

Conversation

alexcrichton
Copy link
Member

This commit primarily starts supporting the DESTDIR environment variable like
the old build system. Along the way this brings config.toml up to date with
support in config.mk with install options supported.

Closes #38441

This commit primarily starts supporting the `DESTDIR` environment variable like
the old build system. Along the way this brings `config.toml` up to date with
support in `config.mk` with install options supported.

Closes rust-lang#38441
@rust-highfive
Copy link
Contributor

r? @brson

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

@brson
Copy link
Contributor

brson commented Jan 12, 2017

@bors r+

@bors
Copy link
Collaborator

bors commented Jan 12, 2017

📌 Commit 16f8372 has been approved by brson

@bors
Copy link
Collaborator

bors commented Jan 12, 2017

⌛ Testing commit 16f8372 with merge 139d741...

bors added a commit that referenced this pull request Jan 12, 2017
rustbuild: Implement DESTDIR support

This commit primarily starts supporting the `DESTDIR` environment variable like
the old build system. Along the way this brings `config.toml` up to date with
support in `config.mk` with install options supported.

Closes #38441
@bors
Copy link
Collaborator

bors commented Jan 12, 2017

☀️ Test successful - status-appveyor, status-travis
Approved by: brson
Pushing 139d741 to master...

@bors bors merged commit 16f8372 into rust-lang:master Jan 12, 2017
@alexcrichton alexcrichton deleted the rustbuild-destdir branch January 13, 2017 23:27
@Keruspe
Copy link
Contributor

Keruspe commented Jan 23, 2017

Any chance to get this backported to 1.15?

@alexcrichton alexcrichton added the beta-nominated Nominated for backporting to the compiler in the beta channel. label Jan 23, 2017
@alexcrichton alexcrichton removed the beta-nominated Nominated for backporting to the compiler in the beta channel. label Jan 30, 2017
@alexcrichton
Copy link
Member Author

Unfortunately we missed the deadline to do backports, so this'll make its way into 1.16

@Keruspe
Copy link
Contributor

Keruspe commented Feb 6, 2017

@alexcrichton I see it's not in the beta yet and the beta-nominated tag is gone.
There's no rush as we're in the beginning of the cycle, this is just a bump in case this would've been forgotten

@alexcrichton
Copy link
Member Author

@Keruspe this PR is now on beta, those tags were for the previous 1.15.0 beta, but we didn't get it out in time for that

@Keruspe
Copy link
Contributor

Keruspe commented Feb 6, 2017

Indeed, either I failed my pull or did so before it hit beta, sorry for the noise

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.

5 participants