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

backport: doc: update install instructions v8 #9899

Closed

Conversation

lukashino
Copy link
Contributor

Cherry-pick of: 5a3ecbd

Follow-up of #9734

Changes:

  • rebase on the current master-6.0.x
  • removed most of the Dependencies section and moved Rust installation guide past the general installation & compilation instructions

Link to redmine ticket

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.

haven't checked how much of my feedback applies to master too

doc/userguide/install.rst Show resolved Hide resolved
doc/userguide/install.rst Show resolved Hide resolved
cargo install --force cbindgen # can be run as a non-root
# Make sure the cargo path is within your PATH environment e.g.:
echo 'export PATH=”${PATH}:~/.cargo/bin”' >> ~/.bashrc
export PATH="${PATH}:/root/.cargo/bin"
Copy link
Member

Choose a reason for hiding this comment

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

same

doc/userguide/install.rst Show resolved Hide resolved
doc/userguide/install.rst Show resolved Hide resolved
doc/userguide/install.rst Show resolved Hide resolved
@jufajardini jufajardini added the typo/doc update No code change : only doc or typo fixes label Dec 1, 2023
@suricata-qa
Copy link

Information:

field baseline test %
TREX_GENERIC_stats_chk
.capture.kernel_drops 0 1 0.00

Pipeline 16865

@lukashino
Copy link
Contributor Author

haven't checked how much of my feedback applies to master too

master will need an update but I would suggest to first merge this and then redo the master again.

Continues in #9976

@lukashino lukashino closed this Dec 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
typo/doc update No code change : only doc or typo fixes
Development

Successfully merging this pull request may close these issues.

4 participants