search

pontem-network / dove

🛠️ Diem/Pontem Move package manager
MIT License
35 stars 15 forks source link

CODE_DESERIALIZATION_ERROR while running "dove run" #131

Closed villesundell closed 3 years ago

villesundell commented 3 years ago

While trying to run "dove run" with Dove 1.3.0, I get:

➜  taohe git:(master) ✗ dove run --file immutable                                                                  
Execution failed with unexpected error CODE_DESERIALIZATION_ERROR

This used to work with 1.2.8.

You can use https://github.com/taoheorg/taohe for testing (master supports Dove 1.3.0).

EDIT: If you are using TaoHe for testing, it's probably noteworthy that dove run --file dummy does not produce any output, but instead terminates silently.

RIg410 commented 3 years ago

Thank you for your feedback. We will fix this problem in dove 1.3.1. You can view https://github.com/pontem-network/move-tools/pull/126

The executor was also significantly redesigned in it. And the ability to execute the public script functions of the module has been added.

The execution by filename has been deleted. And most likely we will make a separate team for this in the near future. Please write to us if you have any ideas and needs from this feature. Now the call will look like this dove run 'immutable(_)' dove run dummy

RIg410 commented 3 years ago

If the script does not create effects, its work data is not output. What would you like to see in the output ?

villesundell commented 3 years ago

Thank you very much: your new calling convention seems fine by me, and as simple it can get :+1:

About output generated by running dummy, 1.2.8 used to print:

dummy ...... ok
[gas: 1, writeset bytes: 0]

So I thought that dove run --file dummy would misbehave since it produced no output :slightly_smiling_face:

RIg410 commented 3 years ago

Returned the gas display. To display information about the execution in dove 1.3.1, you must pass the -v flag.

villesundell commented 3 years ago

This is fixed now, closing :slightly_smiling_face:

However, it seems I misunderstood the new syntax: to me, it seems that the syntax could be simpler, account_address from Dove.toml could be used automatically instead of "_" if one signer is required.