RFR: 8366899: SetupExecute should add the command line to vardeps
Erik Joelsson
erikj at openjdk.org
Thu Sep 18 18:17:54 UTC 2025
On Thu, 18 Sep 2025 17:30:14 GMT, Magnus Ihse Bursie <ihse at openjdk.org> wrote:
>> I'm not going to fight for it, but I would most likely have gone with some variant of what you suggested.
>
> I'm not going to fight for my solution either. :) Which would you prefer, explicitly adding vardeps, or creating a new general extra-deps?
In this case I think I would have kept it simple and just added `$1_VARDEPS` on the prerequisite lines where needed, but again, I don't really mind either way.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/27348#discussion_r2360604864
More information about the build-dev
mailing list