Changeset 105266 in vbox for trunk/src/VBox/Frontends/VBoxManage

Timestamp:

Jul 11, 2024 7:49:37 AM (10 months ago)

Author:

vboxsync

svn:sync-xref-src-repo-rev:

163915

Message:

Recording: Implemented support for a dedicated progress object, which is exposed to API clients. This can be used for better tracking the recording progress as well as for error reporting. The RecordingSettings API also now has a dedicated start() method to start recording, as well as support for attaching to an already ongoing recording by retrieving the progress object at a later time. Adapted FE/Qt (draft, see @todos), FE/VBoxManage and the Validation Kit testdriver to the new APIs. VBoxManage also can attach to an ongoing recording now. The recording progress object also will have multiple operations to get the recording progress for convenience. bugref:10718

Location:

trunk/src/VBox/Frontends/VBoxManage

Files:

• VBoxManageControlVM.cpp (modified) (2 diffs)
• VBoxManageInfo.cpp (modified) (1 diff)

trunk/src/VBox/Frontends/VBoxManage/VBoxManageControlVM.cpp

@@ -1797 +1797 @@
                  || !strcmp(a->argv[1], "videocap") /* legacy command */)
         {
+            if (!strcmp(a->argv[1], "videocap"))
+                RTMsgWarning(ControlVM::tr("Sub command 'videocap' is deprecated -- please use 'recording' instead ."));
+
             if (a->argc < 3)
             {
@@ -1815 +1818 @@
             /* Note: For now all screens have the same configuration. */
 
-            /*
-             * Note: Commands starting with "vcp" are the deprecated versions and are
-             *       kept to ensure backwards compatibility.
-             */
             bool fEnabled;
             if (RT_SUCCESS(parseBool(a->argv[2], &fEnabled)))
             {
-                setCurrentSubcommand(HELP_SCOPE_CONTROLVM_RECORDING);
+                //setCurrentSubcommand(HELP_SCOPE_CONTROLVM_RECORDING);
                 CHECK_ERROR_RET(recordingSettings, COMSETTER(Enabled)(fEnabled), RTEXITCODE_FAILURE);
+
+                if (fEnabled)
+                    RTPrintf(ControlVM::tr("Recording enabled. Use 'start' to start recording.\n"));
+            }
+            else if (!strcmp(a->argv[2], "start"))
+            {
+                //setCurrentSubcommand(HELP_SCOPE_CONTROLVM_RECORDING_START);
+                bool fWait = false;
+                if (a->argc >= 4 && !strcmp(a->argv[3], "--wait"))
+                    fWait = true;
+
+                ComPtr<IProgress> progress;
+                CHECK_ERROR_BREAK(recordingSettings, Start(progress.asOutParam()));
+
+                if (fWait)
+                {
+                    hrc = showProgress(progress, SHOW_PROGRESS_OPS);
+                    CHECK_PROGRESS_ERROR(progress, (ControlVM::tr("Recording failed.")));
+                }
+                else
+                    RTPrintf(ControlVM::tr("Recording started (detacted).\n"));
+            }
+            else if (!strcmp(a->argv[2], "stop"))
+            {
+                //setCurrentSubcommand(HELP_SCOPE_CONTROLVM_RECORDING_STOP);
+                ComPtr<IProgress> progress;
+                CHECK_ERROR_BREAK(recordingSettings, COMGETTER(Progress)(progress.asOutParam()));
+                CHECK_ERROR_BREAK(progress, Cancel());
+            }
+            else if (!strcmp(a->argv[2], "attach"))
+            {
+                //setCurrentSubcommand(HELP_SCOPE_CONTROLVM_RECORDING_ATTACH);
+                ComPtr<IProgress> progress;
+                CHECK_ERROR_BREAK(recordingSettings, COMGETTER(Progress)(progress.asOutParam()));
+                hrc = showProgress(progress, SHOW_PROGRESS_OPS);
             }
             else if (!strcmp(a->argv[2], "screens"))

trunk/src/VBox/Frontends/VBoxManage/VBoxManageInfo.cpp

@@ -2899 +2899 @@
         CHECK_ERROR_RET(machine, COMGETTER(RecordingSettings)(recordingSettings.asOutParam()), hrc);
 
-        BOOL  fEnabled;
+        BOOL fStarted = FALSE;
+        ComPtr<IProgress> progress;
+        hrc = recordingSettings->COMGETTER(Progress)(progress.asOutParam());
+        if (SUCCEEDED(hrc))
+        {
+            hrc = progress->COMGETTER(Completed)(&fStarted);
+            fStarted = !fStarted;
+        }
+        SHOW_BOOL_VALUE_EX("recording_started", Info::tr("Recording status:"), fStarted, Info::tr("started"), Info::tr("stopped"));
+
+        BOOL fEnabled;
         CHECK_ERROR_RET(recordingSettings, COMGETTER(Enabled)(&fEnabled), hrc);
         SHOW_BOOL_VALUE_EX("recording_enabled", Info::tr("Recording enabled:"), fEnabled, Info::tr("yes"), Info::tr("no"));