Re: [PATCH] toplevel: Makefile.def: Make configure-sim depend on all-readline

[Richard Biener via Gcc-patches]

> Am 09.03.2022 um 17:54 schrieb Hans-Peter Nilsson via Gcc-patches 
> <gcc-patches@gcc.gnu.org>:
> 
> Tom Tromey ok'd this for the sourceware side, but thinks I
> need explicit approval on the gcc side.  Ok to commit?
> 
> ------- Start of forwarded message -------
> From: Hans-Peter Nilsson <hans-peter.nils...@axis.com>
> To: "binut...@sourceware.org" <binut...@sourceware.org>,
>    "gdb-patc...@sourceware.org" <gdb-patc...@sourceware.org>
> Subject: [PATCH] toplevel: Makefile.def: Make configure-sim depend on 
> all-readline
> 
> Calling on "global maintainers" as per toplevel/MAINTAINERS
> for "Makefile.*".
> 
> Ok to commit?

Ok 

> (If so, I'll also commit this change to the gcc repo, where
> this dependency is normally unused, i.e. when source trees
> are kept separate.)
> 
> brgds, H-P
> - ----- 8< -----
> 
> Without this, a "make all-sim" without the equivalent of
> libreadline-dev installed on the build system, won't
> properly pick up the in-tree readline build, and you'll see:
> 
> mkdir -p -- ./sim
> Configuring in ./sim
> configure: creating cache ./config.cache
> checking build system type... x86_64-pc-linux-gnu
> checking host system type... x86_64-pc-linux-gnu
> checking target system type... cris-axis-elf
> checking for x86_64-pc-linux-gnu-gcc... gcc
> checking whether the C compiler works... yes
> ...
> checking for library containing tgetent... -ltermcap
> checking for readline in -lreadline... no
> configure: error: the required "readline" library is missing
> make[1]: *** [Makefile:11188: configure-sim] Error 1
> make[1]: Leaving directory '/home/hp/sim/b'
> 
> The sim dependency on readline is apparently (nominally)
> valid as there's a readline call in sim/erc32/sis.c.
> 
> 2022-02-21  Hans-Peter Nilsson  <h...@axis.com>
> 
>    * Makefile.def (dependencies): Make configure-sim depend on
>    all-readline.
> - ---
> Makefile.def | 2 +-
> Makefile.in  | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/Makefile.def b/Makefile.def
> index a504192e6d72..8181a7aa97b5 100644
> - --- a/Makefile.def
> +++ b/Makefile.def
> @@ -570,7 +570,7 @@ dependencies = { module=all-sim; on=all-intl; };
> dependencies = { module=all-sim; on=all-libiberty; };
> dependencies = { module=all-sim; on=all-bfd; };
> dependencies = { module=all-sim; on=all-opcodes; };
> - -dependencies = { module=all-sim; on=all-readline; };
> +dependencies = { module=configure-sim; on=all-readline; };
> 
> // Other host modules.
> dependencies = { module=all-fastjar; on=all-zlib; };
> diff --git a/Makefile.in b/Makefile.in
> index 2b77a4706947..843e150dac63 100644
> - --- a/Makefile.in
> +++ b/Makefile.in
> @@ -63072,7 +63072,7 @@ install-strip-sid: maybe-install-strip-tcl
> install-sid: maybe-install-tk
> install-strip-sid: maybe-install-strip-tk
> configure-sim: maybe-all-gnulib
> - -all-sim: maybe-all-readline
> +configure-sim: maybe-all-readline
> all-fastjar: maybe-all-build-texinfo
> all-libctf: all-libiberty
> all-stage1-libctf: all-stage1-libiberty
> - -- 
> 2.30.2
> ------- End of forwarded message -------