fix deterministic.rs 'seal changed -was: 0xa7a4 now 0x544d'#438
fix deterministic.rs 'seal changed -was: 0xa7a4 now 0x544d'#438hyperji wants to merge 1 commit intokaspanet:masterfrom
Conversation
coderofstuff
left a comment
There was a problem hiding this comment.
This allows compiling the entire repo (specifically the wallet part) with the latest rust version (1.77.0 as of this writing) but breaks older version.
kaspad and the rest of the components that do not depend on wallet are semantically the same and are not affected by this PR.
LGTM.
|
But what is this seal and its purpose and why does it suddenly got broken? Can we really just patch it with a new value? |
|
This pr is included by 1.77 rustc version I suggest to @aspect to make it derive macro only depending on fields and types, he replied that this is general purpose macro and it can be applied to any code. In summary it will work fine until compiler change the logic again. Imho the proper solution is changing the hash and pinning rust toolchain in the repo. And maybe adding msrv |
|
I see, thanks for clarification |
|
Thank you for your contribution. As this was integrated into a more fundamental fix in #442 this current PR has been superseded. |
|
Fixed in #442 I couldn't merge this PR due to other ongoing issues with lints (caused by the new rustc version as well), so the change was applied in the above-mentioned PR that was just merged. |
fix issue: #437
reason:
seal changed -was: 0xa7a4 now 0x544d