-
Notifications
You must be signed in to change notification settings - Fork 5.6k
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
Disable darwin builds #1571
Disable darwin builds #1571
Conversation
telegraf can't be cross-compiled for darwin, it has C dependencies and thus many of the system plugins won't work.
@sparrc Does this mean that we won't be offering Homebrew builds for macOS? I don't think I've seen any build errors for Darwin before. |
it won't create build errors, but the build won't function properly. Homebrew is still available because those get built natively on osx |
Ah, understood. LGTM. |
@sparrc I know this is already merged but can you talk just a little more about this? Just curious what kinds of things you're seeing not functioning. |
I can't find the issue now, but I remember a user having issues with the cpu plugin |
Word. If you run across it again, I'd like to see it. It's kinda monkeying with some build processes we are using to not have the darwin version. This PR was a surprise for us. :) |
is there a reason you can't use homebrew? |
telegraf can't be cross-compiled for darwin, it has C dependencies and thus many of the system plugins won't work.
@sparrc I work with Joel. We're including the telegraf executables in a project to make distribution easier. We specifically want our users to have an easier time trying the project out on their development machines, and adding the extra step of |
I understand, but if you're distributing a project for development machines then it sounds like you may have the resources to build the binary yourself. Unfortunately darwin doesn't have the best support for build system VMs and Telegraf is not a purely Go project (if you consider dependencies) |
I think I agree with you but want to clarify one thing about this particular PR: the removal of this was to remove building telegraf for binary distribution to various OSx versions, and the preferred way of building telegraf on OSX is either homebrew or build from source using make, correct? |
yes, correct, and BTW this was the issue: #1484 and cross-compiling from linux doesn't work because of this code in gopsutil: https://github.com/shirou/gopsutil/blob/master/cpu/cpu_darwin_cgo.go |
Thanks @sparrc! |
Required for all PRs:
telegraf can't be cross-compiled for darwin, it has C dependencies and
thus many of the system plugins won't work.
cc @rossmcdonald