Lockfile is unfortunately named

Bug #916107 reported by Marnanel Thurman
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Plover
Fix Committed
Low
Jordan Metzmeier

Bug Description

Currently the lockfile is at "/tmp/plover". This allows for one instance of Plover per *filesystem*, which is not what we want. It should include the name of localhost, the name of the user, and the display number. It should also begin with a dot to encourage curious users to leave it alone.

Revision history for this message
Jordan Metzmeier (jordanm-k) wrote :

I am attaching a patch that accomplishes all of these requirements.

Revision history for this message
Joshua Harlan Lifton (joshua-harlan-lifton) wrote :

Looks good. I'll apply the patch. Thanks for the fix.

Changed in plover:
assignee: nobody → Jordan Metzmeier (jordanm-k)
status: New → In Progress
Changed in plover:
importance: Undecided → Low
Revision history for this message
Joshua Harlan Lifton (joshua-harlan-lifton) wrote :

Patch applied in commit 63 of trunk.

Changed in plover:
status: In Progress → Fix Committed
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.