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

patchelf.rb reading for everyone :) #8241

Merged
merged 2 commits into from Aug 10, 2020

Conversation

rmNULL
Copy link
Contributor

@rmNULL rmNULL commented Aug 6, 2020

  • Have you followed the guidelines in our Contributing document?
  • Have you checked to ensure there aren't other open Pull Requests for the same change?
  • Have you added an explanation of what your changes do and why you'd like us to include them?
  • Have you written new tests for your changes? Here's an example.
  • Have you successfully run brew style with your changes locally?
  • Have you successfully run brew tests with your changes locally?

enable patchelf.rb reading for everyone.

Those who don't wish to use patchelf.rb can do so by
exporting HOMEBREW_NO_PATCHELF_RB=1 to their env.

@rmNULL rmNULL force-pushed the patchelfrb-reading-for-everyone branch from e576256 to 9f09425 Compare August 6, 2020 23:36
@rmNULL
Copy link
Contributor Author

rmNULL commented Aug 7, 2020

cc @sjackman @woodruffw @MikeMcQuaid

@@ -1,7 +1,7 @@
# frozen_string_literal: true

# enables experimental readelf.rb, patchelf support.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not experimental any more. Let's just remove all usage of HOMEBREW_PATCHELF_RB and HOMEBREW_NO_PATCHELF_RB and delete the legacy code. We can revert this PR if we need to turn it off after that.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Agreed with this; could you make these changes @rmNULL?

@rmNULL rmNULL force-pushed the patchelfrb-reading-for-everyone branch from 9f09425 to 1a7175b Compare August 8, 2020 01:19
@rmNULL rmNULL force-pushed the patchelfrb-reading-for-everyone branch from 1a7175b to 0437193 Compare August 8, 2020 01:25
@woodruffw
Copy link
Member

CI failure looks unrelated:

1st Try error in ./test/cmd/services_spec.rb:4:
expected block to output "Warning: No services available to control with brew services\n" to stderr, but output "Warning: Calling Homebrew.args is deprecated! Use args = <command>_args.parse and pass args along the call chain instead.\nPlease report this issue to the homebrew/services tap (not Homebrew/brew or Homebrew/core), or even better, submit a PR to fix it:\n /private/tmp/homebrew-tests-20200808-15182-19bc5zq/prefix/Library/Taps/homebrew/homebrew-services/lib/services_cli.rb:65\n\nWarning: No services available to control with brew services\n"

@MikeMcQuaid MikeMcQuaid merged commit 510b21f into Homebrew:master Aug 10, 2020
@MikeMcQuaid
Copy link
Member

Thanks @rmNULL!

@BrewTestBot BrewTestBot added the outdated PR was locked due to age label Dec 19, 2020
@Homebrew Homebrew locked as resolved and limited conversation to collaborators Dec 19, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
outdated PR was locked due to age
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants