You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@cloudstack.apache.org by karuturi <gi...@git.apache.org> on 2015/07/24 11:27:51 UTC

[GitHub] cloudstack pull request: fixed finbugs issue due to PR #609

GitHub user karuturi opened a pull request:

    https://github.com/apache/cloudstack/pull/622

    fixed finbugs issue due to PR #609

    applicationCtx need not be static as the bean is singleton

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

    $ git pull https://github.com/karuturi/cloudstack ldap-findbugs

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

    https://github.com/apache/cloudstack/pull/622.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 #622
    
----
commit e0ed0b9d36330baac017381d90d7ab4311743df7
Author: Rajani Karuturi <ra...@gmail.com>
Date:   2015-07-24T09:22:53Z

    fixed finbugs issue due to PR #609
    
    applicationCtx need not be static as the bean is singleton

----


---
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] cloudstack pull request: fixed finbugs issue due to PR #609

Posted by kishankavala <gi...@git.apache.org>.

Github user kishankavala commented on the pull request:

    https://github.com/apache/cloudstack/pull/622#issuecomment-124481535
  
    LGTM


---
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] cloudstack pull request: fixed finbugs issue due to PR #609

Posted by asfgit <gi...@git.apache.org>.

Github user asfgit closed the pull request at:

    https://github.com/apache/cloudstack/pull/622


---
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] cloudstack pull request: fixed finbugs issue due to PR #609

Posted by karuturi <gi...@git.apache.org>.

Github user karuturi commented on the pull request:

    https://github.com/apache/cloudstack/pull/622#issuecomment-124500655
  
    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.
---

[GitHub] cloudstack pull request: fixed finbugs issue due to PR #609

Posted by koushik-das <gi...@git.apache.org>.

Github user koushik-das commented on the pull request:

    https://github.com/apache/cloudstack/pull/622#issuecomment-124484710
  
    lgtm


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