Skip to content

CMake warnings for SphinxExamples #319

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
hjmjohnson opened this issue Jan 17, 2022 · 3 comments
Closed

CMake warnings for SphinxExamples #319

hjmjohnson opened this issue Jan 17, 2022 · 3 comments

Comments

@hjmjohnson
Copy link
Member

Description

CMAKE Warning

 CMake Warning at /home/johnsonhj/Dashboard/src/ITK/CMake/ITKFactoryRegistration.cmake:230 (message):
   ITK_NO_IO_FACTORY_REGISTER_MANAGER CMake variable is deprecated.  Use
   ITK_NO_IMAGEIO_FACTORY_REGISTER_MANAGER
 Call Stack (most recent call first):
   /home/johnsonhj/Dashboard/src/ITK/CMake/UseITK.cmake:43 (itk_generate_factory_registration)
   /home/johnsonhj/Dashboard/src/ITK/Modules/Remote/SphinxExamples/src/IO/ImageBase/RegisterIOFactories/CMakeLists.txt:10 (include)

 CMake Warning at /home/johnsonhj/Dashboard/src/ITK/CMake/ITKFactoryRegistration.cmake:230 (message):
   ITK_NO_IO_FACTORY_REGISTER_MANAGER CMake variable is deprecated.  Use
   ITK_NO_MESHIO_FACTORY_REGISTER_MANAGER
 Call Stack (most recent call first):
   /home/johnsonhj/Dashboard/src/ITK/CMake/UseITK.cmake:43 (itk_generate_factory_registration)
   /home/johnsonhj/Dashboard/src/ITK/Modules/Remote/SphinxExamples/src/IO/ImageBase/RegisterIOFactories/CMakeLists.txt:10 (include)

 CMake Warning at /home/johnsonhj/Dashboard/src/ITK/CMake/ITKFactoryRegistration.cmake:230 (message):
   ITK_NO_IO_FACTORY_REGISTER_MANAGER CMake variable is deprecated.  Use
   ITK_NO_TRANSFORMIO_FACTORY_REGISTER_MANAGER
 Call Stack (most recent call first):
   /home/johnsonhj/Dashboard/src/ITK/CMake/UseITK.cmake:43 (itk_generate_factory_registration)
   /home/johnsonhj/Dashboard/src/ITK/Modules/Remote/SphinxExamples/src/IO/ImageBase/RegisterIOFactories/CMakeLists.txt:10 (include)

Expected coding style

Sphinx Examples should use current coding conventions.

Actual coding style

CMake warnings are issued.

Versions

Master branch if ITK 2022-01-17 with SphinxExamples
@dzenanz
Copy link
Member

dzenanz commented Jan 17, 2022

@blowekamp and @tbirdso last touched that 😄

@dzenanz dzenanz transferred this issue from InsightSoftwareConsortium/ITK Jan 17, 2022
@dzenanz
Copy link
Member

dzenanz commented Jan 17, 2022

I transferred the issue here, as the problem is in the examples.

@tbirdso
Copy link
Contributor

tbirdso commented Jan 17, 2022

Commit to reference is 099b562

Github says the only reference to the deprecated ITK_NO_IO_FACTORY_REGISTER_MANAGER is here:

ITKSphinxExamples/src/IO/ImageBase/RegisterIOFactories/CMakeLists.txt

Line 9 in 345b1d0

set( ITK_NO_IO_FACTORY_REGISTER_MANAGER 1 )

hjmjohnson added a commit that referenced this issue Jan 17, 2022
@hjmjohnson
COMP: Resolve deprecation warning for SphinxExamples
e7fcf6d 
CMake Warning at CMake/ITKFactoryRegistration.cmake:230 (message):
  ITK_NO_IO_FACTORY_REGISTER_MANAGER CMake variable is deprecated.  Use
  ITK_NO_IMAGEIO_FACTORY_REGISTER_MANAGER

Resolves: #319
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@hjmjohnson @dzenanz @tbirdso