Raylibpy 3.7: Support linux + Toggle external raylib library

overdev / raylib-py

A Python binding for the great C library raylib.

Other

185 stars 20 forks source link

Closed adamlwgriffiths closed 3 years ago

adamlwgriffiths commented 3 years ago

Gets raylib 3.7 almost working on Linux (another PR is coming for a separate fix)

Changes:

The ctypes import were pulling in win32 only symbols. Wrap this in an 'if windows' scope.
The ability to provide custom library paths is important for OS such as NixOS. So lets reinstate the older logic that supported passing in a library path via environment variable "RAYLIB_BIN_PATH".
Also support a series of path names being provided which let us search multiple locations.
Add a new environment variable "RAYLIB_BIN_FILENAME" which lets us specify the filename to load for the library. (This isn't required for Nix but I felt that it might be of use)
Corrected type in README.md ("wich" -> "which").

adamlwgriffiths commented 3 years ago

There are some changes I want to make to this before it's merged:

Simplify the file list and just use 1 filename per platform (libraylib.so).
Remove the RAYLIB_BIN_FILENAME option (I doubt it's helpful and just more maintenance).
Look at re-removing the RAYLIB_BIN_PATH and instead have a toggle option that will use the system .so file instead (Ie. rely on LD_LIBRARY_PATH).

adamlwgriffiths commented 3 years ago

I've made the changes and also fixed __all__ including CDLLEx, which isn't valid on Linux.

overdev commented 3 years ago

@adamlwgriffiths Merged!