Conversation
|
Thanks @spencercw |
|
(I am having problems with the IT service provider that provides the build system I use for CI - need to fix this over the next few days for the CI to run) |
|
The mono 10 builds seem to be failing? |
|
Ah, apologies, I had not tested the It would appear the |
|
No idea I am afraid - I would go with "oversight" ! |
|
So the build error on 'build-and-test (8.0.406, 6.12.0.206, styhead, arm)': It's trying to use the .NET 10 files even though we are building with .NET 8. I had an issue like this while testing this change and switching between |
|
Any thoughts on why this is failing? |
|
I've been looking into this a bit more. Essentially what happens is that when the version is switched, empty directories get left behind in The cleanup happens in The Hopefully that makes some kind of sense. It's confusing, and the bitbake behaviour around version switches seems a bit buggy. I think we should just clean the |
|
Thanks for looking at this @spencercw - it is a bit confusing yes. I did think we had a cleaning step in the CI. Aha I see I have that commented out - shall we add that back in? meta-mono/.github/workflows/CI_github.yml Line 85 in b05f077 |
|
I have restored the cleaning step and added |
|
Great stuff thanks - will see if it builds! |
|
Very odd. A completely different pythonnet failure now |
|
I'm struggling to work out how best to deal with this. The current release version (3.0.5) of pythonnet does not support .NET 10. I might be able to backport the relevant patches from master, but that raises the minimum supported version to .NET 8, which would just leave the I'm inclined to leave pythonnet unsupported on .NET 10 until there is an official release, but I'm not sure how best to do that without leaving the CI build in a broken state. Any ideas? |
Not ideal is it but not much we can do about it either. I am inclined to agree we wait until pythonnet is supported properly rather than diving in. Which does mean yes we remove it from the CI build. Shall we remove the recipe here and make a note in an update to the README that pythonnet is not supported with .NET 10? |
|
Ok how's this? |
|
LGTM - let's see what happens (I've seen a chatter about M$ charging per minute for self hosted runners so will have to work out what these builds are costing now and whether there's a need to move to another CI offering... Any thoughts appreciated! |
|
Yeah I saw that. Mad isn't it. I think they've postponed that for the time being, but I'm sure it'll be back before long. We self-host GitLab at my job which works well enough for us. We've still got a bunch of CI stuff on Jenkins as well. |
|
yeah meta-mono was on Jenkins before I moved over to GitHub because I couldn't cope with the agony any more :-D |
|
Looks like the arm version has a dependency on liblttng-ust.so.1. Let's try this. |
|
Great we got a passing build - will let the other complete |
Well that's a new one. Any idea what that's about? |
|
No - excepting that previously and what should happen is that the CI recipes run individually. I see now they seem to be running in parallel which could be causing problems |
|
Hm, that does sound problematic. Maybe this? I'm off for Christmas now so can pick this up again next week. Merry Christmas :) |
|
ok have a good break ! |
|
Nearly there - now there's a test file missing? https://github.com/DynamicDevices/meta-mono/actions/runs/20490977722/job/58883349870#step:10:67 |
|
Urgh. The interpreter path in the binary is wrong: I've added a |
|
Any chance this can be progressed? It needs another CI run. |
|
Ah thanks for prodding me - have kicked one off |
|
10 and 8 built successfully but it seemed to run into trouble with 6 ? https://github.com/DynamicDevices/meta-mono/actions/runs/20571801153/job/61718537286?pr=268 |
This switches from the Visual Studio URLs to the .NET URLs, as per upstream [1]. [1] dotnet/core#9869
|
I don't really understand what's happening here. I'm unable to reproduce the error locally, even inside your Docker image. I have added some commands to print out some extra data which may potentially give a clue. |
|
I'm not sure why your workflows need my approval to be honest - I had things set up so I needed to approve an initial PR which seems sensible but after that it seems to me trust can be extended. Not entirely sure if this is some tightening of the GitHub rules but it's another thing on my list to look at. I was out at FOSDEM last weekend so somewhat overcome and playing catchup here but I was chatting with the Yocto folks and I am very keen to try to bring meta-mono up to follow best Yocto practices, put it through the Yocto layer compliance checks and try to achieve the compliance badge. Not least because I want to know for myself what current best practice looks like so I can follow it with my commercial hat on. So.... apologies again for the long iteration period trying to merge in your - much appreciated - PRs but I'm going to try really hard to figure out what the hell is going on with these build failures :) And, because I don't think people say this enough, thank-you for putting the effort in to push your improvements into this repo sir |
|
RIght ho - so I was following this through and I see we fail again at the same point with the 6.x build. I have a couple of thoughts on ways forward
I am thinking we say "this is no longer supported as it doesn't work in CI and has been removed, use at your own risk" If people actuallly do care they will shout and - perhaps even - fix the issue That enables us to move on and merge in your contribution. Does that make sense or am I missing something? |
|
I think we are getting these CI errors elsewhere - I am not sure why but am having a comprehensive look at what's going on the new PR |
Resolve conflict in CI_github.yml by keeping the cleansstate approach from master and removing debug ls/mkdir commands from the PR branch. Co-authored-by: Cursor <cursoragent@cursor.com>
|
I did a deep dive into trying to work out what's going on and to be honest @spencercw I'm still not entirely sure but it seemed to be related to sstate caching oddities. I've added a couple of commits which get things building, the CI is passing and I'll merge this in now. Thanks once more for your contribution. and apologies that it's taken it this long to get this in! |
|
Thanks for picking this up. I have backported this to scarthgap in #274, so now we can do this all over again 😂 |
|
Haha! Well let's hope some of these CI fixes "stick" !!! :-D |
This switches from the Visual Studio URLs to the .NET URLs, as per upstream [1].
[1] dotnet/core#9869
Note
Adds .NET 10.0.100 support, switches downloads to builds.dotnet.microsoft.com, updates packaging, and includes 10.0.100 in CI.
recipes-mono/dotnet/dotnet_10.0.100.bb(runtime10.0.0) with per-archsha512.recipes-mono/dotnet/dotnet.incSRC_URItohttps://builds.dotnet.microsoft.com/...; dropSRC_FETCH_IDusage and relatedvardeps.packsinto${datadir}/dotnet/and include inFILES:${PN}-dev.RDEPENDS:${PN}-dev = "zlib".INSANE_SKIP:${PN}-dev = "libdir staticdev dev-elf"..github/workflows/CI_github.ymlto includedotnet_version: 10.0.100alongside existing versions.Written by Cursor Bugbot for commit 94b834b. This will update automatically on new commits. Configure here.