diff options
author | Boris Kolpackov <boris@codesynthesis.com> | 2021-04-09 18:37:37 +0200 |
---|---|---|
committer | Boris Kolpackov <boris@codesynthesis.com> | 2021-04-09 18:37:37 +0200 |
commit | c27ecbaefaa4fa6549fa5b9b70b460040e4a200e (patch) | |
tree | 904e1b921823377ea25fa4a40b93c8ec60447cb5 | |
parent | 39736221764168bcd9d0d926a4d4e99f3fe259e6 (diff) |
Revert previous commit (work around batch file issues)
The solution to the issue is now in libbutl.
-rw-r--r-- | bbot/worker/worker.cxx | 27 |
1 files changed, 6 insertions, 21 deletions
diff --git a/bbot/worker/worker.cxx b/bbot/worker/worker.cxx index f012a34..2105939 100644 --- a/bbot/worker/worker.cxx +++ b/bbot/worker/worker.cxx @@ -2165,35 +2165,20 @@ startup () // Use the name=value notation for options to minimize the number of // arguments passed to the environment setup executable. Note that the - // etc/environments/default-*.bat batch files can only handle a limited - // number of arguments. Also quote such options if the environment is a - // batch file to prevent them from being split into two arguments. + // etc/environments/default-*.bat scripts can only handle the limited + // number of arguments. // - auto opt_val = [&pp] (const char* opt, const string& val) -> string - { -#ifdef _WIN32 - if (const char* e = path::traits_type::find_extension ( - pp.effect_string ())) - { - if (icasecmp (e, ".bat") == 0) - return '"' + string (opt) + '=' + val + '"'; - } -#endif - - return string (opt) + '=' + val; - }; - if (ops.systemd_daemon ()) os.push_back ("--systemd-daemon"); if (ops.verbose_specified ()) - os.push_back (opt_val ("--verbose", to_string (ops.verbose ()))); + os.push_back ("--verbose=" + to_string (ops.verbose ())); if (ops.tftp_host_specified ()) - os.push_back (opt_val ("--tftp-host", ops.tftp_host ())); + os.push_back ("--tftp-host=" + ops.tftp_host ()); - os.push_back (opt_val ("--env-script", pp.effect_string ())); - os.push_back (opt_val ("--env-target", tg)); + os.push_back (string ("--env-script=") + pp.effect_string ()); + os.push_back ("--env-target=" + tg); // Note that we use the effective (absolute) path instead of recall since // we may have changed the CWD. |