Skip to content

Conversation

@acioc
Copy link

@acioc acioc commented Mar 12, 2020

Description of changes:
As a result of dafny-lang/dafny#559 we need to add 2 additional dependencies if we are running the latest version of Dafny locally. This will likely be reverted once we add them to dafny-lang/dafny.msbuild#10 and update our version of dafny.msbuild to use the latest version

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@acioc acioc requested review from lavaleri and robin-aws March 12, 2020 22:37
@lavaleri
Copy link
Contributor

AWS CodeBuild CI Report

  • CodeBuild project: DafnyESDK
  • Commit ID: 44a7f16
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

Copy link
Contributor

@lavaleri lavaleri left a comment

Choose a reason for hiding this comment

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

LGTM. Can you get one more person to sanity check that this update unblocks the local build instructions?

@acioc acioc requested a review from MatthewBennington March 12, 2020 23:04
Copy link
Contributor

@lavaleri lavaleri left a comment

Choose a reason for hiding this comment

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

Taking another look, we shouldn't need to add System.Runtime to fix our local build. System.Collections.Immutable should be enough.

Copy link

@MatthewBennington MatthewBennington left a comment

Choose a reason for hiding this comment

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

I'm not sure we should include fixes for versions of Dafny we aren't technically building with. I'm fine with this change (pending my one comment), but once Dafny gets onto real releases, we shouldn't do this.

@acioc
Copy link
Author

acioc commented Mar 13, 2020

Taking another look, we shouldn't need to add System.Runtime to fix our local build. System.Collections.Immutable should be enough.

I ran into a number of issues with Dafny when didn't have System.Runtime. Since this is just for our team though and it works without, I'll remove the extra line for now.

@acioc
Copy link
Author

acioc commented Mar 13, 2020

I'm not sure we should include fixes for versions of Dafny we aren't technically building with. I'm fine with this change (pending my one comment), but once Dafny gets onto real releases, we shouldn't do this.

I agree! I also think we'll want to change our Dafny instructions so you don't need to pull down Dafny and build it in order to get everything working.

@lavaleri
Copy link
Contributor

AWS CodeBuild CI Report

  • CodeBuild project: DafnyESDK
  • Commit ID: 9be256d
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@acioc acioc merged commit 97681d4 into aws:develop Mar 13, 2020
@acioc acioc deleted the new-deps branch March 13, 2020 16:32
josecorella added a commit to josecorella/aws-encryption-sdk-dafny that referenced this pull request Oct 11, 2023
josecorella added a commit that referenced this pull request Oct 11, 2023
josecorella added a commit that referenced this pull request Oct 11, 2023
josecorella added a commit that referenced this pull request Oct 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants