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

Moved the activemq-all dependency from parent pom.xml to platform-docker pom.xml. #628

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

abhinavmishra14
Copy link

@abhinavmishra14 abhinavmishra14 commented Jan 10, 2022

Moved the activemq-all dependency from parent pom.xml to platform-docker pom.xml in order to avoid it being included in the amp. Inclusion in the amp caused issues when amp is deployed on a server.

See the issue description here as well: https://hub.alfresco.com/t5/alfresco-content-services-forum/error-deploying-aio-with-sdk-4-3/m-p/310967/highlight/false#M26453

I have replicated the issue and tested on this demo project : https://github.com/abhinavmishra14/acs7-sdk43-demo

platform docker project: https://github.com/abhinavmishra14/acs7-sdk43-demo/blob/main/acs71-demo-platform-docker/pom.xml#L33

parent pom: https://github.com/abhinavmishra14/acs7-sdk43-demo/blob/main/acs71-demo-platform-docker/pom.xml#L33

Issue: #630

pom.xml in order to avoid it being included in the amp. Inclusion in the
amp caused issues when amp is deployed on a server.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant