#42456 closed update (fixed)
ROOT : Update to 5.34.15
| Reported by: | cjones051073 (Chris Jones) | Owned by: | macports-tickets@… |
|---|---|---|---|
| Priority: | Normal | Milestone: | |
| Component: | ports | Version: | |
| Keywords: | haspatch maintainer | Cc: | mattiafrancescomoro@… |
| Port: | root |
Description
Bump ROOT to 5.34.15
Upstream has also switch the default GUI to a native Cocoa version for OSX 10.8 and newer, so to stay consist this patch does the same. The Cocoa backend seems much more stable now in my tests, and also works when ROOT is imported via python (Previously a stumbling block, as it didn't work). So making it the default now seems reasonable...
Chris
Attachments (1)
Change History (4)
Changed 10 years ago by cjones051073 (Chris Jones)
comment:1 Changed 10 years ago by mf2k (Frank Schima)
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
| Version: | 2.2.1 |
comment:2 follow-up: 3 Changed 10 years ago by mojca (Mojca Miklavec)
It's a pity that you forgot to include the patch for #41572 when increasing the version.
But why not simply making +cocoa default on all versions of OS X? Yes, I know you argued about consistency, but a while back the argument was that cocoa was misbehaving. I doubt that it now only misbehaves on 10.7 and older?
comment:3 Changed 10 years ago by cjones051073 (Chris Jones)
Replying to mojca@…:
It's a pity that you forgot to include the patch for #41572 when increasing the version.
I didn't forget, it was intentional. I only patch things when there is no other option, and in this case there is; Stick with the builtin library. I'll revert once it cleanly builds without the need for any patching.
But why not simply making
+cocoadefault on all versions of OS X? Yes, I know you argued about consistency, but a while back the argument was thatcocoawas misbehaving. I doubt that it now only misbehaves on 10.7 and older?
I prefer not to second guess upstream. ROOT decided to only enable it by default on 10.8 onwards. Presumably there is a reason for this, so I just do the same. It is after all only the default, users are free to change this to whatever they want themselves.
Chris
r116978. Thanks!