Update dfinity&motoko

[eftychis]

No description provided.

[hansl]

@eftychis did you verify that this would allow support for identity in motoko?

[eftychis]

no, I haven't at the moment.

[nomeata]

In Motoko, we started to include the output of a command like this:

~/dfinity/motoko $ git log --oneline 75e091917063f6607600343c4eb79cacb38aa53e..e15d6016197d9d2ca9ae72e00a6d7c3afca0f5bb src/
2722600c IDL-to-Motoko import (#1026)
e3f0efd1 debug_show: Also handle Char (#1060)
43fec1e6 Implement --actor-alias (#1052)
6e4447b8 dec-bug repro & fix (#1050)
88c9f494 Fix for Issue #1051 (Implement IgnoreD) (#1054)
7cea51ba Ropy texts: Introduce concat heap operator (#985)
ca16901a [IDE] Handles more edge-cases for completion prefixes (#1044)
e92d026c Remove dead code from `Typing.error_shared` (#1040)
cd958503 Refine rts_tests for URL decoding (#1043)
fa57d329 Hook up actor imports to Idllib.check_file (#1046)
df30711b Use abbreviations (#1049)
1922d3ad Add the `--actor-idl` flag (#1045)
b285da9c Enable inter-canister calls on ICMode (#1041)
61ae539b Introduce actor references (#1001)
bc1b0c13 Move URL parsing from lib to pipeline (#1042)
24b9155f Correctly decode imported actor URLs (#1034)
3eeeeacd Temporarily disable -(no-)multi-value options (#1038)
acae13f1 Bump wasm dependency to v1.1 (#1032)

in the commit message of such a bump, which makes it clearer what is included.

So this includes inter-canister messaging and actor imports. Caller identification is actually in 75e091917063f6607600343c4eb79cacb38aa53e, so it is already present in SDK before this bump.

[hansl]

Yeah we have seen the commit. We need to make sure it works end to end though. Execution should pass it on, Motoko has syntax for it, we just want to make sure it works with dfx/userlib

…

Sent from my iPhone

On Jan 7, 2020, at 00:37, Joachim Breitner ***@***.***> wrote:  In Motoko, we started to include the output of a command like this: ~/dfinity/motoko $ git log --oneline 75e091917063f6607600343c4eb79cacb38aa53e..e15d6016197d9d2ca9ae72e00a6d7c3afca0f5bb src/ 2722600c IDL-to-Motoko import (#1026) e3f0efd1 debug_show: Also handle Char (#1060) 43fec1e6 Implement --actor-alias (#1052) 6e4447b8 dec-bug repro & fix (#1050) 88c9f494 Fix for Issue #1051 (Implement IgnoreD) (#1054) 7cea51ba Ropy texts: Introduce concat heap operator (#985) ca16901a [IDE] Handles more edge-cases for completion prefixes (#1044) e92d026c Remove dead code from `Typing.error_shared` (#1040) cd958503 Refine rts_tests for URL decoding (#1043) fa57d329 Hook up actor imports to Idllib.check_file (#1046) df30711b Use abbreviations (#1049) 1922d3ad Add the `--actor-idl` flag (#1045) b285da9c Enable inter-canister calls on ICMode (#1041) 61ae539b Introduce actor references (#1001) bc1b0c13 Move URL parsing from lib to pipeline (#1042) 24b9155f Correctly decode imported actor URLs (#1034) 3eeeeacd Temporarily disable -(no-)multi-value options (#1038) acae13f1 Bump wasm dependency to v1.1 (#1032) in the commit message of such a bump, which makes it clearer what is included. So this includes inter-canister messaging and actor imports. Caller identification is actually in 75e091917063f6607600343c4eb79cacb38aa53e, so it is already present in SDK before this bump. — You are receiving this because your review was requested. Reply to this email directly, view it on GitHub, or unsubscribe.

[eftychis]

Created this actor

actor {

  // returns caller id
  public shared { caller = c } func getCaller()  : async Blob {
    c
  };

  // returns self id when called (internally or externally)
  public shared func getSelf()  : async Blob {
    await getCaller();
  };

    public func greet(name : Text) : async Blob {
        let v= await getCaller();
...
     v
    };
};

(had to just return a Blob type instead of my fancier message)
and get a popup window in the example json app with the Blob cropped back:

60,77,132,164,236,237,181,129

Note this is the incorrect number of bytes (but I think after a discussion I had a few minutes ago we just use u64 in the wasm runtime -- needs to be fixed).

[eftychis]

Update: We are not getting the correct id. I have a possible fix. This PR is going to get updated on the dfinity repo side.

[eftychis]

cc @hansl