Porting filecoin-ffi to win32 #357
Draft
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…-update Update FVM (3.0.0-alpha.16) and ipld_encoding (0.3.2)
…ate-fvm Update to FVM 3.0.0-alpha.17
This means we don't construct a v3 config at the top level, then try to convert it to a v2 config. When we do that, we run into issues if, e.g., v3 doesn't define a price-list for a version only supported by v2. This PR also has a bit of cleanup: 1. Removes the manifest CID. 2. Changes the network version from u64 to u32 (simplifies some code). 3. Deletes all the overlay blockstore logic (was only used if the manifest was explicitly passed). Actor "overrides" should now only be done via "redirects".
…i-config feat: switch to an FFI-level config
SC2006: Use $(...) notation instead of legacy backticked `...`.
SC2086: Double quote to prevent globbing and word splitting.
Linux and macOS don't have pc in their `target triple`.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Features
I am looking for someone to test it with Cuda on Windows / Msys2.
Related issues
#291