Opened 5 years ago

Closed 4 years ago

#21169 closed defect (invalid)

Givaro testsuite fails with gcc6

Reported by: vbraun Owned by:
Priority: major Milestone: sage-duplicate/invalid/wontfix
Component: packages: standard Keywords: gcc6 c++11
Cc: cpernet Merged in:
Authors: Reviewers: Clément Pernet
Report Upstream: Fixed upstream, in a later stable release. Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges

Description

test-ffarith.C:193:2: note: in expansion of macro 'TESTE_EG'
  TESTE_EG(e,a);
  ^~~~~~~~
In file included from /usr/include/c++/6.1.1/string:52:0,
                 from /usr/include/c++/6.1.1/bits/locale_classes.h:40,
                 from /usr/include/c++/6.1.1/bits/ios_base.h:41,
                 from /usr/include/c++/6.1.1/ios:42,
                 from /usr/include/c++/6.1.1/ostream:38,
                 from /usr/include/c++/6.1.1/iostream:39,
                 from test-ffarith.C:8:
/usr/include/c++/6.1.1/bits/basic_string.h:5325:5: note: candidate: template<class _CharT, class _Traits, class _Alloc> std::basic_ostream<_CharT, _Traits>& std::operator<<(std::basic_ostream<_CharT, _Traits>&, const std::__cxx11::basic_string<_CharT, _Traits, _Alloc>&)
     operator<<(basic_ostream<_CharT, _Traits>& __os,
     ^~~~~~~~
/usr/include/c++/6.1.1/bits/basic_string.h:5325:5: note:   template argument deduction/substitution failed:
test-ffarith.C:20:12: note:   'std::ostream {aka std::basic_ostream<char>}' is not derived from 'const std::__cxx11::basic_string<_CharT, _Traits, _Alloc>'
  std::cout << F.write(std::cout,a) << "!=" << F.write(std::cout,b) << " failed (at line " <<  __LINE__ << ")" << std::endl; \
             
test-ffarith.C:193:2: note: in expansion of macro 'TESTE_EG'
  TESTE_EG(e,a);
  ^~~~~~~~
Makefile:454: recipe for target 'test-ffarith.o' failed
make[6]: *** [test-ffarith.o] Error 1

Change History (8)

comment:1 Changed 5 years ago by leif

  • Milestone changed from sage-7.3 to sage-7.4

comment:2 Changed 5 years ago by leif

As reported here...

comment:3 Changed 5 years ago by leif

  • Status changed from new to needs_info

Will most probably vanish with #17635.

(LinBox, FFLAS-FFPACK and Givaro will then require C++11, while they currently require -std=c++98, i.e., do not support C++11 at all. :-) )

Should we until then add -std=c++98 in Givaro's spkg-check (and probably also spkg-install) anyway?

comment:4 Changed 5 years ago by cpernet

  • Cc cpernet added

comment:5 Changed 5 years ago by cpernet

  • Report Upstream changed from N/A to Fixed upstream, but not in a stable release.
  • Status changed from needs_info to needs_review

I confirm that since #17635 was merged, givaro test suite (run through sage -f -c givaro) passes. This ticket can be safely closed.

comment:6 Changed 5 years ago by cpernet

  • Report Upstream changed from Fixed upstream, but not in a stable release. to Fixed upstream, in a later stable release.

comment:7 Changed 5 years ago by cpernet

  • Milestone changed from sage-7.4 to sage-duplicate/invalid/wontfix
  • Reviewers set to Clément Pernet
  • Status changed from needs_review to positive_review

As a wontfix ticket, I guess I can positive review it, even if it has no author.

comment:8 Changed 4 years ago by vbraun

  • Resolution set to invalid
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.