public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
* [RFC] Proposal to remove DuetPkg
@ 2018-11-20  5:27 Ni, Ruiyu
  2018-11-20  8:46 ` Laszlo Ersek
  2018-11-20  9:57 ` Leif Lindholm
  0 siblings, 2 replies; 7+ messages in thread
From: Ni, Ruiyu @ 2018-11-20  5:27 UTC (permalink / raw)
  To: edk2-devel@lists.01.org
  Cc: Richardson, Brian, Laszlo Ersek, Leif Lindholm,
	'Andrew Fish (afish@apple.com)', Kinney, Michael D,
	Gao, Liming, Cetola, Stephano

All,
DuetPkg depends on Legacy BIOS to provide a UEFI environment. It was invented in the era when UEFI environment is hard to find. Since now UEFI is very popular in PC area, we could stop the official support of this package and remove it from the master.
Anyone who wants to use it still can just grab it from git history. This is the same as what we did for IPF contents.

If there is no concern by end of November, we will send out a formal patch to remove DuetPkg. (Since DuetPkg is big, we will not generate the huge patch using git command, instead we will just send out the patch mail with correct title/commit message but empty content.)

Thanks,
Ray


^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [RFC] Proposal to remove DuetPkg
  2018-11-20  5:27 [RFC] Proposal to remove DuetPkg Ni, Ruiyu
@ 2018-11-20  8:46 ` Laszlo Ersek
  2018-11-20  9:57 ` Leif Lindholm
  1 sibling, 0 replies; 7+ messages in thread
From: Laszlo Ersek @ 2018-11-20  8:46 UTC (permalink / raw)
  To: Ni, Ruiyu, edk2-devel@lists.01.org
  Cc: Richardson, Brian, Leif Lindholm,
	'Andrew Fish (afish@apple.com)', Kinney, Michael D,
	Gao, Liming, Cetola, Stephano

On 11/20/18 06:27, Ni, Ruiyu wrote:
> All,
> DuetPkg depends on Legacy BIOS to provide a UEFI environment. It was invented in the era when UEFI environment is hard to find. Since now UEFI is very popular in PC area, we could stop the official support of this package and remove it from the master.
> Anyone who wants to use it still can just grab it from git history. This is the same as what we did for IPF contents.
> 
> If there is no concern by end of November, we will send out a formal patch to remove DuetPkg. (Since DuetPkg is big, we will not generate the huge patch using git command, instead we will just send out the patch mail with correct title/commit message but empty content.)
> 
> Thanks,
> Ray
> 

I have no objection.

Thanks,
Laszlo


^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [RFC] Proposal to remove DuetPkg
  2018-11-20  5:27 [RFC] Proposal to remove DuetPkg Ni, Ruiyu
  2018-11-20  8:46 ` Laszlo Ersek
@ 2018-11-20  9:57 ` Leif Lindholm
  2018-11-20 23:20   ` Kinney, Michael D
  1 sibling, 1 reply; 7+ messages in thread
From: Leif Lindholm @ 2018-11-20  9:57 UTC (permalink / raw)
  To: Ni, Ruiyu
  Cc: edk2-devel@lists.01.org, Richardson, Brian, Laszlo Ersek,
	'Andrew Fish (afish@apple.com)', Kinney, Michael D,
	Gao, Liming, Cetola, Stephano

On Tue, Nov 20, 2018 at 05:27:13AM +0000, Ni, Ruiyu wrote:
> All,
> DuetPkg depends on Legacy BIOS to provide a UEFI environment. It was
> invented in the era when UEFI environment is hard to find. Since now
> UEFI is very popular in PC area, we could stop the official support
> of this package and remove it from the master.
> Anyone who wants to use it still can just grab it from git
> history. This is the same as what we did for IPF contents.
> 
> If there is no concern by end of November, we will send out a formal
> patch to remove DuetPkg. (Since DuetPkg is big, we will not generate
> the huge patch using git command, instead we will just send out the
> patch mail with correct title/commit message but empty content.)

Works for me. If you could include a link to the patch in a branch
where it can be reviewed, that would be ideal.

/
    Leif



^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [RFC] Proposal to remove DuetPkg
  2018-11-20  9:57 ` Leif Lindholm
