[jira] [Commented] (APLO-8) apollo-broker-service should avoid sudo if already the right user.

["Lionel Cons (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/APLO-8?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13036075#comment-13036075 ] 

Lionel Cons commented on APLO-8:
--------------------------------

The fix in 1.0-20110519.002024-114 uses == instead of =. The former is bash specific and performs pattern matching, which is not needed here.

> apollo-broker-service should avoid sudo if already the right user.
> ------------------------------------------------------------------
>
>                 Key: APLO-8
>                 URL: https://issues.apache.org/jira/browse/APLO-8
>             Project: ActiveMQ Apollo
>          Issue Type: Bug
>          Components: apollo-cli, apollo-distro
>    Affects Versions: 1.0-beta2
>            Reporter: Hiram Chirino
>            Assignee: Hiram Chirino
>             Fix For: 1.0-beta3
>
>
> Lionel Cons posted to the dev list:
> {quote}
> If the $APOLLO_USER runs apollo-service-broker, the script will
> attempt to use sudo anyway.
> I would suggest to replace:
>  if [ -z "$APOLLO_USER" ] ; then
> by something like:
>  if [ -z "$APOLLO_USER" -a `id -un` != "$APOLLO_USER" ] ; then
> {quote}

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira