Re: [PATCH] ci: Also perform `brew upgrade` on MacOS

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Jun 15, 2021 at 12:43:39PM +0200, Martin Kletzander wrote:
> ci: Also perform `brew upgrade` on MacOS

s/MacOS/macOS/

But see below for why we might have to change the subject even
further.

> @Andrea: if you have a good explanation you'd like to put in the commit message,
> I'd me glad to add it (or you can do that as well).  Thanks

I think something like

  The base OS image might include outdated contents, and we don't
  want to get spurious failures caused by bugs that have already been
  fixed in the respective packages.

  This is particularly important on macOS, because 'brew install foo'
  will fail if 'foo' is already installed but outdated: upgrading all
  packages first ensures we never run into this scenario.

would about sum it up.

> @@ -443,6 +444,7 @@ x64-macos-11-build:
>      CIRRUS_VM_IMAGE_SELECTOR: image
>      CIRRUS_VM_IMAGE_NAME: big-sur-base
>      UPDATE_COMMAND: brew update
> +    UPGRADE_COMMAND: brew upgrade

I believe you also need to add

  UPGRADE_COMMAND: pkg upgrade -y

to the FreeBSD jobs: I don't think Cirrus CI would appreciate having
a completely empty string in the list of commands it's supposed to
run.

With that squashed in,

  Reviewed-by: Andrea Bolognani <abologna@xxxxxxxxxx>

and thanks for taking care of this :)

-- 
Andrea Bolognani / Red Hat / Virtualization




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux