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

Build task for os packages creates target #7485

Merged
merged 2 commits into from
Jun 16, 2016
Merged

Conversation

epixa
Copy link
Contributor

@epixa epixa commented Jun 16, 2016

The target directory is created by the archives task, but it is not
created by the ospackages task, so if you do not have a target
directory and try to skip archives, the build will fail.

The target directory is created by the archives task, but it is not
created by the ospackages task, so if you do not have a target
directory and try to skip archives, the build will fail.
@jbudz
Copy link
Member

jbudz commented Jun 16, 2016

LGTM

@epixa epixa changed the title Build task for creating os packages creates target Build task for os packages creates target Jun 16, 2016
@jbudz
Copy link
Member

jbudz commented Jun 16, 2016

I just merged #7484 where this was introduced, this'll need to go to 4.x now too

@epixa
Copy link
Contributor Author

epixa commented Jun 16, 2016

Whoops. This should really be using the target directory configuration rather than a hardcoded string. Archives should a well. I'll update this PR.

The target directory was being created based on a hardcoded string
rather from the target directory configuration value.
@epixa
Copy link
Contributor Author

epixa commented Jun 16, 2016

Fixed

@jbudz
Copy link
Member

jbudz commented Jun 16, 2016

👍

@epixa epixa merged commit c46172d into elastic:master Jun 16, 2016
@epixa epixa deleted the ospackagestarget branch June 16, 2016 16:52
@epixa epixa added v5.0.0 and removed v5.0.0 labels Jun 28, 2016
airow pushed a commit to airow/kibana that referenced this pull request Feb 16, 2017
Build task for os packages creates target

Former-commit-id: c46172d
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants