[edk2-devel] [PATCH] [MdePkg/Protocols]: New interface, EFI encodings to PCI Plat protocol

[Ni, Ray]

Some unclear parts in the document:
1. Document says title begins with "staging/branch". The "branch" means the "branch" word itself
     or should be replaced with a concrete branch name? I prefer "branch" word. Reasons in previous mail.
    NOTE: this affects the individual patch owner.
2. Who owns the staging/master branch sync to edk2/master? Document says the staging/branch is
     based on staging/master but I guess usually staging patch owner creates staging/branch based on edk2/master.

Thanks,
Ray

> -----Original Message-----
> From: Gao, Liming
> Sent: Friday, July 5, 2019 10:49 AM
> To: Ni, Ray <ray.ni at intel.com>; devel at edk2.groups.io; lersek at redhat.com;
> Javeed, Ashraf <ashraf.javeed at intel.com>
> Cc: Kinney, Michael D <michael.d.kinney at intel.com>
> Subject: RE: [edk2-devel] [PATCH] [MdePkg/Protocols]: New interface, EFI
> encodings to PCI Plat protocol
> 
> Ray:
>   Yes. Please see https://github.com/tianocore/edk2-staging
> 
> Thanks
> Liming
> >-----Original Message-----
> >From: Ni, Ray
> >Sent: Friday, July 05, 2019 10:39 AM
> >To: Gao, Liming <liming.gao at intel.com>; devel at edk2.groups.io;
> >lersek at redhat.com; Javeed, Ashraf <ashraf.javeed at intel.com>
> >Cc: Kinney, Michael D <michael.d.kinney at intel.com>
> >Subject: RE: [edk2-devel] [PATCH] [MdePkg/Protocols]: New interface,
> >EFI encodings to PCI Plat protocol
> >
> >Liming,
> >I understand Laszlo's comment requiring repo name in the patch title to
> >tell everyone where the code will be.
> >But before the patch is pushed, the patch owner in theory cannot
> >guarantee the branch can be successfully created after the R-b is got.
> >Because someone may create a branch with the same name.
> >
> >Is there any public document for the patch which goes to staging repo?
> >
> >Thanks,
> >Ray
> >
> >> -----Original Message-----
> >> From: Gao, Liming
> >> Sent: Friday, July 5, 2019 10:13 AM
> >> To: devel at edk2.groups.io; lersek at redhat.com; Javeed, Ashraf
> >> <ashraf.javeed at intel.com>
> >> Cc: Kinney, Michael D <michael.d.kinney at intel.com>; Ni, Ray
> >> <ray.ni at intel.com>
> >> Subject: RE: [edk2-devel] [PATCH] [MdePkg/Protocols]: New interface,
> >> EFI encodings to PCI Plat protocol
> >>
> >> And, please also specify branch name in edk2-staging.
> >>
> >> The subject title likes : [PATCH edk2-staging\BranchName]
> >> [MdePkg/Protocols]: New interface, EFI encodings to PCI Plat protocol
> >>
> >> >-----Original Message-----
> >> >From: devel at edk2.groups.io [mailto:devel at edk2.groups.io] On Behalf
> >> >Of Laszlo Ersek
> >> >Sent: Friday, July 05, 2019 12:25 AM
> >> >To: devel at edk2.groups.io; Javeed, Ashraf <ashraf.javeed at intel.com>
> >> >Cc: Kinney, Michael D <michael.d.kinney at intel.com>; Gao, Liming
> >> ><liming.gao at intel.com>; Ni, Ray <ray.ni at intel.com>
> >> >Subject: Re: [edk2-devel] [PATCH] [MdePkg/Protocols]: New interface,
> >> >EFI encodings to PCI Plat protocol
> >> >
> >> >On 07/04/19 15:20, Javeed, Ashraf wrote:
> >> >> Please note that this new feature change would be moved to
> >> >> Edk2-staging
> >> >after obtaining the Reviewed-By.
> >> >
> >> >If the feature targets edk2-staging, then please use the following
> >> >subject prefix:
> >> >
> >> >[PATCH edk2-staging]
> >> >
> >> >rather than just
> >> >
> >> >[PATCH].
> >> >
> >> >Thanks.
> >> >Laszlo
> >> >
> >> >


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.

View/Reply Online (#43327): https://edk2.groups.io/g/devel/message/43327
Mute This Topic: https://groups.io/mt/32307674/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-