On Thu, Aug 27, 2015 at 12:54:13PM +0200, Laszlo Ersek wrote:
> Gary,
> 
> On 08/27/15 05:41, Gary Ching-Pang Lin wrote:
> > The HTTP driver now needs the HTTP utilities driver to parse the
> > headers of HTTP requests.
> 
> Can you point out the SVN rev / git commit that makes this necessary?
> I'd like to reference that patch in this commit message.
> 
It's r18316 / 5ca29abe5297. That commit removed HttpUtilities.* from
HttpDxe and make the driver consume EFI_HTTP_UTILITIES_PROTOCOL instead.

Gary Lin

> You don't need to resubmit, I can edit the reference into the commit
> message when I commit the patch.
> 
> Thanks
> Laszlo
> 
> 
> > Add the driver into OVMF so that the
> > HTTP driver can work properly.
> > 
> > Contributed-under: TianoCore Contribution Agreement 1.0
> > Signed-off-by: Gary Ching-Pang Lin <g...@suse.com>
> > ---
> >  OvmfPkg/OvmfPkgIa32.dsc    | 1 +
> >  OvmfPkg/OvmfPkgIa32.fdf    | 1 +
> >  OvmfPkg/OvmfPkgIa32X64.dsc | 1 +
> >  OvmfPkg/OvmfPkgIa32X64.fdf | 1 +
> >  OvmfPkg/OvmfPkgX64.dsc     | 1 +
> >  OvmfPkg/OvmfPkgX64.fdf     | 1 +
> >  6 files changed, 6 insertions(+)
> > 
> > diff --git a/OvmfPkg/OvmfPkgIa32.dsc b/OvmfPkg/OvmfPkgIa32.dsc
> > index 5741eac..d92a327 100644
> > --- a/OvmfPkg/OvmfPkgIa32.dsc
> > +++ b/OvmfPkg/OvmfPkgIa32.dsc
> > @@ -562,6 +562,7 @@ [Components]
> >  !endif
> >  !if $(HTTP_BOOT_ENABLE) == TRUE
> >    NetworkPkg/DnsDxe/DnsDxe.inf
> > +  NetworkPkg/HttpUtilitiesDxe/HttpUtilitiesDxe.inf
> >    NetworkPkg/HttpDxe/HttpDxe.inf
> >    NetworkPkg/HttpBootDxe/HttpBootDxe.inf
> >  !endif
> > diff --git a/OvmfPkg/OvmfPkgIa32.fdf b/OvmfPkg/OvmfPkgIa32.fdf
> > index 0e4ee49..f3dcfff 100644
> > --- a/OvmfPkg/OvmfPkgIa32.fdf
> > +++ b/OvmfPkg/OvmfPkgIa32.fdf
> > @@ -326,6 +326,7 @@ [FV.DXEFV]
> >  !endif
> >  !if $(HTTP_BOOT_ENABLE) == TRUE
> >    INF  NetworkPkg/DnsDxe/DnsDxe.inf
> > +  INF  NetworkPkg/HttpUtilitiesDxe/HttpUtilitiesDxe.inf
> >    INF  NetworkPkg/HttpDxe/HttpDxe.inf
> >    INF  NetworkPkg/HttpBootDxe/HttpBootDxe.inf
> >  !endif
> > diff --git a/OvmfPkg/OvmfPkgIa32X64.dsc b/OvmfPkg/OvmfPkgIa32X64.dsc
> > index 753ab27..1c39cc7 100644
> > --- a/OvmfPkg/OvmfPkgIa32X64.dsc
> > +++ b/OvmfPkg/OvmfPkgIa32X64.dsc
> > @@ -569,6 +569,7 @@ [Components.X64]
> >  !endif
> >  !if $(HTTP_BOOT_ENABLE) == TRUE
> >    NetworkPkg/DnsDxe/DnsDxe.inf
> > +  NetworkPkg/HttpUtilitiesDxe/HttpUtilitiesDxe.inf
> >    NetworkPkg/HttpDxe/HttpDxe.inf
> >    NetworkPkg/HttpBootDxe/HttpBootDxe.inf
> >  !endif
> > diff --git a/OvmfPkg/OvmfPkgIa32X64.fdf b/OvmfPkg/OvmfPkgIa32X64.fdf
> > index 74412d4..3a509da 100644
> > --- a/OvmfPkg/OvmfPkgIa32X64.fdf
> > +++ b/OvmfPkg/OvmfPkgIa32X64.fdf
> > @@ -326,6 +326,7 @@ [FV.DXEFV]
> >  !endif
> >  !if $(HTTP_BOOT_ENABLE) == TRUE
> >    INF  NetworkPkg/DnsDxe/DnsDxe.inf
> > +  INF  NetworkPkg/HttpUtilitiesDxe/HttpUtilitiesDxe.inf
> >    INF  NetworkPkg/HttpDxe/HttpDxe.inf
> >    INF  NetworkPkg/HttpBootDxe/HttpBootDxe.inf
> >  !endif
> > diff --git a/OvmfPkg/OvmfPkgX64.dsc b/OvmfPkg/OvmfPkgX64.dsc
> > index 540333d..778ea28 100644
> > --- a/OvmfPkg/OvmfPkgX64.dsc
> > +++ b/OvmfPkg/OvmfPkgX64.dsc
> > @@ -567,6 +567,7 @@ [Components]
> >  !endif
> >  !if $(HTTP_BOOT_ENABLE) == TRUE
> >    NetworkPkg/DnsDxe/DnsDxe.inf
> > +  NetworkPkg/HttpUtilitiesDxe/HttpUtilitiesDxe.inf
> >    NetworkPkg/HttpDxe/HttpDxe.inf
> >    NetworkPkg/HttpBootDxe/HttpBootDxe.inf
> >  !endif
> > diff --git a/OvmfPkg/OvmfPkgX64.fdf b/OvmfPkg/OvmfPkgX64.fdf
> > index e9ba97b..153dd82 100644
> > --- a/OvmfPkg/OvmfPkgX64.fdf
> > +++ b/OvmfPkg/OvmfPkgX64.fdf
> > @@ -326,6 +326,7 @@ [FV.DXEFV]
> >  !endif
> >  !if $(HTTP_BOOT_ENABLE) == TRUE
> >    INF  NetworkPkg/DnsDxe/DnsDxe.inf
> > +  INF  NetworkPkg/HttpUtilitiesDxe/HttpUtilitiesDxe.inf
> >    INF  NetworkPkg/HttpDxe/HttpDxe.inf
> >    INF  NetworkPkg/HttpBootDxe/HttpBootDxe.inf
> >  !endif
> > 
> 
> 
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel

Reply via email to