[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: scratch/igc cedb01b6c9c: ; * README-IGC: Fix typos and wording.
From: |
Pip Cet |
Subject: |
Re: scratch/igc cedb01b6c9c: ; * README-IGC: Fix typos and wording. |
Date: |
Fri, 13 Dec 2024 12:42:03 +0000 |
"Eli Zaretskii" <eliz@gnu.org> writes:
> branch: scratch/igc
> commit cedb01b6c9c68051e00426415e21a960fdc3de76
> Author: Eli Zaretskii <eliz@gnu.org>
> Commit: Eli Zaretskii <eliz@gnu.org>
>
> ; * README-IGC: Fix typos and wording.
Thanks!
> -It's recommended to use the .gdbinit file in the src/ subdirectory,
> -which configures gdb not to stop in this situation.
> +part of its normal operation. When running Emacs with MPS support in a
> +debugger such as GDB, it is important to take this into account, because
> +the debugger might stop when this signal is raised, and make it look
> +like Emacs segfaulted.
Can we lose that last half-sentence? Emacs did segfault, it's just that
the segfault was expected, so this will only cause confusion.
> +If you use GDB, it's recommended to use the .gdbinit file in the src/
> +subdirectory, which configures GDB not to stop in this situation.
> +Starting GDB from the src directory will cause it automatically load and
> +read the file, subject to the "auto-load safe-path" setting;
> +alternatively, you can force GDB to read that file manually, after
> +starting the debugger:
> +
> + (gdb) source /path/to/emacs-igc/src/.gdbinit
Can we shorten this to just
> +If you use GDB, it's recommended to use the .gdbinit file in the src/
> +subdirectory:
> +
> + (gdb) source /path/to/emacs-igc/src/.gdbinit
? I think those few people who use gdb distributions which allow
auto-loading .gdbinit files will know they can skip this step (and if
they don't, no harm done). The text you pushed makes it sound like we're
advising people to change the auto-load safe-path setting to include the
current directory, and we definitely don't want to do that.
(I intentionally omitted the "If you use GDB" part: we don't want people
to use lldb on GNU/Linux systems, which will result in less useful bug
reports)
Pip
- Re: scratch/igc cedb01b6c9c: ; * README-IGC: Fix typos and wording.,
Pip Cet <=