All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Randy MacLeod" <randy.macleod@windriver.com>
To: Richard Purdie <richard.purdie@linuxfoundation.org>,
	openembedded-core@lists.openembedded.org
Subject: Re: [OE-core] Merge meta-rust to oe-core
Date: Wed, 24 Feb 2021 13:23:55 -0500	[thread overview]
Message-ID: <89c6f05d-5add-8906-c76a-bbaca574497b@windriver.com> (raw)
In-Reply-To: <a373f6be54f63987eb9cb0ce994b807fcca01062.camel@linuxfoundation.org>

On 2021-02-24 4:39 a.m., Richard Purdie wrote:
> On Tue, 2021-02-23 at 22:01 -0500, Randy MacLeod wrote:
>> Clean-up the merge of meta-rust to oe-core by squashing most of the
>> meta-rust history so we are left with:
>>
>>    35da4b252f rust: add a language demo image to test reproducibility
>>    088374371b cargo/rust/rustfmt: exclude from world
>>    7df10d2a22 meta-rust: merge commits
>>    4187796cd6 rust: mv README.md to recipes-devtools/rust/README-rust.md
>>    03b9c60ef4 meta-rust: move code to oe-core from meta-rust layer
>>    2c62874204 libgit2: pull in updates from meta-oe
>>    24cb3db88c libssh2: pull in additional commits from meta-oe
>>    e9a0a3bad2 Add libgit2, libssh2 from meta-oe for rust
>>
>> The full diffstat is below. (1)
>>
>> I didn't really notice the scripts shown below since they are mostly harmless.
>> I'll review them and remove them in v2 if they don't make sense for oe-core.
> 
> This is better, thanks. My feedback from a quick glance:

Thanks for the comments.

> 
> a) Need maintainer entries for the new recipes

Fixed by adding myself but making it clear that other maintainers
are welcome. Once libssh/libgit2 are in master, I'll do the same for
them since they are unassigned in master-next.

> b) The scripts in scripts/ look at the very least inappropriately named
>     for OE-Core, I suspect we don't want/need those.

Working on that next (after 2 more meetings...).

> c) Not entirely sure about the languages demo image, its not how we test
>     any other language

Removed.

> d) There aren't any tests in meta/lib/oeqa!

To come. Is:
   meta/lib/oeqa/selftest/cases/gotoolchain.py
a good first step as I mentioned earlier?

> e) The rust README starts "This OpenEmbedded layer provides" and has a few other
>     issues.

Fixed.



To Do List
==========

1) cargo-bitbake - the README suggest adding this using desktop
    cargo but once we get the SDK merged, it should be added there.
    Someone suggested using devtool for this rather than cargo-bitbake.

2) Fix the SDK libcrypto, libstdc++ glibc-2.33 problem. I'm running a
git bisect using poky, meta-oe/master, meta-rust+SDK patches.

N) Remaining bugs in:

---



Would it be worth adding this to master-next in the next few days
to confirm that rust-hello-world builds for all the supported machines?
I suspect that will work but I've mostly used Ubuntu so it'd be good
to get some coverage testing.


-- 
# Randy MacLeod
# Wind River Linux

  reply	other threads:[~2021-02-24 18:24 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-02-24  3:01 Merge meta-rust to oe-core Randy MacLeod
2021-02-24  3:01 ` [PATCH 1/8] Add libgit2, libssh2 from meta-oe for rust Randy MacLeod
2021-02-24  3:01 ` [PATCH 2/8] libssh2: pull in additional commits from meta-oe Randy MacLeod
2021-02-24  3:01 ` [PATCH 3/8] libgit2: pull in updates " Randy MacLeod
2021-02-24  3:01 ` [PATCH 4/8] meta-rust: move code to oe-core from meta-rust layer Randy MacLeod
2021-02-24  3:01 ` [PATCH 5/8] rust: mv README.md to recipes-devtools/rust/README-rust.md Randy MacLeod
2021-02-24  3:01 ` [PATCH 6/8] meta-rust: merge commits Randy MacLeod
2021-02-24  3:02 ` [PATCH 7/8] cargo/rust/rustfmt: exclude from world Randy MacLeod
2021-02-24  3:02 ` [PATCH 8/8] rust: add a language demo image to test reproducibility Randy MacLeod
2021-02-24  9:39 ` [OE-core] Merge meta-rust to oe-core Richard Purdie
2021-02-24 18:23   ` Randy MacLeod [this message]
2021-02-24 20:26     ` Randy MacLeod
     [not found] ` <1666A58EF14596DD.29651@lists.openembedded.org>
2021-02-24 16:53   ` Richard Purdie

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=89c6f05d-5add-8906-c76a-bbaca574497b@windriver.com \
    --to=randy.macleod@windriver.com \
    --cc=openembedded-core@lists.openembedded.org \
    --cc=richard.purdie@linuxfoundation.org \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.