This repository has been archived by the owner. It is now read-only.

Update to the Apollo 1.3 release. Also install the MQTT plugin by default. #12408

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
3 participants
@chirino
Contributor

chirino commented May 24, 2012

No description provided.

@andypiper

This comment has been minimized.

Show comment Hide comment
@andypiper

andypiper May 31, 2012

Contributor

Tested this Formula today. It's great for a fresh install, but for an upgrade there are a couple of issues:

  • the previous version suggested the use of sudo to run the broker - so the files under /usr/local/var/apollo have the wrong permissions for the regular user to run the broker.
  • you need to create a new broker in order to get the 1.3 function - even after the permissions were fixed, the broker started as a 1.2 broker. Perhaps this is something that an Apollo / ActiveMQ admin or developer natively understands.

In both cases, after deleting the old broker and re-creating it with the 1.3 binary, I had no issues.

It could be that the migration steps should be explained in the output at the end of the brew install / upgrade. It is definitely nice not to run as root with the new release.

Contributor

andypiper commented May 31, 2012

Tested this Formula today. It's great for a fresh install, but for an upgrade there are a couple of issues:

  • the previous version suggested the use of sudo to run the broker - so the files under /usr/local/var/apollo have the wrong permissions for the regular user to run the broker.
  • you need to create a new broker in order to get the 1.3 function - even after the permissions were fixed, the broker started as a 1.2 broker. Perhaps this is something that an Apollo / ActiveMQ admin or developer natively understands.

In both cases, after deleting the old broker and re-creating it with the 1.3 binary, I had no issues.

It could be that the migration steps should be explained in the output at the end of the brew install / upgrade. It is definitely nice not to run as root with the new release.

@adamv

This comment has been minimized.

Show comment Hide comment
@adamv

adamv Jun 13, 2012

Contributor

@andypiper will accept another patch that udpates the caveats; I don't use this software myself to know what to suggest.

Contributor

adamv commented Jun 13, 2012

@andypiper will accept another patch that udpates the caveats; I don't use this software myself to know what to suggest.

@adamv

This comment has been minimized.

Show comment Hide comment
@adamv

adamv Jun 13, 2012

Contributor

Pulled, thanks.

Contributor

adamv commented Jun 13, 2012

Pulled, thanks.

@adamv adamv closed this Jun 13, 2012

@xu-cheng xu-cheng locked and limited conversation to collaborators Feb 16, 2016

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.