View Issue Details

Related ChangesetsJump to NotesJump to History
IDProjectCategoryView StatusDate SubmittedLast Update
0003816Slicer4Core: Base Codepublic2014-08-22 09:382018-03-02 11:01
Reporterlassoan Assigned Tojcfr  
PrioritynormalSeverityminorReproducibilityalways
Status closedResolutionfixed 
Product Version 
Target VersionSlicer 4.7.0Fixed in VersionSlicer 4.7.0 
Summary0003816: Debug-mode build fails on Windows if SimpleITK is enabled
Description

Slicer build fails on Windows in debug mode if SimpleITK is enabled due to some missing python debug files:

fatal error LNK1104: cannot open file 'python27_d.lib'
error PRJ0019: A tool returned an error code from "Performing build step for 'SimpleITK'"

Workaround: disable building of SimpleITK for debug-mode builds by configuring Slicer passing

-DSlicer_USE_SimpleITK:BOOL=OFF

from the command line.

Or adding a boolean entry "Slicer_USE_SimpleITK to CMake gui.

TagsNo tags attached.

Activities

lassoan

lassoan

2014-08-22 09:39

developer   ~0012412

See proposed solution at
http://slicer-devel.65872.n3.nabble.com/Slicer-build-option-quot-Slicer-USE-SimpleITK-quot-now-enabled-by-default-td4032162.html#a4032168
blowekamp

blowekamp

2016-10-12 09:07

developer   ~0014170

This is still a problem. There are 3 option:

1) Disable SimpelITK in debug mode ( see above ).
2) If Slicer is compiled in debug mode, then Python should be compiled in debug mode ( this is what SimpleITK expects )
3) Last I check VTK has a header file ( vtkPython.h? ) which mucks with the preprocessor to allow the mixing of a debug Module with a release Python library. This approach seems to be a bit of an hack, and is unclear if it is "supported". This could be done for SimpleITK if some one is motivated to address this issue.
jcfr

jcfr

2017-05-15 11:14

administrator   ~0014480

Fixed in r26010
See http://viewvc.slicer.org/viewvc.cgi/Slicer4?view=revision&revision=26010
jcfr

jcfr

2017-06-08 01:51

administrator   ~0014682

Fix committed to master branch.

Related Changesets

Import 2017-06-07 23:51:09: master f77b8075

2017-05-15 10:44:24

jcfr

Details Diff 		ENH: Update SimpleITK (Support for debug build)

Fixes 0003816

Tested-by: Andras Lasso <lasso@queensu.ca>

