Slight hack to reduce the chance of config files looking different
authorCarl Hetherington <cth@carlh.net>
Sun, 28 Feb 2021 18:32:55 +0000 (19:32 +0100)
committerCarl Hetherington <cth@carlh.net>
Sun, 28 Feb 2021 20:52:27 +0000 (21:52 +0100)
on Windows and macOS/Linux; allows config_write_utf8_test to work
withouut platform-specific test references.

src/lib/config.cc

index cc9bd6ffdc177d619f810fffe61cc3ff45f0c619..23b06f2e40a7ac9fdeea84eccd05344473c76b64 100644 (file)
@@ -774,8 +774,13 @@ Config::write_config () const
        /* [XML] AutomaticAudioAnalysis 1 to run audio analysis automatically when audio content is added to the film, otherwise 0. */
        root->add_child("AutomaticAudioAnalysis")->add_child_text (_automatic_audio_analysis ? "1" : "0");
 #ifdef DCPOMATIC_WINDOWS
-       /* [XML] Win32Console 1 to open a console when running on Windows, otherwise 0. */
-       root->add_child("Win32Console")->add_child_text (_win32_console ? "1" : "0");
+       if (_win32_console) {
+               /* [XML] Win32Console 1 to open a console when running on Windows, otherwise 0.
+                * We only write this if it's true, which is a bit of a hack to allow unit tests to work
+                * more easily on Windows (without a platform-specific reference in config_write_utf8_test)
+                */
+               root->add_child("Win32Console")->add_child_text ("1");
+       }
 #endif
 
        /* [XML] Signer Certificate chain and private key to use when signing DCPs and KDMs.  Should contain <code>&lt;Certificate&gt;</code>