Linus Torvalds kritiserar Paragon Software och skapar i GitHub onödiga sammanslagningar

Linus Torvalds har väntat länge till låta Paragon Software skicka sin förare NTFS att lägga till i Linux -kärnan och detta är redan gjort och Torvalds har äntligen slagit ihop den nya drivrutinen med Linux 5.15 -kärnkällan.

Men innan det klagade han över att han använde GitHubs funktion för sammanfogningsbegäran i inlägget och säger att GitHub "skapar absolut onödiga sammanslagningar." Uppenbarligen gillar inte skaparen av Linux -kärnan GitHub -sammanslagningar om de gör det. Kompilatorvarningar behandlas nu också som standardfel i kärnbyggnader.

I början av augusti, Torvalds gav Paragon Software ett tryck för att göra dragbegäran»En faktisk sändning av kod som kommer att slås samman med kärnkällorna, så att dess NTFS -läs / skriv -drivrutin kan inkluderas i nästa version 5.15, för vilken sammanfogningsfönstret för närvarande är öppet.

Paragon lämnade in sin "begäran om uttag" och sade:

"Nuvarande version fungerar med normala / komprimerade / spars -filer och stöder acl- och NTFS -logguppspelning." Som sagt, företaget håller fortfarande på att bekanta sig med inlämningsprocessen, och Torvalds hade en hel del kommentarer att göra med avsikt att förbättra framtida dragförfrågningar. Först och främst sa skaparen av Linux -kärnan att dragbegäran borde ha undertecknats. "I en perfekt värld skulle det vara en PGP -signatur som jag kan spåra direkt till dig genom förtroendeskedjan, men jag krävde det aldrig riktigt," sa han.

Sedan märkte att koden i pull -begäran inkluderade sammanslagningsåtaganden som gjorts med GitHub webbgränssnitt.

"Det är en annan av de sakerna jag * verkligen * inte vill se: GitHub skapar onödiga sammanslagningar och du bör aldrig använda GitHub -gränssnitt för att slå ihop någonting." Fusioner är inte en av dessa saker. Tidigare, särskilt 2012, hade Torvalds klagat över vissa aspekter av GitHub.

”Jag gör inte GitHub pull -begäranden. GitHub kasserar all relevant information, till exempel att ha en giltig e -postadress för personen som gör dragbegäran. Diffstatistiken är också bristfällig och onödig, säger han då. Observera att kommandot git request-pull skiljer sig från pull request-funktionen från GitHub. Paragon -förvaret har också meddelanden som saknar information, till exempel "Slå samman gren" torvalds: master "till master". På tal om detta sa Torvalds på lördagen att "Linux -kärnfusioner måste göras * korrekt *".

Förutom att lägga till styrenheten Paragon NTFS till Linux -kärna 5.15, Torvalds har också aktiverat alternativet "-Werror", som är standard för alla kärnuppbyggnader.

Sammanställningsflaggan »-Fel«behandla alla varningar som sammanställningsfel. Genom att marknadsföra alla varningar som fel tvingar Torvalds utvecklare att se till att byggvarningar som annars skulle gå obemärkt förbi eller bara löst påverkas av utvecklare nu prioriteras när de avbryter byggprocessen.

Torvalds kommenterade om förändringen att vi verkligen alltid ska ha en ren byggnad och vi kommer att inaktivera överdriven specifika varningar om det behövs om vi inte kan åtgärda dem. Men medan jag tillämpar detta ganska religiöst på mitt eget träd, gör inte de olika konstruktionsrobotarna som inte nödvändigtvis rapporterar varningarna.

Se till att varningar hanteras, men de flesta av dessa projekt är inte storleken på Linux -kärnan. Å andra sidan WERROR som en Kconfig -switch. Detta kommer att inaktivera flaggan »-Fel» om nyare versioner av kompilatorn introducerar nya varningar som kärnan inte kan korrigera omedelbart eller andra selektiva problem för vilka det inte är möjligt att skapa en kärna utan varningar. Alternativet WERROR är aktiverat som standard för alla kärnuppbyggnader.

Linus avslutade meddelandet av lappen med:

"Förhoppningsvis kommer detta att innebära att jag kommer att få färre dragförfrågningar som innehåller nya varningar som inte uppmärksammades av de olika automationerna vi har implementerat." Jag knackar på trä.


Lämna din kommentar

Din e-postadress kommer inte att publiceras. Obligatoriska fält är markerade med *

*

*

  1. Ansvarig för uppgifterna: Miguel Ángel Gatón
  2. Syftet med uppgifterna: Kontrollera skräppost, kommentarhantering.
  3. Legitimering: Ditt samtycke
  4. Kommunikation av uppgifterna: Uppgifterna kommer inte att kommuniceras till tredje part förutom enligt laglig skyldighet.
  5. Datalagring: databas värd för Occentus Networks (EU)
  6. Rättigheter: När som helst kan du begränsa, återställa och radera din information.