$ git shortlog v1.0rc1..6433028
Bradley Lowekamp (171):
Update CircleCi CMake version to 2.7.2
Add CMake variable CMAKE_DL_LIBS to list for superbuild propagation
Adding initially configured Sphinx documentation
Add logo, title and under development note
disable imgmath due to compatibility issues with readthedocs
Merge topic 'AddInitialSphinxDocumentation'
Use constant reference for FileName
Merge topic 'IOFileNameByReference'
Merge topic 'AddCMAKE_DL_LIBToSuperbuildPropagation'
Merge topic 'UpdateCircleCiCMakeVersion'
Replace 0 magic number with defined constant for WallClock
Use namespace qualification for sitkWallClock
Add RandomSeed header to wrap sitkWallClock enum
Use sitkWallClock enum in test and examples
Silence CMake message for CSharp targets
Update ITK 4.11 along release branch
Merge topic 'UpdateITK4.11-release'
Merge topic 'SilenceCSharpTargets'
Add clamping to optimizer's GetCurrentIteration method
Use target definition over directory includes
Remove required flags and replace with compile options property
Adding CMake macro to migrate cache variable from SITK to SimpleITK
Add missing library target to compile definitions
Remove SITK_EXPRESS_INSTANTIATEDPIXELS option
Migrate CMake options from SITK_ prefix to SimpleITK_
Migrate SITK_FORBID_DOWNLOADS option to SimpleITK_FORBID_DOWNLOADS
Migrate option to SimpleITK_USE_VIRTUALENV from SITK prefix
Migrate to SimpleITK_LUA_BUILD_EXECUTABLE CMake option
Migrate to SimpleITK_GIT_PROTOCOL
Migrate SITK_ prefix for LUA_EXECUTABLE to SimpleITK_ prefix
Change internal variable to prefered SimpleITK_PYTHON project prefix
Prefer SimpleITK over SIMPLEITK for CMake variables
Merge topic 'AddressMSVCConversionWarning'
Correct logic for running C++11 requirement
Merge topic 'RemoveFontFromDoxygen'
Merge topic 'UseCMakeTargetProperties'
Merge topic 'DeprecateSITKCMakeOptionPrefix'
Add missing EXPORTS define to BasicFilters1 library
Merge topic 'UseCMakeTargetProperties'
Organize json scipts into a directory
Adding script to update json files with ITK module names
Adding itk_module field to json files
Separate BasicFilter library creation into function
Add generated cxx file to list with ITK module names
Create BasicFilter libraries corresponding to ITK modules
Remove manual BasicFilters1 library
Rename library by removing BasicFilters form ITK modular libraries.
Update JSONValidate.py path for doxygen cmake code
Include CMake legacy module when used
Fix typeo in argument name for restoring LUA_EXECUTABLE cache variable
Add required virutalenv version for producing wheels
Add CMake option for --plat-name for distutils
Merge topic 'DeprecateSITKCMakeOptionPrefix'
Merge topic 'FixLuaExecutableCacheArgument'
Merge topic 'UseITKModuleForLibraries'
Merge topic 'RemoveDoubleBackticksForDoxygen'
Adding comment to CMake function for building basic filter libraries
Correct ITK Module some transform initializes belong to
Fix basic filter source list to always remove modular sources
Set target properties for using ITK modules instead of including use file
Reduce the set of ITK modules required to build SimpleITK
Do not process JSON filter if ITK modules is not enabled.
Merge topic 'AddPythonPlatNameOption'
Remove bad loop over ITK modules passed to sitk_target_use_itk
Add library directories if needed when using an ITK module
Use local variables for json file path
Delete CMake all variables from cache for list of generated source
Do not add examples as tests unless BUILD_TESTING is enabled
Cast enum to uint32 to avoid signed integer conversion warning
Add clamping to all optimizer's GetCurrentIteration wrapping
Add CMake option for --plat-name for distutils
Merge branch 'release'
Up required Swig version to 3 in find_package command
Merge topic 'UseEnumConstantForWallClockSentinel'
Merge topic 'RefactorExampleIntoExampleDirectories'
Merge topic 'AddressMSVCConversionWarning'
Merge topic 'UseITKModulesToEnableLibraries'
Merge topic 'RequireSwig3'
Add error exception for pep425 wheel construction
Merge topic 'gdcmSeriesFileNames'
Update ITK Superbuild version along ITK 4.11 release
Use CTest script to drive CircleCI builds
Prefer CircleCI environment variables over repository specifics
Merge topic 'UpdateITKOn4.11Release'
Merge topic 'AddErrorExcpetionForPEP425Warning'
Merge topic 'CircleCiUseCTest'
Only install numpy into virtualenv when testing is enabled
Fix correct syntax for CMake configuration variable substitution
Set SITK_UNDEFINED_SYMBOLS_ALLOWED in language projects
CircleCI builds on next branch should be the Continuous model
Correct ctest to ingore pep425 error not warning
Binary files were accidentally added and not md5 hash file
Merge topic 'dicomTagsSpaceBug'
Merge topic 'CMakeUndefVarName'
Merge topic 'SetUndefinedSymbolsForLanguageProjects'
Merge topic 'CircleCINextBuildAreContinuous'
Merge topic 'FixBaselineBinaryAdded'
Merge topic 'NumpyOnlyForTesting'
Merge topic 'AddErrorExcpetionForPEP425Warning'
Merge topic 'UpdateResampleDocumentation'
Update CircleCI yaml to use ctest build flags
Use CMake variable to execute cmake command
Merge topic 'UpdateCircleCIWithCTestBuildFlags'
Merge topic 'FixCMakeEchoCommand'
Add sha512 hash data link files generated ContentLinkSynchronization
Adding script to synchronize md5 and sha512 data link hashes
Fix path to in source ExternalData with Superbuild
Add Kitware's Girder as alternate download site for SHA512 data
Fix the Doxygen mark up for the HashImageFilter
Fix setting ITK Doxygen tag in configuration file
Adding updated ExternalData module with support for multiple hashes
Updating filter converge with ITK 4.11 release
Merge topic 'FixHashFilterDocumentation'
Merge topic 'bugExhaustiveOptimizer'
Merge topic 'UpdateFilterConverage'
Merge topic 'FixITKDoxygenTags'
Add optional old_name to sitk_legacy_naming macro
Rename Superbuild CMake variable to SimpleITK_USE_SYSTEM_*
Remove code for old ITK_WRAPPING option.
Merge topic 'AddSHA512ExternalDataSupport'
Tweaks to R Documentation downloads
Remove debugging message
Merge topic 'FixSeeNodeIndentation'
Merge topic 'RegeneratePythonDocstrings'
Merge topic 'RDocFromMidas'
Merge topic 'AddSHA512ExternalDataSupport'
Rename git default variable to exclude from Superbuild propagation
Merge topic 'UpdateSuperbuildUseSytemVariableNames'
Merge topic 'RenameSimpleITKGitDefaultVar'
Backport ExternalData.cmake for CMake version 3.0
Define RETURN_SELF macro to void when wrapping with SWIG
Fix missing occurrence of rename CMake variable
Only add version suffix and properties to libraries.
Remove AnaconaBuild script
Merge topic 'AddSHA512ExternalDataSupport'
Merge topic 'RenameSimpleITKGitDefaultVar'
Merge topic 'RemoveAnacondaBuildScript'
Add exception for trying to write DICOM with ImageSeriesWriter
Correct the CMake DATA syntax for using a series
Must use a ImageSeriesReader to read a DICOM series
Add Dockerfiles and script to build manylinux complaint wheels
Merge topic 'AddPythonManyliunxDockerScripts'
Merge topic 'OnlyVersionLibraries'
Merge topic 'RDocEquationFixes'
Merge topic 'AddExceptionForWritingDICOMWithSeriesWriter'
Merge topic 'dicomWrite'
Merge topic 'RDocUpdateWarningsFixed'
Merge topic 'DisableWrappedReturnSelf'
Update setup.py classifiers
Merge topic 'UpdatePythonAndJavaDocs'
Merge topic 'UpdateSetup.pyClassifiers'
Add variable options to manylinux docker builds
Merge topic 'UpdateManyLinuxScripts'
Fix the SimpleITK install directory used for superbuild SimpleITK Examples
Use the install directory for include interface
Merge branch 'FixSimpleITKVersionExampleSuperbuild-for-release' into release
Merge branch 'FixIncludeInstallInterface-for-release' into release
Use find_package for install and build path for build for ITK
Pass ITK_DIR to the Examples Superbuild
Updating Superbuild ITK version along release toward 4.11.1
Merge branch 'FixITKTargetsPathInConfig-for-release' into release
Relax tolerance for DiscreteGaussianImageFilter_short test
Merge branch 'RelaxShortDiscreteGaussianTestTolerance-for-release' into release
Relax tolerance on MagnitudeAndPhaseToComplex from i686 results
Release tolerance for LandmarkBasedInitialization for i686
Update to 4.11.1 tagged release
Use exact reported version for required ITK version.
Merge branch 'FixSimpleITKConfigITKVersion-for-release' into release
Merge branch 'AdjustToleraceFori686Debug-for-release' into release
Merge branch 'UpdateITK4.11.1-for-release' into release
ENH: Support building debug SimpleITK against release Python

