Линус Торвальдс критикует Paragon Software и в процессе GitHub создает ненужные слияния

Линус Торвальдс ждал долгое время получить драйвер от Paragon Software NTFS, чтобы добавить его в ядро ​​Linux и это уже сделано и Торвальдс наконец объединил новый драйвер с исходным кодом ядра Linux 5.15.

Но до этого он жаловался на использование функции слияния GitHub. в сообщении говорится, что GitHub «создает совершенно ненужные слияния». Судя по всему, создатель ядра Linux не очень любит слияния GitHub, если они его любят. Кроме того, предупреждения компилятора теперь обрабатываются как ошибки по умолчанию в сборках ядра.

В начале августа Торвальдс подтолкнул Paragon Software сделать запрос на вытягивание»Фактическая отправка кода, который будет объединен с исходными кодами ядра, так что его драйвер чтения / записи NTFS может быть включен в следующую версию 5.15, для которой в настоящее время открыто окно слияния.

Paragon должным образом представил свой «запрос на снятие средств», в котором говорилось:

«Текущая версия работает с обычными / сжатыми / spars файлами и поддерживает воспроизведение журналов acl и NTFS». Тем не менее, компания все еще знакомится с процессом подачи заявок, и Торвальдс хотел сделать несколько замечаний с намерением улучшить будущие запросы на вытягивание. Прежде всего, создатель ядра Linux сказал, что запрос на перенос должен быть подписан. «В идеальном мире это была бы подпись PGP, которую я мог бы проследить прямо до вас через цепочку доверия, но я никогда не требовал этого», - сказал он.

Затем заметил, что код в запросе на перенос включает коммиты слияния, сделанные с помощью веб-интерфейса GitHub.

«Это еще одна из тех вещей, которые я * действительно * не хочу видеть: GitHub создает ненужные слияния, и вам никогда не следует использовать интерфейсы GitHub для слияния чего-либо». Слияния - не одна из таких вещей. В прошлом, особенно в 2012 году, Торвальдс жаловался на определенные аспекты GitHub.

«Я не выполняю пул-реквесты на GitHub. GitHub отбрасывает всю соответствующую информацию, например, даже наличие действительного адреса электронной почты человека, отправляющего запрос на перенос. Также отсутствует и ненужный diffstat », - сказал он тогда. Обратите внимание, что команда git request-pull отличается от функции pull request из GitHub. Кроме того, в репозитории Paragon есть сообщения фиксации, в которых отсутствует информация, например, «Объединить ветку torvalds: master в master». Говоря об этом, Торвальдс сказал в субботу, что «слияние ядра Linux должно выполняться * правильно *».

Помимо добавления контроллера Paragon NTFS для ядра Linux 5.15, Торвальдс также включил параметр сборки "-Werror", который используется по умолчанию для всех сборок ядра.

Флаг компиляции »-Werror«рассматривать все предупреждения как ошибки компиляции. Продвигая все предупреждения как ошибки, Торвальдс вынуждает разработчиков следить за тем, чтобы предупреждения сборки, которые в противном случае могли бы остаться незамеченными или на которые разработчики не могли повлиять, теперь имеют приоритет, поскольку они прерывают процесс сборки.

Торвальдс прокомментировал об изменении что у нас всегда должна быть чистая сборка и мы отключим чрезмерно усердные специальные предупреждения, если это необходимо, если мы не сможем их исправить. Но в то время как я строго придерживаюсь этого правила на своем собственном дереве, различные строительные роботы, которые не обязательно сообщают о предупреждениях, этого не делают.

Убедитесь, что предупреждения обрабатываются, но размер большинства этих проектов меньше размера ядра Linux. С другой стороны, WERROR как переключатель Kconfig. Это отключит флаг »-Werror», если в новых версиях компилятора появятся новые предупреждения, которые ядро ​​не может исправить немедленно, или другие выборочные проблемы, для которых невозможно создать ядро ​​без предупреждений. Параметр WERROR включен по умолчанию для всех сборок ядра.

Линус закончил сообщение патча с:

«Надеюсь, это будет означать, что я буду получать меньше запросов на вытягивание, содержащих новые предупреждения, которые не были замечены различными автоматизаторами, которые мы внедрили». Я стучу по дереву.


Оставьте свой комментарий

Ваш электронный адрес не будет опубликован. Обязательные для заполнения поля помечены *

*

*

  1. Ответственный за данные: Мигель Анхель Гатон
  2. Назначение данных: контроль спама, управление комментариями.
  3. Легитимация: ваше согласие
  4. Передача данных: данные не будут переданы третьим лицам, кроме как по закону.
  5. Хранение данных: база данных, размещенная в Occentus Networks (ЕС)
  6. Права: в любое время вы можете ограничить, восстановить и удалить свою информацию.