<div dir="ltr">Hi Clément,<div><br></div><div>Indeed, this is what I ended up doing - and there are a couple of instances of the same error in various places.</div><div><br></div><div>This appears to be a legitimate error C++ even though this passes with GCC 8.</div><div><br></div><div>On the choice of dropping GCC<8, this seems a bit early, even if the latest flavors of Ubuntu, or Debian Sid already have GCC8 available. GCC6 and GCC7 are recent compilers and people target older distributions in their builds.</div><div><br></div><div>Best,</div><div><br></div><div>Sylvain Corlay</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Mar 5, 2020 at 11:41 AM Clément David <<a href="mailto:Clement.David@esi-group.com">Clement.David@esi-group.com</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">Hello Sylvain,<br>
<br>
I took a look at the trace, it looks like g++ 7 is confused with the xml.h include ; as libxml2 is already protected with ifdef __cplusplus extern "C" you might just move the #include <xml.h> from XMLDocument.hxx:28 out of the extern "C".<br>
<br>
Sorry about that but gcc-7 is not our target for official releases, Debian/Ubuntu/Fedora all ship gcc-8 with a more stable c++17 support. I could only help to debug issues on the conda side.<br>
<br>
Regards,<br>
<br>
--<br>
Clément<br>
<br>
> -----Original Message-----<br>
> From: dev <<a href="mailto:dev-bounces@lists.scilab.org" target="_blank">dev-bounces@lists.scilab.org</a>> On Behalf Of Sylvain Corlay<br>
> Sent: Thursday, March 5, 2020 10:15 AM<br>
> To: List dedicated to the development of Scilab <<a href="mailto:dev@lists.scilab.org" target="_blank">dev@lists.scilab.org</a>><br>
> Subject: Re: [Scilab-Dev] Scilab 6.1.0 is available!<br>
> <br>
> Hello,<br>
> <br>
> Is there any update on the support of GCC 7 for Scilab 6.1?<br>
> <br>
> This is a major blocker for us to package it for the conda ecosystem.<br>
> <br>
> Best,<br>
> <br>
> On Thu, Feb 27, 2020 at 4:12 PM Sylvain Corlay <<a href="mailto:sylvain.corlay@gmail.com" target="_blank">sylvain.corlay@gmail.com</a><br>
> <mailto:<a href="mailto:sylvain.corlay@gmail.com" target="_blank">sylvain.corlay@gmail.com</a>> > wrote:<br>
> <br>
> <br>
> Patching the source to use <experimental/filesystem> resolves that<br>
> particular issue with C++17 in GCC7.<br>
> <br>
> However, the Scilab build now fails with many occurrences of the error:<br>
> <br>
> error: template with C linkage<br>
> <br>
> <br>
> - You can see the raw build logs here: <a href="https://dev.azure.com/conda-" rel="noreferrer" target="_blank">https://dev.azure.com/conda-</a><br>
> forge/84710dde-1620-425b-80d0-<br>
> 4cf5baca359d/_apis/build/builds/125787/logs/7<br>
> - The scilab recipe update including the patches for <filesystem> is<br>
> available here: <a href="https://github.com/conda-forge/scilab-feedstock/pull/11" rel="noreferrer" target="_blank">https://github.com/conda-forge/scilab-feedstock/pull/11</a><br>
> <br>
> Officially supporting older versions of GCC than GCC 8 (which is from<br>
> May 2018) would be greatly appreciated.<br>
> <br>
> <br>
> Best,<br>
> <br>
> <br>
> On Thu, Feb 27, 2020 at 2:28 PM Sylvain Corlay<br>
> <<a href="mailto:sylvain.corlay@gmail.com" target="_blank">sylvain.corlay@gmail.com</a> <mailto:<a href="mailto:sylvain.corlay@gmail.com" target="_blank">sylvain.corlay@gmail.com</a>> > wrote:<br>
> <br>
> <br>
> Hi Clément,<br>
> <br>
> Regarding the Java, we already skip xcos in the build but we<br>
> were really hoping that we could start including it with this version.<br>
> <br>
> For the filesystem thingy, I will be patching the source as part of<br>
> the conda recipe to use <experimental/filesystem> instead and see how it goes -<br>
> although it would be really nice if we could support more compilers out of the<br>
> box.<br>
> <br>
> Sylvain<br>
> <br>
> On Thu, Feb 27, 2020 at 2:25 PM Clément David<br>
> <<a href="mailto:Clement.David@esi-group.com" target="_blank">Clement.David@esi-group.com</a> <mailto:<a href="mailto:Clement.David@esi-group.com" target="_blank">Clement.David@esi-group.com</a>> ><br>
> wrote:<br>
> <br>
> <br>
> Hello Sylvain,<br>
> <br>
> First, thank you for your work on the conda packaging.<br>
> The c++17 requirements is only needed for a single file that is used to implement<br>
> fullpath() (named fullpath.cpp). The used API is reduced to<br>
> std::filesystem::weakly_canonical and std::filesystem::absolute [1]. I guess using<br>
> a light patch might relax the use of the filesystem header, for example,<br>
> something like [2].<br>
> <br>
> About the Java8 requirement, I guess you could<br>
> ./configure --without-xcos as a first approach. I started porting the Java code<br>
> out of javax.xml.bind [3] but that's very repetitive and error prone work.<br>
> <br>
> [1]:<br>
> <a href="https://codereview.scilab.org/#/c/21041/25/scilab/modules/fileio/src/cpp/fullp" rel="noreferrer" target="_blank">https://codereview.scilab.org/#/c/21041/25/scilab/modules/fileio/src/cpp/fullp</a><br>
> ath.cpp<br>
> [2]:<br>
> <a href="https://stackoverflow.com/questions/45867379/why-does-gcc-not-seem-to-" rel="noreferrer" target="_blank">https://stackoverflow.com/questions/45867379/why-does-gcc-not-seem-to-</a><br>
> have-the-filesystem-standard-library<br>
> [3]: <a href="https://codereview.scilab.org/#/c/20630/" rel="noreferrer" target="_blank">https://codereview.scilab.org/#/c/20630/</a><br>
> <br>
> Regards,<br>
> <br>
> --<br>
> Clément<br>
> <br>
> > -----Original Message-----<br>
> > From: dev <<a href="mailto:dev-bounces@lists.scilab.org" target="_blank">dev-bounces@lists.scilab.org</a><br>
> <mailto:<a href="mailto:dev-bounces@lists.scilab.org" target="_blank">dev-bounces@lists.scilab.org</a>> > On Behalf Of Sylvain Corlay<br>
> > Sent: Thursday, February 27, 2020 1:50 PM<br>
> > To: List dedicated to the development of Scilab<br>
> <<a href="mailto:dev@lists.scilab.org" target="_blank">dev@lists.scilab.org</a> <mailto:<a href="mailto:dev@lists.scilab.org" target="_blank">dev@lists.scilab.org</a>> ><br>
> > Subject: Re: [Scilab-Dev] Scilab 6.1.0 is available!<br>
> ><br>
> > Congratulations on the release.<br>
> ><br>
> > I am the author of the conda package for scilab, and<br>
> unfortunately, it does not<br>
> > seem that 6.1.0 can be successfully packaged for<br>
> conda-forge with the new<br>
> > requirement for the <filesystem> C++17 header,<br>
> which requires GCC 8.<br>
> ><br>
> > Conda-forge is still based on GCC 7, which is fairly<br>
> recent, with C++17 enabled by<br>
> > default. Would you consider not using the<br>
> <filesystem> from the C++17 standard<br>
> > so that Scilab can be made available to a wider<br>
> audience?<br>
> ><br>
> > Another blocker to the packaging of Scilab is the<br>
> outdated version of java that is<br>
> > required by the GUI. Is there any plan to support a<br>
> more recent version of<br>
> > OpenJDK?<br>
> ><br>
> > Best,<br>
> ><br>
> > Sylvain Corlay<br>
> ><br>
> > On Tue, Feb 25, 2020 at 2:10 PM Clément David<br>
> <Clement.David@esi-<br>
> > <a href="http://group.com" rel="noreferrer" target="_blank">group.com</a> <<a href="http://group.com" rel="noreferrer" target="_blank">http://group.com</a>><br>
> <mailto:<a href="mailto:Clement.David@esi-group.com" target="_blank">Clement.David@esi-group.com</a> <mailto:<a href="mailto:Clement.David@esi-" target="_blank">Clement.David@esi-</a><br>
> <a href="http://group.com" rel="noreferrer" target="_blank">group.com</a>> > > wrote:<br>
> ><br>
> ><br>
> > Dear Scilab-ers,<br>
> ><br>
> ><br>
> ><br>
> > A brand new Scilab 6.1.0<br>
> <<a href="https://www.scilab.org/download/6.1.0" rel="noreferrer" target="_blank">https://www.scilab.org/download/6.1.0</a>> is<br>
> > released today!<br>
> ><br>
> ><br>
> ><br>
> > This version includes further improvement atop<br>
> Scilab 6.0 for better<br>
> > stability and increased algorithm performance. It also<br>
> includes a reworked<br>
> > display for more compact and meaningful value<br>
> printing; web tools for HTTP,<br>
> > JSON support; better debug support and various<br>
> algorithm rewrite/extension.<br>
> ><br>
> ><br>
> ><br>
> > This first iteration of the 6.1 branch fixes up to 245<br>
> bugs and implements<br>
> > missing features from the 6.0.2 version. We would<br>
> like to give a special thanks<br>
> > to Samuel and Stephane who have been very active<br>
> this year.<br>
> ><br>
> ><br>
> ><br>
> > If you find any critical issue or instability that might<br>
> need a 6.1.x release<br>
> > please alert us <<a href="https://bugzilla.scilab.org/" rel="noreferrer" target="_blank">https://bugzilla.scilab.org/</a>> . If you<br>
> are a toolbox maintainer,<br>
> > please rebuild your code, upgrade it when needed and<br>
> publish it to<br>
> > <a href="http://atoms.scilab.org" rel="noreferrer" target="_blank">atoms.scilab.org</a> <<a href="http://atoms.scilab.org" rel="noreferrer" target="_blank">http://atoms.scilab.org</a>><br>
> <<a href="https://atoms.scilab.org/" rel="noreferrer" target="_blank">https://atoms.scilab.org/</a>> .<br>
> ><br>
> ><br>
> ><br>
> > For the complete list of changes and bugs fixed,<br>
> please take a look at<br>
> > the CHANGES <<a href="https://help.scilab.org/CHANGES" rel="noreferrer" target="_blank">https://help.scilab.org/CHANGES</a>><br>
> file.<br>
> ><br>
> ><br>
> ><br>
> > --<br>
> ><br>
> > Clément on behalf of the Scilab team<br>
> ><br>
> ><br>
> _______________________________________________<br>
> > dev mailing list<br>
> > <a href="mailto:dev@lists.scilab.org" target="_blank">dev@lists.scilab.org</a> <mailto:<a href="mailto:dev@lists.scilab.org" target="_blank">dev@lists.scilab.org</a>><br>
> <mailto:<a href="mailto:dev@lists.scilab.org" target="_blank">dev@lists.scilab.org</a> <mailto:<a href="mailto:dev@lists.scilab.org" target="_blank">dev@lists.scilab.org</a>> ><br>
> > <a href="http://lists.scilab.org/mailman/listinfo/dev" rel="noreferrer" target="_blank">http://lists.scilab.org/mailman/listinfo/dev</a><br>
> ><br>
> <br>
> <br>
> _______________________________________________<br>
> dev mailing list<br>
> <a href="mailto:dev@lists.scilab.org" target="_blank">dev@lists.scilab.org</a> <mailto:<a href="mailto:dev@lists.scilab.org" target="_blank">dev@lists.scilab.org</a>><br>
> <a href="http://lists.scilab.org/mailman/listinfo/dev" rel="noreferrer" target="_blank">http://lists.scilab.org/mailman/listinfo/dev</a><br>
> <br>
<br>
_______________________________________________<br>
dev mailing list<br>
<a href="mailto:dev@lists.scilab.org" target="_blank">dev@lists.scilab.org</a><br>
<a href="http://lists.scilab.org/mailman/listinfo/dev" rel="noreferrer" target="_blank">http://lists.scilab.org/mailman/listinfo/dev</a><br>
</blockquote></div>