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

Installation notes tweak #44

Open
JohnPJenkins opened this issue May 28, 2015 · 1 comment
Open

Installation notes tweak #44

JohnPJenkins opened this issue May 28, 2015 · 1 comment

Comments

@JohnPJenkins
Copy link

It would be a good idea to include the use of CMAKE_INSTALL_PREFIX and make install into the installation instructions, and the compiler/linker flags needed to compile against ROSS: -I/path/to/ROSS/install/include -L/path/to/ROSS/install/lib -lROSS. Additionally, the "Constructing the Model" and top level README pages should be updated accordingly. Not everyone will want to symlink their code into ROSS to build their models :).

@carns
Copy link
Contributor

carns commented May 28, 2015

Rather than document the flags explicitly, it would be better to document the use of ross-config for this stuff. That way as long as ross-config is kept up to date then the documentation will be safe. ross-config can also tell you the CC that was used to compile ROSS (via ross-config --cc) as well if you want to make sure that everything matches up.

If we have a .pc file for package config thing we could document that too or in place of ross-config. It would provide similar functionality.

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

2 participants