[
https://issues.apache.org/jira/browse/MBUILDCACHE-74?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17788594#comment-17788594
]
ASF GitHub Bot commented on MBUILDCACHE-74:
-------------------------------------------
olamy commented on code in PR #110:
URL:
https://github.com/apache/maven-build-cache-extension/pull/110#discussion_r1401411398
##########
src/main/java/org/apache/maven/buildcache/CacheControllerImpl.java:
##########
@@ -503,6 +503,7 @@ public void save(
}
}
} else {
+ localCache.beforeSave(context);
Review Comment:
as this will done for both if and else maybe better to write this only once
before the if?
> Old POM builds are not cleaned up from local cache
> --------------------------------------------------
>
> Key: MBUILDCACHE-74
> URL: https://issues.apache.org/jira/browse/MBUILDCACHE-74
> Project: Maven Build Cache Extension
> Issue Type: Bug
> Affects Versions: 1.0.1
> Reporter: Michael Weirauch
> Priority: Minor
> Labels: pull-request-available
>
> We are operating a multi-module mono repository with an in-tree parent which
> is used by all modules. The parent is modified regularly. While debugging I
> realized that stale cache entries for the parent are still present in the
> local build cache directory allthough "maxBuildsCached" is set to "1".
> I have a fix ready, but I am still trying to figure if I can hook into an
> existing test or create a dedicated one. (I am not that familiar with the
> codebase.)
>
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
