public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: "Andrew Fish" <afish@apple.com>
To: Laszlo Ersek <lersek@redhat.com>
Cc: devel@edk2.groups.io, ray.ni@intel.com,
	Mike Kinney <michael.d.kinney@intel.com>,
	"Bi, Dandan" <dandan.bi@intel.com>,
	"Cetola, Stephano" <stephano.cetola@intel.com>,
	"Gao, Liming" <liming.gao@intel.com>,
	"Carsey, Jaben" <jaben.carsey@intel.com>
Subject: Re: [edk2-devel] [edk2] [RFC] Plan to delete ShellBinPkg from edk2/master
Date: Thu, 04 Apr 2019 08:10:25 -0700	[thread overview]
Message-ID: <ABB0DEB6-D545-4DBB-958C-85595B9F4994@apple.com> (raw)
In-Reply-To: <2c9dd2e2-c261-4216-9541-26f7e4a63988@redhat.com>

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



> On Apr 4, 2019, at 3:45 AM, Laszlo Ersek <lersek@redhat.com> wrote:
> 
> On 04/04/19 06:09, Andrew Fish wrote:
>> 
>> 
>>> On Apr 3, 2019, at 8:42 PM, Ni, Ray <ray.ni@intel.com> wrote:
>>> 
>>> Mike, Laszlo,
>>> It's a good idea to store the shell binaries into the assets of each stable tag.
>>> 
>>> If we go in this way, it means "build" requires network connection to download the
>>> shell binary from the assets of a certain release.
>>> Do you think it's acceptable?
>>> 
>> 
>> Ray,
>> 
>> The other option would be to have a configuration step, like installing Python or the C compilers, that copies the binary. You need a network connection to clone the git repo and to stay in sync with it. I guess you could model that as a git submodule, or actually have a script that grabs the binary you want from a remote system, and fall back to the local copy if you don't have a network connection. 
>> 
>> 
>>> Or we can separate the binary download and build into two phases so build phase
>>> can be independent on network connection.
>>> 
>>> Is there any known practice/solution for such requirement (stable sub-component binaries
>>> needed by a production image generation)?
>>> 
>> 
>> I think to some extent this kind of thing is driven by the customers build rules. Basically what the customer think of as their manifest of parts for software version X.
> 
> I suggested PREBUILD because I took it as a given, from Mike's problem
> statement, that "build" had to ensure, internally, the local
> availability of the shell binary.
> 
> If that's a not requirement, then IMO it's much better to leave it to
> organizations to fetch the prerequisites of their platform builds. I'd
> say that's out of scope for upstream edk2 -- if they need the shell
> binary to be available off-line, at their build time, they can download
> it earlier and cache it locally.
> 

Laszlo,

I guess for edk2 projects the maintainers own the manifest. So the edk2 projects that need the Shell should define how that works. I don't think we need to define a generic solution for 3rd parties as I'd guess Red Hat and Microsoft probably already have tools and strategies to deal with cobbling together software from different packages.  

So I guess we should ask the maintainers of the ekd2 packages does the version of the Shell matter? If no then just pre-install a shell binary as part of the setup. If the version matters then we should look into doing something a little more fancy, and use the pre-installed shell binary as the fallback. 

Is there anyway to tell the Shell version from the Shell PE/COFF? One option could be a build warning if the shell is old and just have the user manually update the shell if needed. 

Thanks,

Andrew Fish


> Thanks
> Laszlo


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

  reply	other threads:[~2019-04-04 15:11 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-04-02  5:38 [RFC] Plan to delete ShellBinPkg from edk2/master Bi, Dandan
2019-04-02  8:49 ` Laszlo Ersek
2019-04-02  9:12   ` Leif Lindholm
2019-04-02 11:29     ` Ryszard Knop
2019-04-02 11:45       ` Laszlo Ersek
2019-04-02 11:50         ` Ryszard Knop
2019-04-02 12:56           ` Laszlo Ersek
2019-04-03  2:17   ` Ni, Ray
2019-04-03 10:09     ` Laszlo Ersek
2019-04-03 15:49       ` Kinney, Michael D
2019-04-03 16:07         ` Laszlo Ersek
2019-04-03 21:47           ` [edk2] " Michael D Kinney
2019-04-04  3:42             ` ray.ni
2019-04-04  4:09               ` [edk2-devel] " Andrew Fish
2019-04-04 10:45                 ` Laszlo Ersek
2019-04-04 15:10                   ` Andrew Fish [this message]
2019-04-04 16:06                     ` Laszlo Ersek
2019-04-04 16:36                       ` Andrew Fish
2019-04-08  8:08                         ` Dandan Bi
2019-04-15 14:58                           ` Liming Gao

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=ABB0DEB6-D545-4DBB-958C-85595B9F4994@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