From 1df8584d13efd6de53692f0b425df90d1687c41e Mon Sep 17 00:00:00 2001 From: Issy Long Date: Thu, 13 Feb 2020 21:28:12 +0000 Subject: [PATCH] bump-formula-pr: Determine Linux-only formulae with `depends_on :linux` - We used to use `# tag "linux"` in Homebrew/linuxbrew-core for the ~50 Linux-only formulae. As of https://github.com/Homebrew/linuxbrew-core/commit/6578a4aa86b80816cf0553f1d45828c0072197a4, we're using `depends_on :linux` to have a consistent syntax between Linux and macOS. - Therefore, we have to change the search string for Linux-only formulae that determines whether or not `brew bump-formula-pr` deletes the bottle line. --- Library/Homebrew/dev-cmd/bump-formula-pr.rb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Library/Homebrew/dev-cmd/bump-formula-pr.rb b/Library/Homebrew/dev-cmd/bump-formula-pr.rb index 0fa0e4debe..4102b51539 100644 --- a/Library/Homebrew/dev-cmd/bump-formula-pr.rb +++ b/Library/Homebrew/dev-cmd/bump-formula-pr.rb @@ -259,7 +259,7 @@ module Homebrew # When bumping a linux-only formula, # one needs to also delete the sha256 linux bottle line. # That's because of running test-bot with --keep-old option in linuxbrew-core. - if formula.path.read.include?('# tag "linux"') + if formula.path.read.include?("depends_on :linux") replacement_pairs << [ /^ sha256 ".+" => :x86_64_linux\n/m, "\\2",