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

ntopng disable-alerts patch #519

Merged
merged 4 commits into from
Jul 31, 2018
Merged

ntopng disable-alerts patch #519

merged 4 commits into from
Jul 31, 2018

Conversation

homer2320776
Copy link
Contributor

The disable-alerts flag of -H was removed and causing issues with commands further down the command line.

Alerts can be disabled via the preferences section of the ntopng GUI.

ntop/ntopng#1490
Alerts can be disabled via ntopng GUI
ntop/ntopng#1490
Added info about disabling alerts via GUI
Copy link
Contributor

@jim-p jim-p left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Rather than comment out the code, just remove it.

The GUI bit can stay for now so people know where to look.

@homer2320776
Copy link
Contributor Author

Removed code from ntopng.inc

@homer2320776
Copy link
Contributor Author

Sorry about the open and close noise, still new to this.

Copy link
Member

@rbgarga rbgarga left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please bump PORTVERSION or PORTREVISION on Makefile to trigger a new package build

@homer2320776
Copy link
Contributor Author

Hopefully I did that correctly.

@rbgarga rbgarga requested a review from jim-p July 17, 2018 18:09
@netgate-git-updates netgate-git-updates merged commit fc923d0 into pfsense:devel Jul 31, 2018
@homer2320776 homer2320776 deleted the patch-1 branch July 31, 2018 13:08
netgate-git-updates pushed a commit that referenced this pull request Feb 15, 2021
These two go together, so they update together. Upstream has merged
all the build-fixes from the previous ports version, and the one
new fix is already reported and merged upstream thanks to OpenBSD.

Release notes are at
	https://github.com/ksnip/ksnip/releases/tag/v1.8.1
with notable changes
	- Allow changing adorner color for rect area selection. (#519)
	- Default font for numbering tool change to Arial. (#200)
	- kImageAnnotator: Horizontally align text inside spin box. (#203)
	- kImageAnnotator: Change zoom with mouse wheel to CTRL+Wheel. (#210)

Reported by:	portscout
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants