Skip to content

Fix for shadowJar out-of-date with configuration caching#708

Merged
johnrengelman merged 1 commit intoGradleUp:masterfrom
mathjeff:configuration-caching-up-to-date
Oct 4, 2021
Merged

Fix for shadowJar out-of-date with configuration caching#708
johnrengelman merged 1 commit intoGradleUp:masterfrom
mathjeff:configuration-caching-up-to-date

Conversation

@mathjeff
Copy link
Copy Markdown
Contributor

@mathjeff mathjeff commented Sep 8, 2021

No description provided.

private List<Transformer> transformers;
private List<Relocator> relocators;
private transient List<Configuration> configurations;
private List<FileCollection> configurations;
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I'm not sure whether the removal of transient can affect correctness of outputs; I was surprised that I didn't notice any differences in output between my first and second builds even though the second build was not UP-TO-DATE.

However, removing transient does seem to allow the task to be up-to-date in the second build.

cc @eskatos since it seems to have been added here: c9e30f1#diff-d5abeaae583093a8a92ef53ffe7f5a21cf185cf0d5e3a76edd236fa26dbf7e9fR34

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Good call! Looks good to me

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Thanks!

private List<Transformer> transformers;
private List<Relocator> relocators;
private transient List<Configuration> configurations;
private List<FileCollection> configurations;
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Good call! Looks good to me

@mathjeff
Copy link
Copy Markdown
Contributor Author

It looks like I don't have permission to submit this or run checks against it; if anybody that does have permission would like to take a look that would be super cool

@johnrengelman johnrengelman added this to the 7.1 milestone Oct 4, 2021
@johnrengelman johnrengelman merged commit 829647a into GradleUp:master Oct 4, 2021
@mathjeff
Copy link
Copy Markdown
Contributor Author

mathjeff commented Oct 4, 2021

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants