Add a fast ugly patch for people in hurry
authorhttp://beaufils.myopenid.com/ <beaufils@web>
Mon, 1 Oct 2012 15:49:28 +0000 (11:49 -0400)
committeradmin <admin@branchable.com>
Mon, 1 Oct 2012 15:49:28 +0000 (11:49 -0400)
doc/bugs/CGI_wrapper_doesn__39__t_store_PERL5LIB_environment_variable.mdwn

index 0da368644055e5003b2e70fe986a377b339ca5a0..81a5abf2862e0430ff566c758e0fe5ebc6ce3fff 100644 (file)
@@ -6,3 +6,23 @@ I think the CGI wrapper should remember PERL5LIB too.
 
 -- Martin
 
 
 -- Martin
 
+Thank's a lot for pointing me to this location in the code. I was looking it for some time.
+
+This brutal patch implement your solution as a temporary fix.
+
+    *** Wrapper.pm.old      2012-08-25 16:41:41.000000000 +0200
+    --- Wrapper.pm  2012-10-01 17:33:17.582956524 +0200
+    ***************
+    *** 149,154 ****
+    --- 149,155 ----
+      $envsave
+            newenviron[i++]="HOME=$ENV{HOME}";
+            newenviron[i++]="PATH=$ENV{PATH}";
+    +       newenviron[i++]="PERL5LIB=$ENV{PERL5LIB}";
+            newenviron[i++]="WRAPPED_OPTIONS=$configstring";
+      
+      #ifdef __TINYC__
+
+As I am not sure that remembering `PERL5LIB` is a good idea, I think that a prettier solution will be to add a config variable (let's say `cgi_wrapper_perllib`) which, if fixed, contains the `PERL5LIB` value to include in the wrapper, or another (let's say `cgi_wrapper_remember_libdir`), which, if fixed, remember the current `PERL5LIB`.
+
+-- Bruno