Skip to content

suricata-update: Initial commit#15367

Closed
Grommish wants to merge 1 commit into
openwrt:masterfrom
Itus-Shield:suricata-update
Closed

suricata-update: Initial commit#15367
Grommish wants to merge 1 commit into
openwrt:masterfrom
Itus-Shield:suricata-update

Conversation

@Grommish
Copy link
Copy Markdown
Contributor

@Grommish Grommish commented Apr 8, 2021

Provide the framework for Suricata-update

Signed-off-by: Donald Hoskins grommish@gmail.com

Maintainer: me
Compile tested: (put here arch, model, OpenWrt version)
Run tested: (put here arch, model, OpenWrt version, tests done)

Description:

Suricata-update framework, uci config and init.d service entry.

Requires Suricata6

@@ -0,0 +1,4 @@
#!/bin/sh /etc/rc.common
/usr/bin/python3 -m pip install --upgrade pip
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

uhhhh what?

Copy link
Copy Markdown
Contributor Author

@Grommish Grommish Apr 8, 2021

Choose a reason for hiding this comment

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

It ensures pip3 is up to date, since it's currently outdated in master. Not a great idea?

The other obvious option is to update the pip package, but to be honest, the python package scares me

@Grommish
Copy link
Copy Markdown
Contributor Author

Grommish commented Apr 8, 2021 via email

@neheb
Copy link
Copy Markdown
Contributor

neheb commented Apr 8, 2021

ping @jefferyto

@jefferyto
Copy link
Copy Markdown
Member

There are lots of Python program packages in the repo, e.g. youtube-dl, docker-compose, etc. If you search for "python3-package.mk" in the repo you will find many other examples. I suggest you take one of these makefiles as a template.

The Python packages folder readme also has a section on building a Python package which explains most (if not all) of the various Python package variables.

If you run into any problems writing a Python package makefile, there are lots of folks here who can help, but I'm afraid your install_suricata_update is definitely not the way to do this.

@Grommish
Copy link
Copy Markdown
Contributor Author

There are lots of Python program packages in the repo, e.g. youtube-dl, docker-compose, etc. If you search for "python3-package.mk" in the repo you will find many other examples. I suggest you take one of these makefiles as a template.

The Python packages folder readme also has a section on building a Python package which explains most (if not all) of the various Python package variables.

If you run into any problems writing a Python package makefile, there are lots of folks here who can help, but I'm afraid your install_suricata_update is definitely not the way to do this.

I absolutely agree. What I have been playing with is simply a file check on the suricata-update command as part of the init.d service and calling pip to install it if it isn't there. This allows me to include the service and have it self-installing without a scripted mess. I've been busy with other things, but yes, this was totally a cluster 👍

I'll get a push up eventually and see if that isn't more acceptable. Until I get rust in, it's pretty much a moot point though. It's still all a WIP.

Provide the framework for Suricata-update

Signed-off-by: Donald Hoskins <grommish@gmail.com>
@BKPepe
Copy link
Copy Markdown
Member

BKPepe commented Jan 4, 2024

Closing, OP is inactive.

@BKPepe BKPepe closed this Jan 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants