EdSchouten
commented
6 years ago Hi there! Great to hear there's interest in this code! \o/
This sounds like a good idea. I would love to keep them integrated in the C library in CloudABI's case, but if we could somehow make it so that I can just put an unmodified copy of this in contrib/ and link that into the C library, that would be fine by me.
Right now these functions are somewhat tied into cloudlibc's localisation framework (through newlocale(LC_TIMEZONE_MASK)). When moving this into a separate library, we'd better create a separate type for the time zone handle. In the case of cloudlibc, locale_t could be adjusted to embed this handle.
Is this something that you've already taken a stab at solving, or shall I go ahead and create a repo under NuxiNL for us to work on this?
CurlyMoo
sdalu
I'm curious what you think about the idea to release localtime_l and mktime_l as two functions in a independent shared library. Until today, i have not found a good library that does what these functions do, purely converting between timezones based on the IANA timezone database.
Because such a library does not exists, i integrated your localtime_l code inside my own program, but that requires (as you do now) a new release as soon as the IANA database is updated. If these functions would exist in a independent shared library, that update mechanism would be far easier, by just updating the shared library.
I also think a lot of C developers would be interested in these two functions alone without having to use the full cloudlibc library.