fedorov
(Andrey Fedorov)
August 11, 2017, 2:03pm
1
I checked the dashboard today, and I found that one of our extension had a (new) build error :
Error copying file "@rpath/libITKFactoryRegistration.dylib" to
"/.../DCMQI-build/dcmqi-build/_CPack_Packages/Darwin/TGZ/26228-macosx-amd64-DCMQI-
git95356d4-2017-07-27/Slicer.app/Contents/lib/Slicer-4.7/libITKFactoryRegistration.dylib".
However, despite of this error, the extension package is available, and it seems to work in Slicer.
@jcfr could this potentially be related to the fixes discussed in No nightly Mac binary since August 1 - #11 by jcfr ?
jcfr
(Jean Christophe Fillion Robin (Kitware))
August 12, 2017, 7:26pm
2
Look like a legitimate error associated with packaging. I will have a look.
To clarify, packaging is technically not part of the “build”, it happens afterward.
That said, we drive the packaging using ctest_build and submit the additional results like they were a build output. That is packaging error are reported like “build” errors.
fedorov
(Andrey Fedorov)
August 12, 2017, 7:44pm
3
Thanks for looking into this @jcfr !
Note that there were no changes to the extension source code between the latest successful build/package (Aug 1, http://slicer.cdash.org/buildSummary.php?buildid=1073259 ) and the date when failure was first observed (Aug 9, http://slicer.cdash.org/buildSummary.php?buildid=1076906 ). In between there were no Mac builds of extensions at all. So it looks like the problem is on the Slicer side.
fedorov
(Andrey Fedorov)
August 15, 2017, 1:40pm
4
@jcfr the problem I mentioned above persists: http://slicer.cdash.org/viewBuildError.php?buildid=1079987
I see the same problem for a large number of other C++ extensions. Is there a plan to address this regression in the Slicer core?
jcfr
(Jean Christophe Fillion Robin (Kitware))
August 21, 2017, 10:27pm
5
One of the MacOSX issue is now addressed by r26296 , I will follow up on the dcmqi error tomorrow.
1 Like
jcfr
(Jean Christophe Fillion Robin (Kitware))
August 24, 2017, 7:45am
6
macOS packaging issue will be addressed by
opened 10:31PM - 12 Mar 20 UTC
closed 10:31PM - 12 Mar 20 UTC
This issue was created automatically from an original Mantis Issue. Further discussion may take place here.
1 Like
fedorov
(Andrey Fedorov)
August 24, 2017, 2:35pm
7
Thank you @jcfr for tracking this down!
I am sure you will let us know if any updates will be needed for the extensions!
jcfr
(Jean Christophe Fillion Robin (Kitware))
August 24, 2017, 6:40pm
8
This should be fixed in r26301
jcfr
(Jean Christophe Fillion Robin (Kitware))
August 25, 2017, 5:56pm
9
And here is a follow up commit addressing the remaining issues in Slicer. See r26305
Corresponding changes in upstream projects are here: