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

Use MIMA #218

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Use MIMA #218

wants to merge 4 commits into from

Conversation

cstamas
Copy link

@cstamas cstamas commented May 9, 2023

Solves #217

A rough draft how could it look like, but I see a big influence of shrinkwrap, hence, IMHO this could be radically simplified.

Also, unsure is "LocalRepository" layrri specific (I guess it is), but just FYI: new feature of resolver is "chained local repository", so it is able to use multiple maven local repositories as well, might be worth to factor it in.

@aalmiray
Copy link
Contributor

aalmiray commented May 9, 2023

FWIW local repository is just a dir with JARs with or without POMs. We need this option to resolve JARs when network is not available. There may not be POMs nor maven-metadata.xml and that's fine.

@cstamas
Copy link
Author

cstamas commented May 10, 2023

Hm, it seems GH env gets detected and as those things are at "standard places" MIMA tries (and fails) to decrypt them?

Failed to decrypt password for server github: org.sonatype.plexus.components.sec.dispatcher.SecDispatcherException: java.io.FileNotFoundException: /home/runner/.m2/settings-security.xml (No such file or directory) @ server: github

My assumption (unverified):

  • settings.xml discovered by MIMA has github credentials encrypted
  • there is no settings-security.xml present, hence MIMA fails
  • it is telling that in module layrry-integrationtest-runner I see no settings.xml at all (hence I believe this is GH Maven env settings that is being discovered)

@aalmiray can you confirm?

EDIT: maveniverse/mima#2

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.

2 participants