RFR: 8278080: Add --with-cacerts-src='user cacerts folder' to enable deterministic cacerts generation [v2]

[Andrew Leonard]

On Thu, 2 Dec 2021 18:46:09 GMT, Erik Joelsson <erikj at openjdk.org> wrote:

>> this was my understanding: https://www.gnu.org/software/make/manual/html_node/Variables-in-Recipes.html
>> 
>> This occurs after make has finished reading all the makefiles and the target is determined to be out of date; so, the recipes for targets which are not rebuilt are never expanded. 
>> 
>> but i'm going to double check I was checking the resultant cacerts correctly in my tests
>
> Oh, I didn't expand the diff far enough to actually see the context correctly when I reviewed this as I would never have imagined the conditional to be placed after the rule. While this will work as so far as using the correct files, incremental builds will not be correct, because the rules are defined in the first pass.
> 
> I very much agree with Magnus that this conditional belongs around line 63.

yes, thanks, feeling rather stupid here! i'll raise an issue to fix

-------------

PR: https://git.openjdk.java.net/jdk/pull/6647