Differential D21192

Restore the ":(attr:filter=lfs)" test for LFS
ClosedPublic
Actions

Authored by epriestley on Apr 30 2020, 4:01 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sat, Apr 20, 5:14 PM

	Unknown Object (File)
	Fri, Apr 19, 12:02 AM

	Unknown Object (File)
	Thu, Apr 18, 1:14 PM

	Unknown Object (File)
	Tue, Apr 16, 7:31 AM

	Unknown Object (File)
	Sat, Apr 6, 4:35 AM

	Unknown Object (File)
	Mon, Apr 1, 2:11 AM

	Unknown Object (File)
	Fri, Mar 29, 8:47 AM

	Unknown Object (File)
	Mar 10 2024, 12:24 AM

View All 69 Files

Subscribers

Details

Reviewers: None
Commits: rARCce7bd679804c: (stable) Restore the ":(attr:filter=lfs)" test for LFS
rARC284139a24ea4: Restore the ":(attr:filter=lfs)" test for LFS

Summary

See D21190. The ".gitattributes" approach fails when ".gitattributes" is in a subdirectory (or global). These are probably unusual cases, but at least one is known in the wild.

Instead:

Restore the ":(attr:filter=lfs)" test, which seems to be the fastest accurate test available in modern Git.
If the test fails, assume the repository is not LFS. This only impacts users running very old versions of Git.

Test Plan

In LFS and non-LFS repositories, created diffs. Saw correct detection again.
Broke the command on purpose, saw LFS detection conclude "no LFS", but not fail disastrously.

Diff Detail

Repository

Branch

lfs2

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 24284
Build 33452: Run Core Tests
Build 33451: arc lint + arc unit

Event Timeline

epriestley created this revision.Apr 30 2020, 4:01 AM

Harbormaster completed remote builds in B24284: Diff 50469.Apr 30 2020, 4:01 AM

epriestley requested review of this revision.Apr 30 2020, 4:01 AM

epriestley mentioned this in D21190: Detect LFS by looking for tracks in ".gitattributes" instead of using "ls-tree".Apr 30 2020, 4:03 AM

This revision was not accepted when it landed; it landed in state Needs Review.Apr 30 2020, 4:04 AM

Closed by commit rARC284139a24ea4: Restore the ":(attr:filter=lfs)" test for LFS (authored by epriestley). · Explain Why

This revision was automatically updated to reflect the committed changes.

epriestley added a commit: rARC284139a24ea4: Restore the ":(attr:filter=lfs)" test for LFS.

epriestley added a commit: rARCce7bd679804c: (stable) Restore the ":(attr:filter=lfs)" test for LFS.

Thanks so much for fixing this!

Revision Contents
Changeset List

Path

Size

src/

repository/

api/

ArcanistGitAPI.php

63 lines

Diff 50469

src/repository/api/ArcanistGitAPI.php

Loading...