Ticket #6902 (closed: fixed)
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
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:24 Changed 5 years ago by Stuart Campbell
This ticket has been transferred to github issue 7748
Pinching this one as I know how to fix it (#6976 did part of the work).