On 22/05/15 15:40, Josef Stribny wrote:
Hi,

so I looked on the results from running your latest changes and I found
one problem.

It's the GemCheckUsesMacros check, which does not seem to work as expected.

Excerpt from the review:

Ruby:
151 [ ]: Specfile should use macros from rubygem-devel package.
152      Note: The specfile doesn't use these macros: %exclude
153      /usr/share/gems/cache/spring-1.3.6.gem,
154      /usr/share/gems/specifications/spring-1.3.6.gemspec


Thanks for a really good catch! It turned out that this was because spec_file.py didn't parse the %files section correctly.

It's actually a pain, but although rpm implements the spec file parsing there is no interface to retrieve the raw text. I have had a rpm patch reviewed adding an interface, but it was rejected :(

The changes are yet not checked in, but later today I will probably merge the ruby-f22 branch.

Cheers!

--alec

_______________________________________________
ruby-sig mailing list
[email protected]
https://admin.fedoraproject.org/mailman/listinfo/ruby-sig

Reply via email to