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

xrootd: update to 5.6.1 #136074

Closed
wants to merge 1 commit into from
Closed

xrootd: update to 5.6.1 #136074

wants to merge 1 commit into from

Conversation

amadio
Copy link
Contributor

@amadio amadio commented Jul 7, 2023

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.

  • Have you followed the guidelines for contributing?
  • Have you ensured that your commits follow the commit style guide?
  • Have you checked that there aren't other open pull requests for the same formula update/change?
  • Have you built your formula locally with brew install --build-from-source <formula>, where <formula> is the name of the formula you're submitting?
  • Is your test running fine brew test <formula>, where <formula> is the name of the formula you're submitting?
  • Does your build pass brew audit --strict <formula> (after doing brew install --build-from-source <formula>)? If this is a new formula, does it pass brew audit --new <formula>?

Formula/xrootd.rb Outdated Show resolved Hide resolved
@carlocab carlocab mentioned this pull request Jul 7, 2023
6 tasks
Copy link
Member

@carlocab carlocab left a 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.

carlocab added a commit to carlocab/homebrew-core that referenced this pull request Jul 7, 2023
@pthariensflame
Copy link
Contributor

Maybe bump this to 5.6.1?

@carlocab
Copy link
Member

@pthariensflame might be blocked here. Feel free to open a new PR for 5.6.1.

@pthariensflame pthariensflame mentioned this pull request Jul 12, 2023
@amadio
Copy link
Contributor Author

amadio commented Jul 12, 2023

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.
@amadio
Copy link
Contributor Author

amadio commented Jul 12, 2023

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,

@amadio amadio changed the title xrootd: enable erasure coding support, update build options xrootd: update to 5.6.1 Jul 12, 2023
@amadio
Copy link
Contributor Author

amadio commented Jul 12, 2023

The failure is during configuration, it doesn't look like this is a problem with the formula itself.

@amadio amadio requested a review from carlocab August 2, 2023 07:35
@github-actions
Copy link
Contributor

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.

@github-actions github-actions bot added the stale No recent activity label Aug 23, 2023
@amadio
Copy link
Contributor Author

amadio commented Aug 23, 2023

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,

@amadio amadio closed this Aug 23, 2023
@chenrui333
Copy link
Member

#143163

@chenrui333 chenrui333 added superseded PR was replaced by another PR build failure CI fails while building the software labels Sep 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build failure CI fails while building the software stale No recent activity superseded PR was replaced by another PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants