Ticket #2256 (closed: fixed)

Opened 10 years ago

Last modified 5 years ago

Remove ConfigService checking for Python and Matlab executable names

Reported by: Martyn Gigg Owned by: Martyn Gigg
Priority: major Milestone: Iteration 27
Component: Mantid Keywords:
Cc: taylorrj@… Blocked By:
Blocking: Tester: Russell Taylor

Description (last modified by Martyn Gigg) (diff)

The ConfigService behaves slightly differently if the executable name is Python or Matlab. At the time this was due to the fact that we needed to know where the Mantid bin directory was if running in a standalone interpeter in a different directory.

Now that we have the MANTIDPATH variable this behaviour should be changed so that Mantid is consistent regardless of the executable name.

The change will mean that when the ConfigService starts it looks for the Mantid.properties file in the following way:

(a) If MANTIDPATH is defined then use this path, else (b) use the executable directory.

Change History

comment:1 Changed 10 years ago by Martyn Gigg

  • Description modified (diff)

comment:2 Changed 10 years ago by Martyn Gigg

(In [8826]) Removed check for the executable name from the ConfigService. It now either uses the path pointed to by MANTIDPATH or the executable's directory if the former is not set. Also renamed a couple of functions relating to this so that it is clearer what their return values point. Re #2256

comment:3 Changed 10 years ago by Russell Taylor

(In [8832]) Trickery no longer needed now that Mantid is consistent in where it looks for properties files. Re #2256.

comment:4 Changed 10 years ago by Peter Peterson

(In [8855]) Fix to help vanilla python. Refs #2256.

comment:5 Changed 10 years ago by Martyn Gigg

  • Status changed from new to accepted

comment:6 Changed 10 years ago by Martyn Gigg

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

comment:7 Changed 10 years ago by Russell Taylor

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

comment:8 Changed 10 years ago by Russell Taylor

  • Status changed from verifying to closed

comment:9 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 3103

Note: See TracTickets for help on using tickets.