Skip to content
This repository has been archived by the owner on Jan 5, 2022. It is now read-only.

Repush to 2.1.1 with compatibility and bug fixes #46

Merged
merged 14 commits into from
May 4, 2014

Conversation

ekzobrain
Copy link
Contributor

  1. Changed all that we discussed in previous (closed) pull request.
  2. Bug fixes
  3. Changed $LOCK_UNIQUE_ID from 'daemon_lock' to 'pid', so File locks would result in filenames like this: /some/path/DaemonName.pid, which is more unix-style compliant. In other lock providers this change doesn't make sense.

The only backward incompatible change in this update is that Core_Lock_Lock::set() is now protected, not public, so custom lock plugins should be updated to conform this. Also it would be good (but not required) to remove checking logic from custom set() methods and limit their functionality to lock setting only.

shaneharter added a commit that referenced this pull request May 4, 2014
Repush to 2.1.1 with compatibility and bug fixes
@shaneharter shaneharter merged commit af2d024 into shaneharter:2.1.1 May 4, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants