Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update gopsutil to v3.21.3 #288

Merged
merged 1 commit into from
May 31, 2022

Conversation

aashishkapur
Copy link
Contributor

I'm getting the following warning when building a package depending on lightstep-tracer-go:
../../pkg/mod/github.com/shirou/gopsutil/v3@v3.21.2/cpu/cpu_darwin_cgo.go:13:5: warning: 'TARGET_OS_MAC' is not defined, evaluates to 0 [-Wundef-prefix=TARGET_OS_]

We can fix this by upgrading gopsutil, see shirou/gopsutil#1042

I'm getting the following warning when building a package depending on lightstep-tracer-go:
../../pkg/mod/github.com/shirou/gopsutil/v3@v3.21.2/cpu/cpu_darwin_cgo.go:13:5: warning: 'TARGET_OS_MAC' is not defined, evaluates to 0 [-Wundef-prefix=TARGET_OS_]

We can fix this by upgrading gopsutil, see shirou/gopsutil#1042
@aashishkapur
Copy link
Contributor Author

@codeboten - mind triggering CI for this branch? Looks like I don't have permissions

@aashishkapur
Copy link
Contributor Author

@codeboten friendly bump on this PR!

@codeboten
Copy link
Contributor

re-opening to trigger circle ci. not sure why the status didnt report here.

@codeboten codeboten closed this May 31, 2022
@codeboten codeboten reopened this May 31, 2022
@aashishkapur
Copy link
Contributor Author

Looks like that did it - thanks! Is there anything I can do to get this merged in?

@aashishkapur
Copy link
Contributor Author

Could I get your review on this PR @codeboten?

Copy link
Contributor

@codeboten codeboten left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the update!

@codeboten codeboten merged commit 32244bd into lightstep:master May 31, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants