M-Labs
/
nac3
8
0
Fork 5
Code Issues 87 Pull Requests 3 Releases Wiki Activity

Correct global name for const object for RPC #289

Merged
sb10q merged 1 commits from 272-key-err-rpc-fix into master 2022-05-17 11:53:13 +08:00
Conversation 1 Commits 1 Files Changed 1
ychenfo commented 2022-05-17 03:12:53 +08:00
Collaborator

Despite #283, this simple patch should be able to fix #272.

Previously the llvm global value name is wrong, causing the wrong key to be sent to the host embedding map.

Despite #283, this simple patch should be able to fix #272. Previously the llvm global value name is wrong, causing the wrong key to be sent to the host embedding map.
ychenfo added 1 commit 2022-05-17 03:12:54 +08:00
sb10q commented 2022-05-17 11:07:44 +08:00
Owner

I don't think #283 and #272 are related. #283 should be a simple matter of fixing the inconsistency in the multiple places where builtin exceptions are defined; in particular, ValueError is not defined consistently.

I don't think #283 and #272 are related. #283 should be a simple matter of fixing the inconsistency in the multiple places where builtin exceptions are defined; in particular, ValueError is not defined consistently.
sb10q merged commit 2cb725b7ac into master 2022-05-17 11:53:13 +08:00
sb10q deleted branch 272-key-err-rpc-fix 2022-05-17 11:53:15 +08:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
UROP0622
   
future-extension
   
high-priority
   
low-priority
No Label
UROP0622
future-extension
high-priority
low-priority
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: M-Labs/nac3#289
There is no content yet.
Delete Branch "272-key-err-rpc-fix"

Deleting a branch is permanent. Although the deleted branch may exist for a short time before cleaning up, in most cases it CANNOT be undone. Continue?