go 1.19 [email protected] 1.18.5

[bayandin]

Created with brew bump-formula-pr.

resource blocks may require updates.

[carlocab]

We want CI-build-dependents-from-source for major version bumps, no? Though we've got so many long-running PR jobs going I'm a bit tempted to skip it for now.

[bayandin]

We want CI-build-dependents-from-source for major version bumps, no?

Hmm, shouldn't the label be set automatically?

Though we've got so many long-running PR jobs going I'm a bit tempted to skip it for now.

We could try, at least once.

[carlocab]

Hmm, shouldn't the label be set automatically?

It should, but I think our triage workflow gets confused as soon as a PR touches more than one file.

[Bo98]

go-critic and golangci-lint need revision bumps.

[carlocab]

octant is pre-existing. #106816

Most of the rest (build constraints, garbage collector, Go version complaints) can probably be fixed by building with a [email protected].

That still leaves a few failures, though.

[andig]

@carlocab are you planning to fix Go 1.18 for part of the packages? I could help tracking the bug ones as long as Go is concerned (not python ...).

[bayandin]

a bunch of them should be fixed by switching to [email protected] (will do it as soon the CI run is done)

[andig]

For https://github.com/go4org/unsafe-assume-no-moving-gc 1.19 is already marked safe, I'll open upstream issues.

[bayandin]

Formulae that require some attention (have unknown failure):

• brew install --build-from-source grafana
• brew install --build-from-source k3d
• brew install --build-from-source liqoctl
• brew install --build-from-source sync_gateway
• brew install --build-from-source traefik@1

[carlocab]

I think grafana is also pre-existing. The error looks similar to #107028.

[andig]

traefik@1 is not related to Go at all- no idea. Must be pre-existing?

[carlocab]

traefik@1 is not related to Go at all- no idea. Must be pre-existing?

Maybe. Do you see a similar error with

brew install -s traefik@1

?

[andig]

traefik@1 is not related to Go at all- no idea. Must be pre-existing?

Maybe. Do you see a similar error with

brew install -s traefik@1

?

Traefik has a different issue for me, but still broken:

==> Installing traefik@1
==> yarn upgrade
==> yarn install
==> yarn run build
Last 15 lines from /Users/andig/Library/Logs/Homebrew/traefik@1/03.yarn:
    at loadLoader (/private/tmp/traefikA1-20220803-18031-17oz0fj/traefik-1.7.34/webui/node_modules/loader-runner/lib/loadLoader.js:32:11)
    at iteratePitchingLoaders (/private/tmp/traefikA1-20220803-18031-17oz0fj/traefik-1.7.34/webui/node_modules/loader-runner/lib/LoaderRunner.js:169:2)
    at iteratePitchingLoaders (/private/tmp/traefikA1-20220803-18031-17oz0fj/traefik-1.7.34/webui/node_modules/loader-runner/lib/LoaderRunner.js:165:10)
    at /private/tmp/traefikA1-20220803-18031-17oz0fj/traefik-1.7.34/webui/node_modules/loader-runner/lib/LoaderRunner.js:176:18
    at loadLoader (/private/tmp/traefikA1-20220803-18031-17oz0fj/traefik-1.7.34/webui/node_modules/loader-runner/lib/loadLoader.js:47:3)
    at iteratePitchingLoaders (/private/tmp/traefikA1-20220803-18031-17oz0fj/traefik-1.7.34/webui/node_modules/loader-runner/lib/LoaderRunner.js:169:2)
    at iteratePitchingLoaders (/private/tmp/traefikA1-20220803-18031-17oz0fj/traefik-1.7.34/webui/node_modules/loader-runner/lib/LoaderRunner.js:165:10)
    at /private/tmp/traefikA1-20220803-18031-17oz0fj/traefik-1.7.34/webui/node_modules/loader-runner/lib/LoaderRunner.js:176:18
    at loadLoader (/private/tmp/traefikA1-20220803-18031-17oz0fj/traefik-1.7.34/webui/node_modules/loader-runner/lib/loadLoader.js:47:3)
    at iteratePitchingLoaders (/private/tmp/traefikA1-20220803-18031-17oz0fj/traefik-1.7.34/webui/node_modules/loader-runner/lib/LoaderRunner.js:169:2)
    at runLoaders (/private/tmp/traefikA1-20220803-18031-17oz0fj/traefik-1.7.34/webui/node_modules/loader-runner/lib/LoaderRunner.js:365:2)
    at NormalModule.doBuild (/private/tmp/traefikA1-20220803-18031-17oz0fj/traefik-1.7.34/webui/node_modules/webpack/lib/NormalModule.js:182:3)
    at NormalModule.build (/private/tmp/traefikA1-20220803-18031-17oz0fj/traefik-1.7.34/webui/node_modules/webpack/lib/NormalModule.js:275:15)
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

[bayandin]

I've pushed everything I have.
Meanwhile, while we're waiting for a free CI-long-timeout slot, feel free to check failed formulae on other systems (I've checked only 12-arm64 runner). Here's a link to the previous run: https://github.com/Homebrew/homebrew-core/actions/runs/2785592300

[bayandin]

So far, after 12-arm64, 11-arm64, linux:

Unknown:

• brew test --retry --verbose func-e (on 11-arm64 only)

 ==> Testing func-e
  ==> /opt/homebrew/Cellar/func-e/1.1.3/bin/func-e run --version
  dyld: Symbol not found: __ZNKSt3__115basic_stringbufIcNS_11char_traitsIcEENS_9allocatorIcEEE3strEv
    Referenced from: /private/tmp/func-e-test-20220805-14034-7m57hu/.func-e/versions/1.22.2/bin/envoy (which was built for Mac OS X 12.3)
    Expected in: /usr/lib/libc++.1.dylib

• brew install --build-from-source sync_gateway (on 12-arm64 only)

 Error: Failure while executing; `python rewrite-manifest.sh --manifest-url file:///private/tmp/sync_gateway-20220805-623-avs958/manifest/default.xml --project-name sync_gateway --set-revision e54a62741bb28f3e54a6599c21c739df9a9dad76` exited with 127. Here's the output:

Switch to 1.18:

• brew install --build-from-source yorkie
• brew test --retry --verbose loki

Pre-existing:

• brew fetch --build-from-source --retry krakend # https://github.com/krakendio/krakend-ce/issues/541
• brew fetch --build-from-source --retry whalebrew # https://github.com/whalebrew/whalebrew/issues/192
• brew install --build-from-source octant # preexisting https://github.com/Homebrew/homebrew-core/pull/106816
• brew install --verbose --build-bottle istioctl # preexisting https://github.com/Homebrew/homebrew-core/pull/107108

[carlocab]

I suggest moving the [email protected] addition along with its dependents (the ones that build, anyway) into a separate PR to help simplify this one.

We also don't need new bottles for the ones that just have a build-only dependency on [email protected] anyway.

[bayandin]

I suggest moving the [email protected] addition along with its dependents (the ones that build, anyway) into a separate PR to help simplify this one.

We also don't need new bottles for the ones that just have a build-only dependency on [email protected] anyway.

Ok, let's try!

[BrewTestBot]

@bayandin has triggered a merge.

[BrewTestBot]

⚠️ @bayandin bottle publish failed.

bottle publish failed

[BrewTestBot]

@bayandin has triggered a merge.