Skip to content

Commit

Permalink
consul: fix to remove outdated caveat
Browse files Browse the repository at this point in the history
The caveat for the `-ui-dir` parameter is no longer correct as since 0.7
and above produce this warning if you try and use it:

```
==> If using Consul version 0.7.0 or later, the web UI is included in the
    binary so use ui to enable it
```

Using -agent -dev implies -ui and so the caveat produces an error trying
to start Consul.

As a result the `pkgshare.install` is superfluous and so that's gone
too.

Closes #15926.

Signed-off-by: ilovezfs <ilovezfs@icloud.com>
  • Loading branch information
leepa authored and ilovezfs committed Jul 23, 2017
1 parent 2f43b3e commit 45cae60
Showing 1 changed file with 0 additions and 7 deletions.
7 changes: 0 additions & 7 deletions Formula/consul.rb
Original file line number Diff line number Diff line change
Expand Up @@ -29,16 +29,9 @@ def install
system "make"
bin.install "bin/consul"
zsh_completion.install "contrib/zsh-completion/_consul"
pkgshare.install "ui" => "web-ui"
end
end

def caveats; <<-EOS.undent
You can activate the UI by running
consul with `-ui-dir #{pkgshare}/web-ui`.
EOS
end

plist_options :manual => "consul agent -dev -advertise 127.0.0.1"

def plist; <<-EOS.undent
Expand Down

0 comments on commit 45cae60

Please sign in to comment.