Dave Chen (4):
Removed font from Doxygen
Fix the indentation of See nodes in the docs
PythonDocstrings.i regenerated to update docs
Update PythonDocstring.i and JavaDoc.i

David T. Chen (2):
Removed double backticks for Doxygen
BUG: Check for undefined varName variable

Richard Beare (3):
Support for downloading archive of R documentation from midas
Improved support for converting equations for R
update R documentation md5

Ziv Yaniv (8):
BUG: removed the loadPrivateTags parameter from GetGDCMSeriesFileNames.
Improved documentation for GetGDCMSeriesFileNames.
BUG: Comparison of image type tag (0008|0008) was incorrect.
Corrected spelling mistake and added details for Resample functions.
BUG: Wrong metric value returned from Exhaustive optimizer.
Adding support for writing DICOM series.
Example moved to new directory structure and added test.
Adding forward declaration for smart pointer.

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

git-svn-id: http://svn.slicer.org/Slicer4/trunk@26010 3bd1e089-480b-0410-8dfb-8563597acbee
mod - SuperBuild/External_SimpleITK.cmake Diff File

Issue History

Date Modified Username Field Change
2014-08-22 09:38 lassoan New Issue
2014-08-22 09:38 lassoan Status new => assigned
2014-08-22 09:38 lassoan Assigned To => jcfr
2014-08-22 09:39 lassoan Note Added: 0012412
2014-08-22 09:50 jcfr Assigned To jcfr => blowekamp
2014-08-22 09:50 jcfr Target Version => Slicer 4.5.0-1
2015-11-02 11:27 jcfr Target Version Slicer 4.5.0-1 => Slicer 4.6.0
2016-10-12 03:56 jcfr Target Version Slicer 4.6.0 => Slicer 4.7.0
2016-10-12 09:07 blowekamp Note Added: 0014170
2017-05-15 11:14 jcfr Note Added: 0014480
2017-05-15 11:14 jcfr Status assigned => resolved
2017-05-15 11:14 jcfr Fixed in Version => Slicer 4.7.0
2017-05-15 11:14 jcfr Resolution open => fixed
2017-06-08 01:51 jcfr Changeset attached => Slicer master f77b8075
2017-06-08 01:51 jcfr Note Added: 0014682
2017-06-08 01:51 jcfr Assigned To blowekamp => jcfr
2018-03-02 11:01 jcfr Status resolved => closed