Merge lp:~axwalk/juju-core/windows-storage-datasource-path into lp:~go-bot/juju-core/trunk
Proposed by
Andrew Wilkins
Status: | Merged |
---|---|
Approved by: | Andrew Wilkins |
Approved revision: | no longer in the source branch. |
Merged at revision: | 2186 |
Proposed branch: | lp:~axwalk/juju-core/windows-storage-datasource-path |
Merge into: | lp:~go-bot/juju-core/trunk |
Diff against target: |
26 lines (+4/-4) 1 file modified
environs/storage/storage.go (+4/-4) |
To merge this branch: | bzr merge lp:~axwalk/juju-core/windows-storage-datasource-path |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Juju Engineering | Pending | ||
Review via email: mp+200953@code.launchpad.net |
Commit message
environs/storage: use path.Join for datasource
I'm trying to get bootstrap working on Windows again,
and found that the simplestreams lookup was going to
the wrong URL (using '\\' instead of '/' as it should).
This was causing tools lookup to fail.
Description of the change
environs/storage: use path.Join for datasource
I'm trying to get bootstrap working on Windows again,
and found that the simplestreams lookup was going to
the wrong URL (using '\\' instead of '/' as it should).
This was causing tools lookup to fail.
To post a comment you must log in.
Reviewers: mp+200953_ code.launchpad. net,
Message:
Please take a look.
Description:
environs/storage: use path.Join for datasource
I'm trying to get bootstrap working on Windows again,
and found that the simplestreams lookup was going to
the wrong URL (using '\\' instead of '/' as it should).
This was causing tools lookup to fail.
https:/ /code.launchpad .net/~axwalk/ juju-core/ windows- storage- datasource- path/+merge/ 200953
(do not edit description out of merge proposal)
Please review this at https:/ /codereview. appspot. com/49610043/
Affected files (+6, -4 lines): storage/ storage. go
A [revision details]
M environs/
Index: [revision details] 20140106181553- 6uzfea71izl68d4 l
=== added file '[revision details]'
--- [revision details] 2012-01-01 00:00:00 +0000
+++ [revision details] 2012-01-01 00:00:00 +0000
@@ -0,0 +1,2 @@
+Old revision: tarmac-
+New revision: <email address hidden>
Index: environs/ storage/ storage. go storage/ storage. go' storage/ storage. go 2013-11-01 06:20:19 +0000 storage/ storage. go 2014-01-09 05:27:24 +0000
=== modified file 'environs/
--- environs/
+++ environs/
@@ -6,7 +6,7 @@
import (
"fmt"
"io"
- "path/filepath"
+ "path"
"launchpad. net/juju- core/environs/ simplestreams" net/juju- core/utils" treamsDataSourc e{basePath, storage, false}
"launchpad.
@@ -91,10 +91,10 @@
return &storageSimpleS
}
-func (s *storageSimpleS treamsDataSourc e) relpath(path string) string { treamsDataSourc e) relpath(storagePath string) Join(s. basePath, relpath) s.basePath, relpath)
- relpath := path
+func (s *storageSimpleS
string {
+ relpath := storagePath
if s.basePath != "" {
- relpath = filepath.
+ relpath = path.Join(
}
return relpath
}