<filename>data/spack-issues/issue-8622.json
{
    "body": "- [x] Speed up locking a bit by disabling the debug info we write to lock files by default.\r\n  - Can still construct a lock with `debug=True`, but we no longer write to/read from the lock file every time we acquire a lock.\r\n- [x] Clean up error messages for when a lock can't be created, or when an exclusive (write) lock can't be taken on a file.\r\n- [x] Add a number of subclasses of LockError to distinguish timeouts from permission issues.\r\n- [x] Add an explicit check to tell the user they're taking a write lock on a read-only file.\r\n  - We had a check for this for when we try to *upgrade* a lock on an RO file, but not for an initial write lock attempt.\r\n- [x] Add more tests for different lock permission scenarios.\r\n- [x] Separate tests for `spack.util.lock` from `llnl.util.lock`, and move `spack.util.multiproc` to `llnl.util.multiproc`\r\n",
    "user": "tgamblin",
    "url": "https://api.github.com/repos/spack/spack/issues/8622",
    "updated_at": "2018-07-21 21:56:18",
    "created_at": "2018-07-01 08:17:47",
    "closed_at": "2018-07-12 17:59:54",
    "state": "closed",
    "title": "Improve lock error messages and permission checks",
    "number": 8622,
    "milestone": null,
    "labels": [
        "locking",
        "refactoring",
        "error-messages"
    ],
    "id": 337273348,
    "html_url": "https://github.com/spack/spack/pull/8622",
    "assignees": [],
    "comments": 0
}