Re: [PATCH v2] travis-ci: fix Perforce install on macOS
From: Junio C Hamano <hidden>
Date: 2017-01-23 18:39:39
Lars Schneider [off-list ref] writes:
The `perforce` and `perforce-server` package were moved from brew [1][2] to cask [3]. Teach TravisCI the new location. Perforce updates their binaries without version bumps. That made the brew install (legitimately!) fail due to checksum mismatches. The workaround is not necessary anymore as Cask [4] allows to disable the checksum test for individual formulas. [1] https://github.com/Homebrew/homebrew-binary/commit/1394e42de04d07445f82f9512627e864ff4ca4c6 [2] https://github.com/Homebrew/homebrew-binary/commit/f8da22d6b8dbcfcfdb2dfa9ac1a5e5d8e05aac2b [3] https://github.com/caskroom/homebrew-cask/pull/29180 [4] https://caskroom.github.io/ Signed-off-by: Lars Schneider <redacted> --- Hi, this small update removes one more unnecessary line and makes the formula name lower case (no functional reason - just looks better ;-). @Junio: Do you prefer such a v2 as "--in-reply-to" to v1 or as separate thread? What eases your workflow?
It does not make that much of a difference if the second one comes within a few days since the first one but in general it probably would help if the follow-up is threaded _and_ made it clear upfront that if somebody is reading v2 then v1 can be ignored ;-). Thanks.