Enable our test_density tool for compilationBRL-CAD
Status: ClosedTime to complete: 48 hrs Mentors: SeanTags: build system, cmake, compile, X11

BRL-CAD provides more than 400 applications.  When we migrated from an Autotools build system to CMake, some tools were not converted and some were disabled long ago for a variety of reasons.  One such tool is one called test_density.

This task requires enabling that tool for compilation, properly defining the build system logic in our CMakeLists.txt file (to link any needed libraries), and fixing any compilation warnings that result when compiling.

Note that the tool is in an incomplete state of development.  This task requires adding whatever little bits are needed so that the program will compile cleanly.  If you cannot get it to compile, you must provide a build log with explanation of the problem(s) you encountered.

Mark the tool as a no-install application, see src/libbn/CMakeLists.txt file for an example on how to do that (bntester).

Code:

  • src/libanalyze/CMakeLists.txt
  • src/libanalyze/test_density.c

Provide a patch of all your changes.

Uploaded Work
File name/URLFile sizeDate submitted
libanalyze.patch5.8 KBDecember 17 2012 18:56 UTC
Comments
Cezaron December 17 2012 18:36 UTCTask Claimed

I would like to work on this task.

Sean on December 17 2012 18:44 UTCTask Assigned

This task has been assigned to Cezar. You have 48 hours to complete this task, good luck!

Cezaron December 17 2012 18:56 UTCReady for review

The work on this task is ready to be reviewed.

Cezaron December 17 2012 18:58 UTCCommit r54076

Commit r54076

Sean on December 17 2012 18:59 UTCmissed one thing

Nicely done, Cezar.  You just missed deleting the commented-out line in CMakeLists.txt that still reference the test program. ;)


 

Sean on December 17 2012 18:59 UTCTask Closed

Congratulations, this task has been completed successfully.