ref-fvm icon indicating copy to clipboard operation
ref-fvm copied to clipboard

Define SignedMessage struct

Open rllola opened this issue 2 years ago • 3 comments

I am using SignedMessage struct from the chainsafe library. Is there any plan to add it to the lib ? It would help with migration without breaking stuff.

See https://github.com/ChainSafe/forest/blob/main/vm/message/src/signed_message.rs#L13

rllola avatar Apr 04 '22 13:04 rllola

Is there any reason you can't define it locally? SignedMessage isn't relevant to the FVM.

Stebalien avatar Apr 04 '22 16:04 Stebalien

(no strong reason not to include it, we'd just rather not include anything we don't need (especially in shared) unless it makes everyone's lives significantly easier)

Stebalien avatar Apr 04 '22 16:04 Stebalien

I am actually trying to define it locally but I have encounter an other issue for which I opened a PR (https://github.com/filecoin-project/ref-fvm/pull/446).

I don't have a strong reason to have it included beside helping migrating from chainsafe lib to this one. I believe Forest will need this too so instead of everyone having its own implementation having one in this standard lib would avoid issues I guess.

rllola avatar Apr 05 '22 09:04 rllola

A few months later, we can confirm that this struct is not relevant or missed in ref-fvm; it's a client-specific type. An idea here would be for Forest to spin off its common types to an independent crate.

raulk avatar Sep 02 '22 22:09 raulk