Home Original page

[PATCH] readelf: Fix typo in --version-info documentation

Jan Beulich jbeulich@suse.com
Mon Nov 10 10:20:24 GMT 2025 
On 07.11.2025 16:53, Dennis Dyall via Binutils wrote:
> From ec86e6f2bfae3b5b8b80af58429477b5201c1398 Mon Sep 17 00:00:00 2001
> From: Dennis Dyallo <dennis.dyall@yubico.com>
> Date: Fri, 7 Nov 2025 16:46:17 +0100
> Subject: [PATCH] readelf: Fix typo in --version-info documentation
> 
> Change "it they exist" to "if they exist" in the description
> of the --version-info option in the readelf man page.

That change is certainly fine, but ...

> --- a/binutils/doc/binutils.texi
> +++ b/binutils/doc/binutils.texi
> @@ -598,7 +598,7 @@ The @option{--output} option can be used to specify a
> path to a
>  directory into which archive members should be extracted.  If this
>  option is not specified then the current directory will be used.
> 
> -Note - although the presence of this option does imply a @option{x}
> +Note - although the presence of this option does imply a @option{x}

... this and the other unrelated changes (likely intending to drop trailing
blanks) came through damaged, resulting in no change at all while at the
same time making the patch fail to apply. When committing for you, I'll
shrink the change to ...

> @@ -5303,7 +5303,7 @@ Displays the contents of the file's dynamic section,
> if it has one.
>  @item -V
>  @itemx --version-info
>  @cindex ELF version sections information
> -Displays the contents of the version sections in the file, it they
> +Displays the contents of the version sections in the file, if they
>  exist.
> 
>  @item -A

... just this hunk, while also taking care of the line wrapping issue
(which, like the whitespace one, likely is a result of a configuration
issue with your mailer).

Jan
More information about the Binutils mailing list