You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@brooklyn.apache.org by geomacy <gi...@git.apache.org> on 2017/04/27 09:48:27 UTC

[GitHub] brooklyn-client pull request #49: Add check that path is not empty.

GitHub user geomacy opened a pull request:

    https://github.com/apache/brooklyn-client/pull/49

    Add check that path is not empty.

    This avoids a panic in a case like br catalog add file://test.zip,
    if you forget to add the path to the file, file:///some/path/test.zip.

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

    $ git pull https://github.com/geomacy/brooklyn-client check-path-not-empty

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

    https://github.com/apache/brooklyn-client/pull/49.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 #49
    
----
commit aeb877223cb3c67ad7f9fc25cd3cef25800f8db7
Author: Geoff Macartney <ge...@cloudsoftcorp.com>
Date:   2017-04-27T09:45:41Z

    Add check that path is not empty.
    
    This avoids a panic in a case like br catalog add file://test.zip,
    if you forget to add the path to the file, file:///some/path/test.zip.

----


---
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-client issue #49: Add check that path is not empty.

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

    https://github.com/apache/brooklyn-client/pull/49
  
    @Graeme-Miller could you review?


---
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-client issue #49: Add check that path is not empty.

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

    https://github.com/apache/brooklyn-client/pull/49
  
    Thanks @Graeme-Miller will merge


---
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-client pull request #49: Add check that path is not empty.

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

    https://github.com/apache/brooklyn-client/pull/49


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