COMP: Support build with Slicer_BUILD_DICOM_SUPPORT "on" then "off"

In commit r21598, the option Module_ITKIODCMTK was conditionally added
depending on the value of Slicer_BUILD_DICOM_SUPPORT.

To support build tree where Slicer_BUILD_DICOM_SUPPORT is first enabled
and then disabled, the options should be directly added so that ITK
is rebuilt with the module turned off.

git-svn-id: http://svn.slicer.org/Slicer4/trunk@22368 3bd1e089-480b-0410-8dfb-8563597acbee
BUG: update WriteCLI to fill a vector of strings, honor mutiple = false

Because Markups define multiple markups (with potentially multiple points) in one node,
it was writting multiple instances of a string + coordinates to the string stream passed
in to WriteCLI. This causes a problem for the parsing of the arguments in CLIs such as
FiducialRegistration which needed a separate string for each point.
With this change, the markup points are writen into a vector of strings, one string
for each point in each Markup.
Added the multipleFlag to be passed to WriteCLI, and if it's false, only write the
first selected markup to the output vector. If that markup has multiple points, it will
write all of those points.
Updated the Annotation control point to use the new WriteCLI signature, but because
of the way the points are written, it will only write the first point in a multi point
annotation (such as a ruler) if multiple is false, so it ignores it and the
CLI module logic takes care of ensuring that only the first child in an
annotation hierachy is written out. Write out a warning message if this is the case.
Added a test for FiducialRegistration, updated the Markups node test 1 to test the single point flag.
Removed trailing white spaces.

Issue #1910 #3345 #3340



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

From: Jean-Christophe Fillion-Robin <jchris.fillionr@kitware.com>

git-svn-id: http://svn.slicer.org/Slicer4/trunk@22370 3bd1e089-480b-0410-8dfb-8563597acbee
BUG: Update SimpleFilters for Marksup Fiducials

ceb1a61 ENH: Update single Point and index types for Markups
4241d7b ENH: update the filter's seed list to use Markups nodes

From: Bradley Lowekamp <blowekamp@mail.nih.gov>

git-svn-id: http://svn.slicer.org/Slicer4/trunk@22371 3bd1e089-480b-0410-8dfb-8563597acbee
STYLE: Refactor function onShowToolButtonTextToggled re-using existing code

See #2164

git-svn-id: http://svn.slicer.org/Slicer4/trunk@22372 3bd1e089-480b-0410-8dfb-8563597acbee
BUG: Apply "ShowToolButtonText" settings to mainwindows on restart

While the slot qSlicerSettingsStylesPanel::onShowToolButtonTextToggled()
is properly called when the settings is registered at startup, it failed
to disable/enable the toolButton property because the mainwindow is not
instantiated.

This commit ensure this is the case in the main windows readSettings()
function.

Fixes #2164

git-svn-id: http://svn.slicer.org/Slicer4/trunk@22373 3bd1e089-480b-0410-8dfb-8563597acbee
COMP: Fix building of Slicer on MacOSX when cronjob is used

Extending PATH with /usr/sbin will allow the OpenSSL config script to
properly find 'sysctl' executable so that the configuration succeed.

git-svn-id: http://svn.slicer.org/Slicer4/trunk@22374 3bd1e089-480b-0410-8dfb-8563597acbee
COMP: Suggest "insteadof" git config only for ITKv3.

Since ITKv4 doesn't use submodule (see fab42d0ef) it only applies to ITKv3.

Fixes #2751

git-svn-id: http://svn.slicer.org/Slicer4/trunk@22375 3bd1e089-480b-0410-8dfb-8563597acbee
COMP: Remove redundant option MRML_USE_Teem and only use MRML_USE_vtkTeem

Fixes #2571

git-svn-id: http://svn.slicer.org/Slicer4/trunk@22376 3bd1e089-480b-0410-8dfb-8563597acbee
ENH: In Recently loaded menu, rename "Clear Menu" into "Clear History"

See #1669

git-svn-id: http://svn.slicer.org/Slicer4/trunk@22377 3bd1e089-480b-0410-8dfb-8563597acbee
ENH: Increase maximum size for Recently loaded menu

See #1669

git-svn-id: http://svn.slicer.org/Slicer4/trunk@22378 3bd1e089-480b-0410-8dfb-8563597acbee