You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@brooklyn.apache.org by justinThompson <gi...@git.apache.org> on 2016/11/01 09:55:03 UTC

[GitHub] brooklyn-dist pull request #57: Adds OS check to the setenv script

GitHub user justinThompson opened a pull request:

    https://github.com/apache/brooklyn-dist/pull/57

    Adds OS check to the setenv script

    This PR does the following:
    - Adds an OS check to the start of the `setenv` script
    - Constructs `JAVA_HOME` based on the OS

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

    $ git pull https://github.com/justinThompson/brooklyn-dist setenv-os-check

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

    https://github.com/apache/brooklyn-dist/pull/57.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 #57
    
----
commit 386e4d6ccf9e4ff16426a5ef4ee271c1f0829bef
Author: Justin Thompson <ju...@cloudsoftcorp.com>
Date:   2016-11-01T09:51:32Z

    added OS check and JAVA_HOME setup to the setenv script

----


---
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.
---

[GitHub] brooklyn-dist pull request #57: Adds OS check to the setenv script

Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:

    https://github.com/apache/brooklyn-dist/pull/57


---
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.
---

[GitHub] brooklyn-dist issue #57: Adds OS check to the setenv script

Posted by neykov <gi...@git.apache.org>.
Github user neykov commented on the issue:

    https://github.com/apache/brooklyn-dist/pull/57
  
    LGTM, merging.


---
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.
---