search

PeterPawn / modfs

modify firmware for NAND-flash based FRITZ!Box routers and install it on such a device
GNU General Public License v2.0
62 stars 15 forks source link

add modscript to enable multiple pages for one fax transmission #31

Closed TheChatty closed 4 years ago

TheChatty commented 4 years ago

I had to force add the new file because due to .gitignore settings.

PeterPawn commented 4 years ago

Ich bleibe mal bei Deutsch (wenn ich mich hinsichtlich Deiner Identität nicht täusche) - ich habe im anderen PR noch einen Kommentar zu der zusätzlichen Verzeichnisebene und der Idee dahinter geschrieben ... möglicherweise war die Idee in der README.md nicht gut genug erläutert (das hatte ich im IPPF getan bei der Einführung).

Basierend auf dieser Idee war jedenfalls der vorherige PR noch schlüssiger ... das Unterverzeichnis "custom" wäre ja jetzt eines für eine solche komplette "contribution" und da wäre dann ein Name, der einen Hinweis darauf gibt, worum es dabei geht, wohl doch besser.

Das "custom" war meinerseits nur ein Beispiel für eine solches "Paket" und sollte mehr illustrieren, daß ich die Funktionsfähigkeit des Codes in Bezug auf die Verzeichnisse unterhalb von "contrib" auch im Auge habe, weil ich sie selbst (eben für dieses "customizing") benutze.

Daher paßt ausgerechnet dieser Verzeichnisname dann auch nicht für Deinen Beitrag - denn den gibt's (a) schon (wie das find dort zeigte) und (b) gibt es keinen Zusammenhang zum "Inhalt" des Patches.

Da würde ich dann eher bei der ersten Variante bleiben - immerhin gibt der Name des Patch-Files ja selbst schon einen Hinweis und da braucht es die Verzeichnisebene dann wohl doch nicht. Die dient ja u.a. auch dazu, damit da jemand auch eigene Lizenzbedingungen o.ä. ablegen kann, die er für die eigenen Beiträge berücksichtigt sehen möchte, daher auch der ausdrückliche Hinweis darauf in der README.md.

Wenn Du also den ersten PR rückgängig gemacht haben möchtest, müßtest Du Dir für den zweiten einen anderen Namen (für die Verzeichnisebene) anstelle von "custom" ausdenken - ansonsten bliebe es beim vorherigen und bereits gemergeten PR. Ich möchte das auch deshalb nur über einen "merge" hinzufügen und ansonsten nicht selbst anpassen, damit da der Urheber eindeutig bleibt und jederzeit in der Historie der Datei nachgelesen werden kann - das ist nicht nur eigene Faulheit (zumal das schneller ginge als die Erklärung dazu - aber es gibt eben Gründe, die dagegen sprechen).

PeterPawn commented 4 years ago

Das mit dem Inhalt der .gitignore war jetzt für mich die beste Lösung, um das Vermischen der zu veröffentlichenden und der "privaten" Daten zu vermeiden - auch deshalb ist die Verwendung von custom in der Verzeichnishierarchie für Dateien, die auch veröffentlicht werden sollen, keine wirklich gute Wahl ... es wird ja custom* ausgeschlossen und mit einem anderen Verzeichnisnamen hast Du dann das Problem mit der Notwendigkeit des --force beim Hinzufügen auch nicht.

TheChatty commented 4 years ago

Lass es gern so wie es ist.