Ticket #11137 (closed: fixed)

Opened 6 years ago

Last modified 5 years ago

Modify MantidPlot startup script to use mantidlibs software collection on RHEL7 and Fedora 20

Reported by: Stuart Campbell Owned by: Stuart Campbell
Priority: critical Milestone: Release 3.3.1
Component: Framework Keywords: PATCH_CANDIDATE
Cc: Blocked By:
Blocking: Tester: Martyn Gigg

Description


Change History

comment:1 Changed 6 years ago by Stuart Campbell

  • Keywords PATCH_CANDIDATE added
  • Milestone set to Release 3.3.1

comment:2 Changed 6 years ago by Stuart Campbell

Use mantidlibs software collection for RHEL7/F20.

Added a test for "Maipo" == RHEL7 and

"Heisenbug" == Fedora 20

so that they will now also use the mantidlibs scl

refs #11137

Changeset: eff27e70a8fe964b3288dfa05eec61e45d1de1c4

comment:3 Changed 6 years ago by Stuart Campbell

Removed Fedora 20 use of software collection.

There seems to be some issues with the newer versions of scl that are shipped with Fedora 20/21. For the moment I am removing the F20 check so that this will only use the software collection for RHEL6/7.

refs #11137

Changeset: 0e65a1864e18e0207f903ede2b4c810da425b715

comment:4 Changed 6 years ago by Stuart Campbell

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

This is being verified as pull request #275.

comment:5 Changed 6 years ago by Stuart Campbell

This is being verified as pull request #276.

comment:6 Changed 6 years ago by Martyn Gigg

  • Status changed from verify to verifying
  • Tester set to Martyn Gigg

comment:7 Changed 6 years ago by Martyn Gigg

Was this based off next? It has the commit that fixes the patch number to 3.3.0 so we shouldn't merge this to master.

comment:8 Changed 6 years ago by Martyn Gigg

I tested this out on our RHEL7 build server and it finds ParaView successfully now.

comment:9 Changed 6 years ago by Martyn Gigg

I guess the easiest way to deal with this would just be to revert 1385fe4 that fixed the patch number.

comment:10 Changed 6 years ago by Stuart Campbell

But it looks like it also has the change to undo it as well. Looking at the diff with master it looks ok

comment:11 Changed 6 years ago by Martyn Gigg

Oh yes, you're right. I should have looked further down. Sorry about that.

It's good to go then.

comment:12 Changed 6 years ago by Martyn Gigg

  • Status changed from verifying to closed

Merge pull request #276 from mantidproject/11137_enable_mantidlibs_rhel7_patch

Use Mantidlibs software collection on RHEL7

Full changeset: 7d5c28aace26785c939c659b28d58ab16bf7fd1b

comment:13 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 11976

Note: See TracTickets for help on using tickets.