-
Notifications
You must be signed in to change notification settings - Fork 63
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
Release new gem version with fix for ip command? #24
Comments
This really should get done - this breaks, for instance, aws-sdk on any OS modern enough to have gotten rid of net-tools (and by extension ifconfig). Are there any blockers to this that need resolving and can be done by the community (ie us lot)? |
+1 would really like to see a gem release! |
Please do a gem release, thanks! |
Wow, I guess I really haven't done a release in ages. For some reason I thought one went out a few months ago, but I guess it was one of my other gems. I'll see if I can't get things ready for a release in the next day or so. |
I am experiencing the same issue. The context is I'm trying to run fluentd + fluent-plugin-s3 in Docker. Docker in recent release has deprecated 'ifconfig' and only provide fluent-plugin-s3 depends on 'fluent-mixin-config-placeholders'. uuidtools is loaded and get confused. |
When I looked at the source code, the bug has already been fixed in 3cdad0a All we need is a release :) |
I'm also waiting next release! |
My workaround in recent Docker was patching a |
Profuse apologies for how long it took to get around to taking care of this. I know it's been a problem for a bunch of people. I've just been bogged down with unrelated startup stuff. 😓 Cleared up by cbf7fe7. |
awesome! Thank you for taking your time managing this project. Very much appreciated! |
Thank you very much! |
Yay! |
Thanks! |
Commit 8c0d99d fixes support for getting the mac address through the
ip
command. After patching it myself, I figured out the solution was already in master. Maybe this fix deserves a new gem release, as theip
command will be more and more common?The text was updated successfully, but these errors were encountered: