public inbox for [email protected]  
help / color / mirror / Atom feed
From: Kartik Ohri <[email protected]>
To: Chapman Flack <[email protected]>
Cc: [email protected]
Subject: Re: PL/Java new build plugin
Date: Fri, 14 Aug 2020 08:14:55 +0530
Message-ID: <CAASLQ4OEbbz1D7eH-fHPcSwA2u4cPqUV_=+TXZz4wW+0UWX-Dw@mail.gmail.com> (raw)
In-Reply-To: <[email protected]>
References: <CAASLQ4Mva7dAXiS_c9OLhV4VzNHF=XNKJosuS5gtZTB6b4TW1Q@mail.gmail.com>
	<CAASLQ4OCLUYXpkk0tTioz8=zc4UEbAm-Whgp_5K7uy-5hZdjKA@mail.gmail.com>
	<[email protected]>
	<CAASLQ4M8cN20L7cyVEgks9qdnfsXH4ci7iQbmyi4v_y0W85eLA@mail.gmail.com>
	<[email protected]>
	<CAASLQ4MLFnD3v_fqrUj+Bwc9MSBG-ATet3J7UXebzXHTH_si3A@mail.gmail.com>
	<[email protected]>
	<CAASLQ4O5Wfk1hH6nO_1PDUbRcpUywvTxq7e99aCJ4RPeMmssAw@mail.gmail.com>
	<[email protected]>
	<[email protected]>
	<CAASLQ4NzHE-YO9XYTQSqHEEz9UQm8dRq1d-jbdAFPVATFiLN2w@mail.gmail.com>
	<CAASLQ4OLhgc=MMMOWv2sxt93xC1Te4vwThBgEy7=QBBz0dEn6g@mail.gmail.com>
	<[email protected]>
	<CAASLQ4NA76safp+fa__9kMo2sG__EiwGdViFECgQ4wfL3=OpXw@mail.gmail.com>
	<[email protected]>
	<CAASLQ4O3ONhDr-XgwDDD3oW+x2VwCDTmuu-T+fz-jcZ=T23zuw@mail.gmail.com>
	<[email protected]>
	<CAASLQ4NSgCJSziVoAAuaLt0Ed-qEo+As54uXco0PZ65n4f4n5A@mail.gmail.com>
	<[email protected]>

On Fri, Aug 14, 2020, 3:02 AM Chapman Flack <[email protected]> wrote:

> On 08/13/20 17:24, Kartik Ohri wrote:
> > I have made the changes to the ProcessBuilder and am now trying to use it
> > build pljava-so using it. I wanted to ask should the working directory
> for
> > the process builder be hard coded as the project's target directory or
> left
> > for the user to specify as an argument.
>
> Hmm, that's a good question.
>
> What if ... instead of script calling new ProcessBuilder(...) to obtain
> an instance, we provide a method (with the same signature), that calls
> the constructor and returns a ProcessBuilder with some common defaults
> already configured?
>
> Then the script can just change any bits it wants different from the
> default.
>
> Regards,
> -Chap
>

Sounds good to me. Will do it this way.
Regards,
Kartik

>


view thread (183+ messages)  latest in thread

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: [email protected]
  Cc: [email protected], [email protected], [email protected]
  Subject: Re: PL/Java new build plugin
  In-Reply-To: <CAASLQ4OEbbz1D7eH-fHPcSwA2u4cPqUV_=+TXZz4wW+0UWX-Dw@mail.gmail.com>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox