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

Fix make-install and make-dist when --disable-docs is set #19896

Merged
merged 1 commit into from
Dec 18, 2014

Conversation

ktossell
Copy link
Contributor

If you configure with --disable-docs, the doc directory does not get generated, so the
cp -r doc dist/ step fails when you make dist{,-tar-bins,-doc} or make install.

@flaper87
Copy link
Contributor

🍰 thanks!

bors referenced this pull request Dec 16, 2014
Using a type alias for iterator implementations is fragile since this
exposes the implementation to users of the iterator, and any changes
could break existing code.

This commit changes the iterators of `VecMap` to use
proper new types, rather than type aliases.  However, since it is
fair-game to treat a type-alias as the aliased type, this is a:

[breaking-change].
bors referenced this pull request Dec 16, 2014
This creates an enormous amount of spew.
@alexcrichton
Copy link
Member

Needs a rebase

If you configure with `--disable-docs`, the `doc` directory does not get generated, so
`cp -r doc dist/` fails when you `make dist{,-tar-bins,-doc}` or `make install`
@ktossell ktossell force-pushed the allow-nodoc-install branch from 25b597a to cbf80f3 Compare December 18, 2014 00:25
@ktossell
Copy link
Contributor Author

Rebased: cbf80f3

bors added a commit that referenced this pull request Dec 18, 2014
If you configure with `--disable-docs`, the `doc` directory does not get generated, so the
`cp -r doc dist/` step fails when you `make dist{,-tar-bins,-doc}` or `make install`.
@bors bors merged commit cbf80f3 into rust-lang:master Dec 18, 2014
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.

4 participants