From: "Yann E. MORIN" <yann.morin.1998@free.fr>
To: Romain Naour <romain.naour@smile.fr>
Cc: yann.morin@orange.com, buildroot@buildroot.org
Subject: Re: [Buildroot] [PATCH 1/2] package/rust: use host libraries from HOST_DIR
Date: Thu, 29 Feb 2024 20:22:13 +0100 [thread overview]
Message-ID: <ZeDZZUks-dQSh5Gs@landeda> (raw)
In-Reply-To: <a8fa2870-4d82-48df-b8ef-03c61a1ba81c@smile.fr>
Romain, All,
On 2024-02-29 14:43 +0100, Romain Naour spake thusly:
> Le 29/02/2024 à 14:35, yann.morin@orange.com a écrit :
> > On 2024-02-29 14:11 +0100, Romain Naour spake thusly:
> >> host-rust package depends on several host packages to provide tools and
> >> libraries but it doesn't take into account HOST_DIR while building rustc
> >> compiler. Indeed, rustc needs zlib and fail to link if zlib is not
> >> installed on the host.
[--SNIP--]
> > But then, we would also need to add host-zlib to
> > HOST_RUST_DEPENDENCIES, no?
> Indeed but in practice host-zlib is already an "indirect" dependency of
> host-rust due to host-openssl (host-libopenssl) dependency.
When a package has a dependency on a library for itself, we must add an
explicit dependency in that package, even if it is transitively brought
by another, like is the case here. So we do want that host-rust depends
on host-zlib explicitly.
> HOST_LIBOPENSSL_DEPENDENCIES = host-zlib
>
> I noticed the missing dependency but forgot to add it in HOST_RUST_DEPENDENCIES.
Mark this patch as Changes-requested, then! ;-)
Regards,
Yann E. MORIN.
--
.-----------------.--------------------.------------------.--------------------.
| Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ |
| +33 561 099 427 `------------.-------: X AGAINST | \e/ There is no |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. |
'------------------------------^-------^------------------^--------------------'
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
prev parent reply other threads:[~2024-02-29 19:22 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-02-29 13:11 [Buildroot] [PATCH 1/2] package/rust: use host libraries from HOST_DIR Romain Naour
2024-02-29 13:11 ` [Buildroot] [PATCH 2/2] package/rust: provide RUSTFLAGS for cargo Romain Naour
2024-02-29 13:35 ` [Buildroot] [PATCH 1/2] package/rust: use host libraries from HOST_DIR yann.morin
2024-02-29 13:43 ` Romain Naour
2024-02-29 19:22 ` Yann E. MORIN [this message]
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=ZeDZZUks-dQSh5Gs@landeda \
--to=yann.morin.1998@free.fr \
--cc=buildroot@buildroot.org \
--cc=romain.naour@smile.fr \
--cc=yann.morin@orange.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