Page MenuHomePhabricator

Correct the behavior of "bin/repository discover --repair"
ClosedPublic

Authored by epriestley on Jan 22 2021, 8:25 PM.
Tags
None
Referenced Files
F12838213: D21513.diff
Thu, Mar 28, 6:01 PM
Unknown Object (File)
Tue, Mar 26, 4:54 AM
Unknown Object (File)
Jan 25 2024, 2:28 AM
Unknown Object (File)
Jan 7 2024, 4:42 AM
Unknown Object (File)
Dec 12 2023, 3:03 PM
Unknown Object (File)
Nov 26 2023, 12:04 AM
Unknown Object (File)
Nov 17 2023, 12:52 PM
Unknown Object (File)
Oct 13 2023, 10:12 PM
Subscribers
None

Details

Summary

Ref T13591. Since D8781, this flag does not function correctly in Git and Mercurial repositories, since ref discovery pre-fills the cache.

Move the "don't look at the database" behavior the flag enables into the cache lookup. D8781 should have been slightly more aggressive and done this, it was just overlooked.

Test Plan
  • Ran bin/repository discover --help and read the updated help text.
  • Ran bin/repository discover --repair in a fully-discovered Git repository.
    • Before: no effect.
    • After: full rediscovery.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This revision was not accepted when it landed; it landed in state Needs Review.Jan 23 2021, 3:51 AM
This revision was automatically updated to reflect the committed changes.