These are chat archives for symengine/symengine

9th
Jan 2016
Ralf Stephan
@rwst
Jan 09 2016 09:49

@isuruf Re your series branch, now that Number is abstract in master your branch no longer compiles with lots of:

../symengine/libsymengine.a(visitor.cpp.o):(.data.rel.ro._ZTIN9SymEngine13ComplexDoubleE[_ZTIN9SymEngine13ComplexDoubleE]+0x10): more undefined references to `typeinfo for SymEngine::Number' follow
../symengine/libsymengine.a(visitor.cpp.o):(.data.rel.ro._ZTVN9SymEngine10SeriesBaseIN7piranha10polynomialINS1_11mp_rationalILi0EEENS1_8monomialIsSt17integral_constantImLm0EEEEEES4_NS_18URatPSeriesPiranhaELb0EEE[_ZTVN9SymEngine10SeriesBaseIN7piranha10polynomialINS1_11mp_rationalILi0EEENS1_8monomialIsSt17integral_constantImLm0EEEEEES4_NS_18URatPSeriesPiranhaELb0EEE]+0xa8): undefined reference to `SymEngine::Number::sub(SymEngine::Number const&) const'

I have no idea how to resolve this.

Isuru Fernando
@isuruf
Jan 09 2016 10:04
I'll check it now
Isuru Fernando
@isuruf
Jan 09 2016 10:14
@rwst, there must have been some merge errors when you merged with master. I pushed to the series branch after merging with master
Ralf Stephan
@rwst
Jan 09 2016 14:00
@isuruf OK but I still had to fix the problem in https://github.com/symengine/symengine/pull/710#issuecomment-170211155
see fix in rwst/symengine@a8beea4
should I do a PR with your branch?
Isuru Fernando
@isuruf
Jan 09 2016 16:53
@rwst, sure go ahead