ENH: added ability to interactively remove fiber lines.

git-svn-id: http://svn.slicer.org/Slicer4/trunk@21986 3bd1e089-480b-0410-8dfb-8563597acbee
COMP: Prevent windows build error shortening DisplayableManager directory

"MRMLDisplayableManager" sub directory associated with module has been
changed into "MRMLDM" to prevent issue related to path length limitation.

See http://slicer-devel.65872.n3.nabble.com/No-Windows-nightly-package-since-May-2-tp4028642p4028646.html

git-svn-id: http://svn.slicer.org/Slicer4/trunk@21987 3bd1e089-480b-0410-8dfb-8563597acbee
BUG: copy opacity when creating a copy of a color node

The MRML color logic was using the the color node copy 
method that just copied the name and rgb, switched it 
to use the one that also copies opacity. 
Expanded the testing to exercise the copy and test the 
results.

Issue #3114


git-svn-id: http://svn.slicer.org/Slicer4/trunk@21988 3bd1e089-480b-0410-8dfb-8563597acbee
COMP: Consistently specify optional external project CMake arguments

Easier maintenance of the build system by using the variable
"EXTERNAL_PROJECT_OPTIONAL_ARGS" to accumulate optional CMake arguments.

git-svn-id: http://svn.slicer.org/Slicer4/trunk@21989 3bd1e089-480b-0410-8dfb-8563597acbee
STYLE: Fix indent in SimpleITK and teem external project

git-svn-id: http://svn.slicer.org/Slicer4/trunk@21990 3bd1e089-480b-0410-8dfb-8563597acbee
COMP: Fix CMake warning related to unused CMAKE_BUILD_TYPE variable

git-svn-id: http://svn.slicer.org/Slicer4/trunk@21991 3bd1e089-480b-0410-8dfb-8563597acbee
COMP: Fix unused CMake variable occurring when configuring DCMTK on windows

Warning reported on the dashboard:

12> Manually-specified variables were not used by the project:
12>    DCMTK_FORCE_FPIC_ON_UNIX
12>    DCMTK_WITH_WRAP

git-svn-id: http://svn.slicer.org/Slicer4/trunk@21992 3bd1e089-480b-0410-8dfb-8563597acbee
BUG: Set the color range when changing number of colors

When copying a color table node and reducing the number
of colors in it, the color table range wasn't updated.
This led to problems in the Editor module when the selected
drawing integer wasn't matched up to the expected color.
Now when the number of colors is set on a user editable
color node, the range is set on the color look up table
and on the GUI slider at the same time.
Also updated the number of colors spin box to only trigger
a reset of the number of colors on the node on the editingFinished signal
as othewise cutting down a table from 300 to 50 colors could
empty out the table and refill it with black rather than 
creating a subset of the original table.

Issue #3115



git-svn-id: http://svn.slicer.org/Slicer4/trunk@21993 3bd1e089-480b-0410-8dfb-8563597acbee
COMP: Fix unused CMake variable Teem_VTK_TOOLKITS_IPATH warning

Following r21963, Teem_VTK_MANGLES is set to OFF. For that reason,
the variable Teem_VTK_TOOLKITS_IPATH is not required anymore.

git-svn-id: http://svn.slicer.org/Slicer4/trunk@21994 3bd1e089-480b-0410-8dfb-8563597acbee
COMP: Remove unused Slicer superbuild boolean variable

The variable "Slicer_BUILD_Extensions" originally introduced
by commit r21550 is not used.

git-svn-id: http://svn.slicer.org/Slicer4/trunk@21995 3bd1e089-480b-0410-8dfb-8563597acbee
COMP: Fix CMake warning related to WITH_MEMCHECK CMake variable

Commented the variable waiting the build system is updated to support
running of the test using valgrind.

git-svn-id: http://svn.slicer.org/Slicer4/trunk@21996 3bd1e089-480b-0410-8dfb-8563597acbee
COMP: Fix unused CMake variable warning related to DOCUMENTATION_* variable

git-svn-id: http://svn.slicer.org/Slicer4/trunk@21997 3bd1e089-480b-0410-8dfb-8563597acbee
COMP: Do not compile zlib using WINAPI calling convention

