<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Jul 22, 2020 at 2:26 PM Christian Mauderer <<a href="mailto:oss@c-mauderer.de">oss@c-mauderer.de</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Ping again.<br>
<br>
It's clearly not BSP specific. So I would like to get an approval for<br>
that before pushing it anywhere.<br></blockquote><div><br></div><div>CentOS 7's gcc appears to have the option and I would assume this is the oldest distribution anyone is using.</div><div><br></div>$ gcc --version<br>gcc (GCC) 4.8.5 20150623 (Red Hat 4.8.5-39)</div><div class="gmail_quote"><br></div><div class="gmail_quote">Go ahead and push it.<br><div><br></div><div>--joel </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
On 15/06/2020 17:36, Christian Mauderer wrote:<br>
> Ping.<br>
> <br>
> Currently shouldn't be a matter for much hosts (less than 20 including<br>
> FreeBSD, Fedora, Arch, openSUSE according to distrowatch [1]) but I<br>
> assume that llvm 10 will spread fast.<br>
> <br>
> <br>
> [1]<br>
> <a href="http://distrowatch.org/search.php?pkg=llvm&relation=greaterequal&pkgver=10&distrorange=InLatest#pkgsearch" rel="noreferrer" target="_blank">http://distrowatch.org/search.php?pkg=llvm&relation=greaterequal&pkgver=10&distrorange=InLatest#pkgsearch</a><br>
> <br>
> <br>
> On 08/06/2020 08:52, Christian Mauderer wrote:<br>
>> llvm version 10 uses features from c++14 standard in the headers. With<br>
>> that, the record/record-main-lttng.cc doesn't build any more. This patch<br>
>> makes sure that c++14 is used if it is available.<br>
>> ---<br>
>>  trace/wscript | 6 +++++-<br>
>>  1 file changed, 5 insertions(+), 1 deletion(-)<br>
>><br>
>> diff --git a/trace/wscript b/trace/wscript<br>
>> index 656f92b..53a1ab4 100644<br>
>> --- a/trace/wscript<br>
>> +++ b/trace/wscript<br>
>> @@ -45,6 +45,7 @@ def configure(conf):<br>
>>      if conf.check(header_name='zlib.h', features='cxx', mandatory=False):<br>
>>          conf.check_cxx(lib = 'z')<br>
>>      conf.check_cxx(lib = 'ws2_32', mandatory=False)<br>
>> +    conf.check_cxx(cxxflags='-std=c++14', mandatory=False, define_name="HAVE_STD_CXX14")<br>
>>      conf.write_config_header('config.h')<br>
>>  <br>
>>  def build(bld):<br>
>> @@ -60,7 +61,10 @@ def build(bld):<br>
>>      conf['warningflags'] = ['-Wall', '-Wextra', '-pedantic']<br>
>>      conf['optflags'] = bld.env.C_OPTS<br>
>>      conf['cflags'] = ['-pipe', '-g'] + conf['optflags']<br>
>> -    conf['cxxflags'] = ['-std=c++11'] + conf['cflags']<br>
>> +    cxxstd = '-std=c++11'<br>
>> +    if bld.env.HAVE_STD_CXX14:<br>
>> +        cxxstd = '-std=c++14'<br>
>> +    conf['cxxflags'] = [cxxstd] + conf['cflags']<br>
>>      conf['linkflags'] = ['-g']<br>
>>      conf['lib'] = []<br>
>>      if bld.env.LIB_WS2_32:<br>
>><br>
> _______________________________________________<br>
> devel mailing list<br>
> <a href="mailto:devel@rtems.org" target="_blank">devel@rtems.org</a><br>
> <a href="http://lists.rtems.org/mailman/listinfo/devel" rel="noreferrer" target="_blank">http://lists.rtems.org/mailman/listinfo/devel</a><br>
> <br>
_______________________________________________<br>
devel mailing list<br>
<a href="mailto:devel@rtems.org" target="_blank">devel@rtems.org</a><br>
<a href="http://lists.rtems.org/mailman/listinfo/devel" rel="noreferrer" target="_blank">http://lists.rtems.org/mailman/listinfo/devel</a><br>
</blockquote></div></div>