[Erledigt] etherwake compatibility to busybox 1.18.x

Kreuzuebersetzer, Diskussion über Änderungen im Tuxbox-CDK und Tuxbox-CVS
GetAway
Contributor
Beiträge: 1509
Registriert: Donnerstag 27. Dezember 2007, 12:59

[Erledigt] etherwake compatibility to busybox 1.18.x

Beitrag von GetAway »

Noticed by Striper for Neutrino-HD.

Der Patch benennt etherwake in ether-wake um, damit Neutrino und Enigma
ether-wake der Busybox benutzen können.

Ich habe den Patch so gestaltet, dass nach dem Einchecken der Ordner im CVS
apps/misc/tools/etherwake in apps/misc/tools/ether-wake umbenannt werden muss.

Außerdem existiert im Netz V1.09, im CVS ist 1.06

Patch: entfernt
rhabarber1848
CDK-Experte
Beiträge: 4335
Registriert: Donnerstag 3. April 2008, 14:05

Re: [Patch] etherwake compatibility to busybox 1.18.4

Beitrag von rhabarber1848 »

Wie wäre es mit einer weiteren cdk/configure-Option (sorry seife ;) ), analog zu --with-wget:

Code: Alles auswählen

--with-etherwake: replace Busybox applet etherwake with original version
denn nur eine von beiden etherwake-Versionen braucht im Image zu sein.
seife
Developer
Beiträge: 4189
Registriert: Sonntag 2. November 2003, 12:36

Re: [Patch] etherwake compatibility to busybox 1.18.4

Beitrag von seife »

Im CVS kann man keine Ordner umbenennen.
Das sollte auch nicht notwendig sein. Ich würde einfach nur das binary nach der installation umbenennen.
GetAway
Contributor
Beiträge: 1509
Registriert: Donnerstag 27. Dezember 2007, 12:59

Re: [Patch] etherwake compatibility to busybox 1.18.4

Beitrag von GetAway »

rhabarber1848 hat geschrieben:Wie wäre es mit einer weiteren cdk/configure-Option (sorry seife ;) ), analog zu --with-wget:

Code: Alles auswählen

--with-etherwake: replace Busybox applet etherwake with original version
denn nur eine von beiden etherwake-Versionen braucht im Image zu sein.
Daran dachte ich auch, war mir nur zu viel Arbeit. Du kannst das eh besser. :D
seife hat geschrieben:Im CVS kann man keine Ordner umbenennen.
Das sollte auch nicht notwendig sein. Ich würde einfach nur das binary nach der installation umbenennen.
Das man die Ordner nicht umbenennen kann, wußte ich nicht. Dann muss in dem Patch nur ein Makefile.am
geändert werden, damit auf den alten Ordner beim Bauen zugegriffen wird.
seife
Developer
Beiträge: 4189
Registriert: Sonntag 2. November 2003, 12:36

Re: [Patch] etherwake compatibility to busybox 1.18.4

Beitrag von seife »

Im CVS gibt es nur "Löschen" => "Neuanlegen". Natürlich mit Verlust der History

Ausserdem wär's gelogen: es ist der etherwake sourcecode und das etherwake binary :)
Drum würde ich einfach einen symlink ether-wake -> etherwake anlegen und gut.
GetAway
Contributor
Beiträge: 1509
Registriert: Donnerstag 27. Dezember 2007, 12:59

Re: [Patch] etherwake compatibility to busybox 1.18.4

Beitrag von GetAway »

Davon abgesehen das du mit dem Verlust der Histoy Recht hast, sollte etherwake eigentlich immer
schon ether-wake heißen, schau mal in den Code. Deswegen bin ich dafür das endlich mal aufzuräumen.
seife
Developer
Beiträge: 4189
Registriert: Sonntag 2. November 2003, 12:36

Re: [Patch] etherwake compatibility to busybox 1.18.4

Beitrag von seife »

Dann kann man ja den symlink weglassen :-) Das Verzeichnis würde ich trotzdem lassen wie es ist. Der Nutzen steht in keinem Verhältnis zum Schmerz ;)
rhabarber1848
CDK-Experte
Beiträge: 4335
Registriert: Donnerstag 3. April 2008, 14:05

Re: [Patch] etherwake compatibility to busybox 1.18.4

Beitrag von rhabarber1848 »

GetAway hat geschrieben:sollte etherwake eigentlich immer schon ether-wake heißen, schau mal in den Code.
Bei Debian heisst die binary etherwake:
# dpkg -L etherwake
/.
/usr
/usr/sbin
/usr/sbin/etherwake
GetAway
Contributor
Beiträge: 1509
Registriert: Donnerstag 27. Dezember 2007, 12:59

Re: [Patch] etherwake compatibility to busybox 1.18.4

Beitrag von GetAway »

Ja.
Striper
Erleuchteter
Erleuchteter
Beiträge: 625
Registriert: Samstag 8. September 2007, 16:17

Re: [Patch] etherwake compatibility to busybox 1.18.4

Beitrag von Striper »

Das Teil heisst ether-wake. Nur weil Debian immer alles krampfhaft anders machen muss, muss man das nicht zwingend nachmachen. ;)
http://linuxmanpages.com/man8/ether-wake.8.php

Ich habe das im Coolstream SVN deshalb umbenannt, weil die Busybox Version auch ether-wake heisst und diese in allen Images verwendet wird. Dort gibt es ja dieses Eigenbau Binary nicht wie im Tuxbox-CVS.
GetAway
Contributor
Beiträge: 1509
Registriert: Donnerstag 27. Dezember 2007, 12:59

Re: [Patch] etherwake compatibility to busybox 1.18.4

Beitrag von GetAway »

