This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: PATCH for new flag `--enable-libs-in-libsubdir'
- To: law at cygnus dot com
- Subject: Re: PATCH for new flag `--enable-libs-in-libsubdir'
- From: Manfred Hollstein <manfred at s-direktnet dot de>
- Date: Wed, 24 Jun 1998 09:32:12 +0200 (MET DST)
- Cc: egcs-patches at cygnus dot com
- References: <13707.58705.347842.433778@saturn.hollstein.net> <21958.898671955@hurl.cygnus.com>
- Reply-To: manfred at s-direktnet dot de, Manfred dot Hollstein at ks dot sel dot alcatel dot de
On Wed, 24 June 1998, 01:05:55, law@hurl.cygnus.com wrote:
>
> In message <13707.58705.347842.433778@saturn.hollstein.net>you write:
> > > Anyway, right now I'm tempted to recommend against this patch, though
> > > maybe we could have some configure option to enable this behavior?
> > > (or document better how to make existing configure options do the
> > > same thing).
> >
> > OK, this patch adds the new configure option `--enable-libs-in-libsubdir'
> > (do you know a better name?), which is disabled by default
> > (== current behaviour). Enabling it has the following effects:
> Maybe --enable-version-specific-libstdc++? Since what it's doing is
> allowing us to keep a set of libstdc++ includes & libraries for each
> specific version of the compiler. I'm not particularly fond of either,
> so use whichever one you like.
I like your suggestion, too. BUT, currently the configure scripts
would fail using such a flag, because they don't replace '+' with
e.g. 'x'. I could fix that, too, if you'd like me to do so:
i.e.:
enableopt=`echo ${option} | sed 's:^--::;s:=.*$::;s:-:_:g'`
should be changed into
enableopt=`echo ${option} | sed 's:^--::;s:=.*$::;s:-:_:g;s:+:x:g'`
This will then use an internal shell variable named
'$enable_version_specific_libstdcxx' instead of
'$enable_version_specific_libstdc++'; the same would be necessary for
the --disable, --with and --without options. Do you think, it should
be fixed, or shall we simply not allow option names with '+' in it?
>
> Could I suggest you also submit a change to the configuration instructions?
>
> (wwwdocs/htdocs/install/configure.html).
>
> Since this is a new option it should note that it's only available for
> egcs-1.1 and later releases.
Yep, I'll add a paragraph for this, too.
>
> Otherwise it seems quite reasonable and should be installed.
>
> Thanks!
>
> jeff
manfred