http://bugzilla.suse.com/show_bug.cgi?id=1091864 http://bugzilla.suse.com/show_bug.cgi?id=1091864#c18 --- Comment #18 from Simon Lees <simonf.lees@suse.com> --- (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 -- You are receiving this mail because: You are on the CC list for the bug.