Skip to content

Include examples/common.cpp in CMakeLists.txt #783

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Closed
wants to merge 1 commit into from

Conversation

iacore
Copy link
Contributor

@iacore iacore commented Apr 5, 2023

This is useful when building 3rdparty applications.

See https://github.com/iacore/llama-tui

This is a bodge. There are better ways to do this, probably.

llama.h)
llama.h
examples/common.cpp
examples/common.h)

Copy link
Member

Choose a reason for hiding this comment

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

llama should not build examples/common.cpp
It is part of the common library which is used only by the examples.

Your project should re-implement the stuff in common

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'll think of another way

@iacore iacore closed this Apr 5, 2023
Deadsg pushed a commit to Deadsg/llama.cpp that referenced this pull request Dec 19, 2023
Co-authored-by: john.shen <john.shen@bioclinica.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants