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

Switch to modern service definition for elasticsearch-full #144

Merged
merged 2 commits into from
Mar 21, 2024

Conversation

PikachuEXE
Copy link
Contributor

Summary: Homebrew has officially deprecated the plist_options command
Example warning message: Warning: Calling plist_options is deprecated! Use service.require_root instead.

See mongodb/homebrew-brew#174 for most details

@cla-checker-service
Copy link

cla-checker-service bot commented Feb 21, 2023

💚 CLA has been signed

@PikachuEXE PikachuEXE marked this pull request as ready for review February 21, 2023 01:10
@PikachuEXE
Copy link
Contributor Author

I have signed the CLA, how to confirm here ._.

@PikachuEXE
Copy link
Contributor Author

Still no one...?

@flanger001
Copy link

Bump @mgreau

@theomelo
Copy link

Bump @marclop @jasontedor

@mischabraam
Copy link

This change is needed more and more...... bump bump

@dskecse
Copy link

dskecse commented Jun 6, 2023

also closes #146

@jasontedor @mgreau is there anything left to approve and merge this PR? could you please have a look?

@drnic
Copy link

drnic commented Jun 29, 2023

Fascinating that a company would stop work on their own homebrew tap and leave it distributing an aging version of their product (v7 vs v8). Is there some history of what happened late 2022 that led @elastic to stop working on this?

@euxx
Copy link

euxx commented Jul 11, 2023

Before the PR be merged, we could apply this PR on locally:

$ cd /usr/local/Homebrew/Library/Taps/elastic/homebrew-tap # or
$ cd /opt/homebrew/Library/Taps/elastic/homebrew-tap # For M1
$ git fetch origin pull/144/head:patch-1
$ git checkout patch-1

@marcosvcloures
Copy link

Keep in mind that the folder for M1 Macs is

$ cd /opt/homebrew/Library/Taps/elastic/homebrew-tap

Copy link

@REFUZIION REFUZIION left a comment

Choose a reason for hiding this comment

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

Just tested these changes, and they function as expected.

@wsmi1289
Copy link

Works for me

@mischabraam
Copy link

Ok, it seems to be working.... what is the wait for?

@drnic
Copy link

drnic commented Oct 30, 2023

They’ve abandoned this project

It’s not mentioned in the downloads docs https://www.elastic.co/downloads/elasticsearch

I guess it’s due to v8 and their move away from their OSS roots?

Copy link

@sdouma sdouma left a comment

Choose a reason for hiding this comment

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

looks good

@sdouma
Copy link

sdouma commented Nov 28, 2023

looks good

I know this does nothing but it makes me laugh

@od-c0d3r
Copy link

od-c0d3r commented Dec 6, 2023

LGTM, ship it

@flanger001
Copy link

They're never going to merge this. We may as well give up now.

@jaredbeck
Copy link

It’s not mentioned in the downloads docs https://www.elastic.co/downloads/elasticsearch

Yeah, the old page (https://www.elastic.co/guide/en/elasticsearch/reference/current/brew.html) is a 404 now.

@dskecse
Copy link

dskecse commented Feb 12, 2024

Now that it's obvious this project has been abandoned and due to this message from its maintainer, I'd like to suggest anyone interested in using the fixed and updated version of ElasticSearch 7.17.14 (as opposed to 7.17.4 from this tap) to check out my Homebrew tap. The [email protected] formula there is mostly a copy of the elasticsearch-full formula from this repo except for the fixes and the version update. To install:

brew tap dskecse/tap
brew install [email protected]

@mgreau
Copy link
Member

mgreau commented Mar 21, 2024

Sorry for the radio silence here, merging.

Copy link
Member

@mgreau mgreau left a comment

Choose a reason for hiding this comment

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

LGTM

@mgreau mgreau merged commit 86ac2cc into elastic:main Mar 21, 2024
@drnic
Copy link

drnic commented Mar 21, 2024

Thanks @mgreau !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.