[Patch] updated configure.in required to enable insight to build on OSX

[Patch] updated configure.in required to enable insight to build on OSX

[Paul Schlie]

Might the following patch please be considered for inclusion into CVS
as soon as possible to enable Insight to build for OSX, Thank you:

--- old/src/configure.in       Fri Jan 28 23:24:22 2005
+++ new/src/configure.in       Sun Feb 20 14:49:03 2005
@@ -309,7 +309,7 @@ case "${host}" in
     noconfigdirs="$noconfigdirs tk itcl libgui gdb dejagnu readline"
     ;;
   *-*-darwin*)
-    noconfigdirs="$noconfigdirs tk itcl libgui"
+    noconfigdirs="$noconfigdirs"
     ;;
 esac

(and seems to execute correctly when invoked within the X11 terminal shell)

Thanks, -paul-

> From: Keith Seitz <keiths@redhat.com>
> Date: Mon, 21 Feb 2005 13:35:29 -0800
> To: Paul Schlie <schlie@comcast.net>
> Cc: <insight@sources.redhat.com>
> Subject: Re: Updated configure.in required to enable insight to build on OSX
>
> Hey, if it works, I don't have a problem. However, according to
> src/MAINTAINERS:
> 
> Makefile.*; configure; configure.in; src-release
>         Please notify the following of any committed patches.
>                 binutils@sources.redhat.com
>                 gdb-patches@sources.redhat.com
> 
> So you would need to send a patch to the binutils and gdb-patches mailing
> lists.
> 
> Keith
> 

Re: [Patch] updated configure.in required to enable insight to build on OSX

[Nick Clifton]

Hi Paul,

>    *-*-darwin*)
> -    noconfigdirs="$noconfigdirs tk itcl libgui"
> +    noconfigdirs="$noconfigdirs"

There is no point in just copying noconfigdirs onto itself so I have 
checked in a variation of your patch with the entry for *-*-darwin* 
entirely removed.

Cheers
   Nick

<toplevel>/ChangeLog
2005-02-22  Paul Schlie  <schlie@comcast.net>

	* configure.in: Allow darwin targeted ports to build tk, itcl and
	libgui.

Re: [Patch] updated configure.in required to enable insight to build on OSX

[Paul Schlie]

Understood, Thank you, -paul-


> From: Nick Clifton <nickc@redhat.com>
> 
> Hi Paul,
> 
>>    *-*-darwin*)
>> -    noconfigdirs="$noconfigdirs tk itcl libgui"
>> +    noconfigdirs="$noconfigdirs"
> 
> There is no point in just copying noconfigdirs onto itself so I have
> checked in a variation of your patch with the entry for *-*-darwin*
> entirely removed.
> 
> Cheers
>    Nick
> 
> <toplevel>/ChangeLog
> 2005-02-22  Paul Schlie  <schlie@comcast.net>
> 
> * configure.in: Allow darwin targeted ports to build tk, itcl and
> libgui.

Re: [Patch] updated configure.in required to enable insight to build on OSX

[Paul Schlie]

Actually come to think of it, might there be any quick convenient way
to disable sim, sid, rda, and libgloss from trying to build without
being explicitly enabled (or even prevented if explicitly disabled)?
As I've had to basically manually re-name or remove those directories,
to prevent them from building when not desired?


> From: Paul Schlie <schlie@comcast.net>
> 
> Understood, Thank you, -paul-
> 
> 
>> From: Nick Clifton <nickc@redhat.com>
>> 
>> Hi Paul,
>> 
>>>    *-*-darwin*)
>>> -    noconfigdirs="$noconfigdirs tk itcl libgui"
>>> +    noconfigdirs="$noconfigdirs"
>> 
>> There is no point in just copying noconfigdirs onto itself so I have
>> checked in a variation of your patch with the entry for *-*-darwin*
>> entirely removed.
>> 
>> Cheers
>>    Nick
>> 
>> <toplevel>/ChangeLog
>> 2005-02-22  Paul Schlie  <schlie@comcast.net>
>> 
>> * configure.in: Allow darwin targeted ports to build tk, itcl and
>> libgui.
> 

Re: [Patch] updated configure.in required to enable insight to build on OSX

[Andrew Cagney]

Paul Schlie wrote:
> Actually come to think of it, might there be any quick convenient way
> to disable sim, sid, rda, and libgloss from trying to build without
> being explicitly enabled (or even prevented if explicitly disabled)?
> As I've had to basically manually re-name or remove those directories,
> to prevent them from building when not desired?

Did you use "cvs update -d" in the top level directory?  Neither the 
BINUTILS, nor GDB modules check any of those directories out.

Andrew

Re: [Patch] updated configure.in required to enable insight to build on OSX

[Paul Schlie]

Actually yes, typically simply co -d src ... and grab everything,
then typically merge gcc into it (which I know isn't blessed, but
have generally found it more reliable for the things that should
likely be in common, to literally be in common; so would be nice
to have a top-level configure maintained, it would seem, as I don't
think I'm the only one?

[personally seems a little archaic for gcc to still remain disassociated
 from the src cvs tree, and end up having to manually check into both?]


> From: Andrew Cagney <cagney@gnu.org>
> Date: Tue, 22 Feb 2005 11:09:59 -0500
> To: Paul Schlie <schlie@comcast.net>
> Cc: Nick Clifton <nickc@redhat.com>, <binutils@sources.redhat.com>,
> <gdb-patches@sources.redhat.com>, <insight@sources.redhat.com>, Keith Seitz
> <keiths@redhat.com>
> Subject: Re: [Patch] updated configure.in required to enable insight to build
> on OSX
> 
> Paul Schlie wrote:
>> Actually come to think of it, might there be any quick convenient way
>> to disable sim, sid, rda, and libgloss from trying to build without
>> being explicitly enabled (or even prevented if explicitly disabled)?
>> As I've had to basically manually re-name or remove those directories,
>> to prevent them from building when not desired?
> 
> Did you use "cvs update -d" in the top level directory?  Neither the
> BINUTILS, nor GDB modules check any of those directories out.
> 
> Andrew