public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: "Zhang, Shenglei" <shenglei.zhang@intel.com>
To: "Justen, Jordan L" <jordan.l.justen@intel.com>,
	"edk2-devel@lists.01.org" <edk2-devel@lists.01.org>,
	"Ni, Ruiyu" <ruiyu.ni@intel.com>
Subject: Re: [PATCH 0/5] Remove EdkShellBinPkg in FDF, DSC and DEC
Date: Thu, 1 Nov 2018 05:17:32 +0000	[thread overview]
Message-ID: <C0706E73DB8C124D9B9C38AA364E5D5E04BAAFA1@shsmsx102.ccr.corp.intel.com> (raw)
In-Reply-To: <154096846205.24288.16651685333658270946@jljusten-skl>

Thanks for your comments about 'USE_OLD_SHELL'. I'll improve the patches in next version.

Shenglei

> -----Original Message-----
> From: Justen, Jordan L
> Sent: Wednesday, October 31, 2018 2:48 PM
> To: edk2-devel@lists.01.org; Zhang, Shenglei <shenglei.zhang@intel.com>
> Subject: Re: [edk2] [PATCH 0/5] Remove EdkShellBinPkg in FDF, DSC and DEC
> 
> I also see USE_OLD_SHELL references in the .dsc files.
> 
> (Try 'git grep USE_OLD_SHELL')
> 
> After the platforms no longer reference EdkShellBinPkg, then the bin
> package will be removed?
> 
> -Jordan
> 
> On 2018-10-30 22:57:06, shenglei wrote:
> > Remove EdkShellBinPkg in CorebootPayloadPkg, EmulatorPkg, Nt32Pkg
> > and OvmfPkg. Replace EdkShellBinPkg with EdkShellPkg in
> > Vlv2TbltDevicePkg.
> >
> > shenglei (5):
> >   CorebootPayloadPkg: Remove EdkShellBinPkg in FDF
> >   EmulatorPkg: Remove EdkShellBinPkg in FDF and DEC
> >   Nt32Pkg: Remove EdkShellBinPkg in FDF,DSC and DEC
> >   OvmfPkg: Remove EdkShellBinPkg in FDF
> >   Vlv2TbltDevicePkg: Remove EdkShellBinPkg with ShellPkg
> >
> >  CorebootPayloadPkg/CorebootPayloadPkg.fdf | 8 --------
> >  EmulatorPkg/EmulatorPkg.dec               | 2 +-
> >  EmulatorPkg/EmulatorPkg.fdf               | 4 ----
> >  Nt32Pkg/Nt32Pkg.dec                       | 2 +-
> >  Nt32Pkg/Nt32Pkg.dsc                       | 4 ++--
> >  Nt32Pkg/Nt32Pkg.fdf                       | 4 ----
> >  OvmfPkg/OvmfPkgIa32.fdf                   | 4 ----
> >  OvmfPkg/OvmfPkgIa32X64.fdf                | 4 ----
> >  OvmfPkg/OvmfPkgX64.fdf                    | 4 ----
> >  Vlv2TbltDevicePkg/PlatformPkg.fdf         | 5 +----
> >  Vlv2TbltDevicePkg/PlatformPkgGcc.fdf      | 5 +----
> >  Vlv2TbltDevicePkg/PlatformPkgGccX64.dsc   | 1 +
> >  Vlv2TbltDevicePkg/PlatformPkgIA32.dsc     | 1 +
> >  Vlv2TbltDevicePkg/PlatformPkgX64.dsc      | 1 +
> >  14 files changed, 9 insertions(+), 40 deletions(-)
> >
> > --
> > 2.18.0.windows.1
> >
> > _______________________________________________
> > edk2-devel mailing list
> > edk2-devel@lists.01.org
> > https://lists.01.org/mailman/listinfo/edk2-devel

      reply	other threads:[~2018-11-01  5:17 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-10-31  5:57 [PATCH 0/5] Remove EdkShellBinPkg in FDF, DSC and DEC shenglei
2018-10-31  5:57 ` [PATCH 1/5] CorebootPayloadPkg: Remove EdkShellBinPkg in FDF shenglei
2018-10-31  5:57 ` [PATCH 2/5] EmulatorPkg: Remove EdkShellBinPkg in FDF and DEC shenglei
2018-10-31  9:12   ` Ni, Ruiyu
2018-10-31  5:57 ` [PATCH 3/5] Nt32Pkg: Remove EdkShellBinPkg in FDF, DSC " shenglei
2018-10-31  5:57 ` [PATCH 4/5] OvmfPkg: Remove EdkShellBinPkg in FDF shenglei
2018-10-31  5:57 ` [PATCH 5/5] Vlv2TbltDevicePkg: Remove EdkShellBinPkg with ShellPkg shenglei
2018-11-04  6:11   ` Wei, David
2018-10-31  6:47 ` [PATCH 0/5] Remove EdkShellBinPkg in FDF, DSC and DEC Jordan Justen
2018-11-01  5:17   ` Zhang, Shenglei [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=C0706E73DB8C124D9B9C38AA364E5D5E04BAAFA1@shsmsx102.ccr.corp.intel.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