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

Remove unused environment variable ELASTICSEARCH_MAJOR #107

Closed
wants to merge 4 commits into from
Closed

Remove unused environment variable ELASTICSEARCH_MAJOR #107

wants to merge 4 commits into from

Conversation

antonbormotov
Copy link
Contributor

No description provided.

@antonbormotov
Copy link
Contributor Author

Anyone from maintainers, how do I fix failing build?

@tianon
Copy link
Member

tianon commented Jun 17, 2016

Failure on 5.0 is unfortunately expected (#98). 😭

This change will need to be made in Dockerfile.template though:

ENV ELASTICSEARCH_MAJOR %%ELASTICSEARCH_MAJOR%%
👍

@antonbormotov
Copy link
Contributor Author

antonbormotov commented Jun 20, 2016

I removed the line from Docker.template and re-ran: ./update.sh 1.5 1.6 1.7 2.0 2.1 2.2 2.3 5.0

@tianon
Copy link
Member

tianon commented Jul 7, 2016

Merged manually via 2bfa452 👍

Thanks!

@tianon tianon closed this Jul 7, 2016
tianon added a commit to infosiftr/stackbrew that referenced this pull request Jul 7, 2016
- `drupal`: 8.1.4
- `elasticsearch`: remove unused `ELASTICSEARCH_MAJOR` (docker-library/elasticsearch#107)
- `java`: add more bits to `PATH` for `alpine` variants
- `memcached`: 1.4.28 (docker-library/memcached#10)
- `percona`: 5.7.13
- `php`: use `/etc/apache2/envvars` and thus the default, stock Debian Apache configuration (docker-library/php#251)
- `rabbitmq`: `hipe_compile` (docker-library/rabbitmq#91), 3.6.3 (docker-library/rabbitmq#93), `RABBITMQ_SSL_VERIFY` + `RABBITMQ_SSL_FAIL_IF_NO_PEER_CERT` support (docker-library/rabbitmq#94)
1gtm pushed a commit to appscode-images/elasticsearch that referenced this pull request Feb 14, 2024
* Contexts for 8.1.2

This commit was created by the elastic-dockerfiles-publisher.

* GH Actions for 8.1 branch

Co-authored-by: mgreau <greaumaxime@gmail.com>
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.

2 participants