Skip to content

Update the .NET SDK in the CI builds to 6.0.300#2780

Closed
Numpsy wants to merge 1 commit into
fsprojects:masterfrom
Numpsy:603_sdk
Closed

Update the .NET SDK in the CI builds to 6.0.300#2780
Numpsy wants to merge 1 commit into
fsprojects:masterfrom
Numpsy:603_sdk

Conversation

@Numpsy

@Numpsy Numpsy commented Jun 16, 2024

Copy link
Copy Markdown
Contributor

Description

A minimal update to the .NET 6 SDK version, to see if it fixes the macOS/ARM CI failures discussed in #2778

@Numpsy

Numpsy commented Jun 16, 2024

Copy link
Copy Markdown
Contributor Author

Hmm, the macOS CI failed and the built output contains

Unhandled exception. System.Reflection.TargetInvocationException: Exception has been thrown by the target of an invocation.
 ---> System.TypeInitializationException: The type initializer for '<StartupCode$Fake-DotNet-Cli-IntegrationTests>.$Fake.DotNet.Cli.IntegrationTests.TemplateTests' threw an exception.
 ---> System.DllNotFoundException: Unable to load shared library 'MonoPosixHelper' or one of its dependencies. In order to help diagnose loading problems, consider setting the DYLD_PRINT_LIBRARIES environment variable: dlopen(libMonoPosixHelper, 0x0001): tried: 'libMonoPosixHelper' (no such file), '/System/Volumes/Preboot/Cryptexes/OSlibMonoPosixHelper' (no such file), '/usr/lib/libMonoPosixHelper' (no such file, not in dyld cache), 'libMonoPosixHelper' (no such file), '/usr/local/lib/libMonoPosixHelper' (no such file), '/usr/lib/libMonoPosixHelper' (no such file, not in dyld cache)

So now I wonder if the old version of Mono.Posix it's using maybe only has Intel CPU support?

@Numpsy

Numpsy commented Jun 17, 2024

Copy link
Copy Markdown
Contributor Author

There are some integration tests in https://github.com/fsprojects/FAKE/blob/master/src/test/Fake.Core.IntegrationTests/Fake.DotNet.sdkAssemblyResolver.fs which are doing explicit things with the 6.0.10X SDK, and I start to wonder if that simply isn't going to work on ARM macs?

@Thorium

Thorium commented Jul 15, 2024

Copy link
Copy Markdown
Member

This could help the CI failure: #2783

@xperiandri
xperiandri enabled auto-merge (rebase) July 25, 2024 11:01
@xperiandri

Copy link
Copy Markdown
Collaborator

But it is already rebased

@Thorium

Thorium commented Jul 25, 2024

Copy link
Copy Markdown
Member

@Numpsy

Numpsy commented Jul 25, 2024

Copy link
Copy Markdown
Contributor Author

I tried updating some of those in https://github.com/Numpsy/FAKE/commits/6.0.131/ a while ago and hit some other failures.
At this point, I don't know if it's worth spending time updating the 6.0 SDKs or if it's better to put the effort into switching everything to .NET 8.0 before 6.0 goes out of support.

@xperiandri

Copy link
Copy Markdown
Collaborator

Probably 8 is better

@Numpsy

Numpsy commented Jul 30, 2024

Copy link
Copy Markdown
Contributor Author

The build at https://github.com/Numpsy/FAKE/actions/runs/10161143249 from the branch at https://github.com/Numpsy/FAKE/tree/bump_flail seems to have passed with the 6.0.30x SDK, though there are still a couple of changes n top of the SDK update in there

@xperiandri xperiandri closed this Jul 31, 2024
auto-merge was automatically disabled July 31, 2024 00:05

Pull request was closed

@Numpsy
Numpsy deleted the 603_sdk branch July 31, 2024 10:00
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