[Bug 1299179] Review Request: editorconfig - tools for text editors

https://bugzilla.redhat.com/show_bug.cgi?id=1299179

--- Comment #4 from Andy Lutomirski luto@kernel.org --- Comments:

You have the ldconfig scriptlets attached to the wrong package -- they belong on -libs.

This is wrong:

%package -n %{name}-libs Summary: Shared libraries for EditorConfig Group: Documentation Requires: %{name}%{?_isa} = %{version}-%{release}

The main package should require the -libs package (which rpm will do for you by figuring it out from DT_NEEDED), not the other way around. In particular, your current variant breaks multilib installs.

If you fix that, please move the %license to -libs. You may need it in -doc, too.

fedora-review is chugging away at this, and I may have more comments when it finishes.