Linus Torvalds critica Paragon Software e nel processo di GitHub crea fusioni inutili

Linus Torvalds ha aspettato per molto tempo a chiedi a Paragon Software di spedire il suo driver NTFS per aggiungerlo al kernel Linux e questo è già fatto e Torvalds ha finalmente unito il nuovo driver con il sorgente del kernel Linux 5.15.

Ma prima si è lamentato dell'utilizzo della funzione di richiesta di unione di GitHub nel post, dicendo che GitHub "crea unioni assolutamente inutili". Apparentemente al creatore del kernel Linux non piacciono molto le unioni di GitHub, se lo fanno. Inoltre, gli avvisi del compilatore ora vengono trattati come errori predefiniti nelle build del kernel.

All'inizio di agosto, Torvalds ha dato a Paragon Software una spinta per effettuare la richiesta pull»Un vero e proprio invio di codice che verrà unito ai sorgenti del kernel, in modo che il suo driver di lettura/scrittura NTFS possa essere incluso nella prossima versione 5.15, per la quale la finestra di unione è attualmente aperta.

Paragon ha debitamente presentato la sua "richiesta di ritiro", dicendo:

"La versione attuale funziona con file normali/compressi/spars e supporta la riproduzione dei log acl e NTFS." Detto questo, la società sta ancora prendendo dimestichezza con il processo di invio e Torvalds aveva parecchi commenti da fare, con l'intento di migliorare le future richieste di pull. Innanzitutto, il creatore del kernel Linux ha affermato che la richiesta pull avrebbe dovuto essere firmata. "In un mondo perfetto, questa sarebbe una firma PGP che posso tracciare direttamente a te attraverso la catena di fiducia, ma non l'ho mai richiesta davvero", ha detto.

Poi notato che il codice nella richiesta pull includeva commit di unione effettuati con l'interfaccia utente web di GitHub.

"È un'altra di quelle cose che * davvero * non voglio vedere: GitHub crea fusioni non necessarie e non dovresti mai usare le interfacce GitHub per unire nulla." le fusioni non sono una di quelle cose. In passato, soprattutto nel 2012, Torvalds si era lamentato di alcuni aspetti di GitHub.

“Non eseguo richieste pull GitHub. GitHub elimina tutte le informazioni rilevanti, ad esempio anche avere un indirizzo email valido per la persona che effettua la richiesta pull. Anche il diffstat è carente e inutile ", ha detto all'epoca. Nota che il comando git request-pull è diverso dalla funzione pull request di GitHub. Inoltre, il repository Paragon ha messaggi di commit a cui mancano informazioni, come »Unisci branch 'torvalds: master' in master«. A proposito, sabato Torvalds ha detto che "le unioni del kernel Linux devono essere eseguite * correttamente *".

Oltre ad aggiungere il controller Paragon NTFS al kernel Linux 5.15, Torvalds ha anche abilitato l'opzione di compilazione "-Werror", che è l'impostazione predefinita per tutte le build del kernel.

Il flag di compilazione »-Errore«tratta tutti gli avvisi come errori di compilazione. Promuovendo tutti gli avvisi come errori, Torvalds obbliga gli sviluppatori a garantire che gli avvisi di build che altrimenti potrebbero passare inosservati o solo vagamente influenzati dagli sviluppatori abbiano ora la priorità poiché interrompono il processo di creazione.

Torvalds ha commentato sul cambiamento che dovremmo sempre avere una struttura pulita e disattiveremo gli avvisi specifici troppo zelanti se necessario, se non possiamo risolverli. Ma mentre lo applico in modo abbastanza religioso sul mio albero, i vari robot da costruzione che non riportano necessariamente gli avvertimenti non lo fanno.

Assicurati che gli avvisi vengano gestiti, ma la maggior parte di questi progetti non ha le dimensioni del kernel Linux. D'altra parte, WERROR come switch Kconfig. Questo disabiliterà il flag »-Werror» se le versioni più recenti del compilatore introducono nuovi avvertimenti che il kernel non può correggere immediatamente o altri problemi selettivi per i quali non è possibile creare un kernel senza avvertimenti. L'opzione WERROR è abilitata per impostazione predefinita per tutte le build del kernel.

Linus ha terminato il messaggio della patch con:

"Speriamo che questo significhi che riceverò meno pull request contenenti nuovi avvertimenti che non sono stati notati dalle varie automazioni che abbiamo implementato." Busso al legno.


Lascia un tuo commento

L'indirizzo email non verrà pubblicato. I campi obbligatori sono contrassegnati con *

*

*

  1. Responsabile dei dati: Miguel Ángel Gatón
  2. Scopo dei dati: controllo SPAM, gestione commenti.
  3. Legittimazione: il tuo consenso
  4. Comunicazione dei dati: I dati non saranno oggetto di comunicazione a terzi se non per obbligo di legge.
  5. Archiviazione dati: database ospitato da Occentus Networks (UE)
  6. Diritti: in qualsiasi momento puoi limitare, recuperare ed eliminare le tue informazioni.