GitHub user snuf opened a pull request:
https://github.com/apache/cloudstack/pull/133
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 host.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/snuf/cloudstack ovm3firefix
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/cloudstack/pull/133.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 #133
----
commit d0e25d96fbdcce830795a5590afdf7f0e3302546
Author: Funs <[email protected]>
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 <[email protected]>
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 [email protected] or file a JIRA ticket
with INFRA.
---
