Remove check for uClibc (FB6591 uses glibc)

[fesc2000]

Not sure if that check had a specific purpose, but it fails on the 6591 which uses glibc.

[PeterPawn]

Eigentlich sollte die Prüfung nur verhindern, daß irgendjemand das Ganze für den Build-Host übersetzt, denn da kann es (schon mangels Urlader-Environment) nicht funktionieren, obwohl es sich vermutlich trotzdem übersetzen läßt. Ich hatte ursprünglich Bedenken im Hinblick auf Mißverständnisse, weil es zusammen mit "decoder" entstand (in der derzeitigen Form) und - anders als das andere Projekt - eben außerhalb der Box nichts bringt.

Findet man vielleicht eine zusätzliche Bedingung, die man bei der 6591 (und ggf. auch anderen künftigen Modellen, wenn AVM auch dort zur glibc wechseln sollte) einbauen könnte?

Klar, die Beschränkung auf die uClibc stellt nicht wirklich sicher, daß es sich um eine FRITZ!Box handelt - es war der bequemste Weg, die Übersetzung für den Build-Host zu verhindern, wenn jemand "den Verwendungszweck" falsch interpretiert.

Wenn das ohnehin unwahrscheinlich ist und einem nichts besseres zur "Erkennung" der 6591 einfällt (vielleicht auch einfach eine -D-Option mit dem FRITZ!Box-Modell beim Kompilieren?), kann sie sicherlich auch komplett entfallen.

Ich hoffe mal, es ist nicht eilig ... bis auf weiteres wirst Du wohl der Einzige sein, der auf seiner 6591 eigene Programme laufen läßt. 😎

[fesc2000]

Nein, gar nicht eilig/wichtig, ich patches es lokal momentan eh weg. Man kann auch ein #if !defined(__UCLIBC__) && !defined(CROSS_COMPILE) draus machen, dann bleibts kompatibel und man setzt CROSS_COMPILE ueber CFLAGS, wenn als make parameter gegeben. Uebrigens, zumindest bei gcc 7.4.0 gibt es folgende Warnung: pkpwd.c:25:30: warning: ‘-Wfatal-errors’ is not an option that controls warnings [-Wpragmas] #pragma GCC diagnostic error "-Wfatal-errors" ^~~~~~~~~~~~~~~~ ich weiss nicht welche gcc version Du verwendetst und ob/wann sich das irgendwan aendert oder geaendert hat. Gruss

…

YourFritz ***@***.***> hat am 20. August 2019 um 10:59 geschrieben: Eigentlich sollte die Prüfung nur verhindern, daß irgendjemand das Ganze für den Build-Host übersetzt, denn da kann es (schon mangels Urlader-Environment) nicht funktionieren, obwohl es sich vermutlich trotzdem übersetzen läßt. Ich hatte ursprünglich Bedenken im Hinblick auf Mißverständnisse, weil es zusammen mit "decoder" entstand (in der derzeitigen Form) und - anders als das andere Projekt - eben außerhalb der Box nichts bringt. Findet man vielleicht eine zusätzliche Bedingung, die man bei der 6591 (und ggf. auch anderen künftigen Modellen, wenn AVM auch dort zur glibc wechseln sollte) einbauen könnte? Klar, die Beschränkung auf die uClibc stellt nicht wirklich sicher, daß es sich um eine FRITZ!Box handelt - es war der bequemste Weg, die Übersetzung für den Build-Host zu verhindern, wenn jemand "den Verwendungszweck" falsch interpretiert. Wenn das ohnehin unwahrscheinlich ist und einem nichts besseres zur "Erkennung" der 6591 einfällt (vielleicht auch einfach eine `-D`-Option mit dem FRITZ!Box-Modell beim Kompilieren?), kann sie sicherlich auch komplett entfallen. Ich hoffe mal, es ist nicht eilig ... bis auf weiteres wirst Du wohl der Einzige sein, der auf seiner 6591 eigene Programme laufen läßt. 😎 -- You are receiving this because you authored the thread. Reply to this email directly or view it on GitHub: #3 (comment)