Fix load library for PS2000A on OSX. #158
Conversation
| @@ -131,7 +131,8 @@ def __init__(self, serialNumber=None, connect=True): | |||
| from ctypes import cdll | |||
| self.lib = cdll.LoadLibrary("lib" + self.LIBNAME + ".so") | |||
| elif platform.system() == 'Darwin': | |||
| self.lib = self.loadLibraryDarwin("lib" + self.LIBNAME + ".dylib") | |||
| @@ -14,7 +14,7 @@ def LoadLibraryDarwin(library): | |||
|
|
|||
| http://ulthiel.com/vk2utl/picoscope-python-interface-under-mac-os-x/ | |||
| """ | |||
| PICO_LIB_PATH = "/Applications/PicoScope6.app/Contents/Resources/lib/" | |||
| PICO_LIB_PATH = "/Applications/PicoScope 6.app/Contents/Resources/lib/" | |||
There was a problem hiding this comment.
Could you please make this more "backward" compatible?? I understand that Picoscope moved the location of their file.
If you could "try both locations" in a loop or something, that would be better.
There was a problem hiding this comment.
As far as I can see, Picoscope changed the path at least 1.5 years ago, so is there anything to be backwards compatible to? Is it possible to install a Picoscope library that uses the old path?
There was a problem hiding this comment.
1.5 years is not a long time for instrumentation. often I'll just leave the software at the version I had when I first got the instrument. it's pretty important to me that the old stuff continues to work at least for a few more months.
|
I think alot of this was done in #144, but maybe we got stuck in "perfection". I can look at cleaning that PR up. unless you want to integreate the essense into it. |
Use the the proper
LoadLibraryDarwinand the lib path currently used by PicoScope 6.13 on OSX.