[Webkit-unassigned] [Bug 227980] New: Resolve land v. land-cowhand differences

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=227980

            Bug ID: 227980
           Summary: Resolve land v. land-cowhand differences
           Product: WebKit
           Version: WebKit Local Build
          Hardware: Unspecified
                OS: Unspecified
            Status: NEW
          Severity: Normal
          Priority: P2
         Component: Tools / Tests
          Assignee: webkit-unassigned at lists.webkit.org
          Reporter: gsnedders at apple.com
                CC: dpino at igalia.com, jbedard at apple.com,
                    ross.kirsling at sony.com,
                    webkit-bug-importer at group.apple.com,
                    wenson_hsieh at apple.com
            Blocks: 221991

>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.


Referenced Bugs:

https://bugs.webkit.org/show_bug.cgi?id=221991
[Bug 221991] [RFC] Remove many webkit-patch commands
-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20210715/a3803793/attachment.htm>