@ 2018-11-20 23:20   ` Kinney, Michael D
  2018-11-20 23:21     ` Kinney, Michael D
  2018-11-21  0:59     ` Andrew Fish
  0 siblings, 2 replies; 7+ messages in thread
From: Kinney, Michael D @ 2018-11-20 23:20 UTC (permalink / raw)
  To: Leif Lindholm, Ni, Ruiyu, Kinney, Michael D
  Cc: edk2-devel@lists.01.org, Gao, Liming, Richardson, Brian,
	Cetola, Stephano, Laszlo Ersek

I agree with this proposal.

I also agree with Leif that large patch series
must be shared on a branch for review.

Mike

> -----Original Message-----
> From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org]
> On Behalf Of Leif Lindholm
> Sent: Tuesday, November 20, 2018 1:57 AM
> To: Ni, Ruiyu <ruiyu.ni@intel.com>
> Cc: edk2-devel@lists.01.org; Gao, Liming
> <liming.gao@intel.com>; Richardson, Brian
> <brian.richardson@intel.com>; Cetola, Stephano
> <stephano.cetola@intel.com>; Kinney, Michael D
> <michael.d.kinney@intel.com>; Laszlo Ersek
> <lersek@redhat.com>
> Subject: Re: [edk2] [RFC] Proposal to remove DuetPkg
> 
> On Tue, Nov 20, 2018 at 05:27:13AM +0000, Ni, Ruiyu
> wrote:
> > All,
> > DuetPkg depends on Legacy BIOS to provide a UEFI
> environment. It was
> > invented in the era when UEFI environment is hard to
> find. Since now
> > UEFI is very popular in PC area, we could stop the
> official support
> > of this package and remove it from the master.
> > Anyone who wants to use it still can just grab it from
> git
> > history. This is the same as what we did for IPF
> contents.
> >
> > If there is no concern by end of November, we will send
> out a formal
> > patch to remove DuetPkg. (Since DuetPkg is big, we will
> not generate
> > the huge patch using git command, instead we will just
> send out the
> > patch mail with correct title/commit message but empty
> content.)
> 
> Works for me. If you could include a link to the patch in
> a branch
> where it can be reviewed, that would be ideal.
> 
> /
>     Leif
> 
> _______________________________________________
> edk2-devel mailing list
> edk2-devel@lists.01.org
> https://lists.01.org/mailman/listinfo/edk2-devel


^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [RFC] Proposal to remove DuetPkg
  2018-11-20 23:20   ` Kinney, Michael D
@ 2018-11-20 23:21     ` Kinney, Michael D
  2018-11-21 11:01       ` Ni, Ruiyu
  2018-11-21  0:59     ` Andrew Fish
  1 sibling, 1 reply; 7+ messages in thread
From: Kinney, Michael D @ 2018-11-20 23:21 UTC (permalink / raw)
  To: Leif Lindholm, Ni, Ruiyu, Kinney, Michael D
  Cc: edk2-devel@lists.01.org, Gao, Liming, Richardson, Brian,
	Cetola, Stephano, Laszlo Ersek

Ray,

One additional note.  There are some tools in
BaseTools that are only used by Duet.  We should
remove those tools too.

Mike

> -----Original Message-----
> From: Kinney, Michael D
> Sent: Tuesday, November 20, 2018 3:21 PM
> To: Leif Lindholm <leif.lindholm@linaro.org>; Ni, Ruiyu
> <ruiyu.ni@intel.com>; Kinney, Michael D
> <michael.d.kinney@intel.com>
> Cc: edk2-devel@lists.01.org; Gao, Liming
> <liming.gao@intel.com>; Richardson, Brian
> <brian.richardson@intel.com>; Cetola, Stephano
> <stephano.cetola@intel.com>; Laszlo Ersek
> <lersek@redhat.com>
> Subject: RE: [edk2] [RFC] Proposal to remove DuetPkg
> 
> I agree with this proposal.
> 
> I also agree with Leif that large patch series
> must be shared on a branch for review.
> 
> Mike
> 
> > -----Original Message-----
> > From: edk2-devel [mailto:edk2-devel-
> bounces@lists.01.org]
> > On Behalf Of Leif Lindholm
> > Sent: Tuesday, November 20, 2018 1:57 AM
> > To: Ni, Ruiyu <ruiyu.ni@intel.com>
> > Cc: edk2-devel@lists.01.org; Gao, Liming
> > <liming.gao@intel.com>; Richardson, Brian
> > <brian.richardson@intel.com>; Cetola, Stephano
> > <stephano.cetola@intel.com>; Kinney, Michael D
> > <michael.d.kinney@intel.com>; Laszlo Ersek
> > <lersek@redhat.com>
> > Subject: Re: [edk2] [RFC] Proposal to remove DuetPkg
> >
> > On Tue, Nov 20, 2018 at 05:27:13AM +0000, Ni, Ruiyu
> > wrote:
> > > All,
> > > DuetPkg depends on Legacy BIOS to provide a UEFI
> > environment. It was
> > > invented in the era when UEFI environment is hard to
> > find. Since now
> > > UEFI is very popular in PC area, we could stop the
> > official support
> > > of this package and remove it from the master.
> > > Anyone who wants to use it still can just grab it
> from
> > git
> > > history. This is the same as what we did for IPF
> > contents.
> > >
> > > If there is no concern by end of November, we will
> send
> > out a formal
> > > patch to remove DuetPkg. (Since DuetPkg is big, we
> will
> > not generate
> > > the huge patch using git command, instead we will
> just
> > send out the
> > > patch mail with correct title/commit message but
> empty
> > content.)
> >
> > Works for me. If you could include a link to the patch
> in
> > a branch
> > where it can be reviewed, that would be ideal.
> >
> > /
> >     Leif
> >
> > _______________________________________________
> > edk2-devel mailing list
> > edk2-devel@lists.01.org
> > https://lists.01.org/mailman/listinfo/edk2-devel


^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [RFC] Proposal to remove DuetPkg
  2018-11-20 23:20   ` Kinney, Michael D
  2018-11-20 23:21     ` Kinney, Michael D
@ 2018-11-21  0:59     ` Andrew Fish
  1 sibling, 0 replies; 7+ messages in thread
