190-webmenu.patch verändern, brache Hilfe

[le_petit]

Ich versuche in der 190-webmenu.patch noch einen Button in der Weboberfläche einzufügen. Aber irgendwie funzt es nicht.

Fritzbox Fon und will das mit BOFH machen!

Ich poste mal die ersten 11 Zeilen des Patches:

diff -ruN usr/www/all/html/de/menus/menu2_fon.html.orig usr/www/all/html/de/menus/menu2_fon.html
--- usr/www/all/html/de/menus/menu2_fon.html.orig	2006-02-22 20:13:43.000000000 +0100
+++ usr/www/all/html/de/menus/menu2_fon.html	2006-03-20 00:31:34.468344704 +0100
@@ -69,5 +69,6 @@
 <? if eq $var:isWizz 1 `
 	<li class="LMenuitem" style="margin-top:24px"><img src="<? echo '$var:pfeil' ?>"> <a href="javascript:jslGoTo('first','direkt')">$var:menuAssistent</a></li>
 ` ?>
+	<li class="LMenuitem" style="margin-top:24px"><img src="<? echo $var:pfeil ?>"> <a href="/cgi-bin/dsmod_status" target="_blank"><? echo $var:menuDSMOD ?></a></li>
+	<li class="LMenuitem"><img src="<? echo $var:pfeil ?>"> <a href="/tsb.htm" target="_blank"><? echo $var:menuLCR ?></a></li>
 	<li class="LMenubottom"></li>
 </ul>

Der Fehler passiert in Step2: Modify
patching file usr/www/all/html/de/menus/menu2_fon.html
patch: **** malformed patch at line 11: </ul>

make: *** [firmware] Fehler 2

Woran liegts? Kann mir das jemand erklären?

 

[derheimi]

Steht doch eigentlich ziemlich genau in der Fehlermeldung, die Du bekommst: Deine Patchdatei ist ungültig, weil die eine bestimmte Syntax haben muss. Ok, Die Zeile 11 ist i.O., es muss also vorher was nicht stimmen.
Tja....hm... die anderen Zeilen sehen aber soweit auch gut...
Die Zahlen in der 4. Zeile, sind die wichtig?
Ja! Die sagen nämlich dem Patch-Programm in welchen Zeilen es den Patch anwenden soll. Und wenn Du da einfach ne Zeile zusätzlich hinzufügst, stimmt das natürlich nicht mehr. Folglich musst Du auch die Zahlen händisch anpassen, wenn Du den Patch schon händisch verändern willst:

@@ -69,5 +69,[B]7[/B] @@

Bedeutet so viel wie: Sehe an Zeile 69 nach und vergleiche 5 Zeilen, füge anschließend an 2 Zeilen ein (--> im geänderten File sind es dann 7 Zeilen).

Tip: Es ist viel einfacher, wenn Du die ursprüngliche Datei kopierst nach dateiname.orig und dann mit "diff -urN dateiname.orig dateiname > mein.patch" so einen Patch erzeugst.

 

[le_petit]

Bedeutet so viel wie: Sehe an Zeile 69 nach und vergleiche 5 Zeilen, füge anschließend an 2 Zeilen ein (--> im geänderten File sind es dann 7 Zeilen).

Das habe ich geschnallt und bekomme es somit hin!

Tip: Es ist viel einfacher, wenn Du die ursprüngliche Datei kopierst nach dateiname.orig und dann mit "diff -urN dateiname.orig dateiname > mein.patch" so einen Patch erzeugst.

Ich habe nun die Originale 190-webmenu.patch genommen habe davon eine 190-webmenu.patch.orig Kopie erstellt und in der 190-webmenu.patch den Link für den LCR-Button eingefügt.

Dann habe ich

diff -urN 190-webmenu.patch.orig 190-webmenu.patch > 190-webmenu.patch.test

ausgeführt. Kann ich dann diesen Testpatch als 190-webmenu.patch kompilieren lassen oder ist da in meiner Reihenfolge noch was falsch?

EDIT:

Ich glaube das ist wohl der falsche Weg. Mit diff vergleiche ich die Unterschiede zweier Dateien und lasse sie mir als Patch ausgeben! Nun weiß ich aber nicht, mit was ich die 190-webmenu.patch.orig für einer Datei vergleiche / was muss in der Datei in meinem Fall drin stehen, damit ich das selber nachvollziehen kann!
Diesen Patch, der dabei entsteht, kann ich dann als 190-webmenu.patch verwenden. Sehe ich das richtig?

 

[derheimi]

Deine prinzipielle Vorgehensweise ist richtig, jetzt kommt es drauf an, was Du genau machen willst?!
Willst Du einen Patch für den [originalen] Patch (1) machen oder willst Du einen Patch für die menu2_fon.html (2), also für die Ausgangsdatei, machen?

Bei (1) müsste man also zunächst den orignalen Patch patchen, und dann den so erzeugten neuen (kombinierten) Patch auf menu2_fon.html anwenden. Bei (2) kannst Du zunächst den originalen Patch auf die HTML anwenden und anschließend (wenn man es mag) auch noch den 2. Patch.
Noch dabei? Sorry - weiß grad nicht, wie ich mich besser ausdrücken kann - Muttersprache schwere Sprache

Also die zweite Variante würde mir persönlich besser gefallen, weil man dann die Patches getrennt voneinander anwenden kann (naja, stimmt nicht ganz, weil die zu ändernden Zeilen zu nah an einander liegen).

 

[olistudent]

Auf alle Fälle hat er jetzt ein Patch für den Patch gemacht. Und so funktioniert das nicht.
Du musst ein Patch von den Files erstellen und nicht vom Patch.

MfG Oliver

 

[ao]

Tip: Es ist viel einfacher, wenn Du die ursprüngliche Datei kopierst nach dateiname.orig und dann mit "diff -urN dateiname.orig dateiname > mein.patch" so einen Patch erzeugst.

Super - vielen Dank für den Tipp!

[++]
Genau das, was ich bisher nicht begriffen hatte!

 

[le_petit]

Irgendwie bin ich noch nicht ganz firm!

Also angenommen ich will einen Patch für die menu2_fon.html, also für die Ausgangsdatei, machen. Der heißt ja dann wieder 190-webmenu.patch!
Mache ich dann

diff -urN menu2_fon.html file > 190-webmenu.patch

ist das so richtig?
Was muss dann in dem file stehen, dass ich dann den DSMOD Button und den LCR Button in dem Patch habe?
Ich hoffe, ich bin auf dem richtigen Weg!

 

[derheimi]

Du nimmst die von Dir noch unveränderte(!) Ausgangsdatei und kopierst sie nach menu2_fon.html.orig. Dann editiert Du die menu2_fon.html nach Deinen Vorstellungen, so wie sie halt im fertigen zustand mal aussehen soll. Dann kommt:

diff -urN menu2_fon.html.orig menu2_fon.html > mein.patch

Und warum muss der Patch dann wieder 190-webmenu.patch heißen? Gib dem doch nen sinnvollen Namen, z.B. 191-webmenu-lcr.patch oder so?!!
Ob in dem Patch auch der DSMOD-Button mit drin ist, kommt halt drauf an, was Du in die geänderte Datei reinschreibst: steht dort der DSMOD-Button schon drin, wird der auch im Patch nicht erwähnt. Nimmst Du aber als Ausgang die ganz ganz originale Datei von AVM und schreibst die Links für beide Buttons rein, dann schon...

 

[le_petit]

Ah, nun hab ichs! Danke für die Erklärung! Werde es mal probieren!