<div dir="ltr"><div dir="ltr"><div class="gmail_default" style="font-size:small"><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Aug 3, 2020 at 3:48 PM Jan Stancek <<a href="mailto:jstancek@redhat.com">jstancek@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><br>
----- Original Message -----<br>
> This commit adds explicit build rules, the main motivation are recent<br>
> build failures caused by library orderings. To fix that this commit<br>
> introduces LTPLDLIBS special variable that is passed to linker before<br>
> the LDLIBS which avoids the need for tricks as<br>
> "LDLIBS := -lltpfoo $(LDLIBS)" in the Makefiles.<br>
> <br>
> This commit also silences the output by default a bit, the verbose<br>
> output could be enabled by VERBOSE=1 env variable, which is probably<br>
> what most of the build systems will do if this gets commited. I guess<br>
> that we can as well silence a bit the "make entering/leaving directory"<br>
> if this the right way to go.<br>
<br>
alias for V=0 / V=1 would be nice<br>
<br>
LTPLDLIBS should probably be mentioned in doc/build-system-guide.txt<br>
<br>
<br>
> <br>
> Signed-off-by: Cyril Hrubis <<a href="mailto:chrubis@suse.cz" target="_blank">chrubis@suse.cz</a>><br>
> ---<br>
>  include/mk/<a href="http://env_post.mk" rel="noreferrer" target="_blank">env_post.mk</a> |  2 ++<br>
>  include/mk/<a href="http://rules.mk" rel="noreferrer" target="_blank">rules.mk</a>    | 29 +++++++++++++++++++++++++++++<br>
>  2 files changed, 31 insertions(+)<br>
>  create mode 100644 include/mk/<a href="http://rules.mk" rel="noreferrer" target="_blank">rules.mk</a><br>
> <br>
> diff --git a/include/mk/<a href="http://env_post.mk" rel="noreferrer" target="_blank">env_post.mk</a> b/include/mk/<a href="http://env_post.mk" rel="noreferrer" target="_blank">env_post.mk</a><br>
> index f4169ad66..bdf8c696d 100644<br>
> --- a/include/mk/<a href="http://env_post.mk" rel="noreferrer" target="_blank">env_post.mk</a><br>
> +++ b/include/mk/<a href="http://env_post.mk" rel="noreferrer" target="_blank">env_post.mk</a><br>
> @@ -107,4 +107,6 @@ $(error You must define $$(prefix) before executing<br>
> install)<br>
>  endif # END $(filter-out install,$(MAKECMDGOALS)),$(MAKECMDGOALS)<br>
>  endif<br>
>  <br>
> +include $(top_srcdir)/include/mk/<a href="http://rules.mk" rel="noreferrer" target="_blank">rules.mk</a><br>
> +<br>
>  endif<br>
> diff --git a/include/mk/<a href="http://rules.mk" rel="noreferrer" target="_blank">rules.mk</a> b/include/mk/<a href="http://rules.mk" rel="noreferrer" target="_blank">rules.mk</a><br>
> new file mode 100644<br>
> index 000000000..e9b9c35ef<br>
> --- /dev/null<br>
> +++ b/include/mk/<a href="http://rules.mk" rel="noreferrer" target="_blank">rules.mk</a><br>
> @@ -0,0 +1,29 @@<br>
> +%.o: %.c<br>
> +ifdef VERBOSE<br>
> +     $(CC) $(CPPFLAGS) $(CFLAGS) -c -o $@ $<<br>
> +else<br>
> +     @$(CC) $(CPPFLAGS) $(CFLAGS) -c -o $@ $<<br>
> +     @echo CC $@<br>
> +endif<br>
<br>
What if we wouldn't print "DIR" (for non-clean targets) and printed relative paths instead?<br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">+1</div></div><div><div class="gmail_default" style="font-size:small">This looks better, I'd give my vote to this.</div></div></div><div><br></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div>Regards,<br></div><div>Li Wang<br></div></div></div></div>