Ticket #2781 (closed: fixed)

Opened 10 years ago

Last modified 5 years ago

Finish off CPack configuration for linux

Reported by: Peter Peterson Owned by: Stuart Campbell
Priority: major Milestone: Iteration 28
Component: Mantid Keywords:
Cc: scu@… Blocked By:
Blocking: Tester: Michael Reuter

Description


Change History

comment:1 Changed 10 years ago by Peter Peterson

  • Owner set to Peter Peterson
  • Status changed from new to accepted

comment:2 Changed 10 years ago by Peter Peterson

(In [10616]) Adding contact information. Refs #2781.

comment:3 Changed 10 years ago by Peter Peterson

(In [10618]) Adding checks for the types of binary packages to create. Refs #2781.

comment:4 Changed 10 years ago by Peter Peterson

(In [10620]) Fixed up the autodetection of package creation tools. Refs #2781.

comment:5 Changed 10 years ago by Peter Peterson

  • Cc scu@… added

comment:6 Changed 10 years ago by Peter Peterson

(In [10623]) Shuffled around the dependencies to make it clearer what the rhel5/rhel6 diffrerences are. Refs #2781.

comment:7 Changed 10 years ago by Peter Peterson

(In [10652]) Renamed all of the color maps to make cpack (and Stu) happy. Refs #2781.

comment:8 Changed 10 years ago by Peter Peterson

(In [10671]) Moved getting the framework version from Framework/CMakeLists.txt into CommonSetup.cmake so cpack has access. Refs #2781.

comment:9 Changed 10 years ago by Peter Peterson

(In [10672]) More smoothing out of everything. The filenames now follow their os's standards. Refs #2781.

comment:10 Changed 10 years ago by Peter Peterson

(In [10687]) Better naming of rpms. Refs #2781.

comment:11 Changed 10 years ago by Peter Peterson

(In [10688]) Needs to be uppercase. Refs #2781.

comment:12 Changed 10 years ago by Peter Peterson

(In [10692]) Removing poco-doc from the list of requirements. Refs #2781.

comment:13 Changed 9 years ago by Peter Peterson

(In [10735]) Tagged icon for installers. This will probably need to be changed. Refs #2781.

comment:14 Changed 9 years ago by Peter Peterson

  • Status changed from accepted to assigned
  • Owner changed from Peter Peterson to Russell Taylor

The only thing that I notice is missing now is to add MantidPlot to the path. I haven't tried anything other than starting it up and look for warnings to appear.

comment:15 Changed 9 years ago by Russell Taylor

  • Status changed from assigned to accepted

comment:16 Changed 9 years ago by Russell Taylor

(In [10770]) Create and install files to set environment variables on Mantid installation (on Red Hat at least). Re #2781.

comment:17 Changed 9 years ago by Russell Taylor

(In [10804]) Comment out to fix SNS rhel builds until I figure out how to do it without being root. Re #2781.

comment:18 Changed 9 years ago by Russell Taylor

(In [10816]) Put this back in - it's working for me without a sudo. Re #2781.

comment:19 Changed 9 years ago by Russell Taylor

(In [10818]) Buildservers unhappy. CMake version? Stem the spam. Re #2781.

comment:20 Changed 9 years ago by Russell Taylor

(In [10827]) Another go, having set a flag in the Jenkins config. Re #2781.

comment:21 Changed 9 years ago by Russell Taylor

(In [11034]) Switch svn info variable used from repo latest to last commit. Should give right answer when building off a tag. Re #2781.

comment:22 Changed 9 years ago by Nick Draper

  • Milestone changed from Iteration 28 to Iteration 29

Bulk move of tickets at the end of iteration 28

comment:23 Changed 9 years ago by Russell Taylor

  • Milestone changed from Iteration 29 to Iteration 28

comment:24 Changed 9 years ago by Russell Taylor

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

I think this is finished...

comment:25 Changed 9 years ago by Russell Taylor

P.S. It needs to be tested on clean machines of all platforms (rhel5, rhel6, ubuntu)

comment:26 Changed 9 years ago by Stuart Campbell

  • Status changed from verify to verifying
  • Tester set to Stuart Campbell

comment:27 Changed 9 years ago by Stuart Campbell

Installed on a clean RHEL 6 machine - looks ok.

comment:28 Changed 9 years ago by Stuart Campbell

On ubuntu 10.10, when doing a plain {{{make package}} I get the following error:

CPack: Create package using DEB
CPack: Install projects
CPack: - Run preinstall target for: Mantid
CPack: - Install project: Mantid
CMake Error at /home/scu/MantidBuild/Debug/cmake_install.cmake:36 (FILE):
  file INSTALL cannot copy file "/home/scu/MantidBuild/Debug/mantid.sh" to
  "/etc/profile.d/mantid.sh".


CPack Error: Error when generating package: mantid

comment:29 Changed 9 years ago by Stuart Campbell

  • Status changed from verifying to reopened
  • Resolution fixed deleted

Building works with sudo make package. But trying to install gives...

scu@ubuntu:~/MantidBuild/Debug$ sudo dpkg -i mantid_1.28.11169-1_amd64.deb 
dpkg: error processing mantid_1.28.11169-1_amd64.deb (--install):
 parse error, in file '/var/lib/dpkg/tmp.ci/control' near line 6 package 'mantid':
 `Depends' field, syntax error after reference to package `boost'
Errors were encountered while processing:
 mantid_1.28.11169-1_amd64.deb

comment:30 Changed 9 years ago by Russell Taylor

  • Owner changed from Russell Taylor to Stuart Campbell
  • Status changed from reopened to assigned

Then, as I don't have an ubuntu machine, it's all yours :) Or you can spin off a new ticket for the next iteration and we'll just not announce a deb yet.

N.B. make install should be possible without being root, but depending on cmake version may need the variable CPACK_SET_DESTDIR set to ON

comment:31 Changed 9 years ago by Stuart Campbell

  • Status changed from assigned to accepted

Created ticket #2887 to fix the CPack DEB package.

comment:32 Changed 9 years ago by Stuart Campbell

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

comment:33 Changed 9 years ago by Michael Reuter

  • Status changed from verify to verifying
  • Tester changed from Stuart Campbell to Michael Reuter

comment:34 Changed 9 years ago by Michael Reuter

  • Status changed from verifying to closed

A new ticket has been made for this on the next iteration.

comment:35 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 3628

Note: See TracTickets for help on using tickets.