Merge pull request #2265 from git-lfs/rev-list-commits-only

git/rev_list_scanner: teach CommitsOnly option

git/rev_list_scanner.go

@@ -87,6 +87,10 @@ type ScanRefsOptions struct {
 	// output of `git-rev-list(1)`. For more information, see the above
 	// documentation on the RevListOrder type.
 	Order RevListOrder
+	// CommitsOnly specifies whether or not the *RevListScanner should
+	// return only commits, or all objects in range by performing a
+	// traversal of the graph. By default, false: show all objects.
+	CommitsOnly bool
 
 	// SkippedRefs provides a list of refs to ignore.
 	SkippedRefs []string
@@ -212,7 +216,10 @@ func NewRevListScanner(left, right string, opt *ScanRefsOptions) (*RevListScanne
 // occurred.
 func revListArgs(l, r string, opt *ScanRefsOptions) (io.Reader, []string, error) {
 	var stdin io.Reader
-	args := []string{"rev-list", "--objects"}
+	args := []string{"rev-list"}
+	if !opt.CommitsOnly {
+		args = append(args, "--objects")
+	}
 
 	if orderFlag, ok := opt.Order.Flag(); ok {
 		args = append(args, orderFlag)

git/rev_list_scanner_test.go

@@ -128,6 +128,13 @@ func TestRevListArgs(t *testing.T) {
 			},
 			ExpectedArgs: []string{"rev-list", "--objects", "--topo-order", "--do-walk", "left", "right", "--"},
 		},
+		"scan commits only": {
+			Left: "left", Right: "right", Opt: &ScanRefsOptions{
+				Mode:        ScanRefsMode,
+				CommitsOnly: true,
+			},
+			ExpectedArgs: []string{"rev-list", "--do-walk", "left", "right", "--"},
+		},
 	} {
 		t.Run(desc, c.Assert)
 	}