Re: Reminder for deleting 3 network drivers in MdeModulePkg

[Leif Lindholm <leif.lindholm@linaro.org>]

Hi Fu Siyuan,

That is fine too.
But I expect the change to be one to go in the week after a stable tag
has been made.

(My own philosophy is that one can be quite aggressive with removals,
because if it turns out to have been a bad idea, git revert can
quickly restore the tree to the state it was.)

Best Regards,

Leif

On Wed, Oct 31, 2018 at 02:58:47AM +0000, Fu, Siyuan wrote:
> Hi, Leif
> 
> Just notice that you are saying edk2-stable201811 tag. 
> 
> We need more time to collect community/marketing feedback on
> deleting these driver, so it may won't be able to catch the stable
> tag in next month.
> 
> BestRegards
> Fu Siyuan
> 
> 
> > -----Original Message-----
> > From: Fu, Siyuan
> > Sent: Wednesday, October 31, 2018 8:49 AM
> > To: Leif Lindholm <leif.lindholm@linaro.org>
> > Cc: edk2-devel@lists.01.org
> > Subject: RE: [edk2] Reminder for deleting 3 network drivers in
> > MdeModulePkg
> > 
> > Hi, Leif
> > 
> > I think checking the branch is a little better than the tag, because we
> > may still have critical bug fixes after the official release.
> > 
> > While any of them are not recommended, always using the NetworkPkg version
> > driver is the best choice.
> > 
> > I will modify the edk2 network wiki page for an updated sample DSC/FDF
> > section, if this patch could pass review w/o objection.
> > https://github.com/tianocore/tianocore.github.io/wiki/NetworkPkg-Getting-
> > Started-Guide
> > 
> > 
> > BestRegards
> > Fu Siyuan
> > 
> > 
> > > -----Original Message-----
> > > From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of
> > > Leif Lindholm
> > > Sent: Tuesday, October 30, 2018 6:47 PM
> > > To: Fu, Siyuan <siyuan.fu@intel.com>
> > > Cc: edk2-devel@lists.01.org
> > > Subject: Re: [edk2] Reminder for deleting 3 network drivers in
> > > MdeModulePkg
> > >
> > > Hi Fu Siyan,
> > >
> > > On Tue, Oct 30, 2018 at 08:11:52AM +0000, Fu, Siyuan wrote:
> > > > People who want to continue use these deleted drivers could go to
> > > > edk2/UDK2018 branch to get them, but it's *highly not recommended*.
> > >
> > > They should also be able to use edk2-stable201811, right?
> > >
> > > > *If you have any opposition on deleting these drivers, please let me
> > > > know before this weekend, thanks.*
> > >
> > > I am all for this change. The diffstat is very nice :)
> > >
> > > Best Regards,
> > >
> > > Leif
> > > _______________________________________________
> > > edk2-devel mailing list
> > > edk2-devel@lists.01.org
> > > https://lists.01.org/mailman/listinfo/edk2-devel