-
Notifications
You must be signed in to change notification settings - Fork 283
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
Default magic_file to 'magic' #137
Comments
How about a environment variable? |
Actually on second thought, I'd rather leave it up to the application to pass this through. Or do you have a situation where you don't own the code calling python-magic? |
Its actually quite simple: there is this binary package, which is compatible with the python build for windows: https://github.com/julian-r/file-windows/. The only problem is, that this |
Okay, just did some investigation to my old memories:
Hope that sounds interesting for you. |
I would love this feature! We are currently deploying to linux but partly developing using windows. |
@julian-r I love that you provide the binaries, but my preference is to not include them in the standard package. Any existing users (by definition) already provide their own binaries, and would in general want to use the system-provided binary. But I think the goal of making this more seamless is a good one. There's a general problem I have here of figuring out how to find the shared lib on various platforms, just look a the mess in magic.py to see all the special cases. This feels like an issue of configuration to me; you have a single codebase and want to run it in different ways (e.g pointing at a new shared lib / magic file). So I'll re-suggest the environment variable idea: PYTHON_MAGIC_SHARED_LIB=something.dll What do you think? |
@julian-r's Windows binaries being available on PyPI are really useful, and I would love them to be maintained somehow |
Merging into #293 |
For windows it would be quite good, because then if the cwd contains the dll and the mgc file everything works out of the box :)
The text was updated successfully, but these errors were encountered: