Re: Problems in objcopy.1, objdump.1, x86_64-linux-gnu-objcopy.1, x86_64-linux-gnu-objdump.1

2018-06-13 Thread Nick Clifton
Hi Eric,

> Problems with objcopy.1:

> -[\fB\-\-add\-symbol\fR 
> \fIname\fR=[\fIsection\fR:]\fIvalue\fR[,\fIflags\fR]
> +[\fB\-\-add\-symbol\fR 
> \fIname\fR=[\fIsection\fR:]\fIvalue\fR[,\fIflags\fR]]

This one has already been fixed in the development sources:

2018-02-20  Ronald Hoogenboom  

* doc/binutils.texi (objcopy): Add missing closing square
parenthesis to listing of objcopy's command line options.


> Problems with objdump.1:

> - 
> \fB\-\-dwarf\fR[=rawline,=decodedline,=info,=abbrev,=pubnames,=aranges,=macro,=frames,=frames\-interp,=str,=loc,=Ranges,=pubtypes,=trace_info,=trace_abbrev,=trace_aranges,=gdb_index,=addr,=cu_index,=links,=follow\-links]
> + 
> \fB\-\-dwarf\fR[=rawline,=decodedline,=info,=abbrev,=pubnames,=aranges,=macro,=frames,=frames\-interp,=str,=loc,=Ranges,=pubtypes,=trace_info,=trace_abbrev,=trace_aranges,=gdb_index,=addr,=cu_index,=links,=follow\-links]]

This one has now been fixed.

Cheers
  Nick

binutils/ChangeLog
2018-06-13  Nick Clifton  

* doc/binutils.texi (objdump): Add missing closing square
parenthesis to listing of objcopy's command line options.

diff --git a/binutils/doc/binutils.texi b/binutils/doc/binutils.texi
index 863ef31e36..b87b1341fc 100644
--- a/binutils/doc/binutils.texi
+++ b/binutils/doc/binutils.texi
@@ -2077,7 +2077,7 @@ objdump [@option{-a}|@option{--archive-headers}]
 [@option{-R}|@option{--dynamic-reloc}]
 [@option{-s}|@option{--full-contents}]
 [@option{-W[lLiaprmfFsoRtUuTgAckK]}|
- 
@option{--dwarf}[=rawline,=decodedline,=info,=abbrev,=pubnames,=aranges,=macro,=frames,=frames-interp,=str,=loc,=Ranges,
=pubtypes,=trace_info,=trace_abbrev,=trace_aranges,=gdb_index,=addr,=cu_index,=links,=follow-links]
+ 
@option{--dwarf}[=rawline,=decodedline,=info,=abbrev,=pubnames,=aranges,=macro,=frames,=frames-interp,=str,=loc,=Ranges,
=pubtypes,=trace_info,=trace_abbrev,=trace_aranges,=gdb_index,=addr,=cu_index,=links,=follow-links]]
 [@option{-G}|@option{--stabs}]
 [@option{-t}|@option{--syms}]
 [@option{-T}|@option{--dynamic-syms}]

___
bug-binutils mailing list
bug-binutils@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-binutils


Problems in objcopy.1, objdump.1, x86_64-linux-gnu-objcopy.1, x86_64-linux-gnu-objdump.1

2018-06-12 Thread esr
This is automatically generated email about markup problems in a man
page for which you appear to be responsible.  If you are not the right
person or list, please tell me so I can correct my database.

See http://catb.org/~esr/doclifter/bugs.html for details on how and
why these patches were generated.  Feel free to email me with any
questions.  Note: These patches do not change the modification date of
any manual page.  You may wish to do that by hand.

I apologize if this message seems spammy or impersonal. The volume of
markup bugs I am tracking is over five hundred - there is no real
alternative to generating bugmail from a database and template.

--
 Eric S. Raymond
Problems with objcopy.1:

(Identical patches should apply to: x86_64-linux-gnu-objcopy.1)

Unbalanced group in command synopsis. You probably forgot
to open or close a [ ] or { } group properly.

--- objcopy.1-unpatched 2018-05-21 09:14:50.980994990 -0400
+++ objcopy.1   2018-05-21 09:21:47.646163311 -0400
@@ -194,7 +194,7 @@
 [\fB\-\-localize\-symbols=\fR\fIfilename\fR]
 [\fB\-\-globalize\-symbols=\fR\fIfilename\fR]
 [\fB\-\-weaken\-symbols=\fR\fIfilename\fR]
