Skip to content
This repository has been archived by the owner on Nov 27, 2023. It is now read-only.

Migrate the oasis-core ledger integration to this repository #16

Merged
merged 5 commits into from
Jul 24, 2020

Conversation

Yawning
Copy link
Contributor

@Yawning Yawning commented Jul 23, 2020

  • Clean up the existing code under the internal package
  • Migrate the signer into a go plugin
  • Migrate the tool to enumerate devices to a command

Notes:

  • We set more than -trimpath when building oasis-node, I'm not sure if that will make the runtime library's plugin loader mad if we don't set the same options when building the plugin.
  • Someone that's not me will need to bump the oasis-core import to the exact version the plugin is supposed to be built against.

@Yawning Yawning force-pushed the yawning/feature/crawling-in-my-skin branch 8 times, most recently from a4b77a3 to 413a802 Compare July 23, 2020 13:40
@Yawning Yawning marked this pull request as ready for review July 23, 2020 13:48
@Yawning Yawning force-pushed the yawning/feature/crawling-in-my-skin branch from 6e650fd to a634ffb Compare July 23, 2020 14:48
tjanez and others added 5 commits July 23, 2020 14:49
This work-arounds an issue where wrong a wrong commit is being checked
out.
 * Improve the common tests
  * Match the oasis-core test style
  * Use `testing.T.Logf`
  * Add more checks for the chunking routine
 * Refactor the chunk generator
 * Make a lot of things private
 * Changed error returns to match oasis-core conventions
 * Use structured logging
 * Other misc cleanups

The tests under `app_test.go` could also use similar improvements,
however I am reluctant to alter test cases I have no way of actually
running to completion.
@Yawning Yawning force-pushed the yawning/feature/crawling-in-my-skin branch from a634ffb to b5fc2b3 Compare July 23, 2020 14:49
Copy link
Member

@tjanez tjanez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Someone that's not me will need to bump the oasis-core import to the exact version the plugin is supposed to be built against.

Yes, we can do that later.

@Yawning Yawning merged commit 3feacef into master Jul 24, 2020
@Yawning Yawning deleted the yawning/feature/crawling-in-my-skin branch July 24, 2020 09:52
tjanez added a commit that referenced this pull request Jul 24, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants