* Re: [edk2-devel] [PATCH] IntelFsp2Pkg/GenCfgOpt.py: Incremental build with UPD in sub DSC.
2020-12-08 2:15 Chiu, Chasel
@ 2020-12-08 3:28 ` Yuwei Chen
0 siblings, 0 replies; 2+ messages in thread
From: Yuwei Chen @ 2020-12-08 3:28 UTC (permalink / raw)
To: devel@edk2.groups.io, Chiu, Chasel
Cc: Ma, Maurice, Desimone, Nathaniel L, Zeng, Star
Reviewed-by: Yuwei Chen <yuwei.chen@intel.com>
Tested-by: Yuwei Chen <yuwei.chen@intel.com>
> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Chiu,
> Chasel
> Sent: Tuesday, December 8, 2020 10:16 AM
> To: devel@edk2.groups.io
> Cc: Chiu, Chasel <chasel.chiu@intel.com>; Ma, Maurice
> <maurice.ma@intel.com>; Desimone, Nathaniel L
> <nathaniel.l.desimone@intel.com>; Zeng, Star <star.zeng@intel.com>
> Subject: [edk2-devel] [PATCH] IntelFsp2Pkg/GenCfgOpt.py: Incremental
> build with UPD in sub DSC.
>
> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3107
>
> Current script only compares main DSC and output file datetime to
> determine if re-generation required or not.
> When UPD defined in sub DSC and was modified current script cannot detect
> and will not re-generate output files which caused incremental build issue.
>
> Since UPD can be defined in any sub DSC the script has been updated to
> compare all DSC datetime with output files to determine re-generation is
> needed or not.
>
> Cc: Maurice Ma <maurice.ma@intel.com>
> Cc: Nate DeSimone <nathaniel.l.desimone@intel.com>
> Cc: Star Zeng <star.zeng@intel.com>
> Signed-off-by: Chasel Chiu <chasel.chiu@intel.com>
> ---
> IntelFsp2Pkg/Tools/GenCfgOpt.py | 13 +++++++++++--
> 1 file changed, 11 insertions(+), 2 deletions(-)
>
> diff --git a/IntelFsp2Pkg/Tools/GenCfgOpt.py
> b/IntelFsp2Pkg/Tools/GenCfgOpt.py index af7e14a10a..a0b8bba81e 100644
> --- a/IntelFsp2Pkg/Tools/GenCfgOpt.py
> +++ b/IntelFsp2Pkg/Tools/GenCfgOpt.py
> @@ -313,6 +313,7 @@ EndList
> self._DscFile = '' self._FvDir = '' self._MapVer = 0+
> self._DscTime = 0 def ParseMacros (self, MacroDefStr): # ['-DABC=1',
> '-D', 'CFG_DEBUG=1', '-D', 'CFG_OUTDIR=Build']@@ -423,6 +424,9 @@
> EndList
> self._DscFile = DscFile self._FvDir = FvDir + # Initial DSC time
> is parent DSC time.+ self._DscTime = os.path.getmtime(DscFile)+
> IsDefSect = False IsPcdSect = False IsUpdSect = False@@ -
> 530,6 +534,12 @@ EndList
> if IncludeDsc == None:
> print("ERROR: Cannot open file '%s'" % IncludeFilePath)
> raise SystemExit++ # Update DscTime when newer DSC
> time found.+ CurrentDscTime =
> os.path.getmtime(os.path.realpath(IncludeDsc.name))+
> if CurrentDscTime > self._DscTime:+ self._DscTime =
> CurrentDscTime+ NewDscLines = IncludeDsc.readlines()
> IncludeDsc.close() DscLines = NewDscLines +
> DscLines@@ -815,9 +825,8 @@ EndList
> if not os.path.exists(OutPutFile): NoFileChange = False else:-
> DscTime = os.path.getmtime(self._DscFile) OutputTime =
> os.path.getmtime(OutPutFile)- if DscTime > OutputTime:+ if
> self._DscTime > OutputTime: NoFileChange = False return
> NoFileChange --
> 2.28.0.windows.1
>
>
>
> -=-=-=-=-=-=
> Groups.io Links: You receive all messages sent to this group.
> View/Reply Online (#68417): https://edk2.groups.io/g/devel/message/68417
> Mute This Topic: https://groups.io/mt/78794912/4546272
> Group Owner: devel+owner@edk2.groups.io
> Unsubscribe: https://edk2.groups.io/g/devel/unsub [yuwei.chen@intel.com]
> -=-=-=-=-=-=
>
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [edk2-devel] [PATCH] IntelFsp2Pkg/GenCfgOpt.py: Incremental build with UPD in sub DSC.
[not found] <164E9C1EF323E769.6597@groups.io>
@ 2020-12-09 12:53 ` Chiu, Chasel
0 siblings, 0 replies; 2+ messages in thread
From: Chiu, Chasel @ 2020-12-09 12:53 UTC (permalink / raw)
To: devel@edk2.groups.io, Chiu, Chasel
Cc: Ma, Maurice, Desimone, Nathaniel L, Zeng, Star
Commit pushed: f95e80d832e923046c92cd6f0b8208cec147138e
> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Chiu, Chasel
> Sent: Tuesday, December 8, 2020 10:16 AM
> To: devel@edk2.groups.io
> Cc: Chiu, Chasel <chasel.chiu@intel.com>; Ma, Maurice
> <maurice.ma@intel.com>; Desimone, Nathaniel L
> <nathaniel.l.desimone@intel.com>; Zeng, Star <star.zeng@intel.com>
> Subject: [edk2-devel] [PATCH] IntelFsp2Pkg/GenCfgOpt.py: Incremental build
> with UPD in sub DSC.
>
> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3107
>
> Current script only compares main DSC and output file datetime to determine if
> re-generation required or not.
> When UPD defined in sub DSC and was modified current script cannot detect and
> will not re-generate output files which caused incremental build issue.
>
> Since UPD can be defined in any sub DSC the script has been updated to compare
> all DSC datetime with output files to determine re-generation is needed or not.
>
> Cc: Maurice Ma <maurice.ma@intel.com>
> Cc: Nate DeSimone <nathaniel.l.desimone@intel.com>
> Cc: Star Zeng <star.zeng@intel.com>
> Signed-off-by: Chasel Chiu <chasel.chiu@intel.com>
> ---
> IntelFsp2Pkg/Tools/GenCfgOpt.py | 13 +++++++++++--
> 1 file changed, 11 insertions(+), 2 deletions(-)
>
> diff --git a/IntelFsp2Pkg/Tools/GenCfgOpt.py
> b/IntelFsp2Pkg/Tools/GenCfgOpt.py index af7e14a10a..a0b8bba81e 100644
> --- a/IntelFsp2Pkg/Tools/GenCfgOpt.py
> +++ b/IntelFsp2Pkg/Tools/GenCfgOpt.py
> @@ -313,6 +313,7 @@ EndList
> self._DscFile = '' self._FvDir = '' self._MapVer = 0+
> self._DscTime = 0 def ParseMacros (self, MacroDefStr): # ['-DABC=1', '-
> D', 'CFG_DEBUG=1', '-D', 'CFG_OUTDIR=Build']@@ -423,6 +424,9 @@ EndList
> self._DscFile = DscFile self._FvDir = FvDir + # Initial DSC time is
> parent DSC time.+ self._DscTime = os.path.getmtime(DscFile)+
> IsDefSect = False IsPcdSect = False IsUpdSect = False@@ -
> 530,6 +534,12 @@ EndList
> if IncludeDsc == None:
> print("ERROR: Cannot open file '%s'" % IncludeFilePath)
> raise SystemExit++ # Update DscTime when newer DSC
> time found.+ CurrentDscTime =
> os.path.getmtime(os.path.realpath(IncludeDsc.name))+ if
> CurrentDscTime > self._DscTime:+ self._DscTime =
> CurrentDscTime+ NewDscLines = IncludeDsc.readlines()
> IncludeDsc.close() DscLines = NewDscLines + DscLines@@
> -815,9 +825,8 @@ EndList
> if not os.path.exists(OutPutFile): NoFileChange = False else:-
> DscTime = os.path.getmtime(self._DscFile) OutputTime =
> os.path.getmtime(OutPutFile)- if DscTime > OutputTime:+ if
> self._DscTime > OutputTime: NoFileChange = False return
> NoFileChange --
> 2.28.0.windows.1
>
>
>
> -=-=-=-=-=-=
> Groups.io Links: You receive all messages sent to this group.
> View/Reply Online (#68417): https://edk2.groups.io/g/devel/message/68417
> Mute This Topic: https://groups.io/mt/78794912/1777047
> Group Owner: devel+owner@edk2.groups.io
> Unsubscribe: https://edk2.groups.io/g/devel/unsub [chasel.chiu@intel.com] -=-
> =-=-=-=-=
>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2020-12-09 12:53 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <164E9C1EF323E769.6597@groups.io>
2020-12-09 12:53 ` [edk2-devel] [PATCH] IntelFsp2Pkg/GenCfgOpt.py: Incremental build with UPD in sub DSC Chiu, Chasel
2020-12-08 2:15 Chiu, Chasel
2020-12-08 3:28 ` [edk2-devel] " Yuwei Chen
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox