Subject: Re: PL/Java new build plugin
To: Kartik Ohri <kartikohri13@gmail.com>
References: 
 <CAASLQ4Mva7dAXiS_c9OLhV4VzNHF=XNKJosuS5gtZTB6b4TW1Q@mail.gmail.com>
 <5F0125BC.5000609@anastigmatix.net>
 <CAASLQ4PtGSS-OP9k_740EBdyT1bVmDi1PUndn1ZNWzyJDRQkMA@mail.gmail.com>
 <5F0172D7.3000708@anastigmatix.net>
 <CAASLQ4NXzEttesB48wxD_wvR5_rNDvXZkKCmNELn247+W_vrMw@mail.gmail.com>
 <5F02122C.7040905@anastigmatix.net>
 <CAASLQ4NTYp42EY1=nnKPRAr6VV-KpQ=hueQn-jHZsEebp4cAaQ@mail.gmail.com>
 <5F021785.905@anastigmatix.net>
 <CAASLQ4OCh2ybECbfJQ5UgSo4r-UxRcmPPhqjrGpEZ=ZPdURNMg@mail.gmail.com>
 <5F021A20.1050903@anastigmatix.net>
 <CAASLQ4MWn3BjiZpijXEjd3sYaeuQnFffafZYe9XWnV8PPXv1nQ@mail.gmail.com>
 <5F02340C.7030002@anastigmatix.net>
 <CAASLQ4NOq2FcTvp1hB-kTnJZgxfRP=u20gfQT=tgpvBq+9Ac-w@mail.gmail.com>
 <CAASLQ4PwL2U6t0DYZK4KF9+2UZT56Npcga4xZWWfo04w2QZxCQ@mail.gmail.com>
 <5F04FA65.5000600@anastigmatix.net>
 <CAASLQ4PEe0_1dE-iOBv27fX1SUH0=ewDm=6w9chHLjWSAkeNgQ@mail.gmail.com>
 <5F071CFC.9090306@anastigmatix.net>
 <CAASLQ4MLFd6zhoh+sWWz+iu4d8+H2ML+y=kL990k7H=kk74pvA@mail.gmail.com>
Cc: pljava-dev@lists.postgresql.org
From: Chapman Flack <chap@anastigmatix.net>
Message-ID: <5F072647.50301@anastigmatix.net>
Date: Thu, 9 Jul 2020 10:14:31 -0400
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101
 Thunderbird/38.7.0
MIME-Version: 1.0
In-Reply-To: 
 <CAASLQ4MLFd6zhoh+sWWz+iu4d8+H2ML+y=kL990k7H=kk74pvA@mail.gmail.com>
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit
Precedence: bulk

On 07/09/20 09:51, Kartik Ohri wrote:

> Any suggestions what could be the problem? As far as I see in the
> build.xml, I can only think of MSVC.RINT but it should not be probably the
> cause.

You could be on to something. I remember thinking that MSVC_RINT is
probably no longer essential to the build, at least for MSVC > 2012
or PG > 9.3. But still, if there's an MSVC-only define that uses it:

https://github.com/amCap1712/pljava/blob/d6608ff/pljava-so/pom.xml#L120

and it isn't being set to something, that might produce a null in the
defines list.

>> to squeeze a little more information out of a NullPointerException
>> by putting the right -XX:... option in the MAVEN_OPTS environment
> 
> Yes, I understand that but as you say it isn't beneficial in this case.

I wonder if it wouldn't be worthwhile to have that in the MAVEN_OPTS
for all the automated builds (with Java >= 14), just to save time if
something else happens to blow up some day,

Regards,
-Chap