SPEC: https://raw.githubusercontent.com/jbadiapa/packages/master/rubygem-fluent-plugin-parser/rubygem-fluent-plugin-parser.spec SRPM: https://github.com/jbadiapa/packages/raw/master/rubygem-fluent-plugin-parser/rubygem-fluent-plugin-parser-0.6.1-1.el7.centos.src.rpm Used this configuration for mock: https://github.com/jbadiapa/packages/blob/master/helperScripts/mock-jbp-centos-7-base-x86_64.cfg The output of the fedora-review can be found: https://github.com/jbadiapa/packages/blob/master/rubygem-fluent-plugin-parser/review.txt
this looks ok to me, please change the requires from rubygem-oj to rubygem(oj). This here should fix the missing provides: https://review.rdoproject.org/r/#/c/7070/
Fixed the rubygem-oj to rubygem(oj) Added some dependencies that were missed.
note, there is no rubygem-rake, no rubygem-bundler, and no rubygem-json in cbs yet.
The three packages are on centos 7 base repository, at least I could see them on: [vagrant@rpmbuild ~]$ yum list rubygem* | egrep -e json -e rake -e bundler rubygem-bundler.noarch 1.7.8-3.el7 @base rubygem-json.x86_64 1.7.7-29.el7 @base rubygem-rake.noarch 0.9.6-29.el7 @base Note: my base repo points at mirrorlist=http://mirrorlist.centos.org/?release=$releasever&arch=$basearch&repo=os&infra=$infra I could see them on cbs as: rubygem-bundler -- http://cbs.centos.org/koji/rpminfo?rpmID=59525 rubygem-json -- http://cbs.centos.org/koji/search?match=glob&type=rpm&terms=rubygem-json* rubygem-rake -- http://cbs.centos.org/koji/search?match=glob&type=rpm&terms=rubygem-rake*
Since you want to have this built against all 0.12.x versions, make the requirement BuildRequires: fluentd < 0.12.31 more something like (either) BuildRequires: fluentd or BuildRequires: fluentd < 0.13 There won't be a fluentd > 0.12.x any time soon in cbs, I guess. Otherwise looks good, the license specified is correct. Please fix the BuildRequires during import.
https://review.rdoproject.org/r/#/c/7478/