[issue29439] _decimal on Android requires linking with libm

2017-02-07 Thread Chi Hsuan Yen
Chi Hsuan Yen added the comment: Actually my device is 6.0. Seems there's nothing interesting between API 23 and 24 on android-changes-for-ndk-developers.md :) Anyway, _decimal should not depend on the whether the python binary references to libm.so or not. Instead, _decimal should explicitly

[issue29439] _decimal on Android requires linking with libm

2017-02-06 Thread Xavier de Gaye
Xavier de Gaye added the comment: > Perhaps test_decimal should fail for CPython if _decimal can't be imported. This is not a problem specific to _decimal, the same problem exists for third-party extension modules, for example pyephem (https://github.com/brandon-rhodes/pyephem/issues/114) and

[issue29439] _decimal on Android requires linking with libm

2017-02-06 Thread Xavier de Gaye
Xavier de Gaye added the comment: With the following module named dlsym.py, the command 'python dlsym.py log10' produces the same successful output on linux and on Android API 21 (when _decimal is not explicitly linked with libm.so, i.e.without changeset b60b46ad8751): 'The symbol "log10"

[issue29439] _decimal on Android requires linking with libm

2017-02-04 Thread Stefan Krah
Stefan Krah added the comment: Perhaps test_decimal should fail for CPython if _decimal can't be imported. All compilers that I tested seem to link fine without -lm in this case. -- ___ Python tracker

[issue29439] _decimal on Android requires linking with libm

2017-02-04 Thread Xavier de Gaye
Xavier de Gaye added the comment: > This wasn't fixed because _decimal didn't build before issue26846 lands. This is misleading, the problem went unnoticed when not linking with '-lm' because: * The _decimal extension module builds without any warning. * test_decimal does not fail. One way to

[issue29439] _decimal on Android requires linking with libm

2017-02-04 Thread Stefan Krah
Changes by Stefan Krah : -- resolution: -> fixed stage: -> resolved status: open -> closed versions: +Python 3.6 ___ Python tracker

[issue29439] _decimal on Android requires linking with libm

2017-02-04 Thread Roundup Robot
Roundup Robot added the comment: New changeset aa26a5712a80849e968171b71b6bc8d9da3ac163 by Stefan Krah in branch 'master': Issue29439: _decimal on Android requires linking with libm. https://github.com/python/cpython/commit/aa26a5712a80849e968171b71b6bc8d9da3ac163

[issue29439] _decimal on Android requires linking with libm

2017-02-04 Thread Roundup Robot
Roundup Robot added the comment: New changeset aa26a5712a80849e968171b71b6bc8d9da3ac163 by Stefan Krah in branch '3.6': Issue29439: _decimal on Android requires linking with libm. https://github.com/python/cpython/commit/aa26a5712a80849e968171b71b6bc8d9da3ac163

[issue29439] _decimal on Android requires linking with libm

2017-02-04 Thread Roundup Robot
Roundup Robot added the comment: New changeset b60b46ad8751 by Stefan Krah in branch '3.6': Issue29439: _decimal on Android requires linking with libm. https://hg.python.org/cpython/rev/b60b46ad8751 -- nosy: +python-dev ___ Python tracker <

[issue29439] _decimal on Android requires linking with libm

2017-02-04 Thread Stefan Krah
Stefan Krah added the comment: Thanks. Strange that on other systems the compilers don't complain (usually they do). -- assignee: -> skrah ___ Python tracker

[issue29439] _decimal on Android requires linking with libm

2017-02-03 Thread Chi Hsuan Yen
New submission from Chi Hsuan Yen: Just like issue21668, _decimal requires -lm on Android. This wasn't fixed because _decimal didn't build before issue26846 lands. More specificially, log10 is called https://hg.python.org/cpython/file/tip/Modules/_decimal/libmpdec/mpdecimal.c#l7862 Added