Skip to content

Conversation

@Chriss-ic
Copy link

Make sure these boxes are checked accordingly before submitting your Pull Request -- thank you.

Contribution style:

Our Contribution agreements:

Changes (if applicable):

Link to ticket: https://redmine.openinfosecfoundation.org/issues/

Describe changes:
-Added a 'Building from Source on Debian' section to debian.rst with categorized configure options and Debian-specific dependencies, addressing Redmine #6824.
-Formatted in reStructuredText for Sphinx compatibility

Provide values to any of the below to override the defaults.

  • To use a Suricata-Verify or Suricata-Update pull request,
    link to the pull request in the respective _BRANCH variable.
  • Leave unused overrides blank or remove.

SV_REPO=
SV_BRANCH=
SU_REPO=
SU_BRANCH=

@victorjulien
Copy link
Member

Git: please put the ticket closed on a line of it's own after the subject line and a newline, like so

doc: Add configure options for Debian source build

Ticket: #6824.

@github-actions
Copy link

NOTE: This PR may contain new authors.

@victorjulien
Copy link
Member

CI check shows a formatting error

Copy link
Member

@victorjulien victorjulien left a comment

Choose a reason for hiding this comment

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

Thanks for looking into this!

See inline comments.

Features
~~~~~~~~
- ``--enable-geoip``: GeoIP support (requires libmaxminddb-dev).
- ``--enable-lua``: Lua scripting (requires lua5.3-dev).
Copy link
Member

Choose a reason for hiding this comment

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

We don't use the system lua anymore, we use the one we bundle with the rust crate.


If you prefer to build Suricata from source on Debian, the `./configure` script allows customization. Run `./configure --help` for the full list. Key options include:

Installation Options
Copy link
Member

Choose a reason for hiding this comment

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

these seem a bit generic, can we put them in a generic "install from source" section and then link to that from here?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants