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

Minor Documentation Improvements #50

Open
SEI-John opened this issue Aug 1, 2023 · 0 comments
Open

Minor Documentation Improvements #50

SEI-John opened this issue Aug 1, 2023 · 0 comments

Comments

@SEI-John
Copy link

SEI-John commented Aug 1, 2023

The documentation has a couple of minor improvements to be made in the section about building a simple application.

  1. my_app.m.cpp:20 bsl::cout requires #include <bsl_iostream.h>
  2. Under "Setup application build dependencies", option 2, "Create a build workspace with the BDE libraries...", it should mention that the top level CMakeLists.txt will need to be modified to include add_subdirectory(<path_to_bde>)
  3. Under "Configure, build and test your application", this command: bbs_build configure build --target my_app needs to be run from where the top level CMakeLists.txt is. It's a little unclear since the previous command cds into the bde directory

Thanks for putting this toolset and documentation together, it's valuable and helpful

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

No branches or pull requests

1 participant