Changeset 24008

Timestamp:

2011-05-09T20:02:09+12:00 (13 years ago)

Author:

ak19

Message:

lucenesearch needs to use a temporary folder to write out a file to. The windows TEMP envvar is not available through the apache webserver without passenv. This seems to be the only option, as the _tempnam and _tmpnam windows methods either try to write to the current working directory or, if they take a dir parameter, will also look in TMP and else use whatever folder you provide. We need to use the TEMP folder *because* we don't know what folder to provide (the collect folder won't be writable if GS is installed on and run from non-writable media like a CD-ROM.

Location:

main/trunk/greenstone2/runtime-src

Files:

• packages/apache-httpd/windows-httpd.conf.in (modified) (1 diff)
• src/colservr/lucenesearch.cpp (modified) (1 diff)

main/trunk/greenstone2/runtime-src/packages/apache-httpd/windows-httpd.conf.in

@@ -50 +50 @@
 # USERPROFILE environment variable in order for open-office to process docs
 # It also requires ProgramFiles to guess where the OpenOffice bin is installed
-PassEnv "USERPROFILE" "ProgramFiles" LANG
+# The TEMP envvar is needed by lucenesearch.cpp so it can write out files 
+# to the TEMP windows folder in case GS is installed on a non-writable media 
+# like a CD-ROM.
+PassEnv "USERPROFILE" "ProgramFiles" LANG "TEMP"
 
 #

main/trunk/greenstone2/runtime-src/src/colservr/lucenesearch.cpp

@@ -170 +170 @@
   // a location designed to be writable
 
-  text_t out_file = filename_cat(getenv("TEMP"),filename);
+  // Need to set the Apache server to pass the TEMP envvar (or TMP) with 
+  // passenv. Little point using _tempnam() function that generates a 
+  // filename and where you need to pass in the dir if *TMP* doesn't exist.  
+  // http://msdn.microsoft.com/en-us/library/hs3e7355%28v=vs.80%29.aspx
+  // dir parameter: "The path used in the file name if there is no TMP
+  // environment variable, or if TMP is not a valid directory." Seems
+  // the inverse of what we want: to use TMP/TEMP if collectdir is invalid.
+
+  text_t out_file = filename_cat(getenv("TEMP"),filename); // TMP or TEMP
 
   cmd += " -out \"" + out_file + "\"";