Skip to content

RABBITMQ_NAME_TYPE should be correctly set in Windows batch files #1514

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

Merged
merged 3 commits into from
Feb 21, 2018

Conversation

lukebakken
Copy link
Collaborator

@lukebakken lukebakken commented Feb 15, 2018

Remove unused -nodename argument to erl, ensure that RABBITMQ_NAME_TYPE is used correctly in Windows batch files

Fixes #1508

@michaelklishin - I confirmed that it's OK to remove -nodename since the rabbitmq-cli commands will get the value from the RABBITMQ_NODENAME env variable.

@lukebakken lukebakken changed the title WIP: Ensure RABBITMQ_NODE_TYPE is used in Windows batch files WIP: Ensure RABBITMQ_NAME_TYPE is used in Windows batch files Feb 15, 2018
@lukebakken lukebakken changed the title WIP: Ensure RABBITMQ_NAME_TYPE is used in Windows batch files Ensure RABBITMQ_NAME_TYPE is used in Windows batch files Feb 15, 2018
@lukebakken lukebakken changed the title Ensure RABBITMQ_NAME_TYPE is used in Windows batch files WIP: RABBITMQ_NAME_TYPE should be used in Windows batch files Feb 20, 2018
@lukebakken lukebakken changed the title WIP: RABBITMQ_NAME_TYPE should be used in Windows batch files RABBITMQ_NAME_TYPE should be correctly set in Windows batch files Feb 20, 2018
@lukebakken lukebakken requested review from michaelklishin and removed request for michaelklishin February 20, 2018 23:24
@michaelklishin michaelklishin merged commit da98806 into v3.7.x Feb 21, 2018
@lukebakken lukebakken deleted the rabbitmq-server-1508 branch February 21, 2018 14:51
@lukebakken lukebakken restored the rabbitmq-server-1508 branch October 20, 2018 00:12
@lukebakken lukebakken deleted the rabbitmq-server-1508 branch October 20, 2018 00:26
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