From: Luis Machado <luis.machado@arm.com>
To: Hans-Peter Nilsson <hp@axis.com>,
binutils@sourceware.org, gdb-patches@sourceware.org
Subject: Re: [PATCH] toplevel: Makefile.def: Make configure-sim depend on all-readline
Date: Tue, 12 Jul 2022 11:11:41 +0100 [thread overview]
Message-ID: <89285421-e99a-ec59-b913-8d7b95b0ee87@arm.com> (raw)
In-Reply-To: <20220308192128.A2D462041A@pchp3.se.axis.com>
Hi,
On 3/8/22 19:21, Hans-Peter Nilsson via Binutils wrote:
> Calling on "global maintainers" as per toplevel/MAINTAINERS
> for "Makefile.*".
>
> Ok to commit?
>
> (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 <hp@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
I still see this, even with this patch applied. It seems sim requires libreadline-dev regardless.
Based on the commit message, I'd expect that to not be the case anymore, right?
next prev parent reply other threads:[~2022-07-12 10:12 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-03-08 19:21 Hans-Peter Nilsson
2022-03-09 16:30 ` Tom Tromey
2022-07-12 10:11 ` Luis Machado [this message]
2022-07-15 22:20 ` Hans-Peter Nilsson
2022-07-19 8:31 ` Christophe Lyon
2022-07-19 17:26 ` Hans-Peter Nilsson
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=89285421-e99a-ec59-b913-8d7b95b0ee87@arm.com \
--to=luis.machado@arm.com \
--cc=binutils@sourceware.org \
--cc=gdb-patches@sourceware.org \
--cc=hp@axis.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).