search

pixie-lang / pixie

A small, fast, native lisp with "magical" powers
GNU General Public License v3.0
2.35k stars 124 forks source link

define exp10 using pow #494

Closed mks-m closed 8 years ago

mks-m commented 8 years ago

in #486 exp10 was added to math.pxi, it doesn't exist on osx and breaks compilation.

i'd love to wrap this in some conditionals but not sure how to detect platform in pixie.

thomasmulvaney commented 8 years ago

There are some functions which might come in handy in here: pixie/vm/libs/platform.py Hope that helps.

On Thu, Feb 11, 2016 at 5:21 PM, Maksym Melnychok notifications@github.com wrote:

in #486 https://github.com/pixie-lang/pixie/pull/486 exp10 was added to math.pxi, it doesn't exist on osx and breaks compilation.

i'd love to wrap this in some conditionals but not sure how to detect

platform in pixie.

You can view, comment on, or merge this pull request online at:

https://github.com/pixie-lang/pixie/pull/494 Commit Summary

  • comment out exp10 import, define it using pow

File Changes

Patch Links:

— Reply to this email directly or view it on GitHub https://github.com/pixie-lang/pixie/pull/494.

thomasmulvaney commented 8 years ago

Looks good! Would you be able to squash this into one commit though please?

mks-m commented 8 years ago

@thomasmulvaney done

thomasmulvaney commented 8 years ago

Thanks!