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

app-admin/logstash-bin: version bump to 2.3.2 #1369

Closed
wants to merge 1 commit into from

Conversation

hydrapolic
Copy link
Contributor

Package-Manager: portage-2.2.28

Package-Manager: portage-2.2.28
@wraeth
Copy link
Contributor

wraeth commented Apr 28, 2016

I have had a look at this ebuild and have a couple of minor notes for future reference:

  • EAPI could be bumped to EAPI6
  • IUSE does not need to be defined if there are no flags
  • DEPEND does not need to be defined if there are no build-time dependencies

Otherwise, this is good - my testing was all successful - and once the QA check comes back I will push.

Thanks!

@hydrapolic
Copy link
Contributor Author

Thanks @wraeth, I'll incorporate the changes into the next bump :)

@hydrapolic hydrapolic deleted the logstash branch May 19, 2016 06:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants