Comment # 18 on bug 1091864 from Simon Lees

(In reply to Marcus R�ckert from comment #12)
> so the thing that is broken with the cmake version of the %make_jobs macro
> (compared to the version in the Qt package)
> 
> is 
> 
> %make_jobs -C build/something
> 
> the params end up after the the %endif of the cmake make_jobs macro
> 
> the main difference between the 2 macros is enabling verbose mode and
> support for ninja. the first step is already handled by the __builder macro.
> but the verbose flag is why we need the %if, so my proposal would be the
> following change.
> 
> ```
> # added the %_make_output_sync from %make_build here as it is useful.
> %__builder_verbose \
> %if "%__builder" == "%__make" \
>     %{?_make_output_sync} VERBOSE=1 \\\
> %else \
>     -v \\\
> %endif \
> %{nil}
> 
> %make_jobs %__builder \\%__builder_verbose %{?_smp_mflags}
> ```
> 
> this makes Fabian's usecase work. now the question is on how do we make this
> change in a backwards compatible way. in ideal world we would just move the
> whole %make_jobs macro to the rpm macros file but that would require a
> maintenance update for rpm and everyone building against :Update.
> 
> or we just copy the macro around and have a comment that asks people to keep
> the macro in sync between the different places. thoughts?

Making it in a backwards compatible way is going to be hard because really the
package should be shared with the SLE one we could split the macro out into a
separate package "cmake-rpm-config" but that would still cause any cmake
package to be listed for a rebuild which maintenance might not be too happy
about, but it would probably be a way forward for Leap 15.1