Патчи для WINE@Etersoft — различия между версиями

Версия 18:58, 22 мая 2012

Чтобы навести некоторый порядок, я постарался записать необходимые правила, которые должны помочь улучшению качества разработки.

О порядке создания изменений для WINE@Etersoft

1. Нужно обязательно проверять, что проблема существует и на wine pure (запуском через wwo или как-то ещё). Это позволит узнать, проблема вызвана нашими изменениями или действительна для всех.
2. Всю разработку нужно вести в ветке eterhack, обновляясь перед началом работы. Это позволит не исправлять баги, которые уже давно исправлены или ведут себя по-другому.
3. После исправления в eterhack патч надо отправить (при необходимости портировав) в сборку стабильного релиза (сейчас eter-2.0.0)

Если исправление не является хаком, или касается какой-то части кода, изменённой нами в eterwine, то отправлять патч надо в eterwine.

Также напоминаю, что хаки, существенно добавляющие код, стоит по возможности реализовывать в виде отдельных функций, чтобы минимизировать изменения оригинального кода. Если требуется исключить часть кода, её следует поместить в конструкцию #if 0 #endif, не удаляя

Важные моменты:

1. Патчи из eter-2.0.0 пропадут, потому что следующая версия будет основана на eterhack. Поэтому важно отправлять их одновременно и туда и туда.
2. Патчи из eterwine автоматически копируются в eterhack

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

Перед созданием хака, специфичного для WINE@Etersoft, нужно приложить максимум усилий для по возможности полного исследования проблемы и её источника, созданя минимальной программы для воспроизведения проблемы, и конечно для реального исправления, а не очередного костыля в коде.