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

Change how binary/lib/dll directories are set #3

Closed
wants to merge 1 commit into from

Conversation

ptheywood
Copy link
Member

Changes CMake archive/binary output directores are set, to be handeleed by updated logic in common.cmake rather than manually in the repository.

Also adds -DFLAMEGPU_ROOT=/path/to/local/FLAMEGPU2 as a mechanism to avoid fetching remote FLAME GPU easily. This will need changing further once we have installation rules.

This change (coupled with the main repo change) does currently lead to libflamegpu.a being placed in build/lib/<config> rather than _deps/flamegpu-builds/lib/<config> which could be reverted to with a change in either repo.

@Robadob
Copy link
Member

Robadob commented Aug 19, 2021

Worth moving these changes into the alifetutorial too, I updated that to copy from this yesterday.

@Robadob
Copy link
Member

Robadob commented Aug 19, 2021

You never did merge my PR. so just modify it I guess.

FLAMEGPU/FLAMEGPU2-tutorial-cudacpp#3

@ptheywood
Copy link
Member Author

You never did merge my PR. so just modify it I guess.

FLAMEGPU/ALifeTutorial#3

I've not got round to testing it yet, but yep the plan was to move it to all the standalone cuda C++ repos once it's narrowed down, which will include the tutorial (especially once we rename it so I don't forget about it in the future)

@ptheywood ptheywood closed this Jan 18, 2023
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.

2 participants