View Issue Details

IDProjectCategoryView StatusLast Update
0032163CommunityOCCT:Configurationpublic2023-01-17 03:51
Reporterwaebbl Assigned Tobugmaster  
PrioritynormalSeverityminor 
Status closedResolutionfixed 
PlatformLinuxOSGentoo 64 bit 
Product Version7.4.0 
Target Version7.5.0Fixed in Version7.5.0 
Summary0032163: Configuration, CMake - flow control statements not properly nested in vtk.cmake
DescriptionWith OCCT-7.4.0 and the release candidate 1 for cmake-3.20.0 there's an issue in adm/cmake/vtk.cmake with unbalanced nesting of endif() and endforeach() flow control statements.

There's one endif() too much before the endforeach() at line 160, which needs to be after the endforeach().

The issue doesn't not pop-up with earlier versions of cmake. At a quick glance, it seems like it's also solved in the current master HEAD, but I haven't verified this yet.
Steps To ReproduceInstall cmake-3.20.0_rc1 and vtk-8.2.0 (not tested with vtk-9*)
Configure occt with vtk support enabled
Additional information
and documentation updates
See https://bugs.gentoo.org/771300 for the bug report.
https://gitlab.kitware.com/cmake/cmake/-/commit/12f6e37eb79ad66c30269a3f19dfc28a9cb834e2 seems to be the commit which introduced a more restrictive handling for this in cmake.
TagsNo tags attached.
Test case number

Relationships

related to 0031582 closedbugmaster Configuration, CMake - OCCT fails to build with VTK 9.0 

Activities

kgv

2021-02-24 22:25

developer   ~0099125

> At a quick glance, it seems like it's also solved in the current master HEAD
Indeed, I have fixed unexpected endif() within 0031582 (part of OCCT 7.5.0 and current master) - please check:
> Corrected unexpected location of endif() and broken indentation.

waebbl

2021-02-25 00:22

reporter   ~0099126

Thanks for checking.

In Gentoo we don't have OCCT 7.5.0 available yet. It's in my queue though. I'll check back once I have the package ready.

waebbl

2021-11-05 16:12

reporter   ~0105050

The issue didn't repeat with 7.5.0 and 7.6.0_beta and looks to be solved. Thanks for investigating.

vglukhik

2023-01-17 03:51

administrator   ~0112871

Closed.

Issue History

Date Modified Username Field Change
2021-02-24 22:02 waebbl New Issue
2021-02-24 22:02 waebbl Assigned To => bugmaster
2021-02-24 22:23 kgv Relationship added related to 0031582
2021-02-24 22:25 kgv Note Added: 0099125
2021-02-24 22:25 kgv Target Version => 7.5.0
2021-02-24 22:34 kgv Summary CMake flow control statements not properly nested => Configuration, CMake - flow control statements not properly nested in vtk.cmake
2021-02-25 00:22 waebbl Note Added: 0099126
2021-11-05 16:12 waebbl Note Added: 0105050
2021-11-05 20:29 kgv Status new => feedback
2021-11-05 20:29 kgv Resolution open => fixed
2021-11-05 20:29 kgv Fixed in Version => 7.5.0
2023-01-17 03:51 vglukhik Status feedback => closed
2023-01-17 03:51 vglukhik Note Added: 0112871