public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: "Ayush Singh" <ayushdevel1325@gmail.com>
To: "devel@edk2.groups.io" <devel@edk2.groups.io>
Cc: "Kinney, Michael D" <michael.d.kinney@intel.com>,
	"mikuback@linux.microsoft.com" <mikuback@linux.microsoft.com>,
	"Gaibusab, Jabeena B" <jabeena.b.gaibusab@intel.com>
Subject: Proposal to move Rust std work to edk2-staging
Date: Sat, 30 Jul 2022 15:31:17 +0530	[thread overview]
Message-ID: <69796452-56d6-75e9-a884-1c13fc68f8d6@gmail.com> (raw)

Hello everyone. The work on Rust std for UEFI has been progressing 
smoothly. Running Rust tests is now possible in it's entirety. The 
number of tests that fail is 133 (4 cause exception + 129 normal fails) 
out of 13,212 tests. Apart from the 4 tests that cause exceptions, most 
of the 129 other tests can be ignored since they are caused by needing 
dynamic linking and/or stack unwinding. So I think, it is time to start 
considering moving the project to a more appropriate repository before I 
open a PR upstream.


However, I do have some concerns regarding moving to edk2-staging:

1. Rust development workflow does not use Mailing lists and Patches. 
This might create friction for the purely `rustc` developers who might 
want to submit changes and are accustomed to PRs.

2. Rust project does not seem to care about maintaining a clean history 
(most of the merges are performed by bots in form of roll-up merges). 
This is much different from how edk2 projects are usually managed. The 
reason where this becomes a problem is having to do re-bases. While Rust 
provides strong guarantees for stability of user facing APIs, the 
internal std APIs are constantly changing. Combine that with no 
clean-history and the upstream PR will probably have to be re-based on 
latest master every few weeks (and it will likely need manual intervention).


I am planning on getting the code base ready for upstream PR before 15th 
August. The main things left to do is to publish new versions of 
`compiler_builtins` and `r-efi` crates, and refactor the code and UEFI 
documentation.


Yours Sincerely

Ayush Singh


             reply	other threads:[~2022-07-30 10:01 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-07-30 10:01 Ayush Singh [this message]
2022-08-01 14:49 ` [edk2-devel] Proposal to move Rust std work to edk2-staging Michael Kubacki

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-list 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=69796452-56d6-75e9-a884-1c13fc68f8d6@gmail.com \
    --to=devel@edk2.groups.io \
    /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