public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
* DSC macro expansion
@ 2017-01-18 23:35 Huber, Brandon
  2017-01-19 10:28 ` Zhu, Yonghong
  0 siblings, 1 reply; 2+ messages in thread
From: Huber, Brandon @ 2017-01-18 23:35 UTC (permalink / raw)
  To: edk2-devel (edk2-devel@lists.01.org)

Hi All,

There seems to be a discrepancy between the DSC spec and the EDKII implementation regarding what macros are allowed in !include statements within a DSC file.

>From the spec (v1.26 section 2.2.5):

Macros, defined in this file, are not permitted in the path or file name of the !include statement, as these files are included prior to processing the file for macros. If the path starts with a "$" character, then one of the system environment variables, $(WORKSPACE), $(EDK_SOURCE), $(EFI_SOURCE), or $(ECP_SOURCE) is being used; only these system environment variables are permitted to start the path of the included file.
Yet from experimentation, all macros defined in DSC files seem to be permitted in !include statements by the build tools. Is the spec out of date, or is the code wrong?

I think the code in question is _GetMacros() in BaseTools/Source/Python/Workspace/WorkspaceDatabase.py<https://github.com/tianocore/edk2/commit/0d2711a69397d2971079121df4326d84736c181e#diff-7cfd5e9099421b207e7718d47950d40dR148> (https://github.com/tianocore/edk2/commit/0d2711a69397d2971079121df4326d84736c181e#diff-7cfd5e9099421b207e7718d47950d40dR148).

Thanks,
-Brandon



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

* Re: DSC macro expansion
  2017-01-18 23:35 DSC macro expansion Huber, Brandon
@ 2017-01-19 10:28 ` Zhu, Yonghong
  0 siblings, 0 replies; 2+ messages in thread
From: Zhu, Yonghong @ 2017-01-19 10:28 UTC (permalink / raw)
  To: Huber, Brandon, edk2-devel (edk2-devel@lists.01.org)

Hi Brandon,

Thanks for the info. I think the spec is out of date, I will propose to update the spec and file a document change bug on Bugzilla system.

Best Regards,
Zhu Yonghong

-----Original Message-----
From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of Huber, Brandon
Sent: Thursday, January 19, 2017 7:35 AM
To: edk2-devel (edk2-devel@lists.01.org) <edk2-devel@lists.01.org>
Subject: [edk2] DSC macro expansion

Hi All,

There seems to be a discrepancy between the DSC spec and the EDKII implementation regarding what macros are allowed in !include statements within a DSC file.

>From the spec (v1.26 section 2.2.5):

Macros, defined in this file, are not permitted in the path or file name of the !include statement, as these files are included prior to processing the file for macros. If the path starts with a "$" character, then one of the system environment variables, $(WORKSPACE), $(EDK_SOURCE), $(EFI_SOURCE), or $(ECP_SOURCE) is being used; only these system environment variables are permitted to start the path of the included file.
Yet from experimentation, all macros defined in DSC files seem to be permitted in !include statements by the build tools. Is the spec out of date, or is the code wrong?

I think the code in question is _GetMacros() in BaseTools/Source/Python/Workspace/WorkspaceDatabase.py<https://github.com/tianocore/edk2/commit/0d2711a69397d2971079121df4326d84736c181e#diff-7cfd5e9099421b207e7718d47950d40dR148> (https://github.com/tianocore/edk2/commit/0d2711a69397d2971079121df4326d84736c181e#diff-7cfd5e9099421b207e7718d47950d40dR148).

Thanks,
-Brandon

_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel


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

end of thread, other threads:[~2017-01-19 10:28 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-01-18 23:35 DSC macro expansion Huber, Brandon
2017-01-19 10:28 ` Zhu, Yonghong

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