Linus Torvalds hat gewartet für eine lange Zeit zu Treiber von Paragon Software ausliefern lassen NTFS zum Hinzufügen zum Linux-Kernel und das ist schon fertig und Torvalds hat den neuen Treiber endlich mit dem Linux 5.15-Kernel-Quellcode zusammengeführt.
Zuvor beschwerte er sich jedoch über die Verwendung der Merge-Request-Funktion von GitHub in der Post und sagte, dass GitHub "absolut unnötige Zusammenführungen erstellt". Anscheinend mag der Schöpfer des Linux-Kernels keine GitHub-Merges, wenn sie es tun. Außerdem werden Compiler-Warnungen jetzt als Standardfehler in Kernel-Builds behandelt.
Anfang August, Torvalds gab Paragon Software einen Schubs, um die Pull-Anfrage zu stellen»Ein tatsächlicher Codeversand, der mit den Kernel-Quellen zusammengeführt wird, damit sein NTFS-Lese-/Schreibtreiber in die nächste Version 5.15 aufgenommen werden kann, für die das Zusammenführungsfenster derzeit geöffnet ist.
Paragon reichte seinen „Rückzugsantrag“ ordnungsgemäß ein und sagte:
"Die aktuelle Version funktioniert mit normalen / komprimierten / Spars-Dateien und unterstützt die Wiedergabe von ACL- und NTFS-Protokollen." Das Unternehmen macht sich jedoch immer noch mit dem Einreichungsprozess vertraut, und Torvalds hatte einige Anmerkungen, um zukünftige Pull-Requests zu verbessern. Zunächst sagte der Schöpfer des Linux-Kernels, dass der Pull-Request hätte signiert werden sollen. "In einer perfekten Welt wäre das eine PGP-Signatur, die ich durch die Vertrauenskette direkt zu Ihnen zurückverfolgen kann, aber ich habe sie nie wirklich gefordert", sagte er.
Dann bemerkte, dass der Code in der Pull-Anfrage Merge-Commits enthielt, die mit der GitHub-Web-UI gemacht wurden.
„Es ist noch eines der Dinge, die ich *wirklich* nicht sehen möchte: GitHub erstellt unnötige Zusammenführungen und Sie sollten niemals GitHub-Schnittstellen verwenden, um etwas zusammenzuführen.“ Zusammenführungen gehören nicht zu diesen Dingen. In der Vergangenheit, insbesondere im Jahr 2012, hatte sich Torvalds über bestimmte Aspekte von GitHub beschwert.
„Ich mache keine GitHub-Pull-Requests. GitHub verwirft alle relevanten Informationen, wie beispielsweise eine gültige E-Mail-Adresse der Person, die den Pull-Request stellt. Auch das Diffstat fehlt und ist unnötig“, sagte er damals. Beachten Sie, dass sich der Befehl git request-pull von der Pull-Request-Funktion von GitHub unterscheidet. Außerdem enthält das Paragon-Repository Commit-Meldungen, denen Informationen fehlen, wie etwa »Zweig 'torvalds: master' in master zusammenführen«. Apropos, Torvalds sagte am Samstag, dass "Linux-Kernel-Merges * korrekt * durchgeführt werden müssen".
Neben dem Hinzufügen des Controllers Paragon NTFS zu Linux-Kernel 5.15, Torvalds hat auch die Build-Option "-Werror" aktiviert, Dies ist die Standardeinstellung für alle Kernel-Builds.
Das Kompilierungsflag »-Werror«alle Warnungen als Kompilierungsfehler behandeln. Indem alle Warnungen als Fehler hochgestuft werden, zwingt Torvalds die Entwickler, sicherzustellen, dass Build-Warnungen, die ansonsten unbemerkt bleiben oder von Entwicklern nur lose beeinflusst werden könnten, jetzt priorisiert werden, wenn sie den Build-Prozess unterbrechen.
Torvalds kommentiert über die Änderung dass wir wirklich immer einen sauberen Build haben sollten und wir werden bei Bedarf übereifrige spezifische Warnungen deaktivieren, wenn wir sie nicht beheben können. Aber während ich dies ziemlich religiös auf meinem eigenen Baum durchsetze, tun es die verschiedenen Bauroboter, die die Warnungen nicht unbedingt melden, nicht.
Stellen Sie sicher, dass Warnungen behandelt werden, aber die meisten dieser Projekte haben nicht die Größe des Linux-Kernels. Auf der anderen Seite WERROR als Kconfig-Schalter. Dadurch wird das Flag »-Werror» ausgeschaltet, wenn neuere Versionen des Compilers neue Warnungen einführen, die der Kernel nicht sofort beheben kann, oder andere punktuelle Probleme, für die es nicht möglich ist, einen Kernel ohne Warnungen zu erstellen. Die Option WERROR ist standardmäßig für alle Kernel-Builds aktiviert.
Linus hat die Nachricht beendet des Patches mit:
"Das bedeutet hoffentlich, dass ich weniger Pull-Requests mit neuen Warnungen erhalte, die von den verschiedenen Automatisierungen, die wir implementiert haben, nicht bemerkt wurden." Ich klopfe auf Holz.