Skip to content

Don't throw away gitaly dir when .backups changes

Jacob Vosmaer requested to merge jv-fix-gitaly-clone into master

What does this Merge Request do?

This fixes a Makefile bug in the code that clones /gitaly. This code should only kick in when /gitaly/.git does not exist. However, because it depended on /.backups, we would do a new gitaly clone every time /.backups changes because that is how make works: if a dependency has an mtime later than the target mtime, consider the target stale and rebuild it. We don't want that mtime logic here.

Merge Request checklist

  • Tests added for new functionality. If not, please raise Issue to follow-up.
  • This change is backward compatible. If not, please include steps to communicate to our users.
  • Documentation added/updated, if needed.
  • gdk doctor test added, if needed.

Merge request reports