Merge pull request #2962 from BrianLakstins/master

Move RunScript module to be configured first

Duplicati/Library/Main/Controller.cs

@@ -771,7 +771,19 @@ namespace Duplicati.Library.Main
             foreach (var k in qp.Keys)
                 conopts[(string)k] = qp[(string)k];
 
+            //// Since Configure in RunScript can alter the RawOptions, make sure it is first in the list for Configure
+            var LoadedModules = new List<KeyValuePair<bool, Interface.IGenericModule>>();
             foreach (var mx in m_options.LoadedModules)
+                if (mx.Value.ToString().ToLower().Contains("runscript"))
+                {
+                    LoadedModules.Insert(0, mx);
+                }
+                else
+                {
+                    LoadedModules.Add(mx);
+                }
+
+            foreach (var mx in LoadedModules)
                 if (mx.Key)
                 {
                     if (mx.Value is Library.Interface.IConnectionModule)

Duplicati/Library/Modules/Builtin/run-script-example.bat

@@ -52,7 +52,7 @@ REM All Duplicati options can be changed by the script by writing options to
 REM stdout (with echo or similar). Anything not starting with a double dash (--)
 REM will be ignored:
 REM echo "Hello! -- test, this line is ignored"
-REM echo "--new-option=""This will be a setting"""
+REM echo --new-option=This will be a setting
 
 REM Filters are supplied in the DUPLICAT__FILTER variable.
 REM The variable contains all filters supplied with --include and --exclude,