rhabarber1848 hat geschrieben:Wie wäre es mit einer weiteren cdk/configure-Option (sorry seife ;) ), analog zu --with-wget:

Code: Alles auswählen

--with-etherwake: replace Busybox applet etherwake with original version
denn nur eine von beiden etherwake-Versionen braucht im Image zu sein.
Neuer Patch mit cdk/configure-option --with-etherwake. Dadurch wird
das Busybox Applet durch das Binary ersetzt.

Für den Dev: CVS Ordnerstruktur bleibt jetzt erhalten.

Edit:
Patch aktualisiert, bitte nochmal neu downloaden.

Patch: etherwake compatibility to busybox 1.18.x_2011_06_30_1619.diff
seife
Developer
Beiträge: 4189
Registriert: Sonntag 2. November 2003, 12:36

Re: [Patch] etherwake compatibility to busybox 1.18.x

Beitrag von seife »

Ich wette, das hast du nicht probiert. Kann IMVHO nämlich nicht bauen (das Makefile wird in einem nicht existenten Verzeichnis erzeugt).

Ausserdem wird das y_tools.sh nicht mehr funktionieren, wenn jemand ein ether-wake nach /var/bin kopiert.
GetAway
Contributor
Beiträge: 1509
Registriert: Donnerstag 27. Dezember 2007, 12:59

Re: [Patch] etherwake compatibility to busybox 1.18.x

Beitrag von GetAway »

Jo, hatte den alten Patch hochgeladen. Mit den y_tools.sh schau ich nochmal. Hatte mich da wohl zu sehr ans SVN gehangen.

Edit:
Patch aktualisiert.
GetAway
Contributor
Beiträge: 1509
Registriert: Donnerstag 27. Dezember 2007, 12:59

Re: [Patch] etherwake compatibility to busybox 1.18.x

Beitrag von GetAway »

Hi,

wenn es an diesem Patch nichts mehr auszusetzen gibt, könnte er ins CVS. Sollte ich noch etwas
vergessen haben, dann bitte darauf hinweisen und ich überarbeite das.
rhabarber1848
CDK-Experte
Beiträge: 4335
Registriert: Donnerstag 3. April 2008, 14:05

Re: [Patch] etherwake compatibility to busybox 1.18.x

Beitrag von rhabarber1848 »

GetAway hat geschrieben:Sollte ich noch etwas vergessen haben, dann bitte darauf hinweisen und ich überarbeite das.
Ich teste den Patch nochmals; keine Sorge, ist nicht untergegangen ;)
rhabarber1848
CDK-Experte
Beiträge: 4335
Registriert: Donnerstag 3. April 2008, 14:05

Re: [Patch] etherwake compatibility to busybox 1.18.x

Beitrag von rhabarber1848 »

Kleine Anmerkung zum Patch, dieser Teil kann entfernt werden:

Code: Alles auswählen

--- a/cdk/configure.ac<>19 Jun 2011 11:57:18 -0000 1.283
+++ b/cdk/configure.ac<>30 Jun 2011 14:28:47 -0000
@@ -1471,6 +1476,7 @@ TUXBOX_RULES_MAKE(vdr)                                                                                                
 TUXBOX_RULES_MAKE(lufs)
 TUXBOX_RULES_MAKE(dropbear)
 TUXBOX_RULES_MAKE(kermit)
+TUXBOX_RULES_MAKE(etherwake)
 TUXBOX_RULES_MAKE(wget)
 TUXBOX_RULES_MAKE(ncftp)
 TUXBOX_RULES_MAKE(screen)
Dieser Teil ist nicht nötig, da etherwake bereits im Tuxbox-CVS enthalten ist, Warnung bei cdk/configure:
checking etherwake rules... Use of uninitialized value in split at ./rules.pl line 45.
Use of uninitialized value $output in concatenation (.) or string at ./rules.pl line 79.
Use of uninitialized value in concatenation (.) or string at ./rules.pl line 230.
Use of uninitialized value $dir in concatenation (.) or string at ./rules.pl line 93.
Use of uninitialized value $dir in concatenation (.) or string at ./rules.pl line 200.
Use of uninitialized value in concatenation (.) or string at ./rules.pl line 230.
done
GetAway
Contributor
Beiträge: 1509
Registriert: Donnerstag 27. Dezember 2007, 12:59

Re: [Patch] etherwake compatibility to busybox 1.18.x

Beitrag von GetAway »

OK, hab's kapiert. Bei wget wird es noch nachgeladen. Daran hatte ich mich orientiert. Danke.
Dafür brauchst aber jetzt keinen neuen Patch?
rhabarber1848
CDK-Experte
Beiträge: 4335
Registriert: Donnerstag 3. April 2008, 14:05

Re: [Patch] etherwake compatibility to busybox 1.18.x

Beitrag von rhabarber1848 »

GetAway hat geschrieben:Dafür brauchst aber jetzt keinen neuen Patch?
Richtig, ich habe es lokal angepasst, weitere Tests stehen noch aus...
Momentan steht VLC bei mir im Mittelpunkt ;)
rhabarber1848
CDK-Experte
Beiträge: 4335
Registriert: Donnerstag 3. April 2008, 14:05

Re: [Erledigt] etherwake compatibility to busybox 1.18.x

Beitrag von rhabarber1848 »

Ein kleines Problem existiert noch mit Yadd-Images, diese beinhalten immer das make target
misc_tools, welches immer die stand-alone-Version von etherwake installiert. Darüberhinaus
gibt es kein make target namens "etherwake".
Folgender Patch berücksichtigt cdk/configure --with-etherwake auch für Yadd-Images:
http://article.gmane.org/gmane.comp.vid ... x.scm/3107
http://article.gmane.org/gmane.comp.vid ... x.scm/3108