Skip to content

cmake : fix deprecated option names not working#8171

Merged
slaren merged 2 commits into
masterfrom
sl/fix-cmake-depr
Jun 27, 2024
Merged

cmake : fix deprecated option names not working#8171
slaren merged 2 commits into
masterfrom
sl/fix-cmake-depr

Conversation

@slaren
Copy link
Copy Markdown
Member

@slaren slaren commented Jun 27, 2024

The deprecated options are not working correctly because the variables are not being set in the correct scope.

LLAMA_OPENMP also cannot work as is because the default is ON, but the way llama_option_depr works, it only works when enabling an option, but not when disabling it. I removed it to avoid confusion.

@slaren slaren changed the title Sl/fix cmake depr cmake : fix deprecated option names not working Jun 27, 2024
@ggerganov ggerganov self-requested a review June 27, 2024 16:31
@github-actions github-actions Bot added the build Compilation issues label Jun 27, 2024
@slaren slaren merged commit 8172ee9 into master Jun 27, 2024
@slaren slaren deleted the sl/fix-cmake-depr branch June 27, 2024 18:04
MagnusS0 pushed a commit to MagnusS0/llama.cpp-normistral-tokenizer that referenced this pull request Jul 1, 2024
* cmake : fix deprecated option names not working

* remove LlAMA_OPENMP
Seunghhon pushed a commit to Seunghhon/llama.cpp that referenced this pull request Apr 26, 2026
* cmake : fix deprecated option names not working

* remove LlAMA_OPENMP
ljubomirj pushed a commit to ljubomirj/llama.cpp that referenced this pull request May 6, 2026
* cmake : fix deprecated option names not working

* remove LlAMA_OPENMP
my-other-github-account pushed a commit to my-other-github-account/llama.cpp that referenced this pull request May 15, 2026
* cmake : fix deprecated option names not working

* remove LlAMA_OPENMP
my-other-github-account pushed a commit to my-other-github-account/llama.cpp that referenced this pull request May 15, 2026
* cmake : fix deprecated option names not working

* remove LlAMA_OPENMP
phibya pushed a commit to ziee-ai/llama.cpp that referenced this pull request May 29, 2026
* cmake : fix deprecated option names not working

* remove LlAMA_OPENMP
AlexiAlp pushed a commit to minghaop/llama.cpp that referenced this pull request Jun 2, 2026
* cmake : fix deprecated option names not working

* remove LlAMA_OPENMP
AlexiAlp pushed a commit to minghaop/llama.cpp that referenced this pull request Jun 2, 2026
* cmake : fix deprecated option names not working

* remove LlAMA_OPENMP
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

build Compilation issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bug: cmake -B build -DLLAMA_CUDA=ON -DLLAMA_NATIVE=ON gives "deprecated" warning but then compiled CPU-only verrsion

2 participants