get_contents: use list instead of set for the partitions

Running get_contents multiple times with the same input files generate
different output files. This is because the list of partitions from
partitions.xml are parsed and stored in a set data structure, which is
unsorted. Looping on the set will return different output.

Convert the set into a list instead which leads to:
1. the output is sorted, and thus the generated file is stable
2. the order from partitions.xml is preserved

While making this change, I decided to put dict elements in the list
instead of a list, it's easier to refer to field by name than by
index.

Signed-off-by: Nicolas Dechesne <[email protected]>

Author: ndechesne

gen_contents.py

@@ -37,14 +37,13 @@ def UpdateMetaData(TemplateRoot, PartitionRoot):
             DefaultStorageType = ChipId.get('storage_type')
 
     PhyPartition = PartitionRoot.findall('physical_partition')
-    PartitionsSet = set()
-    Partitions = PartitionRoot.findall('physical_partition/partition')
-    for partition in Partitions:
+    Partitions = []
+    for partition in PartitionRoot.findall('physical_partition/partition'):
         label = partition.get('label')
         filename = partition.get('filename')
         if label and filename:
-            PartitionsSet.add((label, filename))
-    print(f"PartitionsSet: {PartitionsSet}")
+            Partitions.append({'label': label, 'filename': filename})
+    print(f"Partitions: {Partitions}")
 
     builds = TemplateRoot.findall('builds_flat/build')
     for build in builds:
@@ -56,11 +55,11 @@ def UpdateMetaData(TemplateRoot, PartitionRoot):
         if DownloadFile is not None:
             build.remove(DownloadFile)
             # Partition entires
-            for Partition in PartitionsSet:
+            for Partition in Partitions:
                 new_download_file = ET.SubElement(build, "download_file")
-                new_download_file.set("fastboot_complete", Partition[0])
+                new_download_file.set("fastboot_complete", Partition['label'])
                 new_file_name = ET.SubElement(new_download_file, "file_name")
-                new_file_name.text = Partition[1]
+                new_file_name.text = Partition['filename']
                 new_file_path = ET.SubElement(new_download_file, "file_path")
                 new_file_path.text = "."
             # GPT Main & GPT Backup entries