Following commit r21963, windows 32-bit couldn't be be built because
of missing symbol error associated with python, vtk and itk. Ensuring
ZLIB_WINAPI was defined systematically by configuring it inside "zlibDllConfig.h"
was solving python link error but was then causing further link error while
building ITK, VTK  (conflicting calling convention __fastcall vs __stdcall)

Since originally the flag has been added to be able to compile LibArchive
and since now LibArchive supports zlib build with or without ZLIB_WINAPI,
there is NOW no need to force the flag.

git-svn-id: http://svn.slicer.org/Slicer4/trunk@21998 3bd1e089-480b-0410-8dfb-8563597acbee
BUG: set LUT range widget to trigger setting range on LUT

Updating the GUI element will trigger a signal to update the look up table
on the color node.
Renamed a method from setNumberOfColors to updateNumberOfColors as it wasn't
taking an argument.
TODO: set up an observation on the look up table on the color node

Issue #3115



git-svn-id: http://svn.slicer.org/Slicer4/trunk@21999 3bd1e089-480b-0410-8dfb-8563597acbee
COMP: On MacOSX, do not create duplicate of MRMLIDIOPlugin and MGHIOPlugin

This fix MacOSX warning "/path/to/plugin is already loaded"

Both MRMLIDIOPlugin and MGHIOPlugin were copied so that it exists both
a "dylib" and a "so" version of the file. This was initially committed
on the 2007-09-27 to work around a limitation of ITK preventing it from
loading plugin having ".dylib" extension. See Slicer3 r4454 [1]

Then 2 months later, on the 2007-11-19, a fix has been pushed in ITK so
that plugin having extension "dylib" are considered. See ITK commit 28b4a6ba [2]

This commit account for the ITK fix by removing the code in charge of
copying the plugin.

Lessons learned: When implementing a hack, it is a good practice to report
a bug upstream to track the limitation and also reference the bug in a
comment starting with either "# HACK" for CMake file or "// HACK" for
c/cpp files. It would give a hint to developer re-factoring the code.

[1] http://viewvc.slicer.org/viewvc.cgi/Slicer3?view=revision&revision=4454
[2] https://github.com/Kitware/ITK/commit/28b4a6ba
ENH: Move MacOSX configuration code for CLI factory to a better place.

As a slight improvement, the path is also appended only if it exists.
ENH: Simplify logic by appending path to CLI factory only if they exist.
STYLE: Do not check for version of CMake before including "BundleUtilities"

Since minimum required CMake version for Slicer is 2.8.9, this message
is not needed anymore.
COMP: Ensure Python library directory is considered when fixing up package

When the expression "@SLICER_PYTHON_LIB_DIR@" has been introduced
in commit r21252 [1], it was already resolving to an empty
string because since r15463 [2] the file SlicerBlockCTKAppLauncherSettings is
included in the Applications/SlicerApp/CMakeLists.txt.

Both the fixup script and the launcher settings have been updated to
use the variable "PYTHON_LIBRARY_PATH" introduced in the top-level
CMakeLists.txt by commit r16822 [3]

To prevent such error from happening again, a convenient macro named
"_fixup_paths_append" has been introduced. It basically check if the path
passed as parameter exists and append it to the list of paths. If not,
it throw an error.

[1] http://viewvc.slicer.org/viewvc.cgi/Slicer4?view=revision&revision=21252
[2] http://viewvc.slicer.org/viewvc.cgi/Slicer4?view=revision&revision=15463
[3] http://viewvc.slicer.org/viewvc.cgi/Slicer4?view=revision&revision=16822
COMP: Ensure library paths used for BundleFixup are valid

The helper macro "_fixup_paths_append" is now used consistently to append
path to the "libs_path" variable.

Let's also note the following updates:

 - CMAKE_RUNTIME_OUTPUT_DIRECTORY is not used anymore, instead
${Slicer_BUILD_DIR}/@Slicer_BIN_DIR@ is used.

 - Path "@Slicer_BINARY_DIR@/@Slicer_QtPlugins_DIR@" is not considered
anymore since it didn't exist in the build tree. Instead, the plugin path
are composed using "${Slicer_BUILD_DIR}/@Slicer_BIN_DIR@"