~mfoord/juju-core/tests-replset

Viewing all changes in revision 998.

  • Committer: John Arbash Meinel
  • Author(s): John A Meinel
  • Date: 2013-03-14 11:17:17 UTC
  • mfrom: (997.1.2 windows-compat)
  • Revision ID: john@arbash-meinel.com-20130314111717-9y0rie4ro2dk7zxa
various: use path instead of filepath for Windows

(yes, that sounds backwards, but it is correct)

There are quite a few places where code uses 'filepath' to generate
a path string. However it is generating the path on the local client
but then *sending* that path to the remote location (such as cloudinit).
This means that when running on Windows, it was telling the new
instance to create things like '\var\lib\juju\bootstrap\agent.conf'
which didn't work so well :).

With this patch, and the patch to goyaml, and a small hack to force
the series to "precise" and the Arch to "amd64", I'm able to run
"juju bootstrap" and "juju status" on Windows. The test suite doesn't
run (because we don't have mongod) and I haven't tested deploying
any actual charms (the next place I expect to be a problem).

But this seems a reasonable start. The test suite passes on Linux
still, so the change seems safe enough.

There are still places that use 'filepath' in the code, but
I tried to audit that they were only using those paths locally.
Also, Windows does need '\' style paths for commands that you
execute, though you can use '/' for any files you need to open,
etc.

R=dimitern, dfc
CC=
https://codereview.appspot.com/7813043

expand all expand all

Show diffs side-by-side

added added

removed removed

Lines of Context: