-
Notifications
You must be signed in to change notification settings - Fork 54
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
Clarifying the example in readme #932
Comments
Hmm, I think there is some value of having the example be extremely minimal in order to showcase "hey, look, this is how simple it can be". What about having an expandable element that would say "what is going on under the hood?"? There are also quite a few resources that should be displayed somewhere regarding CMake and scikit-build-core. Here is my last attempt at summarizing the sources and their scope and we could add to it:
For the specific documentation you have in mind, I think it would be best served within this repo's guide, and we should try out to see where and how it best fits. And I think it would be best to mention the resources above in such a PR as well. |
Scikit-build-core installs your package directory automatically if it can find it (and even works in editable mode). Generally it's best to try to avoid installing
It's a bit more complex than that, as it actually always "installs" it to the wheel, which is then unzipped into that location. The main effect being that you can't use absolute paths or react to the contents of site packaegs. That's why there's a whole page on this procedure at https://scikit-build-core.readthedocs.io/en/latest/build.html |
Thank you for the responses. Maybe I'm expecting too much from a readme document. 😅 I guess all I'm asking here is that a bit more context and not full details. Maybe even just some hypothetical distro structure which yields a functional distro package? Aside from readme, an analogue to this Meanwhile, there may be a bug, which triggered me to post this issue. Please see Issue #935 that I just posted. Thanks! |
To a CMake novice (or someone who has not used it in years like me) the example presented in the readme feels a bit under-explained.
It would be helpful if more context is given to the example, specifically where the built package will be installed when
pip install
'ed. Something along the line of "theinstall
command instructs CMake to install the C module to.site_packages/scikit_build_simplest/_module.pyd
when you runpip install scikit_build_simplest
."Also, since this module is not accessible without
scikit_build_simplest/__init__.py
, it may be even better to include another line toCMakeLists.txt
:Thanks!
The text was updated successfully, but these errors were encountered: