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

Some small additions #1

Closed
wants to merge 3 commits into from
Closed

Some small additions #1

wants to merge 3 commits into from

Conversation

SilverFoxx
Copy link

Not sure if you're looking for outside involvement, but I'd be obliged if you could consider these small changes.
I've triple checked they don't break anything :-)

@brav0hax
Copy link
Owner

brav0hax commented May 6, 2013

This is great I was wondering if you can drop the excess./ part and submit
2 pull requests one for macchanger and one for the webserver piece.
Would like to suggest simple format changes and discuss further.

Thanks,
Eric
On May 5, 2013 7:14 AM, "SilverFoxx" notifications@github.com wrote:

Not sure if you're looking for outside involvement, but I'd be obliged if
you could consider these small changes.

I've triple checked they don't break anything :-)

You can merge this Pull Request by running

git pull https://github.com/SilverFoxx/easy-creds master

Or view, comment on, or merge it at:

#1
Commit Summary

  • Remove excess "/"
  • auto-detect dhcp nameserver
  • add option to macchange fakeAP

File Changes

Patch Links:

@SilverFoxx
Copy link
Author

Will do (once I've read the git manual to work out how).

@SilverFoxx SilverFoxx closed this May 6, 2013
@brav0hax
Copy link
Owner

brav0hax commented May 6, 2013

Ha...yeah I feel the same way. Maybe just email me and we can work them out
and then just submit a pull with everything.
On May 6, 2013 6:33 AM, "SilverFoxx" notifications@github.com wrote:

Will do (once I've read the git manual to work out how).


Reply to this email directly or view it on GitHubhttps://github.com//pull/1#issuecomment-17477024
.

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.

None yet

2 participants