fsprojects / FSharp.Control.Reactive

Extensions and wrappers for using Reactive Extensions (Rx) with F#.
http://fsprojects.github.io/FSharp.Control.Reactive
Other
284 stars 60 forks source link

Release with support for System.Reactive >= 6.0. #175

Open OkkeHendriks opened 8 months ago

OkkeHendriks commented 8 months ago

Description

@cloudRoutine @deviousasti @panesofglass

Can one of you please release a new version with support for System.Reactive >= 6.0. There are quite a lot of people waiting on this, see:

pkese commented 7 months ago

+1, having the same issue here


error NU1608: Warning As Error: Detected package version outside of dependency constraint:
FSharp.Control.Reactive 5.0.5 requires System.Reactive (>= 5.0.0 && < 6.0.0)
but version System.Reactive 6.0.0 was resolved.```
deviousasti commented 7 months ago

Let me try unfreezing the pipelines

deviousasti commented 7 months ago

The CI is complaining, MSBuild gripes it seems.

OkkeHendriks commented 7 months ago

Hmm, is there something someone can do about that? 😬

baronfel commented 6 months ago

I can also help out here, would love a 6.0-compatible release.

zhusihan-python commented 4 months ago

same issue here:

TrackTime -> MessageBox.Avalonia 3.1.5.1 -> DialogHost.Avalonia 0.7.7 -> System.Reactive (>= 6.0.0) 
TrackTime -> FSharp.Control.Reactive 5.0.5 -> System.Reactive (>= 5.0.0 && < 6.0.0).    TrackTime
OkkeHendriks commented 4 months ago

I was looking at the build status, the build fails since this commit (but that of course might be a coincidence).

The build log from before that commit, and the log from that commit.

What immediately catches my eye is the msbuild version steps back from Microsoft (R) Build Engine version 17.1.0+ae57d105c for .NET to Microsoft (R) Build Engine version 14.0.25420.1.

But maybe more interesting, the whole build seems to be different. When still working, .\build.ps1 is invoked but when no longer working msbuild is invoked directly? I do not see this reflected in the commit history? Is this configured externally somewhere?

Side note: I cannot get this to build locally using visual studio, or by running ./build.ps1, any instructions for that?

tmoers commented 3 months ago

Any news on this one? I'd gladly help if someone could explain me what to do. I also tried to build locally, but failed to.

tmoers commented 3 months ago

Disclaimer: I know nothing about paket, I'm just trying to get the build working without knowing what I'm doing. Just to get the ball rolling.

@xperiandri The build broke when you added the groups to the paket.references file:

group Net472_or_less
    System.Reactive
    Microsoft.NETCore.Platforms
[snip]

And now I get a bunch of errors like the following when I run dotnet paket install:

Paket failed with
-> Installation Errors :
    - C:\temp\FSharp.Control.Reactive\src\FSharp.Control.Reactive\paket.references uses the group Net472_or_less, but this group was not found in paket.lock.
    - C:\temp\FSharp.Control.Reactive\src\FSharp.Control.Reactive\paket.references uses NuGet package System.Reactive, but it was not found in the paket.lock file in group Net472_or_less.
   However, System.Reactive was found in group Build.
[snip]

When I remove the groups from paket.references, I can run paket install without problems and build.ps1 succeeds.

Alternatively, I can also add the groups to paket.dependencies each with their specific entry for the referenced packages:

group Net472_or_less
    source https://api.nuget.org/v3/index.json
    nuget System.Reactive >= 5.0
    nuget Microsoft.NETCore.Platforms >= 5.0

group NetStandard2_0_or_less
    source https://api.nuget.org/v3/index.json
    nuget System.Reactive >= 5.0
    nuget Microsoft.NETCore.Platforms >= 5.0

<etc>

group Net6_0
    source https://api.nuget.org/v3/index.json
    nuget System.Reactive >= 5.0

As I said, above I know nothing about paket, so I'm unsure how to proceed.

xperiandri commented 3 months ago

Try paket install —f

xperiandri commented 3 months ago

Maybe your solution is correct. Mine worked at the time of PR

marklam commented 2 months ago

I've created a PR #177 that simplifies the packet files & updates the tooling and appveyor build spec.

It seems to build fine from dotnet build, dotnet fake build, and appveyor.

deviousasti commented 2 months ago

I merged in #177 and tried retriggering AppVeyor, but the build is still broken - I don't think I can fix it because it's in @panesofglass account. The build works fine for both @marklam and me (https://ci.appveyor.com/project/deviousasti/fsharp-control-reactive-hj870).

We could probably set up Github Actions - let's get shared ownership of that Nuget package.

xperiandri commented 2 months ago

@sergey-tihon do you have access to it? Can you help?

sergey-tihon commented 2 months ago

FSharp.Control.Reactive is part of fsprojects org, so I can create a Nuget token with the right to push new packages and add to this repo (for GitHub Actions)

marklam commented 2 months ago

I can look at a github action which will version the package correctly for publishing it (version 5.0.6)

sergey-tihon commented 2 months ago

A new repository key added to this repo image

with the right to push new versions of the package image

deviousasti commented 2 months ago

@sergey-tihon Could you give me maintainer access to this repo so I can set up Actions? I have merge permissions, but I can't really see anything in the settings page.

marklam commented 2 months ago

I've created a pr #179 that might (?) help with the action definitions

sergey-tihon commented 2 months ago

@deviousasti done

samritchie commented 1 day ago

Is this waiting for #180? What else needs to happen?