Ticket #531 (closed: fixed)

Opened 12 years ago

Last modified 5 years ago

Rationalise the Mantid.properties files

Reported by: Russell Taylor Owned by:
Priority: major Milestone: Iteration 16
Component: Keywords:
Cc: Blocked By:
Blocking: Tester:

Description

Currently, there are Mantid.properties files in the following locations in the repository:

  • Code/Mantid/release
  • Code/Mantid/debug
  • Code/Mantid/Build/Tests (several different-named ones for tests)
  • Code/Mantid/MatlabAPI/mfiles

This should probably create just one 'master' copy (except perhaps for the unit test versions).

The Windows installer takes its copy from the first of these, but also performs some modifications in Installer/generateWxs.py before writing it out. Moreover, it overwrites any existing modifications that a user may have made.

Change History

comment:1 Changed 12 years ago by Russell Taylor

(In [2306]) Set a couple of paths to something more sensible. Re #531.

comment:2 Changed 12 years ago by Russell Taylor

Just to note that we may still want the ability to differentiate the defaults that Windows and Linux users get. For example, I've just changed the default directory for managed workspaces to ../temp, but for linux it should probably by /tmp.

comment:3 Changed 12 years ago by Nick Draper

I have had a look into the possibility of having a system Mantid.Properties file that we can overwrite at whim, and a user.properties file that can override anything in the system file.

While the POCO classes do not appear to support appending or overwiting property files it does allow repeat defnintions in the same file to overwrite the first settings.

What this means is that if we just append one file stream to the next (with a line feed) then we should get what we want.

comment:4 Changed 12 years ago by Nick Draper

(In [2319]) re #531 Changes to Properties files

Mantid now tries to load TWO properties files Mantid.Properties as before and now a Mantid.user.properties

Any entries in the user.properties file overides those on the main file. The intention is that on reinstalls we will not overwrite the user file, but are free to replace the original .properties file to add new keys etc.

IMPORTANT: The release properties files are now in the Properties subdirectory, the ones in the release directory have been removed.

comment:5 Changed 12 years ago by Nick Draper

(In [2320]) re #531 also removed the debug mantid.properties file

comment:6 Changed 12 years ago by Nick Draper

(In [2322]) re #531 some fixes for linux and a minor change to the wxs generation to include python scripts directory

comment:7 Changed 12 years ago by Nick Draper

(In [2323]) re #531 missed a file

comment:8 Changed 12 years ago by Nick Draper

(In [2324]) re #531 removed several pointless or hopelessly out of date directories from SVN

comment:9 Changed 12 years ago by Nick Draper

(In [2325]) re #531 no need for the services dir

comment:10 Changed 12 years ago by Nick Draper

(In [2326]) re #531 no need for the services dir

comment:11 Changed 12 years ago by Nick Draper

  • Status changed from new to closed
  • Resolution set to fixed

(In [2327]) fixes #531 Mantid.user.properties is now written by mantid if it cannot be found - so there is no need to deliver this in the installer. Also logging levels set higher for files to alleviate speed problem on encrypted hard drives.

comment:12 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 1379

Note: See TracTickets for help on using tickets.