bug#43683: closed (make install-exec does not trigger make $(BUILT_SOURCES))

[GNU bug Tracking System]

Your message dated Tue, 6 Oct 2020 19:18:55 -0600
with message-id <202010070118.0971ItIE025146@freefriends.org>
and subject line Re: bug#43683: make install-exec does not trigger make 
$(BUILT_SOURCES)
has caused the debbugs.gnu.org bug report #43683,
regarding make install-exec does not trigger make $(BUILT_SOURCES)
to be marked as done.

(If you believe you have received this mail in error, please contact
help-debbugs@gnu.org.)


-- 
43683: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=43683
GNU Bug Tracking System
Contact help-debbugs@gnu.org with problems

--- Begin Message --- Subject: make install-exec does not trigger make $(BUILT_SOURCES) Date: Mon, 28 Sep 2020 23:34:54 +0100

I have a project that relies on automatically-built sources, therefore I have created a BUILT_SOURCES variable in my src/Makefile.am file (see GNU Automake § 9.4 Built Sources). Everything works perfectly, however I have noticed that if I launch make install before launching make all the sources are correctly built, but the same does not happen if I launch make install-exec before launching make all. So I have examined the generated src/Makefile, and I have found the reason.

Currently src/Makefile.am generates the following code in src/Makefile:

install: $(BUILT_SOURCES)
        $(MAKE) $(AM_MAKEFLAGS) install-am

install-exec: install-exec-am

Evidently install and install-exec are treated differently, and the second one does not build any source. I believe that the correct generated code for src/Makefile should be instead:

install: $(BUILT_SOURCES)
        $(MAKE) $(AM_MAKEFLAGS) install-am

install-exec: $(BUILT_SOURCES)
        $(MAKE) $(AM_MAKEFLAGS) install-exec-am

Cheers and thanks, GNU folks, for your incredible work.

--madmurphy

--- End Message ---

> --- Begin Message ---
>
>
>
> Subject: 
>
> Re: bug#43683: make install-exec does not trigger make $(BUILT_SOURCES)
>
>
>
>
> Date: 
>
> Tue, 6 Oct 2020 19:18:55 -0600
>
>
>
>
> Hi madmurphy,
>
>     install-exec: $(BUILT_SOURCES)
>             $(MAKE) $(AM_MAKEFLAGS) install-exec-am
>
> Thanks very much for the bug report, patch, and test case. I just
> applied and pushed it, with small modifications. Please report back if
> problems persist. Meanwhile, closing this bug. Thanks again! --karl
>
>
> --- End Message ---