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

docs: mention python _ctypes support #2383

Open
wants to merge 1 commit into
base: devel
Choose a base branch
from

Conversation

djudjuu
Copy link
Collaborator

@djudjuu djudjuu commented Mar 6, 2025

closes #2382
UPDATE: maybe not that needed as its a bit of an edge case to and there is no fitting section in the docs (e.g. general troubleshooting). maybe having an issue with _ctypes text is already some kind of hint for devs running into the same issue

@djudjuu djudjuu linked an issue Mar 6, 2025 that may be closed by this pull request
Copy link

netlify bot commented Mar 6, 2025

Deploy Preview for dlt-hub-docs ready!

Name Link
🔨 Latest commit 4335182
🔍 Latest deploy log https://app.netlify.com/sites/dlt-hub-docs/deploys/67c991d4c639f30008a3bcf4
😎 Deploy Preview https://deploy-preview-2383--dlt-hub-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

```sh
`python -c "import _ctypes"`
```
If this fails, you need to install the [libffi](https://sourceware.org/libffi/)-package
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

more context:
if libffi is mostly included by default,libffi-dev is for compiling software from source. when python versions are managed (and installed) with pyenv (at least thats how i did it), then python gets installed but without the _ctypes module.

Copy link
Collaborator

Choose a reason for hiding this comment

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

@djudjuu did you verify that this is also needed for python 3.12?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I will verify it, but I dont want to mess with my system by removing the libffi-dev package, so I think I will use mutlipass to spin up fresh ubuntu images and test it there. Its probably a good thing to be able to do that anyways

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

so I've verified that it occurs only if, you have build from source (e.g. using pyenv) (i tried with 3.12 and 3.13) when libffi-dev is not on the system. I am not sure this is a likely state, since usually libffi-dev is part of the packages that one has on the system if one compiles C stuff. But somehow that was the case for me, so maybe its not THAT unlikely.

during the 3.1x installation with pyenv I also did get a warning that C support was missing, so in some ways there already is a warning, so maybe this comment in the docs shouldnt be that prominent.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

not sure if that makes it necessary at all to mention it then...maybe lets discuss it live real quick

@djudjuu djudjuu requested a review from sh-rp March 6, 2025 12:26
@djudjuu
Copy link
Collaborator Author

djudjuu commented Mar 6, 2025

hmm. i am seeing my change locally, but when I click on the netllify-deploy preview its not there :/
local:
image

@sh-rp
Copy link
Collaborator

sh-rp commented Mar 6, 2025

hmm. i am seeing my change locally, but when I click on the netllify-deploy preview its not there :/ local: image

you need to switch to the devel docs in the deployed preview to see your changes, check the button with the version on the top.

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.

document streamlit c_types module dependency
2 participants