Mailing-List: contact dev-help@activemq.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@activemq.apache.org
From: fjollberg <git@git.apache.org>
To: dev@activemq.apache.org
Reply-To: dev@activemq.apache.org
Message-ID: <git-pr-199-activemq@git.apache.org>
Subject: [GitHub] activemq pull request #199: Make start script work on busybox
Content-Type: text/plain
Date: Fri, 23 Sep 2016 08:13:32 +0000 (UTC)
archived-at: Fri, 23 Sep 2016 08:13:34 -0000

GitHub user fjollberg opened a pull request:

    https://github.com/apache/activemq/pull/199

    Make start script work on busybox

    The current version of the activemq start script uses options to ps which are unavailable 
    in busybox.
    
    Suggested change refactors the start script to remove duplicated code to find a java process 
    given a pidfile and makes it work on busybox.
    
    I have limited possibilities of verifying the change on different platforms but it is confirmed to
    work on at least Mac OS X in addition to busybox and should work on platforms where 
    previous "ps -p" command worked.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/fjollberg/activemq master

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/activemq/pull/199.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #199
    
----
commit 18dce69a78729e71841fead21c69dd15ffe7db58
Author: Fredrik Jönsson <fjo@kth.se>
Date:   2016-09-23T08:09:54Z

    Remove duplicated code for finding a java process given a pidfile
    and make it work in busybox.

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---