-
Notifications
You must be signed in to change notification settings - Fork 97
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
rospkg identifies macOS sierra and later as "OS X" #209
Comments
When the logic was implemented the official name was "OS X". Please consider contributing a pull request to update it. |
Hey everyone! I just want to know what's the difference of being detected as OSX vs macOS? I mean, in the functional sense does it have some difference? |
I don't think so, but it does need to be "osx" for other things to work. The rosdep lists, for example: https://github.com/ros/rosdistro/blob/master/rosdep/osx-homebrew.yaml. So if we were to change it to "macos" we'd need to change it everywhere. |
So, we can keep this as OSX but first, we need to be sure that this is working on macOS sierra and higher, that's right? I mean, i saw the #210 issue and I just want to be sure if it's related based on the id of OS, but it sounds like a different beast to me (related, but different). |
If it currently works as valid naming conventions, should this issue be closed? Especially since ventura was also recently added without problems. |
From what I understand, this issue is not about supporting newer versions. It's about the name of the OS being "macOS" now and not "OS X" like |
Internally and in stack traces, rospkg identifies macOS sierra as "OS X". The string identifying mac when using this platform is "osx" which is increasingly incorrect. The official name since Sierra is "macOS" so "OS X" is incorrect/misleading.
The text was updated successfully, but these errors were encountered: