I didn't get any feedback yet from the bug reporter here:

https://issues.apache.org/bugzilla/show_bug.cgi?id=47590

But I'm guessing it's a trivial fix along the lines of what we've seen
before, where we pass in default environment, so that the child runs OK
when compiled with the Intel compiler. That trivial patch is attached.

Bill,

I see you are cleaning things up, I'm guessing for a release. Do you
mind if I commit this?

-- 
Bojan
Index: test/testpipe.c
===================================================================
--- test/testpipe.c	(revision 800210)
+++ test/testpipe.c	(working copy)
@@ -146,6 +146,9 @@
                              APR_CHILD_BLOCK);
     ABTS_INT_EQUAL(tc, APR_SUCCESS, rv);
 
+    rv = apr_procattr_cmdtype_set(procattr, APR_PROGRAM_ENV);
+    APR_ASSERT_SUCCESS(tc, "Couldn't set copy environment", rv);
+
     rv = apr_procattr_error_check_set(procattr, 1);
     ABTS_INT_EQUAL(tc, APR_SUCCESS, rv);
 

Reply via email to