Linus Torvalds kritiserer Paragon Software og skaber i processen med GitHub unødvendige fusioner

Linus Torvalds har ventet i lang tid til få Paragon Software til at sende sin driver NTFS for at tilføje det til Linux -kernen og dette er allerede gjort og Torvalds har endelig fusioneret den nye driver med Linux 5.15 kernel source.

Men før det klagede han over at bruge GitHubs fletningsanmodningsfunktion i indlægget og sagde, at GitHub "skaber absolut unødvendige fusioner." Tilsyneladende kan skaberen af ​​Linux -kernen ikke rigtig lide GitHub -fusioner, hvis de gør det. Compiler advarsler behandles nu også som standardfejl i kernel builds.

I begyndelsen af ​​august, Torvalds gav Paragon Software et skub for at foretage trækanmodningen»En egentlig afsendelse af kode, der vil blive flettet med kernelkilderne, så dens NTFS læse / skrive driver kan inkluderes i den næste version 5.15, som fletningsvinduet i øjeblikket er åbent for.

Paragon indsendte behørigt sin "anmodning om tilbagetrækning" og sagde:

"Nuværende version fungerer med normale / komprimerede / spars -filer og understøtter acl- og NTFS -logafspilning." Når det er sagt, bliver virksomheden stadig bekendt med indsendelsesprocessen, og Torvalds havde en del kommentarer at komme med, med det formål at forbedre fremtidige pull -anmodninger. Først og fremmest sagde skaberen af ​​Linux -kernen, at pull -anmodningen skulle have været underskrevet. "I en perfekt verden ville det være en PGP -signatur, som jeg kan spore direkte til dig gennem tillidskæden, men jeg har aldrig rigtig krævet det," sagde han.

Derefter bemærkede, at koden i pull -anmodningen inkluderede fletteforpligtelser, der blev foretaget med GitHub -webgrænsefladen.

"Det er endnu en af ​​de ting, jeg * virkelig * ikke vil se: GitHub skaber unødvendige fusioner, og du bør aldrig bruge GitHub -grænseflader til at flette noget." Fusioner er ikke en af ​​disse ting. Tidligere, især i 2012, havde Torvalds klaget over visse aspekter af GitHub.

“Jeg foretager ikke GitHub pull -anmodninger. GitHub kasserer alle relevante oplysninger, f.eks. Selv at have en gyldig e -mail -adresse til den person, der foretager trækanmodningen. Diffstat er også mangelfuld og unødvendig, ”sagde han dengang. Bemærk, at kommandoen git request-pull er forskellig fra pull request-funktionen fra GitHub. Paragon -depotet har også kommitterende beskeder, der mangler oplysninger, såsom »Flet filial 'torvalds: master' til master«. Taler om det, Torvalds sagde lørdag, at "Linux kernel fusioner skal udføres * korrekt *."

Udover at tilføje controlleren Paragon NTFS til Linux -kerne 5.15, Torvalds har også aktiveret indstillingen "-Werror", som er standard for alle kernel builds.

Kompilationsflaget »-Fejl«behandle alle advarsler som kompilationsfejl. Ved at promovere alle advarsler som fejl, tvinger Torvalds udviklere til at sikre, at bygningsadvarsler, der ellers kan gå ubemærket hen eller kun løst påvirkes af udviklere, nu prioriteres, når de afbryder byggeprocessen.

Torvalds kommenterede om ændringen at vi virkelig altid skal have et rent byggeri og vi vil deaktivere overdrevne specifikke advarsler, hvis det er nødvendigt, hvis vi ikke kan rette dem. Men mens jeg håndhæver dette ganske religiøst på mit eget træ, gør de forskellige konstruktionsrobotter, der ikke nødvendigvis rapporterer advarslerne, det.

Sørg for, at advarsler håndteres, men de fleste af disse projekter er ikke på størrelse med Linux -kernen. På den anden side FEJL som en Kconfig switch. Dette vil deaktivere »-Fejl« -flaget, hvis nyere versioner af kompilatoren introducerer nye advarsler, som kernen ikke kan rette med det samme eller andre selektive problemer, som det ikke er muligt at oprette en kerne for uden advarsler. WERROR -indstillingen er som standard aktiveret for alle kernel -builds.

Linus afsluttede beskeden af plasteret med:

"Forhåbentlig vil dette betyde, at jeg vil modtage færre pull -anmodninger, der indeholder nye advarsler, som ikke blev bemærket af de forskellige automatiseringer, vi har implementeret." Jeg banker på træ.


Efterlad din kommentar

Din e-mailadresse vil ikke blive offentliggjort. Obligatoriske felter er markeret med *

*

*

  1. Ansvarlig for dataene: Miguel Ángel Gatón
  2. Formålet med dataene: Control SPAM, management af kommentarer.
  3. Legitimering: Dit samtykke
  4. Kommunikation af dataene: Dataene vil ikke blive kommunikeret til tredjemand, undtagen ved juridisk forpligtelse.
  5. Datalagring: Database hostet af Occentus Networks (EU)
  6. Rettigheder: Du kan til enhver tid begrænse, gendanne og slette dine oplysninger.