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

Further Fix for Issue #48 #49

Merged
merged 1 commit into from
Apr 11, 2018
Merged

Further Fix for Issue #48 #49

merged 1 commit into from
Apr 11, 2018

Conversation

hoshposh
Copy link

This is an attempt to show what I was trying to articulate in my earlier comments. Rely on the mature apache plugins for pushing project artifacts to the local repo or remote artifact repos with the maven-install-plugin and maven-deploy-plugin. But, still rely on the mojos within this project to perform the golang actions, and also create the .mvn-golang that will be the primary artifact for the project along with the project pom.

Changes:

  • Still rely on the custom mojos to perform the custom golang actions for the project.
  • Rely on the apache maven-install-plugin to perform the actions to install project artifacts to the local repo.
  • Rely on the apache maven-deploy-plugin to perform the deployment actions to place the project artifacts in a artifact repository.

- Still rely on the custom mojos to perform the custom golang actions for the project.

- Rely on the apache maven-install-plugin to perform the actions to install project artifacts to the local repo.

- Rely on the apache maven-deploy-plugin to perform the deployment actions to place the project artifacts in a artifact repository.
@hoshposh
Copy link
Author

I forgot to mention that the comparison between the approaches will be to view the local repo after an install, and you should notice that with these changes the project pom is installed in its original form. With the original fix for this issue, no pom was installed, and prior to that, using install-file a pom generated by the install-file mojo was installed.

@raydac raydac merged commit 3a45d66 into raydac:master Apr 11, 2018
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