Skip to content

Commit

Permalink
rabbitmq-script-wrapper: Don't use "exec" in front of "su"
Browse files Browse the repository at this point in the history
This is useless because su(1) forks a process anyway. Furthermore, it
broke RabbitMQ in Docker.

Fixes #129.
  • Loading branch information
dumbbell committed Apr 23, 2015
1 parent 3d0040f commit f678ab5
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion packaging/common/rabbitmq-script-wrapper
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ elif [ `id -u` = `id -u rabbitmq` -o "$SCRIPT" = "rabbitmq-plugins" ] ; then
fi
exec /usr/lib/rabbitmq/bin/${SCRIPT} "$@"
elif [ `id -u` = 0 ] ; then
exec @SU_RABBITMQ_SH_C@ "/usr/lib/rabbitmq/bin/${SCRIPT} ${CMDLINE}"
@SU_RABBITMQ_SH_C@ "/usr/lib/rabbitmq/bin/${SCRIPT} ${CMDLINE}"
else
/usr/lib/rabbitmq/bin/${SCRIPT}
echo
Expand Down

0 comments on commit f678ab5

Please sign in to comment.