Ticket #6902 (closed: fixed)

Opened 7 years ago

Last modified 5 years ago

Installed Mantid and Built Mantid

Reported by: Owen Arnold Owned by: Russell Taylor
Priority: major Milestone: Release 2.6
Component: Framework Keywords:
Cc: Blocked By:
Blocking: Tester: Michael Reuter

Description

This is a Mac issue.

You cannot have an installed copy of Mantid as well as a built copy of Mantid on the mac. The wrong libraries get loaded during the build process and by ctest.

Change History

comment:1 Changed 7 years ago by Russell Taylor

  • Status changed from new to accepted
  • Owner changed from Martyn Gigg to Russell Taylor

Pinching this one as I know how to fix it (#6976 did part of the work).

comment:2 Changed 7 years ago by Russell Taylor

Re #6902. Remove a hard coded path.

The one used for linux will give the right answer in a more general way and will avoid looking into an installed Mantid when running a locally built one.

Changeset: 6a45449bf051a66f1ca3d2ad6500157681e04c09

comment:3 Changed 7 years ago by Russell Taylor

Do we actually need this section of code on linux???

comment:4 Changed 7 years ago by Russell Taylor

Re #6902. Remove a hard coded path.

The one used for linux will give the right answer in a more general way and will avoid looking into an installed Mantid when running a locally built one.

Changeset: 6a45449bf051a66f1ca3d2ad6500157681e04c09

comment:5 Changed 7 years ago by Martyn Gigg

In reply to to comment 3. I don't think we do now. All of our Linux distros use packages for PyQt & sip and we bundle them in the bin directory any longer. Windows has the packages correctly located in the bundled Python distro.

When we finally sort the Mac bundle out, i.e. bundle Python, we can move the packages to the correct bundled site-packaes directory.

comment:6 Changed 7 years ago by Russell Taylor

Re #6902. Remove unnecessary code from linux build.

We use system-installed sip & PyQt on linux.

Changeset: 424a87b8fd794b5573ac709a527a02b291f9f604

comment:7 Changed 7 years ago by Russell Taylor

Re #6902. Remove unnecessary code from linux build.

We use system-installed sip & PyQt on linux.

Changeset: 424a87b8fd794b5573ac709a527a02b291f9f604

comment:8 Changed 7 years ago by Russell Taylor

Revert "Re #6902. Remove unnecessary code from linux build."

This appears to have had a bad effect on our fragile MantidPlot python tests and I'm not inclined to figure out what's wrong exactly (they still work locally for me).

This reverts commit 424a87b8fd794b5573ac709a527a02b291f9f604.

Changeset: 1ac4bf493f3f456f5669fb97738785ac48c07899

comment:9 Changed 7 years ago by Russell Taylor

Actually, I do see the problem when I build the right branch! Nevertheless, I'm not going to spend time on it (if it ain't broke....).

comment:10 Changed 7 years ago by Russell Taylor

Revert "Re #6902. Remove unnecessary code from linux build."

This appears to have had a bad effect on our fragile MantidPlot python tests and I'm not inclined to figure out what's wrong exactly (they still work locally for me).

This reverts commit 424a87b8fd794b5573ac709a527a02b291f9f604.

Changeset: 1ac4bf493f3f456f5669fb97738785ac48c07899

comment:11 Changed 7 years ago by Russell Taylor

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

Branch is: feature/6902_built_and_installed_on_osx

I can now build and run MantidPlot and unit test locally even if I have an installed Mantid as well. To test: check you can do the same on a Mac!

comment:12 Changed 7 years ago by Michael Reuter

  • Status changed from verify to verifying
  • Tester set to Michael Reuter

comment:13 Changed 7 years ago by Russell Taylor

Re #6902. Remove a hard coded path.

The one used for linux will give the right answer in a more general way and will avoid looking into an installed Mantid when running a locally built one.

Changeset: 6a45449bf051a66f1ca3d2ad6500157681e04c09

comment:14 Changed 7 years ago by Russell Taylor

Re #6902. Remove unnecessary code from linux build.

We use system-installed sip & PyQt on linux.

Changeset: 424a87b8fd794b5573ac709a527a02b291f9f604

comment:15 Changed 7 years ago by Russell Taylor

Revert "Re #6902. Remove unnecessary code from linux build."

This appears to have had a bad effect on our fragile MantidPlot python tests and I'm not inclined to figure out what's wrong exactly (they still work locally for me).

This reverts commit 424a87b8fd794b5573ac709a527a02b291f9f604.

Changeset: 1ac4bf493f3f456f5669fb97738785ac48c07899

comment:16 Changed 7 years ago by Michael Reuter

  • Status changed from verifying to closed

After figuring out the issue of being able to run my build copy, I can now run both a build and installed copy of Mantid without them stepping on each other. Awesome!

comment:17 Changed 7 years ago by Russell Taylor

Re #6902. Remove a hard coded path.

The one used for linux will give the right answer in a more general way and will avoid looking into an installed Mantid when running a locally built one.

Changeset: 6a45449bf051a66f1ca3d2ad6500157681e04c09

comment:18 Changed 7 years ago by Russell Taylor

Re #6902. Remove unnecessary code from linux build.

We use system-installed sip & PyQt on linux.

Changeset: 424a87b8fd794b5573ac709a527a02b291f9f604

comment:19 Changed 7 years ago by Russell Taylor

Revert "Re #6902. Remove unnecessary code from linux build."

This appears to have had a bad effect on our fragile MantidPlot python tests and I'm not inclined to figure out what's wrong exactly (they still work locally for me).

This reverts commit 424a87b8fd794b5573ac709a527a02b291f9f604.

Changeset: 1ac4bf493f3f456f5669fb97738785ac48c07899

comment:20 Changed 7 years ago by Russell Taylor

Re #6902. Remove a hard coded path.

The one used for linux will give the right answer in a more general way and will avoid looking into an installed Mantid when running a locally built one.

Changeset: 6a45449bf051a66f1ca3d2ad6500157681e04c09

comment:21 Changed 7 years ago by Russell Taylor

Re #6902. Remove unnecessary code from linux build.

We use system-installed sip & PyQt on linux.

Changeset: 424a87b8fd794b5573ac709a527a02b291f9f604

comment:22 Changed 7 years ago by Russell Taylor

Revert "Re #6902. Remove unnecessary code from linux build."

This appears to have had a bad effect on our fragile MantidPlot python tests and I'm not inclined to figure out what's wrong exactly (they still work locally for me).

This reverts commit 424a87b8fd794b5573ac709a527a02b291f9f604.

Changeset: 1ac4bf493f3f456f5669fb97738785ac48c07899

comment:23 Changed 7 years ago by Nick Draper

  • Component changed from Mantid to Framework

comment:24 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 7748

Note: See TracTickets for help on using tickets.