Skip to content

Ensure Java compile module path is tracked as a task input#87064

Merged
elasticsearchmachine merged 1 commit intoelastic:masterfrom
mark-vieira:modulepath_classpath_tracking
May 23, 2022
Merged

Ensure Java compile module path is tracked as a task input#87064
elasticsearchmachine merged 1 commit intoelastic:masterfrom
mark-vieira:modulepath_classpath_tracking

Conversation

@mark-vieira
Copy link
Copy Markdown
Contributor

@mark-vieira mark-vieira commented May 23, 2022

This is a follow up to #81066 which ensures that we track the module path as a input on Java compilation tasks. Since we truncate the compile classpath and pass the module path as separate CLI arguments we need to track those separately. We do that by simply annotating a getter on the CommandLineArgumentProvider.

@mark-vieira mark-vieira added >non-issue :Delivery/Build Build or test infrastructure labels May 23, 2022
@mark-vieira mark-vieira requested a review from ChrisHegarty May 23, 2022 19:04
@elasticmachine elasticmachine added the Team:Delivery Meta label for Delivery team label May 23, 2022
@elasticmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-delivery (Team:Delivery)

Copy link
Copy Markdown
Contributor

@ChrisHegarty ChrisHegarty left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@mark-vieira mark-vieira added the auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) label May 23, 2022
@elasticsearchmachine elasticsearchmachine merged commit eddb127 into elastic:master May 23, 2022
@mark-vieira mark-vieira deleted the modulepath_classpath_tracking branch May 23, 2022 20:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) :Delivery/Build Build or test infrastructure >non-issue Team:Delivery Meta label for Delivery team v8.3.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants