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

[backport] Plugin installer support for .tgz file types #6147

Merged
merged 3 commits into from
Feb 8, 2016

Conversation

epixa
Copy link
Contributor

@epixa epixa commented Feb 8, 2016

These file extensions should be interchangeable, so there's no reason
for our plugin installer not to support them. We actually did support
.tgz file types up until we added .zip support.

The first commit is a minor refactor to DRY up the code a bit.

The second commit adds .tgz support.

To test this, confirm that the plugin installer still handles .zip, and .tar.gz,
but also .tgz.

Backports PR #6083 for issue #6074

epixa added 3 commits February 8, 2016 12:37
This should be an in-place refactor without any additional behaviors or
side effects.
These file extensions should be interchangeable, so there's no reason
for our plugin installer not to support them. We actually did support
.tgz file types up until we added .zip support.
The module is generic enough not to require an overly specific name, and
it exposes more things than simply a default function, so it should be a
noun.
@epixa epixa added the backport label Feb 8, 2016
@epixa epixa self-assigned this Feb 8, 2016
@epixa
Copy link
Contributor Author

epixa commented Feb 8, 2016

jenkins, test it

epixa added a commit that referenced this pull request Feb 8, 2016
[backport] Plugin installer support for .tgz file types
@epixa epixa merged commit e769f25 into elastic:4.x Feb 8, 2016
@epixa epixa deleted the 4.x-6074-tgz-plugins branch February 8, 2016 19:41
@elasticsearch-bot
Copy link

courtewing merged this into the following branches!

Branch Commits
4.4 74edb91, 7a23a2b, f31ae6b

elasticsearch-bot pushed a commit that referenced this pull request Feb 8, 2016
This should be an in-place refactor without any additional behaviors or
side effects.

Fixes #6147
elasticsearch-bot pushed a commit that referenced this pull request Feb 8, 2016
These file extensions should be interchangeable, so there's no reason
for our plugin installer not to support them. We actually did support
.tgz file types up until we added .zip support.

Fixes #6147
elasticsearch-bot pushed a commit that referenced this pull request Feb 8, 2016
The module is generic enough not to require an overly specific name, and
it exposes more things than simply a default function, so it should be a
noun.

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

Successfully merging this pull request may close these issues.

2 participants