Ticket #4517 (closed: fixed)

Opened 9 years ago

Last modified 5 years ago

Remove otool usage from Python startup

Reported by: Martyn Gigg Owned by: Martyn Gigg
Priority: major Milestone: Release 2.0
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Russell Taylor

Description

There is currently code that ensures the Python APIs stay separated. This uses otool to pick out the paths of the dependencies but it turns out that otool is only installed with XCode and is not a standard part of OS X unlike ldd on Linux distributions.

Its usage should be removed as it means Mantid cannot be used on a "clean" Mac

Change History

comment:1 Changed 9 years ago by Martyn Gigg

Refs #4517. Remove otool usage from Python.

otool is not a standard tool on OS X unlike ldd on Linux distributions. The dependent libraries are now assumed to be in the MacOS directory rather than using otool.

Changeset: 9fbb09624cc1ac19d33354023389ff80c8dcf88a

comment:2 Changed 9 years ago by Martyn Gigg

  • Status changed from new to accepted

comment:3 Changed 9 years ago by Martyn Gigg

  • Status changed from accepted to verify
  • Resolution set to fixed

comment:4 Changed 9 years ago by Martyn Gigg

Refs #4517. Add a return line.

Changeset: f27cf7c4384dd822e7c09799fb8b564236101ca8

comment:5 Changed 9 years ago by Russell Taylor

  • Status changed from verify to verifying
  • Tester set to Russell Taylor

comment:6 Changed 9 years ago by Russell Taylor

  • Status changed from verifying to closed

Verified by grepping for mention of otool in the source code, and by temporarily renaming otool on my mac and observing that both old and new python appear to still work.

comment:7 Changed 9 years ago by Martyn Gigg

Refs #4517. Remove otool usage from Python.

otool is not a standard tool on OS X unlike ldd on Linux distributions. The dependent libraries are now assumed to be in the MacOS directory rather than using otool.

Changeset: 9fbb09624cc1ac19d33354023389ff80c8dcf88a

comment:8 Changed 9 years ago by Martyn Gigg

Refs #4517. Add a return line.

Changeset: f27cf7c4384dd822e7c09799fb8b564236101ca8

comment:9 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 5364

Note: See TracTickets for help on using tickets.