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

syncing the osx changes into the version of hping that antirez posted to github #1

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

savagejen
Copy link

syncing the osx changes into the version of hping that antirez posted to github

syncing the osx changes into the version of hping that antirez posted
to github
@antirez
Copy link
Owner

antirez commented Jun 21, 2012

Thank you @savagejen I just compiled it in my macbook, was awesome to build it after so many years and to see it running :) I'm reviewing your changes that appear to be all good, removing the comments you left about the warnings and to make me aware of what you changed (thanks), and merging everything back.

@savagejen
Copy link
Author

Thanks! I am looking into how to make xcode play nicely with autotools so that we won't have to include that makefile, but I will have to do that later as right now I am busy with job-hunt stuff.

@antirez
Copy link
Owner

antirez commented Jun 21, 2012

@savagejen cool :) no rush at all... it waited years, can wait a few weeks more ;) a lot of work to do here as well, but it's cool to see it working on osx.

@savagejen
Copy link
Author

Well I am willing to help keep the project moving forward, even if slowly. I've been wanting to do more open source work and playing with the hping code was awesome.

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.

3 participants