fix(xbar/brew-updates): upgrade all actions now only update listed packages

Previously when upgrading all, no package names were passed in, meaning
if a new package has become available since the xbar script last ran,
that would be updated too.

Now instead each listed package in the xbar menu is explicitly passed to
brew.
This commit is contained in:
2022-01-16 03:27:40 +00:00
parent f864236203
commit 4e43aacaf1

View File

@@ -2,7 +2,7 @@
# frozen_string_literal: true # frozen_string_literal: true
# <xbar.title>Brew Updates</xbar.title> # <xbar.title>Brew Updates</xbar.title>
# <xbar.version>v2.2.1</xbar.version> # <xbar.version>v2.2.2</xbar.version>
# <xbar.author>Jim Myhrberg</xbar.author> # <xbar.author>Jim Myhrberg</xbar.author>
# <xbar.author.github>jimeh</xbar.author.github> # <xbar.author.github>jimeh</xbar.author.github>
# <xbar.desc>List and manage outdated Homebrew formulas and casks</xbar.desc> # <xbar.desc>List and manage outdated Homebrew formulas and casks</xbar.desc>
@@ -194,21 +194,23 @@ module Brew
if formulas.size.positive? && casks.size.positive? if formulas.size.positive? && casks.size.positive?
printer.item( printer.item(
"Upgrade All (#{formulas.size + casks.size})", "Upgrade All (#{formulas.size + casks.size})",
terminal: true, refresh: true, shell: [brew_path, 'upgrade'] terminal: true, refresh: true,
shell: [brew_path, 'upgrade'] +
formulas.map(&:name) + casks.map(&:name)
) )
end end
if formulas.size.positive? if formulas.size.positive?
printer.item( printer.item(
"Upgrade All Formulas (#{formulas.size})", "Upgrade All Formulas (#{formulas.size})",
terminal: true, refresh: true, terminal: true, refresh: true,
shell: [brew_path, 'upgrade', '--formula'] shell: [brew_path, 'upgrade', '--formula'] + formulas.map(&:name)
) )
end end
if casks.size.positive? if casks.size.positive?
printer.item( printer.item(
"Upgrade All Casks (#{casks.size})", "Upgrade All Casks (#{casks.size})",
terminal: true, refresh: true, terminal: true, refresh: true,
shell: [brew_path, 'upgrade', '--cask'] shell: [brew_path, 'upgrade', '--cask'] + casks.map(&:name)
) )
end end
end end