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

Removed the dummy remote repository in favour for the snapshot and re… #683

Merged
merged 1 commit into from
Dec 27, 2022

Conversation

markrekveld
Copy link
Contributor

The dummy repository is added to allow for the resolution of depedencies using the jfrog cli resolver configuration for Maven. Some maven build plugins, like the org.codehaus.mojo:versions-maven-plugin will try to resolve artifacts using this dummy repository but will always fail.

This PR implements a solution that allows maven build plugins to resolve artifacts using the jfrog cli resolver configuration by configuring the snapshot and release resolver repositories just like the ArtifactoryEclipseResolversHelper and ArtifactorySonatypeResolversHelper do.
Instead of configuring a dummy and "hoping" it gets proxied to resolver repositories.

@eyalbe4 eyalbe4 self-requested a review December 9, 2022 12:19
@eyalbe4 eyalbe4 merged commit 4230186 into jfrog:master Dec 27, 2022
omerzi added a commit that referenced this pull request Mar 2, 2023
@omerzi
Copy link
Member

omerzi commented Mar 2, 2023

Hi @markrekveld, we have several issues regarding this PR on some of our products, as reflected in the following:

jfrog/jfrog-azure-devops-extension#381

jfrog/jenkins-artifactory-plugin#734

jfrog/jfrog-cli#1839

This required us to revert the changes.

If you would like to improve this PR and contribute a fixing PR, we would greatly appreciate it. Thanks a lot.

@markrekveld
Copy link
Contributor Author

Hi @omerzi

Thanks for letting me know, I'll update the PR and resubmit once done

markrekveld added a commit to Marvelution/build-info that referenced this pull request Mar 2, 2023
@markrekveld
Copy link
Contributor Author

I have opened #715 to address the issues @omerzi linked to

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.

None yet

3 participants