Skip to content

prjxray/util.py: replaced fcntl by flufl.lock to allows using script via NFS#3

Open
trabucayre wants to merge 1 commit intoopenXC7:masterfrom
trabucayre:fix_lock_with_nfs
Open

prjxray/util.py: replaced fcntl by flufl.lock to allows using script via NFS#3
trabucayre wants to merge 1 commit intoopenXC7:masterfrom
trabucayre:fix_lock_with_nfs

Conversation

@trabucayre
Copy link
Copy Markdown

When openXC7 toolchain is shared via NFS to avoid multiple build (with https://github.com/openXC7/toolchain-installer/blob/main/toolchain-sources-builder.sh for example) fasm2frames fails with error:

[Errno 9] Bad file descriptor: /somewhere/openxc7/share/nextpnr/prjxray-db/zynq7/mapping/devices.yaml

This is due to the use of fcntl.flock: this library is not compatible with NFS

This PR replaces fcntl.flock by flufl.lock. This library:

  • handle timeout directly
  • is similar to fcntl.flock but uses a lock file

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant