Enable new math functions?

[bricelam]

Looks like there are some new math functions in SQLite 3.35.0. Should we add SQLITE_ENABLE_MATH_FUNCTIONS to the list of default defines?

https://github.com/ericsink/cb/blob/bd3ec0643c5297c05c765f04a28104e85b59dd0c/bld/cb.cs#L1065-L1072

[ericsink]

When I try integrating 3.35.x (soon), I do plan to try turning that on to see how it goes.

[ericsink]

(But I'm glad you logged this so we have a place to track status of the issue)

[nbevans]

Would be great to get this enabled in the next release. The CEILING function is desperately needed and shouldn't be excluded from the standard build IMO (though that's a quirk of SQLite not SQLitePCLRaw)

[nbevans]

Any chance of it @ericsink ?

I may be naive of what goes on in the SQLitePCL.raw lib builds but it seems highly unlikely that enabling this compile-time option would break anything?

The math functions are included in most builds of SQLite now but not SQLitePCL.raw yet :(

[ericsink]

Hmmm. That looks like it would be fairly straightforward to include those.

I'll take a look soon. A few things have piled up, and it seems like time to crank out another release.

[ericsink]

I had trouble getting the Android builds to link with the math libs, so for now, I released 2.1.5 without this fix, so I can get the Mac arm64 simulator support out there. Hopefully 2.1.6 with math functions will follow shortly.

[ericsink]

Just to remind myself and others about this issue: I've been a bit swamped, and still haven't figured out the problem with math libs linking on android.

[bricelam]

I got the CI passing in https://github.com/ericsink/cb/pull/17!

[ericsink]

Yay!

So it looks like the basic fix was to up the android API level higher.

Thanks!