-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
rustup: Add rustup to macports #27197
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
base: master
Are you sure you want to change the base?
Conversation
|
Would this be used to update the rust that's installed by MacPorts? Seems a little redundant. I see there's a note about uninstalling MacPorts rust, but that should probably be an explicit conflict. Still leaves me wondering if this port has any use. |
Yes, it should be an explicit conflict with the Yes, |
|
I do like the idea of having rustup - but I'd prefer to use it to install/update a non-macports rust. Which is already what I do on my machine under |
Yes, that's what this is aiming to do. Macports installs |
|
Ok, so this won't actually conflict with rust from macports, it's just a different way to manage your rust install. I don't think I have any major objections here. I wonder if the note should be reworded - no need to actually uninstall rust from macports if you're careful and/or set your PATH correctly. Just a bit of thinking here, as nice as the "theory" of having a package manager do everything might be, these days it makes sense to defer a lot of things to other package tools. Package managers like MacPorts can provide rustup, which you can then use to get cargo and manage all things Rust. Likewise for uv and Python... |
Yes! Correct! I removed the note about conflicting with macport's |
reneeotten
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@stuartwesterman there hasn't been any activity/progress on this PR for a while. Reading through the comments there doesn't appear to be clear objections against adding this to MacPorts. So I'd like to get this merged if you are able/willing to take care of a few final comments. Apologies for the delay...
|
I've addressed the review comments. Thank you! |
Description
This adds a
rustupmacports port.Type(s)
Tested on
macOS 15.2
Xcode 16.2 / Command Line Tools x.y.z
Verification
Have you
port lint?sudo port -vst install?