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
Closed

[Mac] New zenmap_auth_wrapper in Objective-C #399

wants to merge 2 commits into from

Conversation

ghost
Copy link

@ghost ghost 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
Copy link

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
Copy link
Author

ghost 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 mentioned this pull request Jul 15, 2016
@ghost
Copy link
Author

ghost 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 pull request was closed.
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

1 participant