Push at 8ac46e4ef76ee56778430bb3dbcc545cce49d208 From: Michael Zimmermann [mailto:[email protected]] Sent: Tuesday, May 17, 2016 3:55 PM To: Gao, Liming <[email protected]> Cc: [email protected] <[email protected]> Subject: Re: [edk2] [PATCH 0/2] add '!include' support to tools_def.txt parser
I sent v3 with relative path support. Thanks Michael On Mon, May 16, 2016 at 8:19 AM, Gao, Liming <[email protected]<mailto:[email protected]>> wrote: Michael: In DSC/FDF, if the file path is relative, Build tool will find it from $(WORKSPACE) and $(PACKAGES_PATH). I expect the same way is in tools_def.txt file. Thanks Liming > -----Original Message----- > From: edk2-devel > [mailto:[email protected]<mailto:[email protected]>] > On Behalf Of > Michael Zimmermann > Sent: Monday, May 16, 2016 2:00 PM > To: Gao, Liming <[email protected]<mailto:[email protected]>> > Cc: [email protected]<mailto:[email protected]> > <[email protected]<mailto:[email protected]>> > Subject: Re: [edk2] [PATCH 0/2] add '!include' support to tools_def.txt parser > > I guess you mean relative to the currently parsed file? > It doesn't, I'll make a new patch with support for that. > > On Mon, May 16, 2016 at 4:30 AM, Gao, Liming > <[email protected]<mailto:[email protected]>> > wrote: > > > Does it support the relative path? > > > > > -----Original Message----- > > > From: edk2-devel > > > [mailto:[email protected]<mailto:[email protected]>] > > > On Behalf Of > > > Michael Zimmermann > > > Sent: Saturday, May 14, 2016 5:49 PM > > > To: [email protected]<mailto:[email protected]> > > > <[email protected]<mailto:[email protected]>> > > > Subject: [edk2] [PATCH 0/2] add '!include' support to tools_def.txt > > parser > > > > > > this is a response to this github issue: > > > https://github.com/tianocore/edk2/issues/77 > > > > > > this patch just adds !include support to tools_def.txt like it's > > supported > > > for FDF and DSC files already. > > > > > > the split of the current tools_def.txt into multiple per-toolchain files > > > can be done in another patch. > > > > > > Thanks > > > Michael > > > > > > Michael Zimmermann (2): > > > BaseTools: fix incorrect documentation about return value of > > > 'LoadToolDefFile' > > > BaseTools: add '!include' support to tools_def.txt parser > > > > > > BaseTools/Source/Python/Common/ToolDefClassObject.py | 51 > > > +++++++++++++++++++++++++++++++++------------------ > > > 1 file changed, 33 insertions(+), 18 deletions(-) > > > -- > > > 2.8.2 > > > _______________________________________________ > > > edk2-devel mailing list > > > [email protected]<mailto:[email protected]> > > > https://lists.01.org/mailman/listinfo/edk2-devel > > > _______________________________________________ > edk2-devel mailing list > [email protected]<mailto:[email protected]> > https://lists.01.org/mailman/listinfo/edk2-devel _______________________________________________ edk2-devel mailing list [email protected] https://lists.01.org/mailman/listinfo/edk2-devel
