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.

Sign up for GitHub

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

Jump to bottom

doc: update install instructions v9 #9976

Closed
wants to merge 1 commit into from
Closed

doc: update install instructions v9 #9976

wants to merge 1 commit into from

Conversation

lukashino
Copy link
Contributor

Cherry-pick of: 5a3ecbd

Follow-up of #9899

Changes:

  • rebase on the current master-6.0.x
  • cleared unneeded dependencies
  • PR comments

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

@lukashino lukashino self-assigned this Dec 5, 2023
@lukashino lukashino requested a review from norg as a code owner December 5, 2023 20:34
@lukashino lukashino mentioned this pull request Dec 5, 2023
Closed
@victorjulien
Copy link
Member

Same comment about the cherry-pick line. Please use the -x format.

@lukashino
Copy link
Contributor Author

sorry about that. Should it even be kept there as the commit has changed so much already?

@victorjulien
Copy link
Member

sorry about that. Should it even be kept there as the commit has changed so much already?

If it feels more like a new work then we can drop the reference I think, esp if it is based your own commit.

@lukashino lukashino mentioned this pull request Dec 6, 2023
Closed
@lukashino
Copy link
Contributor Author

Continues in #9981

@lukashino lukashino closed this Dec 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@norg norg Awaiting requested review from norg norg is a code owner

Assignees

@lukashino lukashino

Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@lukashino @victorjulien