toggle quoted messageShow quoted text
Thanks for the valuable feedbacks and suggestions.
I will re-write script and re-send code review.
From: firstname.lastname@example.org [mailto:email@example.com] On Behalf Of Leif
Sent: Wednesday, August 14, 2019 2:30 AM
To: Gao, Liming <liming.gao@...>
Cc: Chiu, Chasel <chasel.chiu@...>; firstname.lastname@example.org; Feng, Bob C
<bob.c.feng@...>; email@example.com; Cetola, Stephano
<stephano.cetola@...>; Laszlo Ersek (lersek@...)
<lersek@...>; afish@...; Kinney, Michael D
Subject: Re: [edk2-devel] [RFC] BZ 2067 BaseTools/Scripts: Add
GetUtcDateTime.py for edk2-stable201908 stable tag.
I am not fundamentally opposed to merging a simple script that does not affect
other code in the tree.
*But* the fact that we have multiple occurrences of this this time around *is*
a bit of a concern for me. Yes, they won't affect the workings of anything else
as part of the release. But they will not have had any chance to be actually used
For this particular script, I am also not very keen on the implementation. It
manually parses the command line and prints usage instead of using argparse.
And it contains the sys.exit(Main()) antipattern, which does not play well with
Python3 asynchronous i/o (and hence is a bad habit to get into).
Since this script is truly trivial, I am OK for it to be included *if* it is rewritten
using argparse and not calling sys.exit.
On Tue, Aug 13, 2019 at 01:42:23PM +0000, Gao, Liming wrote:
This is a small helper script. I am OK to add it for edk2-stable201908 stabletag.
edk2-stable201908 stable tag.
From: Chiu, Chasel
Sent: Monday, August 12, 2019 3:45 PM
Cc: Gao, Liming <liming.gao@...>; Feng, Bob C
Subject: [RFC] BZ 2067 BaseTools/Scripts: Add GetUtcDateTime.py for
on August 8th:
I would like to add below simple script to 201908 stable tag, review was sent
for patching build time information in any binary.
A script that can return UTC date and time in ascii format which is convenient