Linus Torvalds bekritiseert Paragon Software en creëert in het proces van GitHub onnodige fusies

Linus Torvalds heeft gewacht voor een lange tijd om Paragon Software zijn stuurprogramma laten verzenden NTFS om toe te voegen aan de Linux-kernel en dit is al gedaan en Torvalds heeft eindelijk het nieuwe stuurprogramma samengevoegd met de Linux 5.15-kernelbron.

Maar daarvoor klaagde hij over het gebruik van GitHub's merge request-functie in de post en zei dat GitHub "absoluut onnodige samenvoegingen maakt". Blijkbaar houdt de maker van de Linux-kernel niet echt van GitHub-fusies, als ze dat wel doen. Ook worden compilerwaarschuwingen nu behandeld als standaardfouten in kernelbuilds.

Begin augustus, Torvalds gaf Paragon Software een duw om het pull-verzoek te doen»Een daadwerkelijke verzending van code die zal worden samengevoegd met de kernelbronnen, zodat de NTFS-lees/schrijf-driver kan worden opgenomen in de volgende versie 5.15, waarvoor het samenvoegvenster momenteel open is.

Paragon heeft zijn "verzoek tot intrekking" naar behoren ingediend en zegt:

"De huidige versie werkt met normale / gecomprimeerde / spars-bestanden en ondersteunt acl- en NTFS-logweergave." Dat gezegd hebbende, raakt het bedrijf nog steeds vertrouwd met het indieningsproces, en Torvalds had nogal wat opmerkingen te maken, met de bedoeling toekomstige pull-verzoeken te verbeteren. Allereerst zei de maker van de Linux-kernel dat het pull-verzoek had moeten worden ondertekend. "In een perfecte wereld zou dat een PGP-handtekening zijn die ik rechtstreeks naar je kan herleiden via de vertrouwensketen, maar ik heb er nooit echt om gevraagd", zei hij.

Dan merkte op dat de code in het pull-verzoek merge-commits bevatte die zijn gemaakt met de GitHub-webgebruikersinterface.

"Het is weer een van die dingen die ik * echt * niet wil zien: GitHub maakt onnodige samenvoegingen en je zou nooit GitHub-interfaces moeten gebruiken om iets samen te voegen." Samenvoegingen zijn niet een van die dingen. In het verleden, vooral in 2012, had Torvalds geklaagd over bepaalde aspecten van GitHub.

“Ik doe geen GitHub-pull-verzoeken. GitHub negeert alle relevante informatie, zoals zelfs het hebben van een geldig e-mailadres van de persoon die het pull-verzoek doet. De diffstat is ook gebrekkig en onnodig”, zei hij destijds. Merk op dat het git request-pull commando verschilt van de pull request functie van GitHub. De Paragon-repository heeft ook commit-berichten waarin informatie ontbreekt, zoals »Vertakking 'torvalds: master' samenvoegen tot master«. Daarover gesproken, Torvalds zei zaterdag dat "Linux-kernelsamenvoegingen * correct * moeten worden gedaan".

Naast het toevoegen van de controller Paragon NTFS naar Linux-kernel 5.15, Torvalds heeft ook de "-Werror" build-optie ingeschakeld, wat de standaard is voor alle kernel-builds.

De compilatievlag »-Werror«behandel alle waarschuwingen als compilatiefouten. Door alle waarschuwingen als fouten te promoten, dwingt Torvalds ontwikkelaars ervoor te zorgen dat buildwaarschuwingen die anders onopgemerkt zouden blijven of slechts losjes door ontwikkelaars zouden worden beïnvloed, nu prioriteit krijgen als ze het bouwproces onderbreken.

Torvalds heeft gereageerd over de verandering dat we echt altijd schoon moeten bouwen en we zullen overijverige specifieke waarschuwingen indien nodig uitschakelen, als we ze niet kunnen repareren. Maar terwijl ik dit heel religieus afdwing aan mijn eigen boom, doen de verschillende constructierobots die de waarschuwingen niet noodzakelijkerwijs rapporteren, dat niet.

Zorg ervoor dat waarschuwingen worden afgehandeld, maar de meeste van deze projecten hebben niet de grootte van de Linux-kernel. Aan de andere kant, WERROR als een Kconfig-switch. Dit zal de vlag »-Werror» uitschakelen als nieuwere versies van de compiler nieuwe waarschuwingen introduceren die de kernel niet onmiddellijk kan corrigeren of andere selectieve problemen waarvoor het niet mogelijk is om een ​​kernel te maken zonder waarschuwingen. De WERROR-optie is standaard ingeschakeld voor alle kernel-builds.

Linus maakte het bericht af van de patch met:

"Hopelijk betekent dit dat ik minder pull-verzoeken zal ontvangen met nieuwe waarschuwingen die niet zijn opgemerkt door de verschillende automatiseringen die we hebben geïmplementeerd." Ik klop op hout.


Laat je reactie achter

Uw e-mailadres wordt niet gepubliceerd. Verplichte velden zijn gemarkeerd met *

*

*

  1. Verantwoordelijk voor de gegevens: Miguel Ángel Gatón
  2. Doel van de gegevens: Controle SPAM, commentaarbeheer.
  3. Legitimatie: uw toestemming
  4. Mededeling van de gegevens: De gegevens worden niet aan derden meegedeeld, behalve op grond van wettelijke verplichting.
  5. Gegevensopslag: database gehost door Occentus Networks (EU)
  6. Rechten: u kunt uw gegevens op elk moment beperken, herstellen en verwijderen.