-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
unrecognized command line option '-std=c++14' during Linux builds #9
Comments
How about 20.04 LTS = Focal? |
Tried it, unfortunately no effect. https://travis-ci.org/github/RaRe-Technologies/gensim-wheels/jobs/741274157 |
|
I'm a bit stumped on how to proceed here, so opened spotify/annoy#516 |
Weird. Is it just Annoy that errors out (not part of Gensim proper), or something truly blocking? |
I think it's just Annoy. If we could exclude it from the build process somehow, then we may be able to proceed. I also think I've found the cause of the problem. The build is running inside yet another Docker container, inside Travis, and that Docker container is running Trusty, which has the old gcc. Argh. I'm unfamiliar with this multibuild stuff, so my next step is to figure out who introduced it and go nag them. Failing that, I'll have to learn more about it. |
We may have to move away from Travis altogether, I heard they're sunsetting their open source plans. So I'm not sure whether to invest more time into it. "Github Actions" is one possible replacement – we have a ticket at piskvorky/gensim#2696. |
Yes, I've used github actions before, they are quite good. The problem is: we're coupled to travis via this multibuild stuff, and breaking this coupling will require some effort. |
That is true. I see that effort as a great opportunity to clean up the CI system overall, make it saner – Travis is giving us too much trouble, unreliable. I look forward to it but not sure about the timing :) Certainly not now, in the middle of a release. |
Yeah, while we're at it, we should consider building wheels for every build (or at least for every merged PR). Part of the pain we're feeling now is technical debt caused by a low release cadence. |
Sure, why not. But please finish this beta release first, to avoid any more mishaps. The current frozen-in-mid-release project state is an invitation to further git trouble (not to mention user confusion). |
I've worked around the problem by removing annoy from the build environment. |
full log
I updated the Ubuntu distro to Bionic (18.04LTS) to get a newer compiler, and verified that the gcc version is recent enough:
This is consistent with what TravisCI lists on their page for Bionic, so no surprises there. According to the GCC docs, that version of the compiler should support
-std=c++14
, so I'm not sure what the problem is here.The text was updated successfully, but these errors were encountered: