-
Notifications
You must be signed in to change notification settings - Fork 63
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Explicitly pin the mir-json version that SAW requires (#2115)
SAW's MIR backend requires a particular MIR JSON schema, but it is not entirely obvious which version of the JSON schema to use (#2111). This patch is one step towards addressing this concern. It: * Adds `mir-json` as a submodule. At present, nothing in the repo (CI or otherwise) actually _builds_ this submodule. Its presence is purely to communicate which version of `mir-json` must be used to compile Rust code to JSON that SAW can ingest. * Documents this in the `README`. In the future, we will want to actually build and use `mir-json` in the CI (see #1868 for an in-progress attempt at this), but in the meantime, this is a decent first step. Until we actually start building `mir-json` in the CI and using it, we will need to remember to bump the `mir-json` submodule each time that SAW's JSON schema requirement changes. Addresses one part of #2111.
- Loading branch information
1 parent
30606a2
commit 2684385
Showing
5 changed files
with
74 additions
and
3 deletions.
There are no files selected for viewing
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
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
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
Binary file not shown.