-[\fB\-\-add\-symbol\fR 
\fIname\fR=[\fIsection\fR:]\fIvalue\fR[,\fIflags\fR]
+[\fB\-\-add\-symbol\fR 
\fIname\fR=[\fIsection\fR:]\fIvalue\fR[,\fIflags\fR]]
 [\fB\-\-alt\-machine\-code=\fR\fIindex\fR]
 [\fB\-\-prefix\-symbols=\fR\fIstring\fR]
 [\fB\-\-prefix\-sections=\fR\fIstring\fR]
Problems with objdump.1:

(Identical patches should apply to: x86_64-linux-gnu-objdump.1)

Unbalanced group in command synopsis. You probably forgot
to open or close a [ ] or { } group properly.

--- objdump.1-unpatched 2018-05-21 09:24:20.621123684 -0400
+++ objdump.1   2018-05-21 09:25:15.140753167 -0400
@@ -163,7 +163,7 @@
 [\fB\-R\fR|\fB\-\-dynamic\-reloc\fR]
 [\fB\-s\fR|\fB\-\-full\-contents\fR]
 [\fB\-W[lLiaprmfFsoRtUuTgAckK]\fR|
- 
\fB\-\-dwarf\fR[=rawline,=decodedline,=info,=abbrev,=pubnames,=aranges,=macro,=frames,=frames\-interp,=str,=loc,=Ranges,=pubtypes,=trace_info,=trace_abbrev,=trace_aranges,=gdb_index,=addr,=cu_index,=links,=follow\-links]
+ 
\fB\-\-dwarf\fR[=rawline,=decodedline,=info,=abbrev,=pubnames,=aranges,=macro,=frames,=frames\-interp,=str,=loc,=Ranges,=pubtypes,=trace_info,=trace_abbrev,=trace_aranges,=gdb_index,=addr,=cu_index,=links,=follow\-links]]
 [\fB\-G\fR|\fB\-\-stabs\fR]
 [\fB\-t\fR|\fB\-\-syms\fR]
 [\fB\-T\fR|\fB\-\-dynamic\-syms\fR]
___
bug-binutils mailing list
bug-binutils@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-binutils


Re: Problems in as.1

2016-03-19 Thread Nick Clifton
Hi Eric,

> Problems with as.1:
> 
> Your Synopsis is exceptionally creative.  Unfortunately, that means
> it cannot be translated to structural markup even when things like
> running-text inclusions have been moved elswhere.
> 
> This page was generated from some sort of non-man markup.  Please
> fix the upstream markup so that it generates a well-formed
> manual page with the indicated corrections.
> 
> --- as.1-unpatched2014-06-01 20:05:07.053475183 -0400
> +++ as.1  2014-06-01 20:05:06.269457562 -0400
> @@ -158,6 +158,7 @@
>[\fB\-\-target\-help\fR] [\fItarget-options\fR]
>[\fB\-\-\fR|\fIfiles\fR ...]
>   .PP
> +.SH TARGET-SPECIFIC OPTIONS
>   \&\fITarget AArch64 options:\fR
>  [\fB\-EB\fR|\fB\-EL\fR]
>  [\fB\-mabi\fR=\fI\s-1ABI\s0\fR]


I found that I was unable to call the new section "TARGET-SPECIFIC OPTIONS", 
so instead I have opted for "TARGET".  This is probably due to my non-existent 
Perl programming skills, but anyway.  So I have checked in the attached patch 
in order to fix the problem.

Cheers
  Nick

gas/ChangeLog
2016-03-18  Nick Clifton  <ni...@redhat.com>

* doc/as.texinfo: Place the target specific command line options
into their own man page section.

etc/ChangeLog
2016-03-18  Nick Clifton  <ni...@redhat.com>

* texi2pod.pl: Add TARGET to the list of recognised man page
sections.
diff --git a/etc/texi2pod.pl b/etc/texi2pod.pl
index cd0ffd9..dd3e181 100644
--- a/etc/texi2pod.pl
+++ b/etc/texi2pod.pl
@@ -340,7 +340,7 @@ die "No filename or title\n" unless defined $fn && defined $tl;
 $sects{NAME} = "$fn \- $tl\n";
 $sects{FOOTNOTES} .= "=back\n" if exists $sects{FOOTNOTES};
 
-for $sect (qw(NAME SYNOPSIS DESCRIPTION OPTIONS ENVIRONMENT FILES
+for $sect (qw(NAME SYNOPSIS TARGET DESCRIPTION OPTIONS ENVIRONMENT FILES
 	  BUGS NOTES FOOTNOTES SEEALSO AUTHOR COPYRIGHT)) {
 if(exists $sects{$sect}) {
 	$head = $sect;
diff --git a/gas/doc/as.texinfo b/gas/doc/as.texinfo
index a0584d3..7b36931 100644
--- a/gas/doc/as.texinfo
+++ b/gas/doc/as.texinfo
@@ -246,8 +246,10 @@ gcc(1), ld(1), and the Info entries for @file{binutils} and @file{ld}.
  [@b{--target-help}] [@var{target-options}]
  [@b{--}|@var{files} @dots{}]
 @c
+@c man end
 @c Target dependent options are listed below.  Keep the list sorted.
 @c Add an empty line for separation.
+@c man begin TARGET
 @ifset AARCH64
 
 @emph{Target AArch64 options:}
___
bug-binutils mailing list
bug-binutils@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-binutils