Opened 8 years ago

Closed 8 years ago

#50998 closed defect (fixed)

root5 install fails on OS X 10.11.4 / Xcode 7.3

Reported by: hapaguy (Brian Kurt Fujikawa) Owned by: cjones051073 (Chris Jones)
Priority: Normal Milestone:
Component: ports Version:
Keywords: Cc: mojca (Mojca Miklavec)
Port: root5

Description

The following consistently fails on multiple Macs running OS X 10.11.4 and Xcode 7.3: 

$ sudo port -sv install root5
...
In file included from /opt/local/var/macports/build/_opt_local_var_macports_sources_rsync.macports.org_release_tarballs_ports_science_root5/root5/work/root-5-34-34/core/base/src/ManualBase4.cxx:18:
In file included from include/TClass.h:42:
include/ThreadLocalStorage.h:228:4: error: thread-local storage is not supported for the current target
   TTHREAD_TLS(T*) ptr = NULL;
   ^
...

I am able to install root6 without problems. Also, I was able to successfully install root5 @5.34.34_3 on another Mac before upgrading to OS X 10.11.4 and Xcode 7.3.

Attachments (2)

main.log​ (589.8 KB) - added by hapaguy (Brian Kurt Fujikawa) 8 years ago.
root5-threadlocalstorage.diff​ (7.0 KB) - added by mojca (Mojca Miklavec) 8 years ago.
An upstream patch to fix an issue with C++11 and thread local storage

Download all attachments as: .zip

Change History (12)

Changed 8 years ago by hapaguy (Brian Kurt Fujikawa)

Attachment: main.log​ added

comment:1 Changed 8 years ago by cjones051073 (Chris Jones)

I see the same, reported upstream.

Seems related to C++11 support. Workaround is either use root6, or if you need root5 install without the c++11 variant (which is enabled by default).

Chris

comment:3 Changed 8 years ago by mf2k (Frank Schima)

Cc: jonesc@… removed
Owner: changed from macports-tickets@… to jonesc@…

comment:4 Changed 8 years ago by mojca (Mojca Miklavec)

Cc: mojca@… added

Cc Me!

comment:5 Changed 8 years ago by mojca (Mojca Miklavec)

Version: 2.3.4

A patch ​has been applied upstream.

Changed 8 years ago by mojca (Mojca Miklavec)

Attachment: root5-threadlocalstorage.diff​ added

An upstream patch to fix an issue with C++11 and thread local storage

comment:6 Changed 8 years ago by mojca (Mojca Miklavec)

I attached a patch without testing it. If it works for you, let me know and I'll commit it.

comment:7 Changed 8 years ago by cjones051073 (Chris Jones)

Can I suggest we wait and see if upstream make a new ROOT5 release (which based on an email discussion I had with Pere sounds like they are happy to do). If that is to come soon, I would rather just wait for that.

( Note there is a third workaround option, which is to install using one of the clang variants... )

comment:8 Changed 8 years ago by cjones051073 (Chris Jones)

I have had confirmation a new ROOT5 release is being prepared today. As soon as it is I will prepare the port update.

comment:9 Changed 8 years ago by cjones051073 (Chris Jones)

​https://trac.macports.org/ticket/51057

comment:10 Changed 8 years ago by mojca (Mojca Miklavec)

Resolution: → fixed
Status: new → closed

Committed in r147477.

Note: See TracTickets for help on using tickets.