gnodet commented on PR #1679: URL: https://github.com/apache/maven-resolver/pull/1679#issuecomment-3585609759
> New opt-in **experimental** feature for enhanced local repository: ability to choose "repository key" function. Local repository (simple and enhanced) by default uses the `simple` key (historically) where other places all used `nid`. Have to note, that `simple` is technically equivalent to `nid` as `RemoteRepository.isRepositoryManager()` is not set anywhere in Maven. > > Added key functions: > > * `simple` -> the original code in LRM > * `nid` -> `norm(id)` (path friendly) > * `hurl` -> `sha1(url)` > * `nid_hurl` -> `norm(id)-sha1(url)` > * `gurk` -> `norm(id)-sha(seed)` where "seed" is all config properties of repo > * `ngurk` -> `norm(id)-sha(seed)` where "seed" is all config sans mirror list (just the string "isMirrored" is added for mirrors) Fwiw, since this is switching to a more mathematical expression, I was confused by the `-` which, when I first read, though it was `minus`. I wonder if we could use a different separator such as `+` or `.`. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
