This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]

Re: PATCH for new flag `--enable-libs-in-libsubdir'


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


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]