View Issue Details
|ID||Project||Category||View Status||Date Submitted||Last Update|
|0004247||Slicer4||Core: Extensions||public||2016-08-24 11:38||2017-11-28 15:14|
|Product Version||Slicer 4.5.0-1|
|Target Version||Slicer 4.6.0||Fixed in Version||Slicer 4.7.0|
|Summary||0004247: Extensions do not get built on Windows if any test in the dependent extensions fail|
Which is undesired, because if an extension has 60 tests and one fails (may it be a widget test that fails because the module widget is 5 pixels wider than the set threshold), it does not mean it's not operational, and it doesn't make sense not to even start building the extensions that depend on it.
The extensions that motivated this issue are GelDosimetry and FilmDosimetry, both of which depend on SlicerRT. Sometimes one of the lengthy python self tests fail, or a widget test due to width.
|Tags||No tags attached.|
In Extensions\CMake\SlicerBlockBuildPackageAndUploadExtension.cmake this line causes the target build step to fail:
Build step (in which ctest.exe returns with error code if ctest_test finds that a test fails):
How ctest.exe knows that ctest_test failed (from parsing XML output file)? How it could be prevented to return with error code?
For reference: SlicerRT ticket created for the workaround that needs to be applied due to this issue
This should be fixed by
For future reference the associated github topic as:
I re-enabled the failing SlicerRT tests to see if this change fixes the issue and we get dependent extensions to build even with the failing tests.
Unfortunately the windows build for dependent tests still does not start if any test fails.
I'm increasing the priority of this issue, as no progress has been made over the lsat four months, and it considerably decreases the usability of the extension dependency mechanism.
Thanks for the update.
I wonder how different is the SlicerRt extension configuration from the current tests.
The extension build system tests are added here:
And for all test cases, extension B and C depend on extension A which has one failing test.
A mock web server also confirm that all CDash and Midas queries are submitted as expected.
Is SlicerRt test crashing/hanging forever and preventing the dependent extensions from being built ?
Thanks for investigating! Okay, so the issue is not so trivial and seems SlicerRT specific.
I don't think that the any test hangs or crashes. If I have a module widget generic test fail because the module is wider than the threshold it's checked against, the dependent extensions don't build.
This should be fixed in 26223
Unfortunately nothing changed in this manner. I re-enabled the SlicerRT tests that failed or were prone to failing, and the dependent extension builds do not start on Windows, for example
I believe this has been fixed in r26291
Looking at the dashboard of August 22nd, the GelDosimetry is configured, built and tested.
This is great! Thanks a lot for fixing this.
Sorry I just saw this update somehow. I confirm that the dependent extensions build. Thanks a lot for the fix! Closing the issue.
I have to reopen this issue. I re-enabled the tests that fail (generic tests with module dependencies) or tend to fail (very long python self test), and the dependent extensions son't build on Windows. See this dashboard filtered to SlicerRT and its dependent extensions. You'll see that SlicerRT builds on Windows, but none of the dependencies build on Windows.
Import 2017-06-07 23:51:09: master c226f731
2016-10-12 23:50:38Details Diff
|BUG: Extension build system: Ignore extension CTest return code. Fixes 0004247
This commit ensures each "EXTENSION_UPLOAD_COMMAND" responsible to configure,
build, test and package any given extension is completely "sandboxed" by
using a wrapper script.
This is required because simply setting the RETURN_VALUE parameter
to the ctest_test() command is not enough to avoid the script from
exiting with error code.
# The following wrapper script is required to workaround issue 0004247
# and avoid the all extension build from failing if a test of
# extension fail.
# Note that as soon as CMake >= 3.6.7 is released, it should be possible
# to remove the wrapper script and simply specify CAPTURE_CMAKE_ERROR
# ctest_test option.
# See https://cmake.org/cmake/help/v3.7/command/ctest_test.html
Tested-by: Isaiah Norton <firstname.lastname@example.org>
Tested-by: Johan Andruejol <email@example.com>
Tested-by: Nicole Aucoin <firstname.lastname@example.org>
Tested-by: Steve Pieper <email@example.com>
Reported-by: Csaba Pinter <firstname.lastname@example.org>
Reported-by: Andras Lasso <email@example.com>
Thanks: All of the above
git-svn-id: http://svn.slicer.org/Slicer4/trunk@25439 3bd1e089-480b-0410-8dfb-8563597acbee
|mod - Extensions/CMake/SlicerBlockBuildPackageAndUploadExtensions.cmake||Diff File|
Import 2017-06-07 23:51:09: master 41df9c1a
2016-10-12 23:50:40Details Diff
|BUG: Extension build system: Fix use of wrapper script on windows. Fixes 0004247
This commit ensures the build configuration is passed to the wrapped
It fixes error like this one:
8: SetMakeCommand:C:\D\Support\CMake-3.5.0-rc3\bin\cmake.exe --build . --config "$(Configuration)"
8: SetCTestConfiguration:MakeCommand:C:\D\Support\CMake-3.5.0-rc3\bin\cmake.exe --build . --config "$(Configuration)"
8: Build project
8: MakeCommand:C:\D\Support\CMake-3.5.0-rc3\bin\cmake.exe --build . --config "$(Configuration)"
8: Run command: "C:\D\Support\CMake-3.5.0-rc3\bin\cmake.exe" "--build" "." "--config" "$(Configuration)"
8: Microsoft (R) Build Engine version 12.0.40629.0
8: [Microsoft .NET Framework, version 4.0.30319.34209]
8: Copyright (C) Microsoft Corporation. All rights reserved.
8: Build started 10/12/2016 10:57:46 PM.
8: Project "C:\D\N\Slicer-1-build\Slicer-build\Extensions\CMake\Testing\build_with_upload-build\TestExtA-build\ALL_BUILD.vcxproj" on node 1 (default targets).
8: Project "C:\D\N\Slicer-1-build\Slicer-build\Extensions\CMake\Testing\build_with_upload-build\TestExtA-build\ALL_BUILD.vcxproj" (1) is building "C:\D\N\Slic
er-1-build\Slicer-build\Extensions\CMake\Testing\build_with_upload-build\TestExtA-build\ZERO_CHECK.vcxproj" (2) on node 1 (default targets).
8: C:\Program Files (x86)\MSBuild\Microsoft.Cpp\v4.0\V120\Microsoft.Cpp.Platform.targets(61,5): error MSB8013: This project doesn't contain the Configuration an
d Platform combination of $(Configuration)|x64. [C:\D\N\Slicer-1-build\Slicer-build\Extensions\CMake\Testing\build_with_upload-build\TestExtA-build\ZERO_CHECK.v
git-svn-id: http://svn.slicer.org/Slicer4/trunk@25440 3bd1e089-480b-0410-8dfb-8563597acbee
|mod - Extensions/CMake/SlicerBlockBuildPackageAndUploadExtensions.cmake||Diff File|
|mod - Extensions/CMake/SlicerBlockUploadExtension.cmake||Diff File|
|2016-08-24 11:38||pinter||New Issue|
|2016-08-24 11:38||pinter||Status||new => assigned|
|2016-08-24 11:38||pinter||Assigned To||=> jcfr|
|2016-09-11 10:42||lassoan||Note Added: 0014088|
|2016-09-13 10:45||pinter||Note Added: 0014093|
|2016-10-12 11:45||jcfr||Target Version||Slicer 4.5.1 => Slicer 4.6.0|
|2016-10-13 00:05||jcfr||Note Added: 0014194|
|2016-10-13 00:05||jcfr||Status||assigned => resolved|
|2016-10-13 00:05||jcfr||Fixed in Version||=> Slicer 4.6.0|
|2016-10-13 00:05||jcfr||Resolution||open => fixed|
|2016-10-13 01:24||jcfr||Product Version||Slicer 4.5.1 => Slicer 4.5.0-1|
|2016-10-16 10:30||pinter||Note Added: 0014197|
|2016-10-16 10:30||pinter||Status||resolved => confirmed|
|2016-12-23 16:57||pinter||Note Added: 0014282|
|2016-12-23 16:57||pinter||Priority||normal => high|
|2016-12-23 18:32||jcfr||Note Added: 0014283|
|2017-01-05 11:19||pinter||Note Added: 0014285|
|2017-06-10 08:51||jcfr||Changeset attached||=> Slicer master 41df9c1a|
|2017-06-10 08:51||jcfr||Changeset attached||=> Slicer master c226f731|
|2017-08-10 12:56||jcfr||Note Added: 0015034|
|2017-08-10 14:59||jcfr||Status||confirmed => resolved|
|2017-08-10 14:59||jcfr||Fixed in Version||Slicer 4.6.0 => Slicer 4.7.0|
|2017-08-10 14:59||jcfr||Note Added: 0015041|
|2017-08-21 10:36||pinter||Status||resolved => assigned|
|2017-08-21 10:36||pinter||Note Added: 0015066|
|2017-08-21 10:37||pinter||Summary||Extensions do not get built if any test in the dependent extensions fail => Extensions do not get built on Windows if any test in the dependent extensions fail|
|2017-08-22 23:29||jcfr||Status||assigned => resolved|
|2017-08-22 23:29||jcfr||Note Added: 0015069|
|2017-08-23 09:36||lassoan||Note Added: 0015072|
|2017-11-16 09:34||pinter||Status||resolved => closed|
|2017-11-16 09:34||pinter||Note Added: 0015406|
|2017-11-21 10:23||pinter||Status||closed => feedback|
|2017-11-21 10:23||pinter||Resolution||fixed => reopened|
|2017-11-21 10:23||pinter||Note Added: 0015425|
|2017-11-28 15:14||pinter||Note Edited: 0015425||View Revisions|