Re: cvs commit: jakarta-tomcat-connectors/webapp configure.in

[jean-frederic clere]

Pier Fumagalli wrote:
> 
> Quoting [EMAIL PROTECTED]:
> >
> > Fixed bug I introduced 5 seconds ago. Now the return value of
> > LOCAL_FILTEREXEC should be correctly returned to the parent.
> > (try to modify the last line of APR buildconf to exit with a non-zero
> > value and execute configure, it should report an error)
> 
> Jean Frederic, can you check if it works for you now? Thanks

No, it does not, because the shell forks and ret is not set in father but in the
son process (after the |).
Look to the output:
+++
    APR configure: creating include/arch/unix/apr_private.h
  Execution of ./configure --enable-static --disable-shared --disable-threads
returned
/usr/bin/test: integer expression expected before -ne
                                                                                
+++

The only work-around I see is to write the ret in a file and read it.

+++
Index: configure.in                                                             
===================================================================             
RCS file: /home/cvs/jakarta-tomcat-connectors/webapp/configure.in,v             
retrieving revision 1.12                                                        
diff -u -r1.12 configure.in                                                     
--- configure.in        2001/07/09 15:57:56     1.12                            
+++ configure.in        2001/07/09 16:59:22                                     
@@ -99,7 +99,9 @@                                                               
     done                                                                       
     unset first                                                                
     unset line                                                                 
+    echo ${ret} > retfile                                                      
   }                                                                            
+  ret=`cat retfile`                                                            
   ${echo} "  Execution of $1 returned $ret"                                    
 ])                                                                             
+++

> 
>     Pier