Project

General

Profile

Feature #984

add extra layer of directory search in cpp tests to the build scripts

Added by Santos, Edward over 11 years ago. Updated almost 11 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Build System
Target version:
Start date:
03 May 2012
Due date:
% Done:

100%

Estimated time:
Workflow:
Closed

Description

Scons misses to buid cpp tests as deep in the file directory structure as tests/cpp_unit/Recon/SciFi. An extra layer of directory search is needed.

#1

Updated by Rogers, Chris over 11 years ago

  • Category set to Build System
  • Status changed from Open to In Progress
  • Target version set to Future MAUS release
  • % Done changed from 0 to 50
  • Workflow changed from New Issue to In Development

Please see below for a patch to core_builder.py

I didn't merge it into the trunk yet as it starts building a whole bunch of tests - most of which fail to even compile. So please apply the patch and then try to fix the tests before I can merge it in.

=== modified file 'src/common_py/maus_build_tools/core_builder.py'
--- src/common_py/maus_build_tools/core_builder.py    2012-04-05 08:06:36 +0000
+++ src/common_py/maus_build_tools/core_builder.py    2012-06-06 16:01:20 +0000
@@ -85,7 +85,8 @@
         env.Append(LINKFLAGS=['-undefined', 'suppress','-flat_namespace'])

     test_cpp_files = glob.glob("tests/cpp_unit/*/*cc")+\
-        glob.glob("tests/cpp_unit/*cc")
+                     glob.glob("tests/cpp_unit/*cc")+\
+                     glob.glob("tests/cpp_unit/*/*/*cc")

     env.Program(target = 'tests/cpp_unit/test_cpp_unit', \
                 source = test_cpp_files, \
#2

Updated by Santos, Edward over 11 years ago

Revision 752 of tracker_devel complies.

#3

Updated by Rogers, Chris over 11 years ago

  • Workflow changed from In Development to Awaiting Merge

Great... await the next merge to trunk...

#4

Updated by Rogers, Chris almost 11 years ago

  • Status changed from In Progress to Closed
  • % Done changed from 50 to 100
  • Workflow changed from Awaiting Merge to Closed

This was merged with the other tracker code

#5

Updated by Rogers, Chris almost 11 years ago

  • Target version changed from Future MAUS release to MAUS-v0.4.1

Also available in: Atom PDF