Linus Torvalds har ventet lenge til få Paragon Software til å sende sin sjåfør NTFS for å legge den til Linux -kjernen og dette er allerede gjort og Torvalds har endelig slått sammen den nye driveren med Linux 5.15 -kjernekilden.
Men før det klaget han over bruk av GitHubs fletteforespørselsfunksjon i innlegget og sa at GitHub "skaper absolutt unødvendige sammenslåinger." Tilsynelatende liker ikke skaperen av Linux -kjernen GitHub -sammenslåinger, hvis de gjør det. Kompilatoradvarsler blir nå behandlet som standardfeil i kjerneoppbygginger.
I begynnelsen av august, Torvalds ga Paragon Software et trykk for å gjøre trekkforespørselen»En faktisk sending av kode som vil bli slått sammen med kjernekildene, slik at NTFS -lese / skrive -driveren kan inkluderes i neste versjon 5.15, som flettevinduet for øyeblikket er åpent for.
Paragon sendte behørig sin "tilbaketrekningsforespørsel" og sa:
"Gjeldende versjon fungerer med normale / komprimerte / spars -filer og støtter avspilling av acl- og NTFS -logg." Når det er sagt, blir selskapet fortsatt kjent med innsendingsprosessen, og Torvalds hadde ganske mange kommentarer å komme med, med den hensikt å forbedre fremtidige trekkforespørsler. Først og fremst sa skaperen av Linux -kjernen at pull -forespørselen burde vært signert. "I en perfekt verden ville det være en PGP -signatur som jeg kan spore direkte til deg gjennom tillitskjeden, men jeg har aldri krevd det," sa han.
Deretter la merke til at koden i pull -forespørselen inkluderte sammenslåingsforpliktelser som ble gjort med GitHub -webgrensesnittet.
"Det er nok en av de tingene jeg * virkelig * ikke vil se: GitHub skaper unødvendige sammenslåinger, og du bør aldri bruke GitHub -grensesnitt for å slå sammen noe." Fusjoner er ikke en av disse tingene. Tidligere, spesielt i 2012, hadde Torvalds klaget på visse aspekter ved GitHub.
“Jeg gjør ikke GitHub pull -forespørsler. GitHub forkaster all relevant informasjon, for eksempel til og med å ha en gyldig e -postadresse for personen som foretar trekkforespørselen. Diffstaten mangler også og er unødvendig, ”sa han den gang. Vær oppmerksom på at git request-pull-kommandoen er forskjellig fra pull request-funksjonen fra GitHub. Paragon -depotet har også formidle meldinger som mangler informasjon, for eksempel »Merge branch 'torvalds: master' into master«. Når vi snakket om det, sa Torvalds lørdag at "Linux -kjernefusjoner må utføres * riktig *".
I tillegg til å legge til kontrolleren Paragon NTFS til Linux -kjernen 5.15, Torvalds har også aktivert alternativet "-Feil", som er standard for alle kjerneoppbygginger.
Samlingsflagget »-Feil«behandle alle advarsler som kompilasjonsfeil. Ved å fremme alle advarsler som feil, tvinger Torvalds utviklere til å sikre at bygningsadvarsler som ellers kan gå upåaktet hen eller bare løst påvirkes av utviklere, blir prioritert nå når de avbryter byggeprosessen.
Torvalds kommenterte om endringen at vi virkelig alltid bør ha et rent bygg og vi vil deaktivere overivrige spesifikke advarsler om nødvendig hvis vi ikke kan fikse dem. Men mens jeg håndhever dette ganske religiøst på mitt eget tre, gjør ikke de forskjellige konstruksjonsrobotene som ikke nødvendigvis rapporterer advarslene.
Sørg for at advarsler håndteres, men de fleste av disse prosjektene er ikke på størrelse med Linux -kjernen. På den annen side, FEIL som en Kconfig -bryter. Dette vil slå av »-Feil» -flagget hvis nyere versjoner av kompilatoren introduserer nye advarsler som kjernen ikke kan rette umiddelbart eller andre selektive problemer som det ikke er mulig å opprette en kjerne for uten advarsler. WERROR -alternativet er aktivert som standard for alle kjerneoppbygginger.
Linus avsluttet meldingen av lappen med:
"Forhåpentligvis vil dette bety at jeg vil motta færre trekkforespørsler som inneholder nye advarsler som ikke ble lagt merke til av de forskjellige automatiseringene vi har implementert." Jeg banker på tre.