From: Andrew Fish @ 2018-11-21  0:59 UTC (permalink / raw)
  To: Mike Kinney
  Cc: Leif Lindholm, Ni, Ruiyu, edk2-devel@lists.01.org,
	Cetola, Stephano, Laszlo Ersek, Richardson, Brian, Gao, Liming

Works for me too. 

Thanks,

Andrew Fish

> On Nov 20, 2018, at 3:20 PM, Kinney, Michael D <michael.d.kinney@intel.com> wrote:
> 
> I agree with this proposal.
> 
> I also agree with Leif that large patch series
> must be shared on a branch for review.
> 
> Mike
> 
>> -----Original Message-----
>> From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org]
>> On Behalf Of Leif Lindholm
>> Sent: Tuesday, November 20, 2018 1:57 AM
>> To: Ni, Ruiyu <ruiyu.ni@intel.com>
>> Cc: edk2-devel@lists.01.org; Gao, Liming
>> <liming.gao@intel.com>; Richardson, Brian
>> <brian.richardson@intel.com>; Cetola, Stephano
>> <stephano.cetola@intel.com>; Kinney, Michael D
>> <michael.d.kinney@intel.com>; Laszlo Ersek
>> <lersek@redhat.com>
>> Subject: Re: [edk2] [RFC] Proposal to remove DuetPkg
>> 
>> On Tue, Nov 20, 2018 at 05:27:13AM +0000, Ni, Ruiyu
>> wrote:
>>> All,
>>> DuetPkg depends on Legacy BIOS to provide a UEFI
>> environment. It was
>>> invented in the era when UEFI environment is hard to
>> find. Since now
>>> UEFI is very popular in PC area, we could stop the
>> official support
>>> of this package and remove it from the master.
>>> Anyone who wants to use it still can just grab it from
>> git
>>> history. This is the same as what we did for IPF
>> contents.
>>> 
>>> If there is no concern by end of November, we will send
>> out a formal
>>> patch to remove DuetPkg. (Since DuetPkg is big, we will
>> not generate
>>> the huge patch using git command, instead we will just
>> send out the
>>> patch mail with correct title/commit message but empty
>> content.)
>> 
>> Works for me. If you could include a link to the patch in
>> a branch
>> where it can be reviewed, that would be ideal.
>> 
>> /
>>    Leif
>> 
>> _______________________________________________
>> edk2-devel mailing list
>> edk2-devel@lists.01.org
>> https://lists.01.org/mailman/listinfo/edk2-devel
> _______________________________________________
> edk2-devel mailing list
> edk2-devel@lists.01.org
> https://lists.01.org/mailman/listinfo/edk2-devel



^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [RFC] Proposal to remove DuetPkg
  2018-11-20 23:21     ` Kinney, Michael D
@ 2018-11-21 11:01       ` Ni, Ruiyu
  0 siblings, 0 replies; 7+ messages in thread
From: Ni, Ruiyu @ 2018-11-21 11:01 UTC (permalink / raw)
  To: Kinney, Michael D, Leif Lindholm
  Cc: edk2-devel@lists.01.org, Gao, Liming, Richardson, Brian,
	Cetola, Stephano, Laszlo Ersek

On 11/21/2018 7:21 AM, Kinney, Michael D wrote:
> Ray,
> 
> One additional note.  There are some tools in
> BaseTools that are only used by Duet.  We should
> remove those tools too.

Good points. Thanks for the reminding.
> 
> Mike
> 

-- 
Thanks,
Ray


^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2018-11-21 10:59 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-11-20  5:27 [RFC] Proposal to remove DuetPkg Ni, Ruiyu
2018-11-20  8:46 ` Laszlo Ersek
2018-11-20  9:57 ` Leif Lindholm
2018-11-20 23:20   ` Kinney, Michael D
2018-11-20 23:21     ` Kinney, Michael D
2018-11-21 11:01       ` Ni, Ruiyu
2018-11-21  0:59     ` Andrew Fish

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox