From: Alyssa Ross <hi@alyssa.is>
To: Ville Ilvonen <ville.ilvonen@unikie.com>
Cc: discuss@spectrum-os.org
Subject: Re: Workflow preferences - case aarch64 support
Date: Mon, 20 Jun 2022 12:05:09 +0000 [thread overview]
Message-ID: <20220620120509.w5j24q6ycj44vwah@eve> (raw)
In-Reply-To: <CAP-nJwHPyFie6xSnCU56LrxhsrUd-u+_d0gpR6rN9RQw4mNedA@mail.gmail.com>
[-- Attachment #1: Type: text/plain, Size: 2436 bytes --]
On Mon, Jun 20, 2022 at 01:29:49PM +0300, Ville Ilvonen wrote:
> Hi all,
>
> Good news is that the work on aarch64 support has progressed to the point where
> the Spectrum host can be built natively and run on aarch64 HW.
This is excellent! Very exciting.
> It's still a bunch of patches in various stages of WIP or review to
> spectrum or further
> up the stream.
>
> In short, to "nixpkgs-spectrum":
> 0001-Allow-static-building-of-dtc.patch
> 0002-Workaround-ld-script-segmentation-fault.patch /* may be not needed */
> 0003-Add-imx8-specific-kernel.patch
> and to "spectrum":
> 0001-Add-imx8-kernel.patch
>
> Should anyone want to reproduce, please let me know and I'll try to
> find a solution
> to share them. Probably worth waiting for cross-compilation support unless you
> have powerful aarch64 at your disposal.
>
> In the meanwhile, we try to establish a mechanism to collaborate in branches of
> required upstream repos. It's fairly impractical to fork
> "nixpkgs-spectrum"-repo due to
> its' size, over 2GB,
>
> Which takes me to the discussion point - what's the workflow proposal
> on multiple people
> working on the same functionality in the same branch? If I'm not
> mistaken the email workflow
> creates a new branch for each author? Then someone would have to
> cherry-pick them to the
> same branch to integrate them in dev time hydra builds.
For that situation, I would recommend having a shared Nixpkgs repository
where the people working on the change can work on it together. You're
right that Nixpkgs is big, but the bulk of it only has to be pushed
once, and from that point changes will be incremental. Public git
hosts, if you wanted to use one of those, likely already have somebody's
copy of Nixpkgs on them and might deduplicate against it, so possibly
won't need most git objects to be uploaded anyway. This is broadly how
feature development works for other projects using similar workflows,
like Linux or QEMU.
When it's time to submit the patches to the list for review, anybody can
do that. Authorship information is preserved in the patch metadata, so
it's fine for one person to submit a patch series that includes patches
from multiple authors. git-send-email will also handle automatically
CCing authors on patches that they wrote, so questions / review should
still find their way to the right person.
Does that all make sense?
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
prev parent reply other threads:[~2022-06-20 12:05 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-06-20 10:29 Workflow preferences - case aarch64 support Ville Ilvonen
2022-06-20 12:05 ` Alyssa Ross [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=20220620120509.w5j24q6ycj44vwah@eve \
--to=hi@alyssa.is \
--cc=discuss@spectrum-os.org \
--cc=ville.ilvonen@unikie.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 NNTP newsgroup(s).