API: bugref:11116 Addendum to r174528. Do not disable the serial port, just disconnect and clear the host file path. Plus a grammar fix in the warning message.

baserkan · baserkan · commit 3deb5c990736 · 2026-07-14T11:41:05.000Z
svn:sync-xref-src-repo-rev: r174531
diff --git a/src/VBox/Main/src-server/ApplianceImplImport.cpp b/src/VBox/Main/src-server/ApplianceImplImport.cpp
@@ -1,4 +1,4 @@
-/* $Id: ApplianceImplImport.cpp 114692 2026-07-14 10:57:00Z serkan.bayraktar@oracle.com $ */
+/* $Id: ApplianceImplImport.cpp 114695 2026-07-14 11:41:05Z serkan.bayraktar@oracle.com $ */
 /** @file
  * IAppliance and IVirtualSystem COM class implementations.
  */
@@ -393,7 +393,7 @@ HRESULT Appliance::interpret()
                     if (port_it->portMode == PortMode_RawFile)
                     {
                         i_addWarning(tr("Virtual appliance \"%s\" was configured with serial port(s) "
-                                        "with \"raw file\" mode. These setting will not be imported"), vsysThis.strName.c_str());
+                                        "with \"raw file\" mode. This setting will not be imported."), vsysThis.strName.c_str());
                         break;
                     }
                 }
diff --git a/src/VBox/Main/xml/Settings.cpp b/src/VBox/Main/xml/Settings.cpp
@@ -1,4 +1,4 @@
-/* $Id: Settings.cpp 114692 2026-07-14 10:57:00Z serkan.bayraktar@oracle.com $ */
+/* $Id: Settings.cpp 114695 2026-07-14 11:41:05Z serkan.bayraktar@oracle.com $ */
 /** @file
  * Settings File Manipulation API.
  *
@@ -9706,7 +9706,6 @@ void MachineConfigFile::sanitizeImportedSerialPorts()
         if (port.portMode == PortMode_RawFile)
         {
             port.portMode = PortMode_Disconnected;
-            port.fEnabled = false;
             port.strPath.setNull();
         }
     }

Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,4 @@`
`1`		`-/* $Id: ApplianceImplImport.cpp 114692 2026-07-14 10:57:00Z serkan.bayraktar@oracle.com $ */`
	`1`	`+/* $Id: ApplianceImplImport.cpp 114695 2026-07-14 11:41:05Z serkan.bayraktar@oracle.com $ */`
`2`	`2`	`/** @file`
`3`	`3`	`* IAppliance and IVirtualSystem COM class implementations.`
`4`	`4`	`*/`
`@@ -393,7 +393,7 @@ HRESULT Appliance::interpret()`
`393`	`393`	`if (port_it->portMode == PortMode_RawFile)`
`394`	`394`	`{`
`395`	`395`	`i_addWarning(tr("Virtual appliance \"%s\" was configured with serial port(s) "`
`396`		`- "with \"raw file\" mode. These setting will not be imported"), vsysThis.strName.c_str());`
	`396`	`+ "with \"raw file\" mode. This setting will not be imported."), vsysThis.strName.c_str());`
`397`	`397`	`break;`
`398`	`398`	`}`
`399`	`399`	`}`
Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,4 @@`
`1`		`-/* $Id: Settings.cpp 114692 2026-07-14 10:57:00Z serkan.bayraktar@oracle.com $ */`
	`1`	`+/* $Id: Settings.cpp 114695 2026-07-14 11:41:05Z serkan.bayraktar@oracle.com $ */`
`2`	`2`	`/** @file`
`3`	`3`	`* Settings File Manipulation API.`
`4`	`4`	`*`
`@@ -9706,7 +9706,6 @@ void MachineConfigFile::sanitizeImportedSerialPorts()`
`9706`	`9706`	`if (port.portMode == PortMode_RawFile)`
`9707`	`9707`	`{`
`9708`	`9708`	`port.portMode = PortMode_Disconnected;`
`9709`		`- port.fEnabled = false;`
`9710`	`9709`	`port.strPath.setNull();`
`9711`	`9710`	`}`
`9712`	`9711`	`}`