ToolsDir should be series based
Bug #1399322 reported by
Horacio Durán
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| juju-core |
Won't Fix
|
Low
|
Unassigned | ||
Bug Description
Recently I had to revert a change in a pr because when using the windows client the tools dir path would generated without separator whatsoever https:/
ToolsDir and any other Dir generation should use a target series to decide the path composition (that includes the separator). filepath will just use the separator from the current running code OS which is not right for this case.
My patch only fixes the aforementioned issue, but does not correct the underlying problem where this might be broken for windows workloads since I used path.Join.
| Changed in juju-core: | |
| status: | New → Triaged |
| importance: | Undecided → High |
| Changed in juju-core: | |
| milestone: | 1.22 → 1.23 |
| tags: | added: tech-debt |
| Changed in juju-core: | |
| importance: | High → Medium |
| milestone: | 1.23 → none |
| Changed in juju-core: | |
| importance: | Medium → Low |
| Changed in juju-core: | |
| status: | Triaged → Won't Fix |
To post a comment you must log in.
