Comment 1 for bug 156546

In discussing this with Robert, there are 2 pieces.

a) leave_lock_in_place() was actually supposed to be optional for implementations
b) Pack style repositories shouldn't actually be locking at this level. The physical (on-disk) lock is only taken out when updating the pack names list, and should be done by the smart server, and not the client.