fluidd-core / fluidd

Fluidd, the klipper UI.
https://docs.fluidd.xyz
GNU General Public License v3.0
1.39k stars 424 forks source link

Incorrect System Information (Distribution) for Armbian #1385

Closed halfmanbear closed 6 months ago

halfmanbear commented 7 months ago

Fluidd Version

v1.28.1

Browser

Chrome

Device

Laptop

Operating System

macOS

What happened

Distribution value incorrect from DistroInfo,

Screenshot 2024-03-05 at 12 12 58

What did you expect to happen

Screenshot 2024-03-05 at 12 22 28

How to reproduce

armbian-release files always start with # PLEASE DO NOT EDIT THIS FILE

mks@znp-k1:~$ cat /etc/armbian-release
# PLEASE DO NOT EDIT THIS FILE
BOARD=znp-k1
BOARD_NAME="ZNP-K1"
BOARDFAMILY=rockchip64
BUILD_REPOSITORY_URL=https://github.com/halfmanbear/Armbian-ZNP-K1-build.git
BUILD_REPOSITORY_COMMIT=9e04b40
LINUXFAMILY=rockchip64
ARCH=arm64
IMAGE_TYPE=user-built
BOARD_TYPE=conf
INITRD_ARCH=arm64
KERNEL_IMAGE_TYPE=Image
FORCE_BOOTSCRIPT_UPDATE=
VENDOR="Armbian-unofficial"
VENDORDOCS="https://docs.armbian.com/"
VENDORURL="https://duckduckgo.com/"
VENDORSUPPORT="https://community.armbian.com/"
VENDORBUGS="https://armbian.atlassian.net/"
BOOTSCRIPT_FORCE_UPDATE="no"
BOOTSCRIPT_DST="boot.cmd"
VERSION=24.2.0-trunk
REVISION=24.2.0-trunk
BRANCH=current
mks@znp-k1:~$ uname -a
Linux znp-k1 6.1.67-current-rockchip64 #2 SMP PREEMPT Mon Dec 11 09:41:01 GMT 2023 aarch64 GNU/Linux
mks@znp-k1:~$ sudo lsb_release -a
No LSB modules are available.
Distributor ID: Debian
Description:    Armbian-unofficial 24.2.0-trunk bookworm
Release:    12
Codename:   bookworm

Additional information

https://github.com/mainsail-crew/mainsail/pull/1021

pedrolamas commented 7 months ago

Hi @halfmanbear, thank you for opening this ticket.

Can you please check and confirm what version of Moonraker you have installed? It should be visible on Fluidd Settings, in the Updates section.

halfmanbear commented 7 months ago

Yes I'm currently running v0.8.0-319-gf0ae5179

pedrolamas commented 7 months ago

Yes I'm currently running v0.8.0-319-gf0ae5179

By any chance are you running a fork of Moonraker? I can't find that version on the official Moonraker repository...

I would expect the suffix gf0ae5179 to match a commit in https://github.com/Arksine/moonraker, but I can't see any matching

matmen commented 7 months ago

I think the g isn't supposed to be there (it's not a hex character) - I assume it's the latest commit https://github.com/Arksine/moonraker/commit/f0ae5179

pedrolamas commented 7 months ago

This is indeed a bug, though one could argue that the problem is actually upstream in Moonraker, or even on with the distro dependency that it then uses to retrieve the distribution values.

Having said that, I too experienced this issue in the past with the BTT CB1 and even submitted a fix to distro repository that was merged and released in version 1.9.0 - at the time this fix was enough for the CB1, but might not be the case with other Armbian releases!

pedrolamas commented 7 months ago

FWIW, I've just pushed a pull request upstream to the distro repository to have this fixed directly there.

pedrolamas commented 6 months ago

I don't see this getting fixed upstream any time soon, so I've now pushed a fix on our side.