Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[16.9] use proper assembly and package versions for FSharp.Core #10249

Merged
merged 3 commits into from
Oct 15, 2020

Conversation

brettfo
Copy link
Member

@brettfo brettfo commented Oct 12, 2020

A recent version number refactor is failing internal builds due to how we calculate those. The bug manifests itself when creating binding redirects and was rather tricky to track down, but the real issue is that we want to maintain assembly version 5.0.0.0 on FSharp.Core but update the NuGet package version to 5.0.1.

I've also added another CI step to fake an internal build with a build ID to mimic the way this failed in the signed build.

PR is draft until a full internal build passes. Internal build is good.

@brettfo brettfo force-pushed the binding-redirect branch 5 times, most recently from 9484086 to c3e230f Compare October 14, 2020 18:57
@brettfo brettfo changed the title [16.9] split out FSCoreVersion and FSCoreAssemblyVersion use proper assembly and package versions for FSharp.Core Oct 14, 2020
@brettfo brettfo force-pushed the binding-redirect branch 2 times, most recently from 70a199a to 76443a7 Compare October 14, 2020 19:05
@brettfo brettfo requested a review from TIHan October 14, 2020 23:10
@brettfo brettfo marked this pull request as ready for review October 14, 2020 23:11
@brettfo brettfo changed the title use proper assembly and package versions for FSharp.Core [16.9] use proper assembly and package versions for FSharp.Core Oct 14, 2020
@brettfo brettfo merged commit 052aeba into dotnet:release/dev16.9 Oct 15, 2020
@brettfo brettfo deleted the binding-redirect branch October 15, 2020 03:12
nosami pushed a commit to xamarin/visualfsharp that referenced this pull request Feb 23, 2021
…et#10249)

* add PR build to mock official build

* fix assembly and package versions

* improve build error message
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