public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: "Andrew Fish" <afish@apple.com>
To: edk2-devel-groups-io <devel@edk2.groups.io>, matthewfcarlson@gmail.com
Cc: rfc@edk2.groups.io
Subject: Re: [edk2-devel] Basetools as a pip module
Date: Tue, 01 Sep 2020 17:49:37 -0700	[thread overview]
Message-ID: <4C114D94-5320-4825-9400-1044FC5FA9AB@apple.com> (raw)
In-Reply-To: <CALSfKnm=cnO5VvDZY9hSe0GhdL=GCzqqU=3j2gPiVFNpCt1P7g@mail.gmail.com>

[-- Attachment #1: Type: text/plain, Size: 2410 bytes --]



> On Sep 1, 2020, at 4:35 PM, Matthew Carlson <matthewfcarlson@gmail.com> wrote:
> 
> Hello all,
> 
> A recent topic on the RFC mailing list went out and the work on moving Basetools/Sources/Python to a separate repo has started. See the RFC conversation here: https://edk2.groups.io/g/rfc/topic/74009714#270 <https://edk2.groups.io/g/rfc/topic/74009714#270>
> 
> The repo in question is here: https://github.com/tianocore/edk2-basetools <https://github.com/tianocore/edk2-basetools>
> 
> The current plan is shortly after the stable tag is created, a series of patches will come into edk2 that redirects the build system into the new python module as well as adds additional documentation. You can see a sample of this work here: https://github.com/matthewfcarlson/edk2 <https://github.com/matthewfcarlson/edk2> as this has a branch that has the work required to use the basetools pip module. The patches won't delete the Basetools/Sources/Python folder but will allow users to select between them. After a certain grace period, the python folder will be deleted and the pip module will be the de facto way of using basetools.
> 
> Three questions need to be answered:
> 
> 1. After the patches that enable the pip module land, how long should the grace period be?
> 2. During the grace period, should basetools commits land in both places or just in the edk2-basetools directory?
> 3. How should the user be able to select which basetools to use (the one in EDK2 or the pip module)? Currently the approach being considered is a simple environmental variable? One of the key considerations is transparency since it won't be apparent what is being used for a particular build without some sort of mechanism to notify the developer. With two seperate versions of Basetools, it becomes very easy for the version of basetools you're using to not be the one you expect.
> 

Matthew,

I’ll throw out some current developer centric ideas. 

1) If you `source edksetup.sh` (edksetup.bat) you get the current behavior, and you add an argument you get the pip flavor? So maybe `edksetup.bat pip-basetools`?
2) We have similar issues to this with env variables and the build command dumps them out when it runs. Can we use the current EDK_TOOL_PATH? Or maybe add an extra print to show that the pip module is being used?

Thanks,

Andrew Fish


> Thank you.
> -Matthew Carlson
> 


[-- Attachment #2: Type: text/html, Size: 3846 bytes --]

  reply	other threads:[~2020-09-02  0:49 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-09-01 23:35 Basetools as a pip module Matthew Carlson
2020-09-02  0:49 ` Andrew Fish [this message]
2020-09-02  8:49   ` [edk2-devel] " Laszlo Ersek
2020-09-02 19:06     ` Matthew Carlson
2020-09-03  2:14       ` Andrew Fish
2020-09-03  2:46         ` Matthew Carlson

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=4C114D94-5320-4825-9400-1044FC5FA9AB@apple.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