Summary: | app-admin/fluentd ebuild request | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | William Hubbs <williamh> |
Component: | New packages | Assignee: | William Hubbs <williamh> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | hydrapolic, ruby, sluidfoe |
Priority: | Normal | ||
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
URL: | https://www.fluentd.org | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- |
Description
William Hubbs
2018-04-04 17:12:37 UTC
Looking at the dependencies at https://rubygems.org/gems/fluentd We already have dev-ruby/coolio, dev-ruby/http_parser_rb, dev-ruby/msgpack, dev-ruby/tzinfo, and dev-ruby/yajl-ruby. fluentd should not depend on dev-ruby/tzinfo-data. We do not package this because dev-ruby/tzinfo-1* will actually use sys-libs/timezone-data without it, and it makes much more sense to have one consistent source of that stuff. tzinfo-data is a crutch only to be used when your system timezone stuff is completely out of whack. You'll need to edit the gemspec to remove this dependency, most likely. dev-ruby/serverengine and dev-ruby/sigdump are in my overlay. I can move them over to the main tree, but I'll likely need to do some cleanups. It looks like dig_rb isn't needed. It backports the dig method for Arrays and Hashes from ruby 2.3 back to earlier versions. Since ruby 2.2 is going to go away very soon in Gentoo my proposal would be to remove this gem from the gemspec and run without it because it is a no-op for ruby 2.3+ That leaves strptime to be added. dev-ruby/strptime is now available. (In reply to Hans de Graaff from comment #1) > Looking at the dependencies at https://rubygems.org/gems/fluentd > > We already have dev-ruby/coolio, dev-ruby/http_parser_rb, dev-ruby/msgpack, > dev-ruby/tzinfo, and dev-ruby/yajl-ruby. Do all of these have the appropriate use_ruby settings? > fluentd should not depend on dev-ruby/tzinfo-data. We do not package this > because dev-ruby/tzinfo-1* will actually use sys-libs/timezone-data without > it, and it makes much more sense to have one consistent source of that > stuff. tzinfo-data is a crutch only to be used when your system timezone > stuff is completely out of whack. > You'll need to edit the gemspec to remove this dependency, most likely. > > dev-ruby/serverengine and dev-ruby/sigdump are in my overlay. I can move > them over to the main tree, but I'll likely need to do some cleanups. Can you let me know when this can be done? > It looks like dig_rb isn't needed. It backports the dig method for Arrays > and Hashes from ruby 2.3 back to earlier versions. Since ruby 2.2 is going > to go away very soon in Gentoo my proposal would be to remove this gem from > the gemspec and run without it because it is a no-op for ruby 2.3+ > > That leaves strptime to be added. Where is the best place to edit a gemspec in a ruby ebuild? Also, what settings do I need for the ruby-fakegem eclass variables before I inherit the eclass in the fluentd ebuild? Where is the best place in a ruby ebuild to do things like install init scripts? (In reply to William Hubbs from comment #3) > > We already have dev-ruby/coolio, dev-ruby/http_parser_rb, dev-ruby/msgpack, > > dev-ruby/tzinfo, and dev-ruby/yajl-ruby. > > Do all of these have the appropriate use_ruby settings? What do you consider to be appropriate? They should all have at least ruby23 which is the current stable version. > > dev-ruby/serverengine and dev-ruby/sigdump are in my overlay. I can move > > them over to the main tree, but I'll likely need to do some cleanups. > > Can you let me know when this can be done? dev-ruby/serverengine is now moved as well. > Where is the best place to edit a gemspec in a ruby ebuild? all_ruby_prepare() which is a prepare phase that applies to all ruby targets. > Also, what settings do I need for the ruby-fakegem eclass variables before I > inherit the eclass in the fluentd ebuild? This is all documented in the eclass :-) USE_RUBY is mandatory. Note that the eclass adds code for USE=doc and FEATURES=test by default, so you may need to tweak this if the defaults don't work. You can use RUBY_FAKEGEM_RECIPE_DOC and RUBY_FAKEGEM_RECIPE_TEST for that. > Where is the best place in a ruby ebuild to do things like install init > scripts? all_ruby_install(), see for example www-servers/thin. Several of the dependencies do not have ruby25 in USE_RUBY. Can I just add it? (In reply to William Hubbs from comment #5) > Several of the dependencies do not have ruby25 in USE_RUBY. > Can I just add it? Only if all dependencies have it as well and all tests pass. If you want to take a look, here is the list: dev-ruby/coolio dev-ruby/http_parser_rb dev-ruby/msgpack dev-ruby/tzinfo dev-ruby/yajl-ruby Thanks much. (In reply to William Hubbs from comment #7) > If you want to take a look, here is the list: > > dev-ruby/coolio > dev-ruby/http_parser_rb > dev-ruby/msgpack > dev-ruby/tzinfo > dev-ruby/yajl-ruby > > Thanks much. All of these now include ruby25 Thanks much for your help. When I add fluentd, is it ok for me to add ruby as a co-maintainer? I have added thapp-admin/fluentd to the tree with Ruby as a co-maintainer. If you do not want to co-maintain, feel free to remove the team. I will leave this bug open until I hear either way. Thanks a lot for your help. William (In reply to William Hubbs from comment #9) > When I add fluentd, is it ok for me to add ruby as a co-maintainer? That's fine to help out with ruby issues, but if we end up the only maintainer we'll drop that. (In reply to Hans de Graaff from comment #4) > Note that > the eclass adds code for USE=doc and FEATURES=test by default, so you may > need to tweak this if the defaults don't work. USE=doc fails: * Running compile phase for all ... fatal: Not a git repository (or any parent up to mount point /var/tmp) Stopping at filesystem boundary (GIT_DISCOVERY_ACROSS_FILESYSTEM not set). rake aborted! Don't know how to build task 'rdoc' (see --tasks) (See full trace by running task with --trace) FEATURES=test is also broken, looks like it require the currently unpackaged oj gem. 'm doing a high priority work related task at the moment, so feel free to work on those issues for now. I'll look later if you haven't. commit 4340ad2606cb5ffba23e93b01f5c9ae323ff2efd Author: William Hubbs <williamh@gentoo.org> Date: Mon Apr 9 19:52:48 2018 -0500 app-admin/fluentd: new package, data collector and unified logging layer Package-Manager: Portage-2.3.24, Repoman-2.3.6 (END) |