Use foreman-devel module

[nofaralfasi]

No description provided.

[stejskalleos]

So, the error is clear:

RPM build errors:
error: Installed (but unpackaged) file(s) found:
   /usr/share/gems/gems/hammer_cli_foreman_ansible-0.5.0/locale/en/LC_MESSAGES/hammer_cli_foreman_ansible.mo
...

Checking the repository, .mo files are correctly added in the spec file, and we don't ignore them in the .gitignore file.

I don't understand why it's failing, @ofedoren @evgeni any thoughts?

[evgeni]

You probably need a similar change like we had in hammer_rh_cloud here to explicitly include the locale folder:
theforeman/foreman-packaging#9864
theforeman/foreman-packaging#9982

https://github.com/theforeman/foreman-packaging/blob/1b1d77395a48029a4b41f3e95f3cd726adfe1ff6/packages/plugins/rubygem-hammer_cli_foreman_ansible/rubygem-hammer_cli_foreman_ansible.spec#L54-L60

[evgeni]

can you please update this to

Suggested change

	- http://koji.katello.org/releases/yum/foreman-nightly/el8/x86_64/
	- http://yum.theforeman.org/releases/nightly/el8/x86_64/

[evgeni]

technically, for hammer plugins, you do not need the foreman-devel module.

I think it's more correct to use it (and I'd merge this PR), but today the compilation should work with both just fine.

[evgeni]

And it was actually pointed out in #21:

Packit fails the only fails I'm glad to observe :) indication that we finally will have translations 🤦 after we update rpmspec.

→ once you got a new gem released, we update the packaging and things will be green here too.