Skip to content

Conversation

mikemorris
Copy link
Contributor

@mikemorris mikemorris commented Nov 16, 2021

Bumps transitive dep https://github.com/go-ole/go-ole to v1.2.6 with fixes for windows/arm and windows/arm64 builds.

Resolves below warning when running make lint

--> Running go golangci-lint
WARN [runner] Can't run linter goanalysis_metalinter: inspect: failed to load package agent: could not load export data: no export data for "github.com/hashicorp/consul/command/agent"

Refs shirou/gopsutil#1181 for additional detail on a non-blocking upstream transitive dependency update.

Also resolves shirou/gopsutil#976 when attempting to compile on macOS with a non-Apple LLVM (such as installed via Homebrew). This error can be hit when running go build or go test directly rather than via the Makefile.

@mikemorris mikemorris requested review from rboyer and a team November 16, 2021 17:47
@github-actions github-actions bot added the pr/dependencies PR specifically updates dependencies of project label Nov 16, 2021
@mikemorris mikemorris force-pushed the deps/upgrade-gopsutil-windows-arm branch from 4d49c1e to 52826ad Compare November 16, 2021 18:01
@vercel vercel bot temporarily deployed to Preview – consul-ui-staging November 16, 2021 18:01 Inactive
@vercel vercel bot temporarily deployed to Preview – consul November 16, 2021 18:01 Inactive
@mikemorris mikemorris force-pushed the deps/upgrade-gopsutil-windows-arm branch from 52826ad to 2149cee Compare November 16, 2021 19:54
@vercel vercel bot temporarily deployed to Preview – consul November 16, 2021 19:54 Inactive
@vercel vercel bot temporarily deployed to Preview – consul-ui-staging November 16, 2021 19:54 Inactive
@rboyer
Copy link
Member

rboyer commented Nov 16, 2021

@mikemorris was this just basically: go get github.com/shirou/gopsutil/v3@latest ?

@mikemorris mikemorris changed the title deps: update gopsutil to fix windows/arm and windows/arm64 builds deps: update gopsutil to fix Windows ARM and macOS non-Apple LLVM builds Nov 16, 2021
Bumps transitive dep go-ole to v1.2.6 with fixes
@mikemorris mikemorris force-pushed the deps/upgrade-gopsutil-windows-arm branch from 2149cee to 3dea9e9 Compare November 16, 2021 20:18
@vercel vercel bot temporarily deployed to Preview – consul November 16, 2021 20:18 Inactive
@vercel vercel bot temporarily deployed to Preview – consul-ui-staging November 16, 2021 20:18 Inactive
@mikemorris mikemorris merged commit 25826e3 into main Nov 16, 2021
@mikemorris mikemorris deleted the deps/upgrade-gopsutil-windows-arm branch November 16, 2021 20:40
@hc-github-team-consul-core
Copy link
Collaborator

🍒 If backport labels were added before merging, cherry-picking will start automatically.

To retroactively trigger a backport after merging, add backport labels and re-run https://circleci.com/gh/hashicorp/consul/504160.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr/dependencies PR specifically updates dependencies of project
Projects
None yet
Development

Successfully merging this pull request may close these issues.

error: 'TARGET_OS_MAC' is not defined, evaluates to 0 [-Werror,-Wundef-prefix=TARGET_OS_]
3 participants