BurntSushi / jiff

A date-time library for Rust that encourages you to jump into the pit of success.
The Unlicense
1.77k stars 35 forks source link

Support riscv32imac-unknown-none-elf #162

Open lu-zero opened 1 day ago

lu-zero commented 1 day ago

There are few uses of Arc and those are missing for the target. Would it be acceptable to add portable-atomic and portable-atomic-util and the required knobs to support it?

BurntSushi commented 1 day ago

I am okay with it as long as they are only included and used for targets that specifically need them.

Another option is to just not use an Arc on those targets and instead use a Box. It might lead to higher memory usage and longer clones. This is probably not great, specifically, for the case of TimeZone which is really designed to be cheap to clone. So probably the target specific dependency is worth it.