Linus Torvalds krytykuje Paragon Software i w procesie GitHub tworzy niepotrzebne fuzje

Linus Torvalds czekał przez długi czas aby firma Paragon Software wysłała swój sterownik NTFS, aby dodać go do jądra Linux i to już jest zrobione a Torvalds w końcu połączył nowy sterownik ze źródłami jądra Linux 5.15.

Ale wcześniej narzekał na korzystanie z funkcji żądania scalenia GitHub w poście, mówiąc, że GitHub „tworzy absolutnie niepotrzebne scalenia”. Najwyraźniej twórca jądra Linuksa tak naprawdę nie lubi fuzji GitHub, jeśli tak. Ponadto ostrzeżenia kompilatora są teraz traktowane jako błędy domyślne w kompilacjach jądra.

Na początku sierpnia Torvalds zachęcił firmę Paragon Software do złożenia żądania ściągnięcia»Rzeczywiste wysłanie kodu, który zostanie scalony ze źródłami jądra, tak aby jego sterownik odczytu/zapisu NTFS mógł zostać dołączony do następnej wersji 5.15, dla której okno scalania jest aktualnie otwarte.

Firma Paragon należycie złożyła „wniosek o wycofanie”, mówiąc:

„Obecna wersja działa z plikami normalnymi / skompresowanymi / spars i obsługuje odtwarzanie dzienników acl i NTFS”. To powiedziawszy, firma wciąż zapoznaje się z procesem składania wniosków, a Torvalds miał sporo komentarzy do zgłoszenia, z zamiarem ulepszenia przyszłych żądań ściągnięcia. Przede wszystkim twórca jądra Linuksa powiedział, że pull request powinien być podpisany. „W idealnym świecie byłby to podpis PGP, który mogę prześledzić bezpośrednio do ciebie poprzez łańcuch zaufania, ale nigdy tak naprawdę tego nie wymagałem” – powiedział.

Następnie zauważyłem, że kod w żądaniu ściągnięcia zawierał zatwierdzenia scalania wykonane za pomocą internetowego interfejsu użytkownika GitHub.

„To kolejna z tych rzeczy, których * naprawdę * nie chcę widzieć: GitHub tworzy niepotrzebne scalenia i nigdy nie powinieneś używać interfejsów GitHub do scalania czegokolwiek.” Fuzje nie są jedną z tych rzeczy. W przeszłości, zwłaszcza w 2012 roku, Torvalds narzekał na pewne aspekty GitHub.

„Nie wykonuję pull requestów na GitHub. GitHub odrzuca wszystkie istotne informacje, takie jak nawet posiadanie ważnego adresu e-mail osoby, która wysyła żądanie ściągnięcia. diffstat jest również niewystarczający i niepotrzebny ”- powiedział wtedy. Zauważ, że polecenie git request-pull różni się od funkcji pull request dostępnej w GitHub. Ponadto repozytorium Paragon zawiera komunikaty commit, w których brakuje informacji, takie jak »Scal gałąź 'torvalds: master' w master«. Skoro o tym mowa, Torvalds powiedział w sobotę, że "Scalanie jądra Linuksa musi być wykonane * poprawnie *".

Oprócz dodania kontrolera Paragon NTFS do jądra Linux 5.15, Torvalds włączył również opcję budowania "-Werror", co jest wartością domyślną dla wszystkich kompilacji jądra.

Flaga kompilacji »-Błąd«traktuj wszystkie ostrzeżenia jako błędy kompilacji. Promując wszystkie ostrzeżenia jako błędy, Torvalds zmusza programistów do zapewnienia, że ​​ostrzeżenia o kompilacji, które w innym przypadku mogłyby pozostać niezauważone lub tylko w niewielkim stopniu dotknięte przez programistów, mają teraz priorytet, ponieważ przerywają proces kompilacji.

Torvalds skomentował o zmianie że naprawdę powinniśmy zawsze mieć czystą budowę i w razie potrzeby wyłączymy nadgorliwe konkretne ostrzeżenia, jeśli nie będziemy w stanie ich naprawić. Ale chociaż egzekwuję to dość religijnie na moim własnym drzewie, różne roboty budowlane, które niekoniecznie zgłaszają ostrzeżenia, nie robią tego.

Upewnij się, że ostrzeżenia są obsługiwane, ale większość z tych projektów nie ma rozmiaru jądra Linuksa. Z drugiej strony WERROR jako przełącznik Kconfig. Spowoduje to wyłączenie flagi »-Werror», jeśli nowsze wersje kompilatora wprowadzą nowe ostrzeżenia, że ​​jądro nie może natychmiast naprawić lub inne selektywne problemy, dla których nie można utworzyć jądra bez ostrzeżeń. Opcja WERROR jest domyślnie włączona we wszystkich kompilacjach jądra.

Linus dokończył wiadomość łatki z:

„Mam nadzieję, że będzie to oznaczać, że będę otrzymywać mniej żądań ściągnięcia zawierających nowe ostrzeżenia, które nie zostały zauważone przez różne wdrożone przez nas automatyzacje”. Pukam w drewno.


Bądź pierwszym który skomentuje

Zostaw swój komentarz

Twój adres e-mail nie zostanie opublikowany. Wymagane pola są oznaczone *

*

*

  1. Odpowiedzialny za dane: Miguel Ángel Gatón
  2. Cel danych: kontrola spamu, zarządzanie komentarzami.
  3. Legitymacja: Twoja zgoda
  4. Przekazywanie danych: Dane nie będą przekazywane stronom trzecim, z wyjątkiem obowiązku prawnego.
  5. Przechowywanie danych: baza danych hostowana przez Occentus Networks (UE)
  6. Prawa: w dowolnym momencie możesz ograniczyć, odzyskać i usunąć swoje dane.