Skip to content

nmap: do not build zenmap by default#577

Closed
npotts wants to merge 3 commits into
openwrt:masterfrom
npotts:master
Closed

nmap: do not build zenmap by default#577
npotts wants to merge 3 commits into
openwrt:masterfrom
npotts:master

Conversation

@npotts
Copy link
Copy Markdown

@npotts npotts commented Nov 30, 2014

Removed the default to build zenmap. This causes the build problem as described in https://dev.openwrt.org/ticket/18436.

Signed-off-by: Nick Potts nicholas.potts@the-popular-google-email-system.com

Removed the default to build zenmap.  This causes the build problem as described in https://dev.openwrt.org/ticket/18436.
@hnyman
Copy link
Copy Markdown
Contributor

hnyman commented Nov 30, 2014

Two comments:

  • you should add the sign off line to the commit itself
  • you never mention the package that you are trying to fix, apparently "nmap", right?

@ffainelli
Copy link
Copy Markdown
Member

Right, we would want the subject to look like:

nmap: do not build zenmap by default

@npotts
Copy link
Copy Markdown
Author

npotts commented Dec 1, 2014

should I revert the commit and resubmit this patch?

@npotts npotts changed the title Removed the default to build zenmap nmap: do not build zenmap by default Dec 1, 2014
@sbyx
Copy link
Copy Markdown
Member

sbyx commented Dec 2, 2014

Just do a git rebase -i in your branch and change the commit title and force push the branch.

npotts and others added 2 commits December 2, 2014 18:31
@npotts npotts closed this Dec 3, 2014
@npotts npotts reopened this Dec 3, 2014
@npotts
Copy link
Copy Markdown
Author

npotts commented Dec 3, 2014

I think I did it via git commit --amend.

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.

6 participants