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
[chore] CMake: deprecate custom processor count logic #868
Conversation
CMake 3 shipped with a built-in module. Also quote `KO_DOWNLOAD_DIR`.
Today it looks like this: https://github.com/Kitware/CMake/blob/283330976f2379eec214bfe1d51a27c29cc1f8fc/Modules/ProcessorCount.cmake For 3.5 it looks like this: https://github.com/Kitware/CMake/blob/64130a7e793483e24c1d68bdd234f81d5edb2d51/Modules/ProcessorCount.cmake Either way, same idea but more complete with "automatic" updates & improvements. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! :)
include(ProcessorCount) | ||
ProcessorCount(N) | ||
# 0 if unknown | ||
set(PROCESSOR_COUNT ${N}) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm assuming N falls-back to 0, right? ;).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes it does (reading the links provided helps. :D).
Also includes a bunch of CMake refactoring (koreader/koreader-base#865 koreader/koreader-base#867 koreader/koreader-base#868)
Also includes a bunch of CMake refactoring (koreader/koreader-base#865 koreader/koreader-base#867 koreader/koreader-base#868)
* Properly account for MuPDF feeding us premultiplied alpha * Bump base to pickup necessary backend changes Also includes a bunch of CMake refactoring (koreader/koreader-base#865 koreader/koreader-base#867 koreader/koreader-base#868)
…ader#4807) * Properly account for MuPDF feeding us premultiplied alpha * Bump base to pickup necessary backend changes Also includes a bunch of CMake refactoring (koreader/koreader-base#865 koreader/koreader-base#867 koreader/koreader-base#868)
CMake 2.8.5 shipped with a built-in module.
Also quote
KO_DOWNLOAD_DIR
.