FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

[WIP][RFC] Add a mini-guide for debugging packages and programs, to help with better issue reports by ericonr · Pull Request #530 · void-linux/void-docs · GitHub

Repository navigation

[WIP][RFC] Add a mini-guide for debugging packages and programs, to help with better issue reports - #530

Open
ericonr wants to merge 3 commits into
void-linux:masterfrom
ericonr:debug
Open

ericonr wants to merge 3 commits into
void-linux:masterfrom
ericonr:debug

Conversation

ericonr commented Dec 2, 2020 •
edited
Loading

Copy link
Copy Markdown
Member

Some of this might fit better in void-packages, like Issue templates and such. I'm open to suggestions.

Just use a void-docs.md file directly.
Comment thread src/contributing/debug.md Outdated
- `backtrace`, which shows the function calls made until the application got to
that place;
- `info threads`, which shows information about threads in the program;
- `break <function>`, which puts a breakpoint in a function that you may think

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

I don't think any of this is nessecary.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Perhaps keep only the backtrace part? Because that's what we want to get if someone says "this segfaults when I launch it".

Comment thread src/contributing/debug.md Outdated
Comment on lines +40 to +41
for example, to access files that don't exist, and their programmer didn't
implement a proper fallback or error message.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Could be shortened.

Comment thread src/contributing/debug.md Outdated
Comment on lines +49 to +51
GDB is especially useful when an application crashes with `SIGABRT` or `SIGSEGV`
(see [signal(7)](https://man.voidlinux.org/signal.7), since it will stop
execution at that point and you can print a backtrace showing all the function

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Don't think this is needed either

Comment thread src/contributing/debug.md Outdated
Comment on lines +33 to +34
After this is done, check if the issue persists. It should be noted that issues
like this can indicate hardware issues in your storage media or RAM.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

It might, but it also might not. I feel like this would lead to a lot of people going OH NO MY RAM IS DEAD when they actually powered off the computer in the middle of an upgrade or smth.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Good point.

Comment thread src/contributing/debug.md Outdated
Comment thread src/contributing/debug.md Outdated

## Look at error messages attentively THIS NEEDS WORK

Python programs, for example, complain loudly about missing modules and other

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Maybe a list of different types of programs? I don't like this format.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

I'm making a second version.

Comment thread src/contributing/debug.md
Comment on lines +55 to +59
```
$ DEBUGINFOD_URLS="https://debugingod.s.voidlinux.org" gdb --args <program> [arguments]
```

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality
  1. This is not an example, its more of a 'syntax' i guess
  2. I would note the debuginfod server env variable to make it work separately.

Comment thread src/contributing/index.md Outdated
Comment on lines +5 to +7
If you find an issue with a package and you manage to fix it, you can add your
solution to the official package repository. New packages and updates are also
welcome! To contribute to the Void packages repository, start by reading the

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Wordy

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Indeed, but I'm not sure I improved it.

Comment thread src/contributing/index.md Outdated
Comment on lines +13 to +15
void-packages repository. This makes maintainers aware of the issue, allows
others to fix it, and can work as a place for anyone to share workarounds in the
meantime. Remember to check the existing issues for your problem! There are also

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

This sentence could be shortened to something like 'This gives the problem visibility to expedite fixing' or something like that

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

I moved it around a bit, not 100% sure yet.

Comment thread src/contributing/index.md Outdated
void-packages repository. This makes maintainers aware of the issue, allows
others to fix it, and can work as a place for anyone to share workarounds in the
meantime. Remember to check the existing issues for your problem! There are also
some [things you can do to improve your bug report](./debug.md).

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

improve your bug report could be changed to something along the lines of assist us in finding the probelm

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Done.

ericonr commented Dec 7, 2020

Copy link
Copy Markdown
Member Author

@fosslinux thanks for the review! I have updated it.

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants


Back | FazBrowse Home | New Git URL