Re: [Bioc-devel] surfaltr build on Bioc 3.17 error advice

2023-03-24 Thread Hervé Pagès
Hi Pooja, Looks like std::auto_ptr is no longer valid in C++17: https://stackoverflow.com/questions/69116001/how-do-i-re-enable-c17-removed-features-in-clang Weird thing is that the compiler is invoked with -std=gnu++17 on all platforms but compilation of msa only fails on Mac. It looks like u

Re: [Bioc-devel] surfaltr build on Bioc 3.17 error advice

2023-03-24 Thread Pooja Gangras
Hi Hervé, I reached out to Ulrich who is the maintainer for msa package. He found your suggestions very helpful and used the option 2 you suggested to fix the error. Looks like the changes he made have fixed everything on windows and Linux platforms but the macOS platform is still showing an error

Re: [Bioc-devel] surfaltr build on Bioc 3.17 error advice

2023-03-21 Thread Hervé Pagès
On 21/03/2023 12:48, Pooja Gangras wrote: > Hi Herve, > > Thank you for the quick response. > > So what surprises me is that msa was compiling and building just fine > on the devel branch until now. As I said, the switch from C++14 to C++17 happened not too long ago in the devel version of R. An

Re: [Bioc-devel] surfaltr build on Bioc 3.17 error advice

2023-03-21 Thread Pooja Gangras
Hi Herve, Thank you for the quick response. So what surprises me is that msa was compiling and building just fine on the devel branch until now. And hence so was surfaltr, without any errors. There was a warning in surfaltr which one can see on the 3.16 release, I had fixed it in the devel branch

Re: [Bioc-devel] surfaltr build on Bioc 3.17 error advice

2023-03-21 Thread Hervé Pagès
Hi Pooja, Generally speaking there are 3 things you can do when a dep breaks your package: 1. Consider getting rid of that dep. 2. Contact the author/maintainers of the dep to let them know about the problem. If you can suggest a fix (e.g. by sending a PR on GitHub), that's even better, as

[Bioc-devel] surfaltr build on Bioc 3.17 error advice

2023-03-21 Thread Pooja Gangras
Hi, I got an email yesterday alerting me of the error in the build in the new BioC release. Upon looking into the error further I found out that the error is occurring because a dependency 'msa' package is not being built in the new release likely due to some issues with the C++ compiler (just gue