Linus Torvalds critica a Paragon Software e no processo do GitHub cria fusões desnecessárias

Linus Torvalds está esperando por muito tempo para fazer com que a Paragon Software envie seu driver NTFS para adicioná-lo ao kernel Linux e isso já está feito e Torvalds finalmente fundiu o novo driver com a fonte do kernel Linux 5.15.

Mas antes disso ele reclamou sobre o uso do recurso de solicitação de mesclagem do GitHub na postagem, dizendo que o GitHub "cria mesclagens absolutamente desnecessárias". Aparentemente, o criador do kernel do Linux realmente não gosta de mesclagens do GitHub, se gostarem. Além disso, os avisos do compilador agora são tratados como erros padrão nas compilações do kernel.

No início de agosto, Torvalds deu à Paragon Software um empurrão para fazer a solicitação de pull»Um envio real de código que será mesclado com as fontes do kernel, de forma que seu driver de leitura / gravação NTFS possa ser incluído na próxima versão 5.15, para a qual a janela de mesclagem está aberta no momento.

A Paragon apresentou devidamente o seu "pedido de retirada", dizendo:

"A versão atual funciona com arquivos normais / compactados / spars e oferece suporte à reprodução de log ACL e NTFS." Dito isso, a empresa ainda está se familiarizando com o processo de envio e Torvalds tinha alguns comentários a fazer, com a intenção de melhorar as solicitações de pull futuras. Em primeiro lugar, o criador do kernel do Linux disse que a solicitação de pull deveria ter sido assinada. "Em um mundo perfeito, seria uma assinatura PGP que posso rastrear diretamente até você através da cadeia de confiança, mas nunca exigi isso", disse ele.

Depois notou que o código na solicitação pull incluía confirmações de mesclagem feitas com a IU da web do GitHub.

"É mais uma daquelas coisas que eu * realmente * não quero ver: o GitHub cria mesclagens desnecessárias e você nunca deve usar interfaces do GitHub para mesclar nada." As fusões não são uma dessas coisas. No passado, especialmente em 2012, Torvalds reclamava sobre certos aspectos do GitHub.

“Eu não faço solicitações de pull do GitHub. O GitHub descarta todas as informações relevantes, como até mesmo ter um endereço de e-mail válido para a pessoa que fez a solicitação de pull. O diffstat também é deficiente e desnecessário ”, disse na época. Observe que o comando git request-pull é diferente da função de solicitação pull do GitHub. Além disso, o repositório Paragon tem mensagens de commit que não contêm informações, como »Merge branch 'torvalds: master' into master«. Falando nisso, Torvalds disse no sábado que "as fusões do kernel do Linux devem ser feitas * direito *."

Além de adicionar o controlador Paragon NTFS para Linux kernel 5.15, Torvalds também habilitou a opção de construção "-Werror", que é o padrão para todas as compilações de kernel.

O sinalizador de compilação »-Werror«tratar todos os avisos como erros de compilação. Ao promover todos os avisos como erros, Torvalds força os desenvolvedores a garantir que os avisos de construção que poderiam passar despercebidos ou apenas afetados vagamente pelos desenvolvedores agora sejam priorizados conforme interrompem o processo de construção.

Torvalds comentou sobre a mudança que devemos sempre ter uma construção limpa e desativaremos os avisos específicos de excesso de zelo, se necessário, se não pudermos corrigi-los. Mas, embora eu aplique isso religiosamente em minha própria árvore, os vários robôs de construção que não necessariamente relatam os avisos, não o fazem.

Certifique-se de que os avisos sejam tratados, mas a maioria desses projetos não são do tamanho do kernel do Linux. Por outro lado, WERROR como um switch Kconfig. Isso desligará o sinalizador »-Werror» se novas versões do compilador introduzirem novos avisos de que o kernel não pode corrigir imediatamente ou outros problemas seletivos para os quais não é possível criar um kernel sem avisos. A opção WERROR é habilitada por padrão para todas as compilações de kernel.

Linus terminou a mensagem do patch com:

"Espero que isso signifique que receberei menos solicitações de pull contendo novos avisos que não foram percebidos pelas várias automações que implementamos." Eu bato na madeira.