/juju

Fix lp:1454658 TestUseLumberjack fails on windows #2323

Merged
merged 1 commit into from May 13, 2015

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@waigani @jujubot
@waigani
Contributor

waigani commented May 13, 2015

Wrap file paths in filepath.FromSlash.

(Review request: http://reviews.vapour.ws/r/1684/)
@waigani
Fix lp:1454658 TestUseLumberjack fails on windows 
Wrap file paths in filepath.FromSlash.
96c64ce
@waigani
Contributor

waigani commented May 13, 2015

$$fix-1454658$$
@jujubot
Contributor

jujubot commented May 13, 2015

Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju
@jujubot
Contributor

jujubot commented May 13, 2015

Build failed: Does not match ['fixes-1454658']
build url: http://juju-ci.vapour.ws:8080/job/github-merge-juju/3249
@waigani
Contributor

waigani commented May 13, 2015

$$fixes-1454658$$
@jujubot
Contributor

jujubot commented May 13, 2015

Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju

jujubot added a commit that referenced this pull request May 13, 2015

@jujubot
Merge pull request #2323 from waigani/fix-1454658-lumberjack-124 
Fix lp:1454658 TestUseLumberjack fails on windows

Wrap file paths in filepath.FromSlash.

(Review request: http://reviews.vapour.ws/r/1684/)
b5c0b9a

@jujubot jujubot merged commit b5c0b9a into juju:1.24 May 13, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment