public inbox for [email protected]  
help / color / mirror / Atom feed
From: Chapman Flack <[email protected]>
To: Kartik Ohri <[email protected]>
Cc: [email protected]
Subject: Re: PL/Java new build plugin
Date: Thu, 13 Aug 2020 17:32:19 -0400
Message-ID: <[email protected]> (raw)
In-Reply-To: <CAASLQ4NSgCJSziVoAAuaLt0Ed-qEo+As54uXco0PZ65n4f4n5A@mail.gmail.com>
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>

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





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: <[email protected]>

* 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