Skip to content

Commit

Permalink
Require /bin/bash in packaging
Browse files Browse the repository at this point in the history
This commit adds a hard requirement to the RPM and Debian packages for
/bin/bash to be present, and adds a note regarding this to the migration
docs.

Relates elastic#18259
  • Loading branch information
jasontedor committed May 11, 2016
1 parent 1d80542 commit 2bf585e
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 0 deletions.
2 changes: 2 additions & 0 deletions distribution/build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -317,6 +317,8 @@ configure(subprojects.findAll { ['deb', 'rpm'].contains(it.name) }) {
preUninstall file("${scripts}/prerm")
postUninstall file("${scripts}/postrm")

requires('/bin/bash')

into '/usr/share/elasticsearch'
fileMode 0644
dirMode 0755
Expand Down
5 changes: 5 additions & 0 deletions docs/reference/migration/migrate_5_0/packaging.asciidoc
Original file line number Diff line number Diff line change
Expand Up @@ -37,4 +37,9 @@ from the tar or zip distributions, and /etc/elasticsearch/jvm.options if install
from the Debian or RPM packages. You can specify an alternative location by setting
the environment variable `ES_JVM_OPTIONS` to the path to the file.

==== /bin/bash is now required

Previously, the scripts used to start Elasticsearch and run plugin
commands only required a Bourne-compatible shell. Starting in
Elasticsearch 5.0.0, the bash shell is now required and `/bin/bash` is a
hard-dependency for the RPM and Debian packages.

0 comments on commit 2bf585e

Please sign in to comment.