commit 99e8b983ba4fc03dcb2cb89e8c96fce26e77b304
author Jeremy Kerr <jk@codeconstruct.com.au> Sun Jun 04 15:22:54 2023 +0800
committer ManojKiran Eda <manojkiran.eda@gmail.com> Tue Jul 25 12:03:50 2023 +0000
tree 1bad93cf4cb014b73041d7478f72bb2e8153546a
parent b1475015b0d1c8b58e2cb28aaa2d63bde6693ba5

tools: fw-update: allow ISO 8610 date formats

Currently, the ReleaseDateTime data is expected to be in a
"%d/%m/%Y %H:%M:%S" format.

However, this is ambiguous (ie., is locale-specific).

This change adds supoort to allow an ISO 8601 format of:

 "%Y-%m-%dT%H:%M:%S"

and a relaxed version with the 'T' separator replaced with a space:

 "%Y-%m-%d %H:%M:%S"

Update the readme and example json to switch to the preferred format.

Signed-off-by: Jeremy Kerr <jk@codeconstruct.com.au>
Change-Id: Ica3ae824979df6b666b8abb619ceaedffd90b374

tools/fw-update/pldm_fwup_pkg_creator.py

diff --git a/tools/fw-update/pldm_fwup_pkg_creator.py b/tools/fw-update/pldm_fwup_pkg_creator.py
index 0209f1a..b61afa5 100755
--- a/tools/fw-update/pldm_fwup_pkg_creator.py
+++ b/tools/fw-update/pldm_fwup_pkg_creator.py
@@ -186,9 +186,22 @@
     )
 
     if release_date_time_str is not None:
-        release_date_time = datetime.strptime(
-            release_date_time_str, "%d/%m/%Y %H:%M:%S"
-        )
+        formats = [
+            "%Y-%m-%dT%H:%M:%S",
+            "%Y-%m-%d %H:%M:%S",
+            "%d/%m/%Y %H:%M:%S",
+        ]
+        release_date_time = None
+        for fmt in formats:
+            try:
+                release_date_time = datetime.strptime(
+                    release_date_time_str, fmt
+                )
+                break
+            except ValueError:
+                pass
+        if release_date_time is None:
+            sys.exit("Can't parse release date '%s'" % release_date_time_str)
     else:
         release_date_time = datetime.now()