0LNetworkCommunity / libra-legacy-v6

Apache License 2.0
249 stars 116 forks source link

Simplify Libra.move (or replace) #18

Open 0o-de-lally opened 4 years ago

0o-de-lally commented 4 years ago

We can radically simplify Libra.move

Since there are changes to stdlib/Libra.move, perhaps we should fork that file, and call it something like Coins.move, and place in /OL/. That way we could keep the original Libra.move, and pull in FB's changes to follow what is changing. (Although eventually think we need to not include it in genesis).

Some changes to Libra.move

liangping commented 4 years ago

We can not have two Libra.move, even though they are in different directory. it will come up with duplicated module error.

0o-de-lally commented 4 years ago

Ok good point. My thought was to radically simplify Libra.move, or eliminate it as a dependency in favor of a "Coins.move". But I think we should let it stabilize a bit before these optimizations.

0o-de-lally commented 4 years ago

@nelaturuk Maybe you can revisit this now that we've got a good cleanup done, and removed association.