Skip to content
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

Unbreak the Ubuntu 16.04.2 LTS buildbot #33888

Closed
mkoeppe opened this issue May 24, 2022 · 13 comments
Closed

Unbreak the Ubuntu 16.04.2 LTS buildbot #33888

mkoeppe opened this issue May 24, 2022 · 13 comments

Comments

@mkoeppe
Copy link
Contributor

mkoeppe commented May 24, 2022

Buildbot needs to be configured with

./configure --without-system-gcc)

to force building gcc

CC: @vbraun

Component: build

Issue created by migration from https://trac.sagemath.org/ticket/33888

@mkoeppe mkoeppe added this to the sage-9.7 milestone May 24, 2022
@vbraun

This comment has been minimized.

@mkoeppe
Copy link
Contributor Author

mkoeppe commented May 30, 2022

comment:2

Just why? The default compiler on Debian 9 is supported.

@vbraun
Copy link
Member

vbraun commented May 30, 2022

comment:3

When building with our gcc the build then fails later when trying to compile ncurses: http://build.sagemath.org/#/builders/69/builds/1/steps/10/logs/ncurses

/var/lib/buildbot/slave/sage_git/build/local/bin/g++ -DHAVE_CONFIG_H -I. -I../../c++ -I../include -I../../c++/../include  -D_GNU_SOURCE -DNDEBUG -g -O2  -fPIC -c ../../c++/cursesf.cc -o ../obj_s/cursesf.o
In file included from ../../c++/cursesw.h:36,
                 from ../../c++/cursesp.h:39,
                 from ../../c++/cursesf.h:39,
                 from ../../c++/cursesf.cc:35:
./etip.h:375:33: error: ISO C++17 does not allow dynamic exception specifications
  375 | #define THROWS(s)               throw(s)
      |                                 ^~~~~
../../c++/cursesw.h:777:50: note: in expansion of macro 'THROWS'
  777 |   virtual void   err_handler(const char *) const THROWS(NCursesException);
      |                                                  ^~~~~~
./etip.h:376:33: error: ISO C++17 does not allow dynamic exception specifications
  376 | #define THROW2(s,t)             throw(s,t)
      |                                 ^~~~~
../../c++/cursesp.h:89:32: note: in expansion of macro 'THROW2'
   89 |   void OnError (int err) const THROW2(NCursesException const, NCursesPanelException)
      |                                ^~~~~~
./etip.h:376:33: error: ISO C++17 does not allow dynamic exception specifications
  376 | #define THROW2(s,t)             throw(s,t)
      |                                 ^~~~~
../../c++/cursesf.h:64:38: note: in expansion of macro 'THROW2'
   64 |   inline void OnError(int err) const THROW2(NCursesException const, NCursesFormException) {
      |                                      ^~~~~~
./etip.h:376:33: error: ISO C++17 does not allow dynamic exception specifications
  376 | #define THROW2(s,t)             throw(s,t)
      |                                 ^~~~~
../../c++/cursesf.h:112:39: note: in expansion of macro 'THROW2'
  112 |   inline void OnError (int err) const THROW2(NCursesException const, NCursesFormException) {
      |                                       ^~~~~~
./etip.h:376:33: error: ISO C++17 does not allow dynamic exception specifications
  376 | #define THROW2(s,t)             throw(s,t)
      |                                 ^~~~~
../../c++/cursesf.h:394:39: note: in expansion of macro 'THROW2'
  394 |   inline void OnError (int err) const THROW2(NCursesException const, NCursesFormException) {
      |                                       ^~~~~~
./etip.h:375:33: error: ISO C++17 does not allow dynamic exception specifications
  375 | #define THROWS(s)               throw(s)
      |                                 ^~~~~
../../c++/cursslk.h:120:45: note: in expansion of macro 'THROWS'
  120 |   inline void Error (const char* msg) const THROWS(NCursesException) {
      |                                             ^~~~~~
Makefile:359: recipe for target '../obj_s/cursesf.o' failed
make[6]: *** [../obj_s/cursesf.o] Error 1

@vbraun
Copy link
Member

vbraun commented May 30, 2022

comment:4

Note: has gcc 5.4.0 which will be dropped in #33316

vbraun@k8s:~/Buildbot$ cat /etc/debian_version 
stretch/sid
vbraun@k8s:~/Buildbot$ gcc --version
gcc (Ubuntu 5.4.0-6ubuntu1~16.04.12) 5.4.0 20160609
Copyright (C) 2015 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

@mkoeppe
Copy link
Contributor Author

mkoeppe commented May 30, 2022

comment:5

Debian 9 has gcc 6.3.0 as the default gcc.

Your gcc says "Ubuntu 5.4.0-6ubuntu1~16.04.12". That's not Debian.

@vbraun
Copy link
Member

vbraun commented May 30, 2022

comment:6

I guess thats Ubuntu 16.04.2 then, ESM until April 2026

@vbraun vbraun changed the title Unbreak the Debian 9 buildbot Unbreak the Ubuntu 16.04.2 LTS buildbot May 30, 2022
@mkoeppe
Copy link
Contributor Author

mkoeppe commented May 30, 2022

comment:7

Yes, #33316 drops support for ubuntu-xenial with its default toolchain.

@mkoeppe
Copy link
Contributor Author

mkoeppe commented May 30, 2022

comment:8

The upgrade path is to use ubuntu-xenial-toolchain-gcc_9, see #32074.

@vbraun
Copy link
Member

vbraun commented May 30, 2022

comment:9

Still would be nice if your own ncurses would compile with our own gcc... Maybe check #32137?

@mkoeppe
Copy link
Contributor Author

mkoeppe commented May 30, 2022

comment:10

ready for review

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Jul 18, 2022

comment:11

Is your bot fixed? Then please close the ticket

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Aug 6, 2022

comment:12

I assume we can close it as outdated

@mkoeppe mkoeppe removed this from the sage-9.7 milestone Aug 6, 2022
@kwankyu
Copy link
Collaborator

kwankyu commented Aug 10, 2022

comment:13

Replying to @mkoeppe:

I assume we can close it as outdated

It seems Ubuntu 16.04.2 is not there anymore.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants