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

[Mac] New zenmap_auth_wrapper in Objective-C #399

Closed
wants to merge 2 commits into from

Conversation

@ghost
Copy link

commented Jun 2, 2016

The Objective-C zenmap_auth.m file replaces zenmap_auth.c.
Its main purpose is to call applescript scripts to launch Zenmap with a "helper tool" (an authorisation wrapper), that will launch Zenmap with/without administrator privileges.

Waiting for your comments!
Cheers,
VincentD

@dmiller-nmap

This comment has been minimized.

Copy link

commented Jul 12, 2016

This works like a charm, thanks! I think that instead of changing $CC to clang directly, we should change gcc to clang at the top of the script where CC is set. Nothing else in the script uses CC, so there's no reason to set it to something that won't be used.

@ghost

This comment has been minimized.

Copy link
Author

commented Jul 15, 2016

The pleasure is mine. You are right I am going to commit the corresponding changes right away. Thanks for your time and review!

@ghost ghost referenced this pull request Jul 15, 2016
@ghost

This comment has been minimized.

Copy link
Author

commented Jul 15, 2016

Since I deleted the branch by mistake, I just opened a new PR: #456, which closes this one.

@ghost ghost closed this Jul 15, 2016
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant
You can’t perform that action at this time.