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

Update dependencies and add net452 support. #379

Merged
merged 2 commits into from
Feb 3, 2020

Conversation

gdziadkiewicz
Copy link
Collaborator

Why?

I want to use FSharpx.Extras in my legacy net452 project.

How?

I restored net45 support to FSharpx.Collections (support was dropped in 2.0.0) and now I'm able to build FSharpx.Extras for net452 (FSharpx.Async works on net452).

@gdziadkiewicz gdziadkiewicz changed the title Update dependencies and add net452 support. WIP Update dependencies and add net452 support. Nov 22, 2019
@gdziadkiewicz gdziadkiewicz force-pushed the feature/EnableNet452 branch 3 times, most recently from 846cfe9 to 65080f0 Compare December 30, 2019 21:32
@gdziadkiewicz gdziadkiewicz changed the title WIP Update dependencies and add net452 support. Update dependencies and add net452 support. Dec 30, 2019
@gdziadkiewicz
Copy link
Collaborator Author

@panesofglass Can you take a look?

"isRoot": true,
"tools": {
"fake-cli": {
"version": "5.19.0",
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Decided to go with this approach due to error described in fsprojects/FAKE#2447 .

]
},
"paket": {
"version": "5.226.0",
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

and used the opportunity to change paket approach to the prefered one https://fsprojects.github.io/Paket/get-started.html#NET-Core-preferred

dotnet: 2.1.300
sudo: false # use the new container-based Travis infrastructure
mono: none
dotnet: 3.1.100
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Required for local tools

@@ -15,12 +17,12 @@ group Build

group Test
source https://api.nuget.org/v3/index.json
framework: netcoreapp21
framework: net46, netcoreapp21
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I decided to go with net46 as I didn't want to downgrade fsunit version.

<Optimize>true</Optimize>
<Tailcalls>true</Tailcalls>
<GenerateDocumentationFile>true</GenerateDocumentationFile>
<AutoGenerateBindingRedirects>true</AutoGenerateBindingRedirects>
</PropertyGroup>
<PropertyGroup Condition=" '$(OS)' != 'Windows_NT' ">
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@gdziadkiewicz
Copy link
Collaborator Author

@forki @mausch @panesofglass Can you help with getting this merged and released? This will have additional benefit of solving #382 build problems.

@panesofglass panesofglass merged commit 0b68a71 into fsprojects:master Feb 3, 2020
@gdziadkiewicz gdziadkiewicz deleted the feature/EnableNet452 branch February 23, 2020 09:58
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.

2 participants