From 4cce5200037b773ffa638135ff6e706648eeddc8 Mon Sep 17 00:00:00 2001
From: Jari Nippula <jarix@ssrc.tii.ae>
Date: Mon, 23 Sep 2024 14:17:34 +0300
Subject: [PATCH] Fix participant profile ame replacement

Allow profile_name be in any position in participant tag definition
instead of assuming it is the first argument

Cleanup one unnecessary list generation in line iteration
---
 combine_default_profiles.py | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/combine_default_profiles.py b/combine_default_profiles.py
index 4e10a03d..6e69abf3 100755
--- a/combine_default_profiles.py
+++ b/combine_default_profiles.py
@@ -34,14 +34,14 @@ def cleanup_temporary_files():
 keep_config = True
 with open(default_profiles_file, "r") as in_f:
   with open(agent_refs_tmp_file, "w") as out_f:
-    for line in in_f.readlines():
+    for line in in_f:
       line_str = line.strip()
 
       # Replace participant profile name
-      if line_str.startswith("<participant profile_name="):
+      if line_str.startswith("<participant") and "profile_name=" in line_str:
         line = re.sub(
-            '<participant profile_name="\S+"',
-            '<participant profile_name="default_xrce_participant"',
+            'profile_name="\S+"',
+            'profile_name="default_xrce_participant"',
             line)
 
       # Remove data_reader and data_writer configs