Ticket #10058 (assigned)

Opened 6 years ago

Last modified 5 years ago

Using class_maker.py on MantidGeometry breaks the build

Reported by: Michael Wedel Owned by: Martyn Gigg
Priority: minor Milestone: Backlog
Component: Tools Keywords:
Cc: Blocked By:
Blocking: Tester:

Description (last modified by Michael Wedel) (diff)

When adding a new class to MantidGeometry with the class_maker.py script, the CMakeLists-file is altered. The following line, which is located near the end of the CMakeLists-file:

   set (SRC_FILES ${SRC_FILES} ${OPENCASCADE_SRC} )

is replaced by an additional line at the beginning of set ( SRC_FILES ...:

   ${SRC_FILES} ${OPENCASCADE_SRC}

which is at the beginning of the file.

After this modification, building Mantid results in the following linker errors (Ubuntu):

../bin/libMantidGeometry.so: undefined reference to `Mantid::Geometry::OCGeometryHandler::OCGeometryHandler(Mantid::Geometry::Object*)'
../bin/libMantidGeometry.so: undefined reference to `Mantid::Geometry::OCGeometryHandler::getTriangleFaces()'
../bin/libMantidGeometry.so: undefined reference to `Mantid::Geometry::OCGeometryHandler::NumberOfTriangles()'
../bin/libMantidGeometry.so: undefined reference to `Mantid::Geometry::OCGeometryHandler::getTriangleVertices()'
../bin/libMantidGeometry.so: undefined reference to `Mantid::Geometry::OCGeometryHandler::NumberOfPoints()'
../bin/libMantidGeometry.so: undefined reference to `Mantid::Geometry::OCGeometryHandler::~OCGeometryHandler()'

Another problem is that the generated unit test file contains this line:

using namespace Mantid::API;

which will only compile if something that imports something from Mantid::API is tested.

Change History

comment:1 Changed 6 years ago by Michael Wedel

  • Description modified (diff)

comment:2 Changed 6 years ago by Michael Wedel

  • Component changed from Framework to Tools

comment:3 Changed 6 years ago by Martyn Gigg

  • Priority changed from major to minor
  • Owner set to Martyn Gigg
  • Status changed from new to assigned

comment:4 Changed 6 years ago by Nick Draper

Moved to the backlog at the code freeze of R3.3

comment:5 Changed 6 years ago by Nick Draper

  • Milestone changed from Release 3.3 to Backlog

comment:6 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 10900

Note: See TracTickets for help on using tickets.