Skip to content

Commit 90b9364

Browse files
hairyhummichaelklishin
authored andcommitted
Batch file syntax errors. Config check error format
(cherry picked from commit 220ec77)
1 parent 5bd62b9 commit 90b9364

File tree

3 files changed

+16
-12
lines changed

3 files changed

+16
-12
lines changed

scripts/rabbitmq-server.bat

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -89,7 +89,7 @@ if "!RABBITMQ_CONFIG_FILE!" == "!RABBITMQ_CONFIG_FILE_NOEX!" (
8989
if exist "!RABBITMQ_ADVANCED_CONFIG_FILE!" (
9090
echo "WARNING: Using RABBITMQ_ADVANCED_CONFIG_FILE: !RABBITMQ_ADVANCED_CONFIG_FILE!"
9191
)
92-
set RABBITMQ_CONFIG_ARG_FILE="!RABBITMQ_ADVANCED_CONFIG_FILE!"
92+
set RABBITMQ_CONFIG_ARG_FILE=!RABBITMQ_ADVANCED_CONFIG_FILE!
9393
)
9494
)
9595

@@ -99,9 +99,9 @@ rem the file should exist
9999
rem the file should be a valid erlang term file
100100

101101
rem Config file extension is .config
102-
if "!RABBITMQ_CONFIG_FILE_NOEX!.config" == "!RABBITMQ_CONFIG_FILE" (
102+
if "!RABBITMQ_CONFIG_FILE_NOEX!.config" == "!RABBITMQ_CONFIG_FILE!" (
103103
set RABBITMQ_CONFIG_ARG_FILE=!RABBITMQ_CONFIG_FILE!
104-
) else if "!RABBITMQ_CONFIG_FILE_NOEX!.conf" == "!RABBITMQ_CONFIG_FILE" (
104+
) else if "!RABBITMQ_CONFIG_FILE_NOEX!.conf" == "!RABBITMQ_CONFIG_FILE!" (
105105
set RABBITMQ_CONFIG_ARG_FILE=!RABBITMQ_ADVANCED_CONFIG_FILE!
106106
) else if not "" == "!RABBITMQ_CONFIG_FILE!" (
107107
if not "!RABBITMQ_CONFIG_FILE_NOEX!" == "!RABBITMQ_CONFIG_FILE!" (
@@ -114,7 +114,7 @@ if "!RABBITMQ_CONFIG_FILE_NOEX!.config" == "!RABBITMQ_CONFIG_FILE" (
114114

115115
CALL :get_noex !RABBITMQ_CONFIG_ARG_FILE! RABBITMQ_CONFIG_ARG_FILE_NOEX
116116

117-
if not "$!RABBITMQ_CONFIG_ARG_FILE_NOEX!.config" == "!RABBITMQ_CONFIG_ARG_FILE!" (
117+
if not "!RABBITMQ_CONFIG_ARG_FILE_NOEX!.config" == "!RABBITMQ_CONFIG_ARG_FILE!" (
118118
if "!RABBITMQ_CONFIG_ARG_FILE!" == "!RABBITMQ_ADVANCED_CONFIG_FILE!" (
119119
echo "ERROR: Wrong extension for RABBITMQ_ADVANCED_CONFIG_FILE: !RABBITMQ_ADVANCED_CONFIG_FILE!"
120120
echo "ERROR: extension should be .config"
@@ -133,7 +133,7 @@ if exist !RABBITMQ_CONFIG_ARG_FILE! (
133133
)
134134

135135
rem Set -conf and other generated config parameters
136-
if "!RABBITMQ_CONFIG_FILE_NOEX!.conf" == "!RABBITMQ_CONFIG_FILE" (
136+
if "!RABBITMQ_CONFIG_FILE_NOEX!.conf" == "!RABBITMQ_CONFIG_FILE!" (
137137
set RABBITMQ_GENERATED_CONFIG_ARG=-conf "!RABBITMQ_CONFIG_FILE!" ^
138138
-conf_dir "!RABBITMQ_GENERATED_CONFIG_DIR!" ^
139139
-conf_script_dir !CONF_SCRIPT_DIR:\=/! ^

scripts/rabbitmq-service.bat

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -172,7 +172,7 @@ if "!RABBITMQ_CONFIG_FILE!" == "!RABBITMQ_CONFIG_FILE_NOEX!" (
172172
if exist "!RABBITMQ_ADVANCED_CONFIG_FILE!" (
173173
echo "WARNING: Using RABBITMQ_ADVANCED_CONFIG_FILE: !RABBITMQ_ADVANCED_CONFIG_FILE!"
174174
)
175-
set RABBITMQ_CONFIG_ARG_FILE="!RABBITMQ_ADVANCED_CONFIG_FILE!"
175+
set RABBITMQ_CONFIG_ARG_FILE=!RABBITMQ_ADVANCED_CONFIG_FILE!
176176
)
177177
)
178178

@@ -182,9 +182,9 @@ rem the file should exist
182182
rem the file should be a valid erlang term file
183183

184184
rem Config file extension is .config
185-
if "!RABBITMQ_CONFIG_FILE_NOEX!.config" == "!RABBITMQ_CONFIG_FILE" (
185+
if "!RABBITMQ_CONFIG_FILE_NOEX!.config" == "!RABBITMQ_CONFIG_FILE!" (
186186
set RABBITMQ_CONFIG_ARG_FILE=!RABBITMQ_CONFIG_FILE!
187-
) else if "!RABBITMQ_CONFIG_FILE_NOEX!.conf" == "!RABBITMQ_CONFIG_FILE" (
187+
) else if "!RABBITMQ_CONFIG_FILE_NOEX!.conf" == "!RABBITMQ_CONFIG_FILE!" (
188188
set RABBITMQ_CONFIG_ARG_FILE=!RABBITMQ_ADVANCED_CONFIG_FILE!
189189
) else if not "" == "!RABBITMQ_CONFIG_FILE!" (
190190
if not "!RABBITMQ_CONFIG_FILE_NOEX!" == "!RABBITMQ_CONFIG_FILE!" (
@@ -197,7 +197,7 @@ if "!RABBITMQ_CONFIG_FILE_NOEX!.config" == "!RABBITMQ_CONFIG_FILE" (
197197

198198
CALL :get_noex !RABBITMQ_CONFIG_ARG_FILE! RABBITMQ_CONFIG_ARG_FILE_NOEX
199199

200-
if not "$!RABBITMQ_CONFIG_ARG_FILE_NOEX!.config" == "!RABBITMQ_CONFIG_ARG_FILE!" (
200+
if not "!RABBITMQ_CONFIG_ARG_FILE_NOEX!.config" == "!RABBITMQ_CONFIG_ARG_FILE!" (
201201
if "!RABBITMQ_CONFIG_ARG_FILE!" == "!RABBITMQ_ADVANCED_CONFIG_FILE!" (
202202
echo "ERROR: Wrong extension for RABBITMQ_ADVANCED_CONFIG_FILE: !RABBITMQ_ADVANCED_CONFIG_FILE!"
203203
echo "ERROR: extension should be .config"
@@ -216,7 +216,7 @@ if exist !RABBITMQ_CONFIG_ARG_FILE! (
216216
)
217217

218218
rem Set -conf and other generated config parameters
219-
if "!RABBITMQ_CONFIG_FILE_NOEX!.conf" == "!RABBITMQ_CONFIG_FILE" (
219+
if "!RABBITMQ_CONFIG_FILE_NOEX!.conf" == "!RABBITMQ_CONFIG_FILE!" (
220220
set RABBITMQ_GENERATED_CONFIG_ARG=-conf "!RABBITMQ_CONFIG_FILE!" ^
221221
-conf_dir "!RABBITMQ_GENERATED_CONFIG_DIR!" ^
222222
-conf_script_dir !CONF_SCRIPT_DIR:\=/! ^

src/rabbit_config.erl

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -257,9 +257,13 @@ assert_config(Filename, Env) ->
257257
case filelib:is_regular(Filename) of
258258
true ->
259259
case file:consult(Filename) of
260-
{ok, []} -> {error, "ERROR: Config file ~s should not be empty: ~s"};
260+
{ok, []} -> {error,
261+
{"ERROR: Config file ~s should not be empty: ~s",
262+
[Env, Filename]}};
261263
{ok, [_]} -> ok;
262-
{ok, [_|_]} -> {error, "ERROR: Config file ~s must contain ONE list ended by <dot>: ~s"};
264+
{ok, [_|_]} -> {error,
265+
{"ERROR: Config file ~s must contain ONE list ended by <dot>: ~s",
266+
[Env, Filename]}};
263267
{error, {1, erl_parse, Err}} ->
264268
{error, {"ERROR: Unable to parse erlang terms from ~s file: ~s~n"
265269
"ERROR: Reason: ~p~n"

0 commit comments

Comments
 (0)