public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: "Rebecca Cran" <rebecca@bsdio.com>
To: devel@edk2.groups.io, "Gerd Hoffmann" <kraxel@redhat.com>
Cc: gaoliming <gaoliming@byosoft.com.cn>,
	"'Bob Feng'" <bob.c.feng@intel.com>,
	"'Yuwei Chen'" <yuwei.chen@intel.com>,
	"Andrew Fish" <afish@apple.com>,
	"Leif Lindholm" <quic_llindhol@quicinc.com>,
	"Kinney, Michael D" <michael.d.kinney@intel.com>
Subject: Re: [edk2-devel] RFC: Updating IA32/X64 cross-compilation to work like other archs
Date: Tue, 05 Nov 2024 07:06:46 -0700	[thread overview]
Message-ID: <6d84ca57-e15a-4b59-8c97-d2ef106f5d59@app.fastmail.com> (raw)
In-Reply-To: <znmb2775n33qokocztz6b4zgav6ot6fspv546hjfwtrcnmerd5@kk2jwwm24mjd>

Thanks, I’ll merge this after the stable tag.

On Tue, Nov 5, 2024, at 5:45 AM, Gerd Hoffmann wrote:
> On Thu, Oct 17, 2024 at 08:21:54PM -0600, Rebecca Cran wrote:
>> Cross-compilation to platforms other than IA32 and X64 work by setting an
>> environment variable such as GCC_AARCH64_PREFIX that contains the path/name
>> of the cross-compiler without the "gcc", "objcopy" etc. suffix.
>> 
>> On IA32 and X64 however, the environment variable GCC_BIN has been used
>> instead. With fast AArch64 and other non-x86 platforms now being available,
>> I'd like to propose updating IA32 and X64 to work the same as other
>> platforms by using e.g. GCC_IA32_PREFIX and GCC_X64_PREFIX.
>> 
>> 
>> I've attached a proposed patch: I'm attaching it here to gather feedback
>> before opening a PR since it does change how builds are done and I want to
>> check if it's something people want to proceed with.
>
> I think it'll be nice if edk2 is consistent here, using the same naming
> scheme for all platforms, so I'm in favor of the change.
>
> take care,
>   Gerd
>
>
>
> 


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#120731): https://edk2.groups.io/g/devel/message/120731
Mute This Topic: https://groups.io/mt/109076070/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-



      reply	other threads:[~2024-11-05 14:06 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-10-18  2:21 [edk2-devel] RFC: Updating IA32/X64 cross-compilation to work like other archs Rebecca Cran
2024-11-05 12:45 ` Gerd Hoffmann
2024-11-05 14:06   ` Rebecca Cran [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-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=6d84ca57-e15a-4b59-8c97-d2ef106f5d59@app.fastmail.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