Re: external diff headers

Ben Collins-Sussman wrote:

> Branko =?ISO-8859-2?Q?=C8ibej?= <brane@xbc.nu> writes:
>
>> But you can portably open a pipe to a child process in APR. Karl, you
>> did that with the patch and merge bits in the client lib. I think you
>> want apr_create_pipe.
>
>
> Karl is calling apr_create_process() after attaching various
> apr_file_t's to the process' in/out/err streams. This is how we call
> the external diff/patch commands. (See libsvn_wc/log.c
> (svn_wc_run_cmd_in_directory)).
>
> I don't understand how apr_create_pipe will solve the problem; it
> doesn't do what popen does -- does it?

It opens an anonymous pipe, same as popen. I /think/ you do an
apr_create_pipe and use the returned handle for the output file of the
child process (apr_setprocattr_childout).

-- 
Brane �ibej
    home:   <brane_at_xbc.nu>             http://www.xbc.nu/brane/
    work:   <branko.cibej_at_hermes.si>   http://www.hermes-softlab.com/
     ACM:   <brane_at_acm.org>            http://www.acm.org/