Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
elasticsearch: do not symlink to plugin script
This commit modifies the Elasticsearch formula to not symlink to the
plugin script so that the path is not polluted with a script named
"plugin".

Closes Homebrew/legacy-homebrew#47941.

Signed-off-by: Dominyk Tiller <dominyktiller@gmail.com>
  • Loading branch information
jasontedor authored and DomT4 committed Jan 11, 2016
1 parent ea786ff commit ec1cead
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions Formula/elasticsearch.rb
Expand Up @@ -61,7 +61,7 @@ def install
(etc/"elasticsearch/scripts").mkdir unless File.exist?(etc/"elasticsearch/scripts")
(libexec/"config").rmtree

bin.write_exec_script Dir[libexec/"bin/*"]
bin.write_exec_script Dir[libexec/"bin/elasticsearch"]
end

def post_install
Expand All @@ -77,6 +77,7 @@ def caveats; <<-EOS.undent
Logs: #{var}/log/elasticsearch/#{cluster_name}.log
Plugins: #{libexec}/plugins/
Config: #{etc}/elasticsearch/
plugin script: #{libexec}/bin/plugin
EOS
end

Expand Down Expand Up @@ -112,7 +113,7 @@ def plist; <<-EOS.undent
end

test do
system "#{bin}/plugin", "list"
system "#{libexec}/bin/plugin", "list"
pid = "#{testpath}/pid"
begin
system "#{bin}/elasticsearch", "-d", "-p", pid, "--path.data", testpath
Expand Down

0 comments on commit ec1cead

Please sign in to comment.