busybox-tools CleanUp

[dileks]

In this ticket I would like to collect all-about busybox-tools cleanup (starting with r7718).

A little bit of documentation would appreciated, not only in the SCM comments?
Personally, I would like to see like in Linux-kernel development a "Documentation" folder in freetz root-dir.
What about a "Documentation/busybox-cleanup.txt"? For example: As far as I can see, the decompressors are now handled via busybox (short: bb).

In r7720 I see a little mistake in fwmod:

4	TAR_TOOL="tar"
174	UNTAR="$TAR"

Should be UNTAR=""${TOOLS_DIR}/${TAR_TOOL}"?

File ".build-prerequisites" could be listed/sorted alphabetically for better human-readability?

Eine Datei als Dokumentation zu einem Vorgang, der (hoffentlich) bald abgeschlossen ist? Da finde ich das Ticket-System doch sinnvoller.

Dokumentation ist sinnvoller um Dinge zu beschreiben, die vermutlich auch länger so bleiben, ob es jetzt in einem Verzeichnis Documentation oder anderswo steht. Das Wiki ist z.B. auch ein geeigneter Ort für Dokumentation. Da steht sogar schon einiges.

[PATCH] fwmod: Fix comment for $UNTAR

I rechecked fwmod file and was wrong…

154:TAR_TOOL="tar"
170:TAR="${TOOLS_DIR}/${TAR_TOOL}"
174:UNTAR="$TAR"

We had several problems by using de-/compressors from bb or from local system, so such a change should surely be documented, but where? README? CHANGELOG? Wiki is a wiki and can not be shipped with a final release tarball. The whole project lacks of docs (not speaking of English docs), sorry if I am too honest.

See [7725]

Welche Probleme gab es denn mit Programmen von BB? Gab es einen Bug-Report an die BB-Liste darüber?

er13:

• busybox tar: kein pax (aka POSIX 1003.1-2001) tar format unterstützt, s. r7390 (transmission-tarball kann nicht entpackt werden)
  
• busybox unzip: keine File-Permissions unterstützt, s. r7199 (cyassl-Scripte sind nach dem Entpacken nicht ausführbar)

[PATCH] fwmod: Fix comment for usage of $UNTAR (see #1535)

[PATCH] make/Makefile.in: Fix comment for usage of unzip

With r7726:

Download succeeded - "https://github.com/lacombar/kconfig/tarball/kconfig-sa-2.6.39"  ->  saved to folder "dl"
tools/freetz_download: line 205: tools/md5sum: No such file or directory
MD5 mismatch for dl/kconfig-sa-2.6.39: 609c3981f2466620ee42fe0bfa33388b

Use host's md5sum in freetz_download?

[PATCH] freetz_download: Temporarily use system's md5sum (see #1535)

md5sum is a busybox applet, maybe "make busybox-tools-dirclean" should help

This was with a fresh checkout trunk, at the point kconfig(-sa) is downloaded & checked, there is no bb built before, so its own md5sum binary can not be used

In [7727]:

• use systems md5sum if busybox is not yet build, refs #1535
• thanks dileks_ for reporting this

In [7728]:

• skip MD5 check if no md5sum is available (busybox aplet or system md5sum), refs #1535

Replying to oliver:

• busybox unzip: keine File-Permissions unterstützt, s. r7199 (cyassl-Scripte sind nach dem Entpacken nicht ausführbar)

Zumindest die File-Permissions sind da, wenn man im menuconfig DESKTOP auswählt.

In [7826]:

• use systems md5sum if busybox is not yet build, refs #1535
• see ​http://www.ip-phone-forum.de/showthread.php?t=240590&p=1764045&viewfull=1#post1764045

Replying to cuma:
Wie wäre es denn mit

PATH=$(dirname $0):$PATH
md5sum ...

Dann sucht die Shell automatisch nach md5sum in der richtigen Reihenfolge.

Wobei möglicherweise hier der Fehler darin lag, dass $(dirname $0) als Ergebnis . gelifert hat und nicht tools.

Ich hatte das einfach aus r7727 kopiert. Beim PATH erweitern weiss ich nicht was dadurch für Probleme auftauchen können. Soll eigentlich nur temporär sein, wie lange weiss ich auch nicht
Kannst du aber gerne fixen

Was ist hier noch zu tun? Den Vorschlag von Ralf einarbeiten?

Funktioniert jetzt wie es soll, deswegen von mir aus lassen wie es ist.

Den temporären Fix könnte man wieder rausnehmen

Was meinst du mit rausnehmen? Der wird doch nach wie vor benötigt, wenn wir nicht eine andere Lösung einbauen?

In [7826]: use systems md5sum if busybox is not yet build, refs #1535

Mittlerweile müsste doch jeder die BusyBox inklusive md5sum Applet neu gebaut haben

—> http://freetz.org/browser/trunk/tools/make/Config.busybox#L237

Für den busybox-tools-Download wird es doch immer so sein, dass kein busybox md5-Applet da ist?

Da ist was dran. Dann stimmt der kommentar im Quellcode aber nicht. Ich (und Post im IPPF) meinte das mehr im Sinne von kein change_check → alte HostBusyBox? ohne md5sum

In [8661]:

fix comment for md5sum (closes #1535)