Changeset 397 for AE

Timestamp:

Jul 7, 2009, 12:10:17 AM (15 years ago)

Author:

iritscen

Message:

Fixed key_config copying bug and Mac GDF pref-writing bug.

File:

• AE/Installer/trunk/source/main_window.cpp (modified) (2 diffs)

AE/Installer/trunk/source/main_window.cpp

@@ -76 +76 @@
         string escape_me = "& ;()|<>\"'\\#*?$";
         for(int i = 0; i < input.size(); i++)  {
-                for(int j = 0; j < escape_me.size(); j++) if (input[i] == escape_me[j]) output += '//';
+                for(int j = 0; j < escape_me.size(); j++) if (input[i] == escape_me[j]) output += '\\';
                 output += input[i];
         }
@@ -357 +357 @@
                 copy((path)"packages/VanillaBSL/IGMD", (path)"../GameDataFolder");
                 setProgressBar( 1000 );
+
+                copy("../../persist.dat","../persist.dat");
+                copy("../../key_config.txt","../key_config.txt"); 
                 
-                // CIP:last of all, set up the edition folder as a playable Oni installation by placing the latest application (+ Daodan DLL on Windows) in edition/
-                // or we could just include it in the zip in the proper place already.
-
-                copy("../../persist.dat","../persist.dat");
-                copy("../../keyconfig.txt","../keyconfig.txt"); 
 #ifndef WIN32
                 /* On Mac only, set the current GDF to the AE GDF by writing to Oni's global preferences file (thankfully a standard OS X ".plist" XML file).
-                   If there are no Oni prefs (only possible if Oni has not been run even once), then the above line will fail silently, no harm done,
-                   and when the user does run Oni for the first time, using the copy of the app in the AE GDF, Oni will set the prefs to use that GDF at that point */
-                path fullAEpath = system_complete("."); // get full path for Installer
-                char prefsCommand[300] = "defaults write com.godgames.oni RetailInstallationPath -string '";
-                strcat(prefsCommand, fullAEpath.parent_path().parent_path().string().c_str()); // get path of edition/ folder (Oni wants the folder that *contains* the GDF)
+                   Tests for presence of prefs with [ -f ] before doing anything so it doesn't create a partial prefs file -- just in case user has never
+                   run Oni before :-p */
+                string fullAEpath = escapePath(system_complete(".").parent_path().parent_path().string()); // get full path for edition/
+                char prefsCommand[300] = "[ -f ~/Library/Preferences/com.godgames.oni.plist ] && defaults write com.godgames.oni RetailInstallationPath -string '";
+                strcat(prefsCommand, fullAEpath.c_str()); // get path of edition/ folder (Oni wants the folder that *contains* the GDF)
                 strcat(prefsCommand, "'"); // path string is enclosed in single quotes to avoid the need to escape UNIX-unfriendly characters
                 system(prefsCommand);