-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Framework: .net add armada370 to unsupported architectures #4528
Conversation
Thanks. Does this need adding to the new radarr / lidarr packages too? I had a quick look and Google suggested that the other armada architectures might be ok. |
Yea I just quickly opened a PR to discuss the issue more openly (rather than deep down an old PR) and so that I don't forget. I need to sleep but will get to it soon. I know it's like a one line change, but we need to test this a bit. Most code just uses the nice generic architecture groupings. Thanks I was hoping some other users/devs might have other armada architectures to test on. But I suspect you are right and that it's the only one. Just want to keep this open for a short while just to see. cc/ @hgy59 |
Sounds good, just wanted to mention it while I remembered. FTR I found some output of cpuinfo here: shirou/gopsutil#881 (comment) It looks like
I think anything I think that just leaves |
I installed Radarr today and landed here after looking what was going on. I get that the CPU is not supported by .NET core, but it seems the Radarr version available for this architecture is not reflecting that. I could get Radarr working by manually installing https://packages.synocommunity.com/radarr/13/radarr.v13.f15047%5Barmada370-armada375-armada38x-alpine-alpine4k-armadaxp-comcerto2k-monaco%5D.spk, but it would be good to avoid updating to the unsupported version again via the package manager.
|
@publicarray Actually, could it be that there is actually a way to make .NET core work for this type of architecture?! |
I think radarr is dead on armada370 to be honest. Right now I don't think we'd want to commit to supporting a custom build of .net for it. |
OK fair enough. Then I guess this can be merged? |
@piejanssens I have tried compiling a version of dotnet that should support the armada370. Would you be able to help me test it? |
@ta264 Yes |
Can you come by the radarr discord channel please? https://radarr.video/discord |
@hgy59 @piejanssens I may have a solution to this. Please could you test the radarr package from the run here: https://github.com/Servarr/spksrc/suites/6540598550/artifacts/244139785 |
I don't have access to the DS213J at this point, but I'll test within a few days. |
@ta264 It crashes once I access its webpage:
|
@piejanssens Please could you join the Radarr discord so we can try to narrow it down? https://radarr.video/discord |
@mreid-tt this is another topic related to your work on cleaning up the dotnet core dependent packages. |
Superseded by #5680 |
Motivation: From a user report ds213j returns
Illegal instruction (core dumped)
#4464 (comment). Other armada architectures might also be affected. Thanks @ta264 for the report and analysisLinked issues: #4464 (comment)
Checklist
all-supported
completed successfully