opam icon indicating copy to clipboard operation
opam copied to clipboard

Avoid head -c GNUism in reftests

Open madroach opened this issue 1 year ago • 2 comments

head -c is not posix-compliant, use cut -b instead.

madroach avatar Jun 04 '24 19:06 madroach

There's some problems with this implementation from what I can tell:

  1. It adds the overhead of reading the lock directory and checking if it's up to date on every single iteration of the watch mode. Has this overhead been measured?

  2. Auto re-locking does not get triggered when local opam repositories are updated.

  3. It does not use a lock so a user manually running dune pkg lock can race against the auto lock.

Apart from that, I have some questions:

  1. How does one force dune to update remote opam repositories before running the auto lock?

  2. What happens when one doing is multiple updates to project files in succession? Does it mean this triggers multiple auto locks that all get staggered?

rgrinberg avatar Jun 15 '24 01:06 rgrinberg

This PR can be closed as it was done in but in a different way in #10851.

maiste avatar Sep 23 '24 13:09 maiste