View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0004407 | Slicer4 | Core: Building (CMake, Superbuild) | public | 2017-08-04 08:57 | 2017-09-21 18:10 |
Reporter | pieper | Assigned To | jcfr | ||
Priority | normal | Severity | block | Reproducibility | have not tried |
Status | closed | Resolution | fixed | ||
Platform | osx on factory | OS | OS Version | ||
Product Version | |||||
Target Version | Fixed in Version | Slicer 4.7.0 | |||
Summary | 0004407: new libarchive version won't built on factory clang | ||||
Description | Upgraded libarchive [1] causes build error [2] on factory macs in this code [3]. Apparently the assumption in the code that all versions of clang will have __builtin_bswap16 is not true for the version of clang on the factory. https://github.com/Slicer/Slicer/commit/10faee88ed79ddf00d268c0354c96be23563b340 | ||||
Steps To Reproduce | Only appears on the factory clang, not on my local mac which uses clang : Apple LLVM version 8.1.0 (clang-802.0.42) | ||||
Additional Information | It seems LHA code is related to this format: https://en.wikipedia.org/wiki/LHA_(file_format) if we can't fix the build we could go back to having our own fork of libarchive and just turn off this format. | ||||
Tags | No tags attached. | ||||
I think the easiest will be to stub out the LHA support in our own fork. That way we don't need to change the factory clang, which is likely to have other side effects. @jcfr if you agree I can do take care of this. |
|
Agreed. Since this is related to older compiler and we will move the build to a newer system soon, disabling LHA is the way to go. Thanks for your help. Would be great if you could submit a patch. Here is the convention for Slicer specific branches on our fork. See https://www.slicer.org/wiki/Documentation/Nightly/Developers/Versioning#Patches_for_tagged_release Let me know if you have any questions, |
|
@dzenanz Could you confirm that the current Slicer trunk also build on your workstation ? |
|
Thanks for the pointers - I'll try to work on it this afternoon or maybe this weekend. Of course if anyone else wants to take this on feel free! I hope it's just this one issue. Just to confirm Jc the factory is still using the version of clang as described in [1] and [2]? Independent of libarchive we'll probably be wanting to update that at some point. Have we tested if it works with vtk8, c++11, and Qt5? [1] https://www.slicer.org/wiki/Documentation/Nightly/Developers/Factory |
|
I will have a look later this morning and keep you posted
Yes
Slicer will be moved to a newer macOS. We testing this build on "factory-south" instead of "factory" |
|
Fix integrated in r26206 |
|
Thanks Jc! |
|
You are welcome. Note: I have only tested on Linux and MacOSX. And I am confident it will not break windows. If there are issues, I will follow up on Monday |
|
Date Modified | Username | Field | Change |
---|---|---|---|
2017-08-04 08:57 | pieper | New Issue | |
2017-08-04 08:57 | pieper | Status | new => assigned |
2017-08-04 08:57 | pieper | Assigned To | => jcfr |
2017-08-04 10:10 | pieper | Note Added: 0015016 | |
2017-08-04 10:18 | jcfr | Note Added: 0015017 | |
2017-08-04 10:29 | jcfr | Note Added: 0015018 | |
2017-08-04 10:30 | jcfr | Note Edited: 0015018 | View Revisions |
2017-08-04 10:30 | jcfr | Note Edited: 0015018 | View Revisions |
2017-08-04 10:57 | pieper | Note Added: 0015019 | |
2017-08-04 11:04 | jcfr | Note Added: 0015020 | |
2017-08-04 16:37 | jcfr | Status | assigned => resolved |
2017-08-04 16:37 | jcfr | Resolution | open => fixed |
2017-08-04 16:37 | jcfr | Fixed in Version | => Slicer 4.7.0 |
2017-08-04 16:37 | jcfr | Note Added: 0015021 | |
2017-08-04 16:46 | pieper | Note Added: 0015023 | |
2017-08-04 16:47 | jcfr | Note Added: 0015024 | |
2017-09-21 18:10 | pieper | Status | resolved => closed |