Re: C++ conversion status update

On 04/13/2016 03:16 PM, Joel Brobecker wrote:
>> OK.  We can just add an entry to lynxos to GDB_AC_BUILD_WITH_CXX
>> to make it default to C, still.  Like this, against
>> the version in the users/palves/cxx-conversion branch:
>>
>> diff --git i/gdb/build-with-cxx.m4 w/gdb/build-with-cxx.m4
>> index aa3661f..e42719e 100644
>> --- i/gdb/build-with-cxx.m4
>> +++ w/gdb/build-with-cxx.m4
>> @@ -30,6 +30,7 @@ AC_DEFUN([GDB_AC_BUILD_WITH_CXX],
>>      *-*nto* | \
>>      *-*bsd* | \
>>      xtensa*-*-linux* | \
>> +    *-*-lynxos* | \
>>      null)
>>        enable_build_with_cxx=no ;;
>>      *)
>>
>> That'll give you until all the other hosts are confirmed-converted,
>> at least.
> 
> We could do that indeed; or, IIRC, there is a configure option
> to force the use of C instead of C++.

It's --enable-build-with-cxx=yes|no.  That hunk above decides
the default for when the --enable-build-with-cxx option is not
specified.

 I don't mind requiring
> the user to force it; it's a good reminder (and ties in nicely
> with another comment below).

Hmm.  That's an excellent point.  If others are OK with it, I'm in
favor too.

>>> If we do, perhaps it's time to change the major version number
>>> to 8.0?
>>>
>>
>> Or maybe defer that until we actually declare C mode unsupported.
>> I myself hope to do that before the next release, but ...  :-)
> 
> :-). I know it sounds self-serving, but I think we should be giving
> users a one-release grace period. Make C++ the default, but with
> C support for those who can't build with C++. That way, they can
> still build, but they are clearly on notice about the impending
> change. Then, as soon as the release branch is cut, we can flip
> the switch. That way, I don't think you'll have to keep the compat
> mode for much longer than you were planning to.

That may well be.

Thanks,
Pedro Alves