[edk2-devel] [PATCH v1 1/1] BaseTools/Ecc: Replace deprecated function time.clock()

[Michael Kubacki]

Hi Phil,

It has been a policy from Microsoft contributors to not include a year 
in the copyright header you will find this in many other existing files 
in edk2, I am simply aligning to that practice.

Bob is the maintainer and as maintainer he chose to submit the patch (it 
has been on the list since May 8th). It has been relatively common to 
reply to the patch thread to indicate it was submitted with commit 
details. I noticed the details were not updated in the BZ either in this 
case so I have done that and closed the BZ.

Thanks,
Michael


On 5/14/2020 12:37 AM, Philippe Mathieu-Daudé wrote:
> +Michael & Laszlo.
> 
> On 5/13/20 4:15 AM, Bob Feng wrote:
>> Reviewed-by: Bob Feng <bob.c.feng at intel.com>
>>
>> -----Original Message-----
>> From: michael.kubacki at outlook.com <michael.kubacki at outlook.com>
>> Sent: Saturday, May 9, 2020 10:23 AM
>> To: devel at edk2.groups.io
>> Cc: Feng, Bob C <bob.c.feng at intel.com>; Gao, Liming 
>> <liming.gao at intel.com>
>> Subject: [PATCH v1 1/1] BaseTools/Ecc: Replace deprecated function 
>> time.clock()
>>
>> From: Michael Kubacki <michael.kubacki at microsoft.com>
>>
>> REF:https://bugzilla.tianocore.org/show_bug.cgi?id=2707
>>
>> Ecc fails with Python 3.8 because it uses the deprecated time.clock() 
>> function - https://docs.python.org/3.7/library/time.html#time.clock
>>
>> This change updates EccMain.py to use time.perf_counter().
>>
>> Cc: Bob Feng <bob.c.feng at intel.com>
>> Cc: Liming Gao <liming.gao at intel.com>
>> Signed-off-by: Michael Kubacki <michael.kubacki at microsoft.com>
>> ---
>>   BaseTools/Source/Python/Ecc/EccMain.py | 5 +++--
>>   1 file changed, 3 insertions(+), 2 deletions(-)
>>
>> diff --git a/BaseTools/Source/Python/Ecc/EccMain.py 
>> b/BaseTools/Source/Python/Ecc/EccMain.py
>> index 560bb9fd7a31..72edbea3b883 100644
>> --- a/BaseTools/Source/Python/Ecc/EccMain.py
>> +++ b/BaseTools/Source/Python/Ecc/EccMain.py
>> @@ -2,6 +2,7 @@
>>   # This file is used to be the main entrance of ECC tool  #  # 
>> Copyright (c) 2009 - 2018, Intel Corporation. All rights reserved.<BR>
>> +# Copyright (c) Microsoft Corporation.<BR>
> 
> When trying to apply this patch I got:
> 
> error: patch failed: BaseTools/Source/Python/Ecc/EccMain.py:2
> 
> Because it has already been merged as commit 
> 242ab73d7f255d5d859eaf74a23b9d68c686d177.
> 
> 1/ The copyright comment misses a year range
> 
> 2/ It is annoying to not know a patch has been merged.
> What is the expected workflow? Pull repository, look at patches merged, 
> then go on the list to review the un-merged ones? I suppose this won't 
> matter once workflow is switched to GitHub. Still, not very practical.
> 
> Regards,
> 
> Phil.
> 
>>   # SPDX-License-Identifier: BSD-2-Clause-Patent  #
>> @@ -406,9 +407,9 @@ if __name__ == '__main__':
>>       EdkLogger.Initialize()
>>       EdkLogger.IsRaiseError = False
>> -    StartTime = time.clock()
>> +    StartTime = time.perf_counter()
>>       Ecc = Ecc()
>> -    FinishTime = time.clock()
>> +    FinishTime = time.perf_counter()
>>       BuildDuration = time.strftime("%M:%S", 
>> time.gmtime(int(round(FinishTime - StartTime))))
>>       EdkLogger.quiet("\n%s [%s]" % (time.strftime("%H:%M:%S, %b.%d 
>> %Y", time.localtime()), BuildDuration))
>> -- 
>> 2.16.3.windows.1
>>
>>
>> 
>>
> 

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

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