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

Timestamp:

2011-09-06T15:40:29+12:00 (13 years ago)

Author:

ak19

Message:

1. On Windows, Depositor needs env var OS for the call to util::setup_greenstone_env() from bin\script\build to work (it will try to run setup.bat; if OS is set, it will be Windows_NT and if it's not set, it will default to Win95 and try to run a COMMAND statement that conflicts with the Depositor's building process. The fix is to let Apache have access to the env var OS, by using PassEnv. 2. Fixed oversight of not closing filehandle in util::setup_greenstone_env().

File:

• main/trunk/greenstone2/runtime-src/packages/apache-httpd/windows-httpd.conf.in (modified) (1 diff)

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

@@ -53 +53 @@
 # to the TEMP windows folder in case GS is installed on a non-writable media 
 # like a CD-ROM.
-PassEnv "USERPROFILE" "ProgramFiles" LANG "TEMP"
+# Need to pass OS too, so that when the depositor builds and 
+# util::setup_greenstone_env() is called, which then calls setup.bat, 
+# the %OS% var is set. If empty, it defaults to Win95 instead of being 
+# Windows_NT, and then things fail in a memory allocation line in setup.bat.
+PassEnv "USERPROFILE" "ProgramFiles" "TEMP" "OS" "LANG"
 
 #