227980 2021-07-15 00:15:28 -0700 Resolve land v. land-cowhand differences 2023-12-06 11:02:13 -0800 1 1 1 Unclassified WebKit Tools / Tests WebKit Local Build Unspecified Unspecified RESOLVED INVALID InRadar P2 Normal --- 221991 1 gsnedders webkit-unassigned dpino jbedard ross.kirsling webkit-bug-importer wenson_hsieh oldest_to_newest 1777142 0 gsnedders 2021-07-15 00:15:28 -0700 From Bug 221991: (In reply to Diego Pino from comment #22) > (In reply to Jonathan Bedard from comment #18) > > > Any reason you're using `land-cowhand` over `land`? My understanding was > > that `land-cowhand` skipped the pre-commit tests, but `land` no longer has > > pre-commit tests in it. > > No other reason than tradition. When I asked other more experienced WebKit > developers how I could land a patch without filing a bug I was taught > `land-cowhand` and I've been keep using it since then for that kind of task. (In reply to Wenson Hsieh from comment #25) > FWIW, I generally prefer `land-cowhand` over `land`, since it automatically > generates and presents a pretty-printed version of the patch that I can > check for sanity. Looking at the code, there's a fair difference between land and land-cowhand: Land: steps = [ steps.AddSvnMimetypeForPng, steps.UpdateChangeLogsWithReviewer, steps.ValidateReviewer, steps.ValidateChangeLogs, steps.Build, steps.Commit, steps.CloseBugForLandDiff, ] LandCowhand: steps = [ steps.SortXcodeProjectFiles, steps.PrepareChangeLog, steps.EditChangeLog, steps.CheckStyle, steps.ConfirmDiff, steps.Build, steps.Commit, steps.CloseBugForLandDiff, ] These are very different, and indicate very different intended uses, as far as I can tell. land is intended to be used when landing a reviewed patch, whereas land-cowhand is intended to be used when landing an unreviewed patch. While I'm skeptical about putting much in the way of resources into this given the forthcoming transition to GitHub PRs, I suspect we should try and reduce the difference between the two (e.g. adding steps.ConfirmDiff to Land), and potentially rename land-cowhand (again) to something like land-unreviewed to make it clear what the intended use. 1777143 1 webkit-bug-importer 2021-07-15 00:15:40 -0700 <rdar://problem/80617267> 1997254 2 gsnedders 2023-12-05 15:52:43 -0800 Nobody in the past two years has complained about LandCowhand being deprecated (and we have admittedly moved to git since!), so I suggest we close this as invalid when we just remove the deprecated command. 1997438 3 gsnedders 2023-12-06 11:02:13 -0800 LandCowhand no longer exists (removed via bug 221991).