new package: utils/filelock #3859

Merged
merged 1 commit into from Dec 2, 2015

Conversation

Projects
None yet
2 participants
Contributor

davecheney commented Nov 30, 2015

Juju contains two types that manage on disk locks, the main one is in
juju/utils/fslock, and a second, and less featureful version in container/

This PR moves the second version to its own package (as it has nothing to do
with the container package). This move is temporary, and after refactoring
utils/filelock will be merged into juju/utils/fslock (or fslock merged into this
package).

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

Contributor

davecheney commented Dec 1, 2015

$$merge$$

Contributor

jujubot commented Dec 1, 2015

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

Contributor

jujubot commented Dec 1, 2015

Build failed: Does not match ['fixes-1471941']
build url: http://juju-ci.vapour.ws:8080/job/github-merge-juju/5595

new package: utils/filelock
Juju contains two types that manage on disk locks, the main one is in
juju/utils/fslock, and a second, and less featureful version in container/

This PR moves the second version to its own package (as it has nothing to do
with the container package). This move is temporary, and after refactoring
utils/filelock will be merged into juju/utils/fslock (or fslock merged into this
package).
Contributor

davecheney commented Dec 2, 2015

$$merge$$

Contributor

jujubot commented Dec 2, 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 Dec 2, 2015

Merge pull request #3859 from davecheney/move-filelock
new package: utils/filelock

Juju contains two types that manage on disk locks, the main one is in
juju/utils/fslock, and a second, and less featureful version in container/

This PR moves the second version to its own package (as it has nothing to do
with the container package). This move is temporary, and after refactoring
utils/filelock will be merged into juju/utils/fslock (or fslock merged into this
package).

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

@jujubot jujubot merged commit 53917d9 into juju:master Dec 2, 2015

@davecheney davecheney deleted the davecheney:move-filelock branch Dec 2, 2015

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