On 1/28/2009 4:43 PM, Lorenzo Mancini wrote: > Giovanni Bajo wrote: >> On 1/27/2009 7:24 PM, Lorenzo Mancini wrote: > >>> The proposed patch is attached. It basically removes the >>> install_name_tool magic and inserts the workpath at the beginning of >>> DYLD_LIBRARY_PATH if we are running under Mac OS X. It should work for >>> both one-file and one-dir deploys. >> The patch looks great and it's fine by me. Your analysys is compelling, too! >> >> The only doubt is what version of OSX is required for DYLD_LIBRARY_PATH >> to work. Was it introduced in 10.0, or is it a later feature? Because >> obviously our generated file would work only starting from that version. > > With some googling I've found reports of people setting > DYLD_LIBRARY_PATH on Mac OS X 10.1 [1] [2]. > > I didn't find any document about the introduction of DYLD_LIBRARY_PATH > in any specific Mac OS X version, so I'd say it's safe to assume that it > was there on 10.0 too. > > I've also tested the patch with one-dir and one-file deploys of a simple > script using PyQt4, to ensure there are no regressions. The same tests > run under an unpatched PyInstaller show indeed that the same one-file > deploy cannot work for the aforementioned reasons. > > This improvement to Mac OS X support calls for the upcoming release of > 1.4! :)
Well done, thanks! -- Giovanni Bajo Develer S.r.l. http://www.develer.com --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "PyInstaller" group. To post to this group, send email to [email protected] To unsubscribe from this group, send email to [email protected] For more options, visit this group at http://groups.google.com/group/PyInstaller?hl=en -~----------~----~----~----~------~----~------~--~---
