-
-
Notifications
You must be signed in to change notification settings - Fork 12.4k
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
xrootd: update to 5.6.1 #136074
xrootd: update to 5.6.1 #136074
Conversation
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.
As-is, this will break installs for M1 macOS users, so we should fix that before merging.
Needed for Homebrew#136074.
Maybe bump this to 5.6.1? |
@pthariensflame might be blocked here. Feel free to open a new PR for 5.6.1. |
Yes, please bump to 5.6.1, I've made a fix to find the kernel uuid so the ossp-uuid dependency can be dropped. |
Option ENABLE_CRYPTO has been removed in 5.6, it is now always enabled/required. CMake build system moved to use the new module FindPython.cmake, so need to replace PYTHON_EXECUTABLE with Python_EXECUTABLE.
Please note the changes in build options. Please prefer this to #136388, as python may be configured incorrectly without the change from this pull request. Cheers, |
The failure is during configuration, it doesn't look like this is a problem with the formula itself. |
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. |
I will release 5.6.2 soon, so let's close this one and I will open a new one once the new version is out. Cheers, |
Option ENABLE_CRYPTO has been removed in 5.6, it is now always enabled/required. CMake build system moved to use the new module FindPython.cmake, so need to replace PYTHON_EXECUTABLE with Python_EXECUTABLE.
brew install --build-from-source <formula>
, where<formula>
is the name of the formula you're submitting?brew test <formula>
, where<formula>
is the name of the formula you're submitting?brew audit --strict <formula>
(after doingbrew install --build-from-source <formula>
)? If this is a new formula, does it passbrew audit --new <formula>
?