Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Gestalt selector gestaltSystemVersion deprecation warning #8270

Closed
cyberduck opened this issue Oct 21, 2014 · 2 comments
Closed

Gestalt selector gestaltSystemVersion deprecation warning #8270

cyberduck opened this issue Oct 21, 2014 · 2 comments

Comments

@cyberduck
Copy link
Collaborator

@cyberduck cyberduck commented Oct 21, 2014

2c875c6 created the issue

Gestalt has been deprecated since 10.8 and therefore it will return the wrong version number on 10.10.
On Yosemite, Gestalt returns 10.9 instead of 10.10.

How to reproduce:

  1. Install OSX 10.10 Yosemite
  2. Start Cyberduck (any version)
  3. Watch the console logs

Starting Cyberduck on Yosemite OSX 10.10 leads to the following warnings in the console:

Cyberduck Version 4.5.3 (15087)

21/10/14 13:41:34,762 Cyberduck[356]: WARNING: The Gestalt selector gestaltSystemVersion is returning 10.9.0 instead of 10.10.0. Use NSProcessInfo's operatingSystemVersion property to get correct system version number.
Call location:
21/10/14 13:41:34,763 Cyberduck[356]: 0   CarbonCore                          0x00007fff90e5bdc3 ___Gestalt_SystemVersion_block_invoke + 113
21/10/14 13:41:34,763 Cyberduck[356]: 1   libdispatch.dylib                   0x00007fff8cf20c13 _dispatch_client_callout + 8
21/10/14 13:41:34,763 Cyberduck[356]: 2   libdispatch.dylib                   0x00007fff8cf20b26 dispatch_once_f + 117
21/10/14 13:41:34,763 Cyberduck[356]: 3   CarbonCore                          0x00007fff90e044da _Gestalt_SystemVersion + 987
21/10/14 13:41:34,764 Cyberduck[356]: 4   CarbonCore                          0x00007fff90e040c7 Gestalt + 144
21/10/14 13:41:34,764 Cyberduck[356]: 5   librococoa.dylib                    0x000000010ecec89c UKGetSystemVersionComponents + 33
21/10/14 13:41:34,764 Cyberduck[356]: 6   librococoa.dylib                    0x000000010ece84d7 -[UKCrashReporter checkForCrash] + 155

Cyberduck Version 4.5.3 (15415)

21/10/14 13:53:57,436 Cyberduck[4499]: WARNING: The Gestalt selector gestaltSystemVersion is returning 10.9.0 instead of 10.10.0. Use NSProcessInfo's operatingSystemVersion property to get correct system version number.
Call location:
21/10/14 13:53:57,438 Cyberduck[4499]: 0   CarbonCore                          0x00007fff90e5bdc3 ___Gestalt_SystemVersion_block_invoke + 113
21/10/14 13:53:57,438 Cyberduck[4499]: 1   libdispatch.dylib                   0x00007fff8cf20c13 _dispatch_client_callout + 8
21/10/14 13:53:57,438 Cyberduck[4499]: 2   libdispatch.dylib                   0x00007fff8cf20b26 dispatch_once_f + 117
21/10/14 13:53:57,438 Cyberduck[4499]: 3   CarbonCore                          0x00007fff90e044da _Gestalt_SystemVersion + 987
21/10/14 13:53:57,438 Cyberduck[4499]: 4   CarbonCore                          0x00007fff90e040c7 Gestalt + 144
21/10/14 13:53:57,438 Cyberduck[4499]: 5   librococoa.dylib                    0x000000010e73fddb UKGetSystemVersionComponents + 33
21/10/14 13:53:57,438 Cyberduck[4499]: 6   librococoa.dylib                    0x000000010e73b9df -[UKCrashReporter checkForCrash] + 135

Application is working properly, but I think messages in console should be avoided.

@cyberduck
Copy link
Collaborator Author

@cyberduck cyberduck commented Oct 21, 2014

@dkocher commented

The problem is that NSProcessinfo operatingSystemVersion is only available in OS X v10.10 and later.

Loading

@cyberduck
Copy link
Collaborator Author

@cyberduck cyberduck commented Dec 8, 2014

@dkocher commented

In c83e767.

Loading

@cyberduck cyberduck closed this Dec 8, 2014
@iterate-ch iterate-ch locked as resolved and limited conversation to collaborators Nov 26, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants