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

Update suricata.inc - fix suricata_Getdirsize tab char #1308

Merged
merged 2 commits into from Oct 16, 2023

Conversation

graham-collinson
Copy link
Contributor

Assuming this was intended to be the tab character. PHP 8 changed strpos so that Integer is no longer cast to a character by strpos.

Assuming this was intended to be the tab character.  PHP 8 changed strpos so that Integer is no longer cast to a character by strpos.
@graham-collinson
Copy link
Contributor Author

redmine issue https://redmine.pfsense.org/issues/14855

@bmeeks8
Copy link
Contributor

bmeeks8 commented Oct 9, 2023

I am good with this fix, but it needs an additional update to the port revision version in the Makefile so that the package will rebuild and show as an available update. Please increment the PORTREVISION parameter in the Makefile to be "2" (it is currently "1" in the DEVEL snapshot branch package). That will make the current Snapshot branch Suricata GUI package version be 7.0.0_2.

Thank you for the contribution!

@graham-collinson
Copy link
Contributor Author

@bmeeks8 Thanks for the pointer. That ok now?

@bmeeks8
Copy link
Contributor

bmeeks8 commented Oct 9, 2023

@bmeeks8 Thanks for the pointer. That ok now?

Perfect! Now we wait on the Netgate developer team to review and merge. I can drop them an email to hopefully speed it along. They are a bit preoccupied I think with the upcoming 23.09 release.

As the package maintainer, they usually ask me to weigh in with my thoughts on community pull request submissions, so that's why I proactively jumped in. But the Netgate team is the only one with permission to merge a pull request.

@netgate-git-updates netgate-git-updates merged commit 40f4192 into pfsense:devel Oct 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants