All of lore.kernel.org
 help / color / mirror / Atom feed
From: Glenn Washburn <development@efficientek.com>
To: grub-devel@gnu.org, Daniel Kiper <dkiper@net-space.pl>
Cc: Oskari Pirhonen <xxc3ncoredxx@gmail.com>,
	Glenn Washburn <development@efficientek.com>
Subject: [PATCH v4] docs: Add missing assumption
Date: Tue, 11 Jul 2023 12:07:29 -0500	[thread overview]
Message-ID: <20230711170729.1497818-1-development@efficientek.com> (raw)

Also reword a prior sentence to be more clear.

Fixes: 5a3d2b4742df (docs: Add debugging chapter to development documentation)
Signed-off-by: Glenn Washburn <development@efficientek.com>
---
Range-diff against v3:
1:  437c408b31cb ! 1:  c228e0412720 docs: Add missing assumption
    @@ docs/grub-dev.texi: GDB is started from the @file{grub-core} directory in the GR
     -If not building GRUB, the distribution may have a package which installs
     +When using a pre-built GRUB, the distribution may have a package which installs
      this GDB script along with debug symbol binaries, such as Debian's
    --@samp{grub-pc-dbg} package. The GDB script is intended to be used
    + @samp{grub-pc-dbg} package. The GDB script is intended to be used
     -like so, assuming:
    -+@samp{grub-pc-dbg} package. The GDB script is intended to by used
     +like so, assuming that @samp{/path/to/script} is the path to the directory
     +containing the gdb_grub script and debug symbol files:
      

 docs/grub-dev.texi | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/docs/grub-dev.texi b/docs/grub-dev.texi
index 0834ca562ad6..a695b02f06df 100644
--- a/docs/grub-dev.texi
+++ b/docs/grub-dev.texi
@@ -660,10 +660,11 @@ GDB is started from the @file{grub-core} directory in the GRUB2 build
 directory. GRUB2 developers have made this more simple by including a GDB
 script which does much of the setup. This file is at @file{grub-core/gdb_grub}
 in the build directory and is also installed via @command{make install}.
-If not building GRUB, the distribution may have a package which installs
+When using a pre-built GRUB, the distribution may have a package which installs
 this GDB script along with debug symbol binaries, such as Debian's
 @samp{grub-pc-dbg} package. The GDB script is intended to be used
-like so, assuming:
+like so, assuming that @samp{/path/to/script} is the path to the directory
+containing the gdb_grub script and debug symbol files:
 
 @example
 cd $(dirname /path/to/script/gdb_grub)
-- 
2.34.1



             reply	other threads:[~2023-07-11 17:08 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-07-11 17:07 Glenn Washburn [this message]
2023-07-12  0:08 ` [PATCH v4] docs: Add missing assumption Oskari Pirhonen
2023-07-13 18:15   ` Daniel Kiper

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20230711170729.1497818-1-development@efficientek.com \
    --to=development@efficientek.com \
    --cc=dkiper@net-space.pl \
    --cc=grub-devel@gnu.org \
    --cc=xxc3ncoredxx@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.