Linus Torvalds critique Paragon Software et dans le processus de GitHub crée des fusions inutiles

Linus Torvalds attendait pendant longtemps pour demander à Paragon Software d'expédier son pilote NTFS à ajouter au noyau Linux et c'est déjà fait et Torvalds a finalement fusionné le nouveau pilote avec la source du noyau Linux 5.15.

Mais avant cela, il s'est plaint de l'utilisation de la fonctionnalité de demande de fusion de GitHub dans le post, disant que GitHub "crée des fusions absolument inutiles". Apparemment, le créateur du noyau Linux n'aime pas vraiment les fusions GitHub, s'ils le font. De plus, les avertissements du compilateur sont désormais traités comme des erreurs par défaut dans les versions du noyau.

Début août, Torvalds a donné un coup de pouce à Paragon Software pour faire la demande de tirage» Un véritable envoi de code qui sera fusionné avec les sources du noyau, afin que son pilote de lecture/écriture NTFS puisse être inclus dans la prochaine version 5.15, pour laquelle la fenêtre de fusion est actuellement ouverte.

Paragon a dûment soumis sa « demande de retrait », en déclarant :

"La version actuelle fonctionne avec les fichiers normaux/compressés/spars et prend en charge la lecture des journaux acl et NTFS." Cela dit, la société se familiarise toujours avec le processus de soumission, et Torvalds avait quelques commentaires à faire, dans le but d'améliorer les futures demandes de tirage. Tout d'abord, le créateur du noyau Linux a déclaré que la pull request aurait dû être signée. "Dans un monde parfait, ce serait une signature PGP que je peux retracer directement jusqu'à vous via la chaîne de confiance, mais je ne l'ai jamais vraiment exigée", a-t-il déclaré.

Puis remarqué que le code de la demande d'extraction incluait des commits de fusion effectués avec l'interface utilisateur Web GitHub.

"C'est une autre de ces choses que je ne veux * vraiment * pas voir : GitHub crée des fusions inutiles et vous ne devriez jamais utiliser les interfaces GitHub pour fusionner quoi que ce soit. " les fusions ne font pas partie de ces choses. Par le passé, notamment en 2012, Torvalds s'était plaint de certains aspects de GitHub.

« Je ne fais pas de requêtes d'extraction GitHub. GitHub supprime toutes les informations pertinentes, telles que même avoir une adresse e-mail valide pour la personne faisant la demande d'extraction. Le diffstat fait également défaut et est inutile », avait-il déclaré à l'époque. Notez que la commande git request-pull est différente de la fonction pull request de GitHub. De plus, le référentiel Paragon contient des messages de validation qui manquent d'informations, tels que »Fusionner la branche 'torvalds: master' dans master«. En parlant de cela, Torvalds a déclaré samedi que "les fusions du noyau Linux doivent être effectuées * correctement *".

En plus d'ajouter le contrôleur Paragon NTFS vers le noyau Linux 5.15, Torvalds a également activé l'option de construction "-Werror", qui est la valeur par défaut pour toutes les versions du noyau.

Le drapeau de compilation »-Werror«traiter tous les avertissements comme des erreurs de compilation. En promouvant tous les avertissements comme des erreurs, Torvalds oblige les développeurs à s'assurer que les avertissements de build qui pourraient autrement passer inaperçus ou seulement vaguement affectés par les développeurs sont désormais prioritaires lorsqu'ils interrompent le processus de build.

Torvalds a commenté sur le changement que nous devrions vraiment toujours avoir une construction propre et nous désactiverons les avertissements spécifiques trop zélés si nécessaire, si nous ne pouvons pas les corriger. Mais alors que j'applique cela assez religieusement sur mon propre arbre, les différents robots de construction qui ne signalent pas nécessairement les avertissements ne le font pas.

Assurez-vous que les avertissements sont traités, mais la plupart de ces projets n'ont pas la taille du noyau Linux. D'autre part, WERROR en tant que commutateur Kconfig. Cela désactivera le drapeau »-Werror» si les nouvelles versions du compilateur introduisent de nouveaux avertissements que le noyau ne peut pas corriger immédiatement ou d'autres problèmes sélectifs pour lesquels il n'est pas possible de créer un noyau sans avertissements. L'option WERROR est activée par défaut pour toutes les versions du noyau.

Linus a terminé le message du patch avec :

"J'espère que cela signifiera que je recevrai moins de demandes d'extraction contenant de nouveaux avertissements qui n'ont pas été remarqués par les différentes automatisations que nous avons implémentées." Touchons du bois.


Le contenu de l'article adhère à nos principes de éthique éditoriale. Pour signaler une erreur, cliquez sur c'est par ici !.

Soyez le premier à commenter

Laisser un commentaire

Votre adresse e-mail ne sera pas publiée.

*

*

  1. Responsable des données: Miguel Ángel Gatón
  2. Finalité des données: Contrôle du SPAM, gestion des commentaires.
  3. Légitimation: votre consentement
  4. Communication des données: Les données ne seront pas communiquées à des tiers sauf obligation légale.
  5. Stockage des données: base de données hébergée par Occentus Networks (EU)
  6. Droits: à tout moment, vous pouvez limiter, récupérer et supprimer vos informations.