[RFA/doco] Document that the GNAT runtime should not be stripped

[RFA/doco] Document that the GNAT runtime should not be stripped

[Joel Brobecker]

Hello,

A feature we recently contributed to GDB allows the debugger to stop
the program execution when an exception is raised. This feature requires
that the GNAT runtime not be stripped of its debugging information.
This issue was brought to our attention by Daniel J, who used the Debian
version of GNAT where the GNAT runtime is stripped.

I therefore suggest the following addition to the install guide.

2007-01-05  Joel Brobecker  <brobecker@adacore.com>

        * doc/install.texi (Final install): Document the fact that
        the GNAT runtime should not be stripped.

Would that be OK to apply?

Thank you,
-- 
Joel

[RFA/doco] Document that the GNAT runtime should not be stripped

[Joel Brobecker]

[-- Attachment #1: Type: text/plain, Size: 675 bytes --]

[humpf, as usual, I forgot the patch - sorry]

Hello,

A feature we recently contributed to GDB allows the debugger to stop
the program execution when an exception is raised. This feature requires
that the GNAT runtime not be stripped of its debugging information.
This issue was brought to our attention by Daniel J, who used the Debian
version of GNAT where the GNAT runtime is stripped.

I therefore suggest the following addition to the install guide.

2007-01-05  Joel Brobecker  <brobecker@adacore.com>

        * doc/install.texi (Final install): Document the fact that
        the GNAT runtime should not be stripped.

Would that be OK to apply?

Thank you,
-- 
Joel

[-- Attachment #2: doc.diff --]
[-- Type: text/plain, Size: 673 bytes --]

Index: install.texi
===================================================================
--- install.texi	(revision 120469)
+++ install.texi	(working copy)
@@ -2062,7 +2062,10 @@
 @end smallexample
 
 We strongly recommend to install into a target directory where there is
-no previous version of GCC present.
+no previous version of GCC present.  Also, the GNAT runtime should not
+be stripped, as this would break certain features of the debugger that
+depend on this debugging information (catching Ada exceptions for
+instance).
 
 That step completes the installation of GCC; user level binaries can
 be found in @file{@var{prefix}/bin} where @var{prefix} is the value

Re: [RFA/doco] Document that the GNAT runtime should not be stripped

[Gerald Pfeifer]

[-- Attachment #1: Type: TEXT/PLAIN, Size: 386 bytes --]

On Fri, 5 Jan 2007, Joel Brobecker wrote:
> [humpf, as usual, I forgot the patch - sorry]

Yeah, I was wondering. :-)

> 2007-01-05  Joel Brobecker  <brobecker@adacore.com>
> 
>         * doc/install.texi (Final install): Document the fact that
>         the GNAT runtime should not be stripped.

Okay.  If you want to apply this to active release branches, that's
fine as well.

Gerald

[-- Attachment #2: Type: TEXT/PLAIN, Size: 673 bytes --]

Index: install.texi
===================================================================
--- install.texi	(revision 120469)
+++ install.texi	(working copy)
@@ -2062,7 +2062,10 @@
 @end smallexample
 
 We strongly recommend to install into a target directory where there is
-no previous version of GCC present.
+no previous version of GCC present.  Also, the GNAT runtime should not
+be stripped, as this would break certain features of the debugger that
+depend on this debugging information (catching Ada exceptions for
+instance).
 
 That step completes the installation of GCC; user level binaries can
 be found in @file{@var{prefix}/bin} where @var{prefix} is the value

Re: [RFA/doco] Document that the GNAT runtime should not be stripped

[Joel Brobecker]

> > 2007-01-05  Joel Brobecker  <brobecker@adacore.com>
> > 
> >         * doc/install.texi (Final install): Document the fact that
> >         the GNAT runtime should not be stripped.
> 
> Okay.  If you want to apply this to active release branches, that's
> fine as well.

Thank you! It's a good idea to apply this to the release branch too,
so I'm checking it out. I will commit there as soon as the checkout
is complete.

Thanks again,
-- 
Joel

Re: [RFA/doco] Document that the GNAT runtime should not be stripped

[Joel Brobecker]

> > Okay.  If you want to apply this to active release branches, that's
> > fine as well.
> 
> Thank you! It's a good idea to apply this to the release branch too,
> so I'm checking it out. I will commit there as soon as the checkout
> is complete.

Actually, the commit attempt failed. Maybe I used the wrong branch?
svn://gcc.gnu.org/svn/gcc/branches/gcc-4_1-branch/gcc

It's not terribly important that this patch makes it to the release
branch, however.

-- 
Joel

Re: [RFA/doco] Document that the GNAT runtime should not be stripped

[Ian Lance Taylor]

Joel Brobecker <brobecker@adacore.com> writes:

> > > Okay.  If you want to apply this to active release branches, that's
> > > fine as well.
> > 
> > Thank you! It's a good idea to apply this to the release branch too,
> > so I'm checking it out. I will commit there as soon as the checkout
> > is complete.
> 
> Actually, the commit attempt failed. Maybe I used the wrong branch?
> svn://gcc.gnu.org/svn/gcc/branches/gcc-4_1-branch/gcc

That's the right branch, but I think that in order to commit a change
you need to use svn+ssh:, not svn:.

Ian

Re: [RFA/doco] Document that the GNAT runtime should not be stripped

[Joel Brobecker]

> > Actually, the commit attempt failed. Maybe I used the wrong branch?
> > svn://gcc.gnu.org/svn/gcc/branches/gcc-4_1-branch/gcc
> 
> That's the right branch, but I think that in order to commit a change
> you need to use svn+ssh:, not svn:.

Duh! Of course. I was so focused on the branch name that I overlooked
this. Thanks!

Patch also checked in the branch.

Thank you,
-- 
Joel