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

Changed prerequisites.qbk to avoid confusion while building boost. #3214

Merged
merged 1 commit into from Mar 6, 2018

Conversation

Projects
None yet
2 participants
@Jakub-Golinowski
Copy link
Contributor

Jakub-Golinowski commented Mar 5, 2018

Proposed Changes

  • Moved the important box about Windows platform to the top
  • Added information about possible necessity of using --build-type=complete in the b2 command on Windows platform in the above mentioned important box
  • deleted the --build-type=complete from the cross-platform code box

Any background context you want to provide?

linux users do not use option --build-type=complete so it should not be listed in the cross-platform box. --build-type=complete was added initially for Windows users therefore I moved it to Windows important box.

@hkaiser

hkaiser approved these changes Mar 5, 2018

Copy link
Member

hkaiser left a comment

Very nice, thanks!

@hkaiser hkaiser merged commit 3353a70 into STEllAR-GROUP:master Mar 6, 2018

2 checks passed

ci/circleci Your tests passed on CircleCI!
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.
You signed in with another tab or window. Reload to refresh your session. You signed out in another tab or window. Reload to refresh your session.