Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Cache Maven distribution downloaded by Maven Wrapper #448

Open
Marcono1234 opened this issue Jan 29, 2023 · 6 comments · May be fixed by #453
Open

Cache Maven distribution downloaded by Maven Wrapper #448

Marcono1234 opened this issue Jan 29, 2023 · 6 comments · May be fixed by #453
Labels
feature request New feature or request to improve the current logic

Comments

@Marcono1234
Copy link
Contributor

Description:
Cache the Maven distribution downloaded by the official Maven Wrapper.

The cache key should probably be derived from the .mvn/wrapper/maven-wrapper.properties file (see documentation), similar to how it is done already for the Gradle Wrapper.

The directory to cache is ~/.m2/wrapper/dists (see documentation).

Justification:
Without caching, projects using the Maven Wrapper would have to download the Maven distribution (~8MB) every time the project is built.

For Gradle the setup-java action already supports caching the Gradle distribution downloaded by the wrapper:

'**/gradle-wrapper.properties',

path: [join(os.homedir(), '.gradle', 'caches'), join(os.homedir(), '.gradle', 'wrapper')],

Are you willing to submit a PR?

Probably not

@Marcono1234 Marcono1234 added feature request New feature or request to improve the current logic needs triage labels Jan 29, 2023
@IvanZosimov
Copy link
Contributor

Hi, @Marcono1234 👋 Thank you for the feature request! We will investigate it and get back to you with our decision as soon as possible.

yeikel added a commit to yeikel/setup-java that referenced this issue Feb 12, 2023
yeikel added a commit to yeikel/setup-java that referenced this issue Feb 12, 2023
yeikel added a commit to yeikel/setup-java that referenced this issue Feb 12, 2023
yeikel added a commit to yeikel/setup-java that referenced this issue Feb 12, 2023
@yeikel
Copy link

yeikel commented Feb 16, 2023

@IvanZosimov I submitted #453 for this

@fadma10
Copy link

fadma10 commented Apr 20, 2023

it's runing

@seepine
Copy link

seepine commented Jul 14, 2023

@yeikel Hi, maybe you can support gradlew also, this is really useful of cache mvnw and gradlew

@yeikel
Copy link

yeikel commented Jul 14, 2023

@yeikel Hi, maybe you can support gradlew also, this is really useful of cache mvnw and gradlew

I think that we should track that separately

@seepine
Copy link

seepine commented Jul 14, 2023

@yeikel Hi, maybe you can support gradlew also, this is really useful of cache mvnw and gradlew

I think that we should track that separately

You are right, maybe not work of cache gradlew due to excessive cache size, i find it work just now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request New feature or request to improve the current logic
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants