Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

adding own static packages appendig 'dirty' in version string #321

Open
WileC opened this issue Aug 25, 2020 · 2 comments
Open

adding own static packages appendig 'dirty' in version string #321

WileC opened this issue Aug 25, 2020 · 2 comments

Comments

@WileC
Copy link
Contributor

WileC commented Aug 25, 2020

Wenn ich ein eigenes addon in die /addon/static.pkg eintrage und den make durchlaufen lassen, wird zwar alles ordnungsgemäß in die Firmware gepackt, aber der Version-String durch das /tools/freetz-version mit einen "-dirty" versehen ...

Liegt das daran, dass das Verzeichnis /addon/ nicht in der .gitignore drin steht? .. Das Verzeichnis dort eintragen macht aber auch nicht wirklich sinn, da sonst Änderungen durch das Repo nicht übernommen werden oder?

@f-666
Copy link
Member

f-666 commented Aug 26, 2020

Effektiv ja. Ich glaube schon die Veränderung der .gitignore macht das Repository "dirty".

@PeterPawn
Copy link
Contributor

dirty / unclean / tainted - was wäre die richtige "Beschreibung" des Umstands, daß es sich nicht um einen "clean build" handelt, bei dem nur die Pakete und Einstellungen aus dem "Original" verwendet wurden? AVM hängt ein "M" an die Build-Nummer an und schaltet bestimmte Funktionen in den Kernel-Quellen auch nur dann frei, wenn dieses "M" vorhanden ist (Kernel-Funktion avm_fw_is_internal zur Abfrage).

Letztlich geht es ja auch in Freetz nur darum, schon an der Versionsangabe zu erkennen, ob Änderungen erfolgt sind - und das ist natürlich auch dann schon gegeben, wenn man eigene Pakete hinzugefügt hat.

Wobei das wieder einfach "zu umgehen" wäre - man erstellt seinen eigenen Fork vom Ursprung und packt da sein eigenes Paket in den "addon"-Zweig. Dann checkt man dieses eigene Repo aus und baut sein(e) Image(s) mit dem - da es hier keine (weitere) Änderung gibt, schlägt auch freetz-version nicht zu.

Wenn man sich nur am "dirty" stört, kann man das in der freetz-version ja auch deaktivieren (https://github.com/Freetz/freetz/blob/master/tools/freetz-version#L54). Der Test ist ohnehin nicht zu 100% aussagekräftig, da zusätzliche Patches für Pakete nicht automatisch zum "-dirty" führen - weil eben "untracked changed" dank -uno ignoriert werden und weitere Patch-Files für ein Paket wären ja "untracked".

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants