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

rospkg identifies macOS sierra and later as "OS X" #209

Open
rotu opened this issue Sep 9, 2020 · 7 comments
Open

rospkg identifies macOS sierra and later as "OS X" #209

rotu opened this issue Sep 9, 2020 · 7 comments

Comments

@rotu
Copy link
Contributor

rotu commented Sep 9, 2020

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.

@dirk-thomas
Copy link
Member

When the logic was implemented the official name was "OS X". Please consider contributing a pull request to update it.

@dirk-thomas dirk-thomas added this to the Untargeted milestone Oct 14, 2020
@erickisos
Copy link

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?

@christophebedard
Copy link
Member

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.

@erickisos
Copy link

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).

@christophebedard
Copy link
Member

It does currently work! See #216 for Big Sur and #229 for Monterey.

@CursedRock17
Copy link

CursedRock17 commented Apr 5, 2023

If it currently works as valid naming conventions, should this issue be closed? Especially since ventura was also recently added without problems.

@christophebedard
Copy link
Member

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 rospkg reports

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants