[enh] add support for multiple outputs

Allows Silkaj to make a transaction to multiple receivers using a new separator in --output option:

--output=<public key>[:checksum][,<public key>[:checksum]]

So the transaction's --amount is sent to each output, which means if we send --amount 10 with 3 outputs we actually send 30.

I've tried to integrate it without many modifications as I don't know the possible impacts elsewhere.

changed milestone to %0.6.0

assigned to @moul

changed the description

changed target branch from master to dev

Firt of all. Thanks for your contribution!

---

Silkaj workflow is: master <— dev <– branch. I changed the target of this PR to dev branch. There is conflicts to solve.

---

Otherwise, I would like to keep the constancy with amount command with which we can specify pubkeys splited with colons. Unfortunatly, there is already a pubkey’s sum check which already uses colons to seperate the pubkey from the checksum. If we don’t find a better solution, we will have to keep it that way for now.

---

This is the first review. Then, I will need to test it.

added 26 commits

• 4ded4148...36a15c0c - 25 commits from branch clients/python:dev
• b2319de2 - [enh] add support for multiple outputs

Compare with previous version

Ok, tested. It works!

And the commit has been rebased on dev branch.

added 2 commits

• bab25f5c - [enh] #83 (closed): add support for multiple outputs
• 5af9bc89 - [enh] #83 (closed): tx: refactor multiple outputs:

Compare with previous version

Ok, we are close to merge it.

I published a commit which refactor. @c-geek, you can test it.

I have to figure out for the three upper points.

added 1 commit

• 86fdc27f - [enh] #83 (closed), #72 (closed): handle the enough sources’ check for multiple outputs,

Compare with previous version

resolved all discussions

added 4 commits

• 8f6a2fcf - [mod] #72 (closed): Remove duplicate check for outputBackChange pubkey.
• b3a6344a - [enh] #83 (closed): handle the enough sources’ check for multiple outputs.
• 5e76a2c5 - [enh] #83 (closed), #72 (closed): add proper pubkey’s checksum check.
• 2259d3df - [mod] #83 (closed), #72 (closed): keep constancy with amount cmd, using ':' as pubkeys separator

Compare with previous version

added 1 commit

• ac5c1b1a - [mod] #83 (closed): remove duplicate 'enough money' check.

Compare with previous version

added 1 commit

• e093a8c4 - [mod] remove useless import.

Compare with previous version

Ok, we should be good now. There is commits not related to this PR. But, yeah, we made it!

I changed:

• pubkeys separator from , to :.
• pubkey from its checksum separator from : to !.

Cool :-)

mentioned in commit 4a4d80ae