this gem only works on Linux? #18
Comments
|
maybe, but also the source code contains a serious syntax error, here:
This must be
instead. |
|
Got it. A new release will be made in a few hours. |
|
@victorhazbun Is there some other gem that you would suggest, or is there a pull request that you can make which resolves the issue? |
|
Exception fix pushed. Will look into cap2 issue as time allows. Does anyone else have issues or can provide the environment that they're using? |
|
@victorhazbun what makes you think you need the cap2 gem? Otherwise, I think it's just a syntax error, and @chernesk can close this. |
|
@djberg96 The Just installed this (Arch Linux), and also get this error, although easily remedied by installing the This gem could benefit a lot from some clearer documentation, especially in regards to platform and protocol specifics, such as dependencies, etc. Was hoping for a cleaner approach than using |
There was a missing comma after a |
|
@ForeverZer0 You are right that cap2 is an undocumented dependency, not sure when it was added, but the gemspec should be updated. But, one way or another, icmp pings require root privs (on unixy systems anyway) since you're making a raw socket. Your command line ping command runs suid root, btw. As for your needs, are none of the other ping types viable? |
|
I'll update the gemspec today and make sure that the comma is taken care of.
…On Fri, May 31, 2019, at 7:24 AM, Daniel Berger wrote:
@ForeverZer0 <https://github.com/ForeverZer0> You are right that cap2 is an undocumented dependency, not sure when it was added, but the gemspec should be updated.
But, one way or another, icmp pings require root privs (on unixy systems anyway) since you're making a raw socket. Your command line ping command runs suid root, btw.
As for your needs, are none of the other ping types viable?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub <#18>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AACIOEY2R6UXDX3ASCM5NGTPYEDFXANCNFSM4ERNQDWA>.
|
|
@chernesk looks like you already fixed the missing comma a while back. |
|
I was responding from email, so I hadn't taken a look. Anything else we want to change while I'm in there? |
|
@chernesk Not sure, but I do see some http-ping test failures when I run the tests locally on my RHEL7 laptop. I'm not sure how active fakeweb is. It was considered a dead project for a time, though I can see that there were some updates a couple years ago. |
|
@chernesk Setting up some sort of CI would probably be a good idea - perhaps Travis (Linux) and Appveyor (Windows). |
|
Agreed. I'll work on that as soon as I can.
…On Fri, May 31, 2019, at 8:19 AM, Daniel Berger wrote:
@chernesk <https://github.com/chernesk> Setting up some sort of CI would probably be a good idea - perhaps Travis (Linux) and Appveyor (Windows).
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub <#18>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AACIOE3A5JM5ETZA2VHIH7DPYEJWDANCNFSM4ERNQDWA>.
|
|
@djberg96 Totally missed the syntax issue, you have better eyes than me xD I realize I CAN use the other classes without root, and TBH, for 95% or more of situations, the app will require root anyways (just a custom mirrorlist generator for |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I installed it and started building a "Network ping sweeping", but as soon as I run my program I got this issue:
It seems like I need to install the cap2 gem which only runs on Linux.
The text was updated successfully, but these errors were encountered: