Skip to content

docs: improve debuginfo install description (fix issue #1943)#1980

Merged
BareosBot merged 2 commits intobareos:masterfrom
bruno-at-bareos:dev/bruno/master/issue1943-docs-install-debuginfo
Oct 22, 2024
Merged

docs: improve debuginfo install description (fix issue #1943)#1980
BareosBot merged 2 commits intobareos:masterfrom
bruno-at-bareos:dev/bruno/master/issue1943-docs-install-debuginfo

Conversation

@bruno-at-bareos
Copy link
Contributor

@bruno-at-bareos bruno-at-bareos commented Oct 14, 2024

Please check

  • Short description and the purpose of this PR is present above this paragraph
  • Your name is present in the AUTHORS file (optional)

If you have any questions or problems, please give a comment in the PR.

Helpful documentation and best practices

Checklist for the reviewer of the PR (will be processed by the Bareos team)

Make sure you check/merge the PR using devtools/pr-tool to have some simple automated checks run and a proper changelog record added.

General
  • Is the PR title usable as CHANGELOG entry?
  • Purpose of the PR is understood
  • Commit descriptions are understandable and well formatted
  • Required backport PRs have been created
  • Correct milestone is set

@bruno-at-bareos bruno-at-bareos linked an issue Oct 14, 2024 that may be closed by this pull request
@bruno-at-bareos bruno-at-bareos linked an issue Oct 15, 2024 that may be closed by this pull request
@bruno-at-bareos bruno-at-bareos force-pushed the dev/bruno/master/issue1943-docs-install-debuginfo branch from 59f8561 to 44d4b99 Compare October 15, 2024 14:23
@sebsura sebsura self-requested a review October 17, 2024 09:30
@bruno-at-bareos bruno-at-bareos force-pushed the dev/bruno/master/issue1943-docs-install-debuginfo branch from 44d4b99 to b0cf018 Compare October 17, 2024 09:59
@bruno-at-bareos bruno-at-bareos added this to the 24.0.0 milestone Oct 17, 2024
Copy link
Contributor

@sebsura sebsura left a comment

Choose a reason for hiding this comment

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

The changes are great. I added some comments to the documentation that you added.

@bruno-at-bareos bruno-at-bareos force-pushed the dev/bruno/master/issue1943-docs-install-debuginfo branch from 778867e to 33d5e22 Compare October 17, 2024 12:59
@bruno-at-bareos
Copy link
Contributor Author

@sebsura as discussed, I've redone the section, by putting in first place how to install all, then only describe specific daemon with fd as an example.
To make the commits more organized, I've done a rebase, sorry if that force you for a full review.

@bruno-at-bareos bruno-at-bareos force-pushed the dev/bruno/master/issue1943-docs-install-debuginfo branch from 33d5e22 to 2e84ad7 Compare October 21, 2024 12:35
@bruno-at-bareos bruno-at-bareos force-pushed the dev/bruno/master/issue1943-docs-install-debuginfo branch from 2e84ad7 to 8db7af3 Compare October 21, 2024 14:21
Copy link
Contributor

@sebsura sebsura left a comment

Choose a reason for hiding this comment

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

Great changes. Thanks for your work!

@bruno-at-bareos
Copy link
Contributor Author

Backport created now last build for merge

Fix issue bareos#1943

Explain first how to install all, then add a note how-to do so for
only Bareos Filedaemon.
Additionnaly explain how to allow ptrace operations.
Cleanup trailing new line

Signed-off-by: Bruno Friedmann <bruno.friedmann@bareos.com>
Co-authored-by: Sebastian Sura <124262655+sebsura@users.noreply.github.com>
@bruno-at-bareos bruno-at-bareos force-pushed the dev/bruno/master/issue1943-docs-install-debuginfo branch from 8db7af3 to 12fcc59 Compare October 22, 2024 08:09
@BareosBot BareosBot merged commit f84f1d0 into bareos:master Oct 22, 2024
@bruno-at-bareos bruno-at-bareos deleted the dev/bruno/master/issue1943-docs-install-debuginfo branch November 25, 2024 10:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

docs: update debuginfo-install documentation

3 participants