MIME-Version: 1.0
References: 
 <CAASLQ4Mva7dAXiS_c9OLhV4VzNHF=XNKJosuS5gtZTB6b4TW1Q@mail.gmail.com>
 <5F230337.3040406@anastigmatix.net>
 <CAASLQ4Ny+VWWH=_AR+=7gjXinUScybraOJxBUaG5AmLZmx+NPQ@mail.gmail.com>
 <5F232EBB.9030208@anastigmatix.net>
 <CAASLQ4OknEQTpSeoBfyAZR1na-EyL+AtN9t0fPGyv5DfWJOSWA@mail.gmail.com>
 <CAASLQ4NHROvOq-rUz6Jn0dUd7oAq3_P655WNeqKksNBqFtAGgw@mail.gmail.com>
 <CAASLQ4OCLUYXpkk0tTioz8=zc4UEbAm-Whgp_5K7uy-5hZdjKA@mail.gmail.com>
 <5F278CCE.1090006@anastigmatix.net>
 <CAASLQ4M8cN20L7cyVEgks9qdnfsXH4ci7iQbmyi4v_y0W85eLA@mail.gmail.com>
 <5F2897AF.9040909@anastigmatix.net>
 <CAASLQ4MLFnD3v_fqrUj+Bwc9MSBG-ATet3J7UXebzXHTH_si3A@mail.gmail.com>
 <5F2A29E4.2000803@anastigmatix.net>
 <CAASLQ4O5Wfk1hH6nO_1PDUbRcpUywvTxq7e99aCJ4RPeMmssAw@mail.gmail.com>
 <5F2EB313.8090706@anastigmatix.net> <5F2EE97B.1030108@anastigmatix.net>
 <CAASLQ4NzHE-YO9XYTQSqHEEz9UQm8dRq1d-jbdAFPVATFiLN2w@mail.gmail.com>
 <CAASLQ4OLhgc=MMMOWv2sxt93xC1Te4vwThBgEy7=QBBz0dEn6g@mail.gmail.com>
 <5F30B425.4020402@anastigmatix.net>
In-Reply-To: <5F30B425.4020402@anastigmatix.net>
From: Kartik Ohri <kartikohri13@gmail.com>
Date: Wed, 12 Aug 2020 00:19:23 +0530
Message-ID: 
 <CAASLQ4NA76safp+fa__9kMo2sG__EiwGdViFECgQ4wfL3=OpXw@mail.gmail.com>
Subject: Re: PL/Java new build plugin
To: Chapman Flack <chap@anastigmatix.net>
Cc: pljava-dev@lists.postgresql.org
Content-Type: multipart/alternative; boundary="00000000000060a31805ac9e8808"
Precedence: bulk

--00000000000060a31805ac9e8808
Content-Type: text/plain; charset="UTF-8"

>
> ProcessBuilder is already close to a nice-enough interface for constructing
> a command to run, but there should probably be a convenience method for
> actually running it; that method could take care of the last steps,
> setting the I/O redirection, handling the output, checking the exit
> status, and so on.
>

I have currently added this
https://github.com/amCap1712/pljava/commit/174dd71365986094a500d6dda4c832de78f90783.
Is this fine? I had to change the way error handling worked to avoid
creating a new functional interface.

--00000000000060a31805ac9e8808
Content-Type: text/html; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

<div dir=3D"ltr"><div class=3D"gmail_quote"><blockquote class=3D"gmail_quot=
e" style=3D"margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204)=
;padding-left:1ex">ProcessBuilder is already close to a nice-enough interfa=
ce for constructing<br>
a command to run, but there should probably be a convenience method for<br>
actually running it; that method could take care of the last steps,<br>
setting the I/O redirection, handling the output, checking the exit<br>
status, and so on.<br></blockquote><div><br></div><div>I have currently add=
ed this=C2=A0<a href=3D"https://github.com/amCap1712/pljava/commit/174dd713=
65986094a500d6dda4c832de78f90783">https://github.com/amCap1712/pljava/commi=
t/174dd71365986094a500d6dda4c832de78f90783</a>. Is this fine? I had to chan=
ge the way error handling worked to avoid creating a new functional interfa=
ce.=C2=A0</div></div></div>

--00000000000060a31805ac9e8808--