-
-
Notifications
You must be signed in to change notification settings - Fork 60
Add note about possible Homebrew conflict #591
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
Conversation
Add note about HomeBrew and PATH variable adjustment.
|
Caution Review failedThe pull request is closed. WalkthroughAdded instructional guidance to MacRelease.md documentation recommending adding MacPorts bin directory to PATH to prevent conflicts with HomeBrew on macOS systems. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes
Poem
✨ Finishing touches🧪 Generate unit tests (beta)
📜 Recent review detailsConfiguration used: CodeRabbit UI Review profile: CHILL Plan: Pro 📒 Files selected for processing (1)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 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.
Pull request overview
This PR adds documentation about potential conflicts between Homebrew and MacPorts on macOS, specifically guiding users to prioritize MacPorts binaries in their PATH variable.
- Adds a note warning about Homebrew utility conflicts with MacPorts
- Provides a command example to prepend MacPorts bin directory to the PATH variable
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
|
||
| - Currently, Mac builds are done on a High Sierra virtual machine, and the binary packages for High Sierra are used, so that is the mininum OS version for the MacOSX Minsky release. Arm64 builds are done on Ventura for the same reason. | ||
| - Ports installs of Python do not create a python3 package for pkg-config, rather they are a more specific name like python-3.12.pc. Use find to find the location of the specific .pc file, and link it to python3.pc in the same place, which will allow python builds to work. | ||
| - If you have HomeBrew installed, it's utilities may conflict with that of MacPorts. To avoid problems in that case, iit is useful to add the MacPorts bin directory to the front of the PATH variable: |
Copilot
AI
Dec 7, 2025
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.
There's a double 'i' in "iit" - should be "it".
| - If you have HomeBrew installed, it's utilities may conflict with that of MacPorts. To avoid problems in that case, iit is useful to add the MacPorts bin directory to the front of the PATH variable: | |
| - If you have HomeBrew installed, it's utilities may conflict with that of MacPorts. To avoid problems in that case, it is useful to add the MacPorts bin directory to the front of the PATH variable: |
|
|
||
| - Currently, Mac builds are done on a High Sierra virtual machine, and the binary packages for High Sierra are used, so that is the mininum OS version for the MacOSX Minsky release. Arm64 builds are done on Ventura for the same reason. | ||
| - Ports installs of Python do not create a python3 package for pkg-config, rather they are a more specific name like python-3.12.pc. Use find to find the location of the specific .pc file, and link it to python3.pc in the same place, which will allow python builds to work. | ||
| - If you have HomeBrew installed, it's utilities may conflict with that of MacPorts. To avoid problems in that case, iit is useful to add the MacPorts bin directory to the front of the PATH variable: |
Copilot
AI
Dec 7, 2025
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.
The possessive form should use "its" (without an apostrophe) instead of "it's" (which is a contraction for "it is"). Should read: "its utilities may conflict".
| - If you have HomeBrew installed, it's utilities may conflict with that of MacPorts. To avoid problems in that case, iit is useful to add the MacPorts bin directory to the front of the PATH variable: | |
| - If you have HomeBrew installed, its utilities may conflict with that of MacPorts. To avoid problems in that case, iit is useful to add the MacPorts bin directory to the front of the PATH variable: |
Add note about HomeBrew and PATH variable adjustment.
This change is
Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.