#35 Merged at db47d90
Repository
anthony_tuininga
Branch
default

Fix to allow resolving dependencies on OSX where dependency paths are relative.

Author
  1. Almar Klein
Reviewers
Description

Fix to allow resolving dependencies on OSX where dependency paths are relative.

This enables using cx_freeze in a conda environment.

Comments (6)

  1. Almar Klein author

    The replacement was first done in another part in the code. You suggested to move it here.

    No, not ready for merge yet, because I have not had a chance to test it. I had trouble with cloning the repo locally, because I made the initial commit via the web interface. I probably should not do that.

    Anyway, I will fire up a VM, test this code, and let you know the result.

  2. Almar Klein author

    Current code is tested. I also had to replace @rpath. The problem with that @rpath is that it signifies multiple search directories that are specified in the executable. So to make this work in "the right way" would be a bit more involved. However, the current solution works at least with conda-based distributions; I just successfully freezed IEP using Pyzo :)

    1. Thomas Kluyver

      This looks clean, and doesn't interfere with other platforms, so I have no objection to it. I don't know enough about Macs to judge the improvement, but I'll trust your testing.

  3. Almar Klein author

    It should be pretty safe, because without the replacements cx_Freeze wont be able to copy the necessary files anyway. This code helps in most cases. But with the @rparth probably not all cases.