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

Basic helloworld exmaple #465

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Basic helloworld exmaple #465

wants to merge 3 commits into from

Conversation

magres
Copy link

@magres magres commented Aug 23, 2021

Could be useful for newbies.


Building instructions:

cmake -DQt5_DIR=<Qt5 path> -DCoin_DIR=<Coin3d path> -DSoQt_DIR=<SoQt path> <example source path>
Copy link
Collaborator

@VolkerEnderlein VolkerEnderlein Oct 4, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

According to this CMake question on StackOverflow you should use either <package name>_ROOT variables as command line arguments or define the <package name>_DIR variables as environment variables. That's why I mostly recommend using the CMAKE_PREFIX_PATH argument on the command line call to CMake (for instance: cmake -S <sourcedir> -B <builddir> -DCMAKE_PREFIX_PATH="<Qt5 path>;<Coin3d path>;<SoQt path>" -G "<selected CMake generator>") You might omit the <SoQt path>, as in general SoQt will be installed in the Coin3d path.

The CMake build command should explicitly mention the config: cmake --build <builddir> --config Release

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@magres 👆

@VolkerEnderlein
Copy link
Collaborator

Thanks for the example. I'll happily merge your PR if you adapt the build instructions.

@luzpaz
Copy link
Contributor

luzpaz commented Aug 27, 2022

bumping

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.

3 participants