cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From snuf <>
Subject [GitHub] cloudstack pull request: Ovm3firefix: fixes for unit tests on Wind...
Date Fri, 27 Mar 2015 10:29:35 GMT
GitHub user snuf opened a pull request:

    Ovm3firefix: fixes for unit tests on Windows and for running management on Windows.

    Some of the unit tests fail on Windows as File.separator was used, using it might seem
like a good idea as suggested by Sonarcube, but all the paths generated with it are actually
only run on Linux hosts as the agent for OVM3 runs on linux. So using the File.seperator is
a real bad idea, not just for the tests but also for someone running the manager on a windows

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

    $ git pull ovm3firefix

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

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

    This closes #133
commit d0e25d96fbdcce830795a5590afdf7f0e3302546
Author: Funs <>
Date:   2015-03-27T10:14:35Z

    Fix paths so they also work with tests on windows.
    Sonarcube is great, but has no context and can be wrong, in this
    case File.separator is nice if you're running platform independent,
    the agent on OVM3 is however NOT platform independent and will break
    if we feed it windows separators.

commit e7d7643dd71b2175a8f553e1659141371c631656
Author: Funs <>
Date:   2015-03-27T10:19:16Z

    Only one thing can come from the agent


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 or file a JIRA ticket
with INFRA.

View raw message